-
Notifications
You must be signed in to change notification settings - Fork 531
Fix ct image releasing #11660
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
Fix ct image releasing #11660
Conversation
…out existing images #11659
pdurbin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it should do the trick! Thanks, @poikilotherm!
|
As discussed at https://dataverse.zulipchat.com/#narrow/channel/375812-containers/topic/6.2E7.20release/near/529904650 @poikilotherm is thinking about using this PR to close multiple issues:
So I'm pulling it out of "ready for QA" to give him time. Thanks, Oliver! ❤️ |
…1661 Adjusted tag parameter names for both `app` and `configbaker` images to follow a consistent schema (`.tag.1`, `.tag.2`). Updated `pom.xml` to define and include additional tags under new parameters. See also: fabric8io/docker-maven-plugin#1865
…ce scripts We cannot use the Maven property external configuration bits (which is enabled implicitly by default), as we are using `<external>` with a Compose file. So using this option has no effect on the build! (Same as we cannot add tags this way...)
Updated Dockerfile to include the BASE_IMAGE argument. The ARG value is not available anymore _after_ the FROM statement, so we need to repeat it for the workaround to work in full.
…11661 Updated `pom.xml` across backports to include new parameters (`.tag.1`, `.tag.2`) for both `app` and `configbaker` images. Configured additional tags for usage in container maintenance builds and external Compose workflows. Backports target the last three releases, v6.6, v6.7, and v6.7.1.
|
@pdurbin this should be done. Tested it locally. Please try it yourself as well. Thx! |
pdurbin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This all looks reasonable to me. Approved. Thanks, @poikilotherm! ❤️
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
|
@pdurbin @ofahimIQSS we shall run the maintenance action once merged. Thx! |
This comment has been minimized.
This comment has been minimized.
1 similar comment
|
📦 Pushed preview images as 🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name. |
|
As per discussion on Zulip, I am going ahead and push the button on this one. Not including it in 6.8 means adding another backport and no proper images at release time. If things go sideways, blame it on the remote guy.
|

What this PR does / why we need it:
Add another build trigger for base images in case there is no "r0" tag on the hub, the base image needs no updates, but we're maintaining a release.
Which issue(s) this PR closes:
Special notes for your reviewer:
None
Suggestions on how to test this:
Copy the added bits into a separate test script at
.github/workflows/scripts/containers/test.sh:Running the script should print:
Changing
IS_DEV=1or changing theBASE_IMAGE_REFto point to e.g.6.6-nobleshould printis_new_release=0.Also, please run the maintenance script locally and in DRY_RUN mode and watch for the tag options to be added correctly. You should also run a DAMP_RUN and see if the build log contains the additional tags (the docker build statement should carry them!)
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Nope.
Is there a release notes update needed for this change?:
Nope.
Additional documentation:
None.