Skip to content

[Synthetics] Revert base image rename from ubuntu to ubi#8452

Merged
emilioalvap merged 1 commit intoelastic:8.18from
emilioalvap:issue-8434-8.18
Jun 17, 2025
Merged

[Synthetics] Revert base image rename from ubuntu to ubi#8452
emilioalvap merged 1 commit intoelastic:8.18from
emilioalvap:issue-8434-8.18

Conversation

@emilioalvap
Copy link
Copy Markdown
Contributor

@emilioalvap emilioalvap commented Jun 11, 2025 •
edited
Loading

What does this PR do?

Reverts base image rename introduced in #8078.

Why is it important?

Checklist

  • I have read and understood the pull request guidelines of this project.
  • 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
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

How to test this PR locally

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

@emilioalvap emilioalvap requested a review from a team as a code owner June 11, 2025 13:24
@emilioalvap emilioalvap requested review from blakerouse and pkoutsovasilis and removed request for a team June 11, 2025 13:24
@elastic-sonarqube
Copy link
Copy Markdown

elastic-sonarqube bot commented Jun 11, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jun 11, 2025

💚 Build Succeeded

cc @emilioalvap

pkoutsovasilis
pkoutsovasilis approved these changes Jun 11, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@pkoutsovasilis pkoutsovasilis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok now I see what happened. The complete container variant for versions >= 9.0.0 is ubi based and unfortunately during the backport mentioned in the PR description this got applied here as well. LGTM thanks for addressing this one @emilioalvap

@emilioalvap emilioalvap mentioned this pull request Jun 16, 2025
Merged
8 tasks
@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Jun 17, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jun 17, 2025

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@emilioalvap emilioalvap merged commit 7917dc4 into elastic:8.18 Jun 17, 2025
12 of 14 checks passed
@pkoutsovasilis pkoutsovasilis mentioned this pull request Jun 18, 2025
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blakerouse blakerouse Awaiting requested review from blakerouse blakerouse is a code owner automatically assigned from elastic/elastic-agent-control-plane

1 more reviewer

@pkoutsovasilis pkoutsovasilis pkoutsovasilis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@emilioalvap emilioalvap

Labels

backport-skip skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@emilioalvap @elasticmachine @pkoutsovasilis @pierrehilbert