Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ironbank: rename ubi9 to ubi image #37153

Merged
merged 2 commits into from
Nov 27, 2023
Merged

ironbank: rename ubi9 to ubi image #37153

merged 2 commits into from
Nov 27, 2023

Conversation

v1v
Copy link
Member

@v1v v1v commented Nov 20, 2023 •
edited
Loading

elastic/elastic-agent#3790 will need to be merged when this one is merged to help with the changes in the DRA (elastic/infra)

Proposed commit message

Rename docker images from ubi9 to ubi from 8.12 onwards and 7.17.16

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
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

@v1v v1v added the backport-7.17 Automated backport to the 7.17 branch with mergify label Nov 20, 2023
@v1v v1v requested a review from a team as a code owner November 20, 2023 14:35
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label Team:Automation Label for the Observability productivity team labels Nov 20, 2023
@v1v v1v self-assigned this Nov 20, 2023
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Nov 20, 2023
@v1v v1v marked this pull request as draft November 20, 2023 14:35
@v1v v1v requested review from a team November 20, 2023 14:35
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Duration: 181 min 15 sec

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify Mergify
Copy link
Contributor

mergify bot commented Nov 21, 2023

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b feature/replace-ubi9-and-use-ubi upstream/feature/replace-ubi9-and-use-ubi
git merge upstream/main
git push upstream feature/replace-ubi9-and-use-ubi

@v1v
Copy link
Member Author

v1v commented Nov 21, 2023

/package

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Duration: 174 min 49 sec

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@v1v v1v marked this pull request as ready for review November 21, 2023 12:02
@v1v v1v mentioned this pull request Nov 21, 2023
Merged
7 tasks
@v1v
Copy link
Member Author

v1v commented Nov 21, 2023

elastic/elastic-agent#3790 should be merged at the same time

@v1v v1v merged commit 296183c into elastic:main Nov 27, 2023
6 checks passed
mergify bot pushed a commit that referenced this pull request Nov 27, 2023
@v1v @mergify
ironbank: rename ubi9 to ubi image (#37153)
835eb28 
(cherry picked from commit 296183c)

# Conflicts:
#	.ci/packaging.groovy
#	Jenkinsfile
@mergify mergify bot mentioned this pull request Nov 27, 2023
Merged
cmacknz pushed a commit to cmacknz/beats that referenced this pull request Jan 10, 2024
@mergify
[7.17](backport elastic#37153) ironbank: rename ubi9 to ubi image (el…
cc0fbbe 
…astic#37205)
Scholar-Li pushed a commit to Scholar-Li/beats that referenced this pull request Feb 5, 2024
@v1v @Scholar-Li
ironbank: rename ubi9 to ubi image (elastic#37153)
c5fc5ef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amannocci amannocci amannocci approved these changes

@kuisathaverat kuisathaverat kuisathaverat approved these changes

@pierrehilbert pierrehilbert pierrehilbert approved these changes

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

@fearful-symmetry fearful-symmetry Awaiting requested review from fearful-symmetry fearful-symmetry is a code owner automatically assigned from elastic/elastic-agent-data-plane

Assignees

@v1v v1v

Labels
backport-7.17 Automated backport to the 7.17 branch with mergify Team:Automation Label for the Observability productivity team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@v1v @elasticmachine @pierrehilbert @kuisathaverat @amannocci