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.

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

dra: rename ubi8 for ubi9 #11990

Merged
merged 1 commit into from
Nov 2, 2023
Merged

dra: rename ubi8 for ubi9 #11990

merged 1 commit into from
Nov 2, 2023

Conversation

v1v
Copy link
Member

@v1v v1v commented Nov 2, 2023

Motivation/summary

APM Server in 7.17 uses the packaging system from elastic/beats but elastic/beats has not been updated to use ubi9 yet.

This is the workaround until the upstream project moves to ubi9

Checklist

For functional changes, consider:

  • Is it observable through the addition of either logging or metrics?
  • Is its use being published in telemetry to enable product improvement?
  • Have system tests been added to avoid regression?

How to test these changes

Create some files and run the script locally

$ mkdir -p build/distributions
$ touch "build/distributions/apm-server-7.17.15-aarch64.rpm"
touch "build/distributions/apm-server-ubi8-7.17.15-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-oss-7.17.15-linux-arm64.docker.tar.gz"
touch "build/distributions/apm-server-oss-7.17.15-SNAPSHOT-linux-arm64.docker.tar.gz"
touch "build/distributions/apm-server-7.17.15-linux-arm64.docker.tar.gz"
touch "build/distributions/apm-server-7.17.15-SNAPSHOT-linux-arm64.docker.tar.gz"
touch "build/distributions/apm-server-oss-7.17.15-SNAPSHOT-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-oss-7.17.15-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-7.17.15-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-7.17.15-SNAPSHOT-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-ubi8-7.17.15-linux-arm64.docker.tar.gz"
touch "build/distributions/apm-server-ubi8-7.17.15-SNAPSHOT-linux-amd64.docker.tar.gz"
touch "build/distributions/apm-server-ubi8-7.17.15-SNAPSHOT-linux-arm64.docker.tar.gz"

for file in apm-server-ubi8* ; do  mv $file ${file//ubi8/ubi9} ; done
image

I triggered this build manually

Related issues

@v1v v1v requested review from a team November 2, 2023 08:28
@v1v v1v merged commit 68aaf6f into 7.17 Nov 2, 2023
8 checks passed
@v1v v1v deleted the feature/rename-ubi-9 branch November 2, 2023 08:53
v1v added a commit to v1v/apm-server that referenced this pull request Nov 2, 2023
@v1v v1v mentioned this pull request Nov 2, 2023
3 tasks
v1v added a commit that referenced this pull request Nov 2, 2023
* Revert "dra: rename ubi8 for ubi9 (#11990)"
* Revert "fix: correct base Docker image names (#11926) (#11928)"
* Revert "[7.17] ironbank: ubi9:9.2 (backport #11785) (#11813)"
v1v added a commit to v1v/apm-server that referenced this pull request Nov 3, 2023
kruskall pushed a commit that referenced this pull request Nov 10, 2023
* Revert "ironbank: remove support for ubi9 (#11991)"

This reverts commit abd2f78.

* Revert "dra: rename ubi8 for ubi9 (#11990)"

This reverts commit 68aaf6f.

* update dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants