Skip to content

fix(synthetics): chromium dependencies for Synthetics monitor#48569

Merged
mcapell merged 3 commits intoelastic:mainfrom
mcapell:mcapell/fix/wolfi-missing-deps-playwright
Jan 29, 2026
Merged

fix(synthetics): chromium dependencies for Synthetics monitor#48569
mcapell merged 3 commits intoelastic:mainfrom
mcapell:mcapell/fix/wolfi-missing-deps-playwright

Conversation

@mcapell
Copy link
Contributor

@mcapell mcapell commented Jan 27, 2026

Proposed commit message

Add missing dependency to run chromium on wolfi images.

Checklist

  • My code follows the style guidelines of this project
    - [ ] I have commented my code, particularly in hard-to-understand areas
    - [ ] I have made corresponding changes to the documentation
    - [ ] I have made corresponding change to the default configuration files
    - [x] I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Within the ./x-pack/heartbeat directory, build the docker image with:

DEV=true PLATFORMS=linux/arm64 PACKAGES=docker mage package

Without the dependency, the build fails with the following error:

Dockerfile:129
--------------------
 128 |     RUN (npm i -g --loglevel verbose --production --engine-strict  @elastic/synthetics@stack_release || sh -c 'tail -n +1 ${NPM_CONFIG_PREFIX}/_logs/* && exit 1')
 129 | >>> RUN CHROMIUM_BIN=$(ls -1 /usr/share/heartbeat/.cache/ms-playwright/chromium_headless_shell-*/chrome-linux/headless_shell | head -n1);\
 130 | >>>     $CHROMIUM_BIN --version
 131 |
--------------------
ERROR: failed to build: failed to solve: process "/bin/sh -c CHROMIUM_BIN=$(ls -1 /usr/share/heartbeat/.cache/ms-playwright/chromium_headless_shell-*/chrome-linux/headless_shell | head -n1);    $CHROMIUM_BIN --version" did no
t complete successfully: exit code: 127

Related issues

Use cases

Screenshots

Logs

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 27, 2026
@github-actions
Copy link
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
@mcapell mcapell requested a review from emilioalvap January 27, 2026 17:34
@mcapell mcapell marked this pull request as ready for review January 27, 2026 17:34
@mcapell mcapell requested a review from a team as a code owner January 27, 2026 17:34
@mcapell mcapell requested review from mauri870 and rdner January 27, 2026 17:34
@mcapell mcapell added the Team:obs-ds-hosted-services Label for the Observability Hosted Services team label Jan 27, 2026
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 27, 2026
@mergify
Copy link
Contributor

mergify bot commented Jan 27, 2026

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @mcapell? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.
@mcapell mcapell added needs_team Indicates that the issue/PR needs a Team:* label backport-active-all Automated backport with mergify to all the active branches labels Jan 27, 2026
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 27, 2026
Add missing dependency to run chromium on wolfi images.
@mcapell mcapell force-pushed the mcapell/fix/wolfi-missing-deps-playwright branch from 3e4e7a8 to 2aae7e5 Compare January 28, 2026 14:18
@github-actions
Copy link
Contributor

@Mergifyio backport 8.19 9.2 9.3

@mergify
Copy link
Contributor

mergify bot commented Jan 29, 2026

mergify bot pushed a commit that referenced this pull request Jan 29, 2026
* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)
mergify bot pushed a commit that referenced this pull request Jan 29, 2026
* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)
mergify bot pushed a commit that referenced this pull request Jan 29, 2026
* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)
mcapell added a commit that referenced this pull request Jan 29, 2026
#48599)

* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)

Co-authored-by: Marc Capell <marc.capell@elastic.co>
mcapell added a commit that referenced this pull request Jan 29, 2026
#48598)

* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)

Co-authored-by: Marc Capell <marc.capell@elastic.co>
mcapell added a commit that referenced this pull request Jan 29, 2026
#48597)

* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)

Co-authored-by: Marc Capell <marc.capell@elastic.co>
@pierrehilbert pierrehilbert added the backport-9.1 Automated backport to the 9.1 branch label Feb 2, 2026
mergify bot pushed a commit that referenced this pull request Feb 2, 2026
* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)

# Conflicts:
#	dev-tools/packaging/templates/docker/Dockerfile.tmpl
pierrehilbert added a commit that referenced this pull request Feb 4, 2026
…thetics monitor (#48641)

* fix(synthetics): chromium dependencies for Synthetics monitor (#48569)

* fix(synthetics): chromium dependencies for Synthetics monitor

Add missing dependency to run chromium on wolfi images.

(cherry picked from commit d282e8b)

# Conflicts:
#	dev-tools/packaging/templates/docker/Dockerfile.tmpl

* Update Node.js version in Dockerfile template

---------

Co-authored-by: Marc Capell <marc.capell@elastic.co>
Co-authored-by: Pierre HILBERT <pierre.hilbert@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-9.1 Automated backport to the 9.1 branch backport-active-all Automated backport with mergify to all the active branches Team:obs-ds-hosted-services Label for the Observability Hosted Services team

4 participants