Skip to content

cmd-remote-build-container: add --add-openshift-build-labels#4124

Merged
jlebon merged 1 commit into
coreos:mainfrom
jlebon:pr/add-more-ocp-labels
Jun 4, 2025
Merged

cmd-remote-build-container: add --add-openshift-build-labels#4124
jlebon merged 1 commit into
coreos:mainfrom
jlebon:pr/add-more-ocp-labels

Conversation

@jlebon

@jlebon jlebon commented Jun 2, 2025

Copy link
Copy Markdown
Member

Currently, oc looks for non-standard labels for git information, which it then places in e.g. image-references in the release payload.

Ideally oc would also check for the standard
org.opencontainers.image.{revision,source} labels too for this purpose but this assumption is wired in a bunch of places so let's just conform for now to it.

Add a new option to inject these labels. This can then be used when building the node image extensions image. In turn, this makes the release controller page for a release have working links for our images to the openshift/os commit they were built from, just like the other images.

Currently, `oc` looks for non-standard labels for git information, which
it then places in e.g. `image-references` in the release payload.

Ideally `oc` would also check for the standard
`org.opencontainers.image.{revision,source}` labels too for this purpose
but this assumption is wired in a bunch of places so let's just conform
for now to it.

Add a new option to inject these labels. This can then be used when
building the node image extensions image. In turn, this makes the
release controller page for a release have working links for our images
to the openshift/os commit they were built from, just like the other
images.
jlebon added a commit to jlebon/fedora-coreos-pipeline that referenced this pull request Jun 2, 2025
This will fix a small UX issue in the release controller: it will add
working links for our images to the openshift/os commit they were built
from, just like the other images.

See also coreos/coreos-assembler#4124 which this
requires.

@ravanelli ravanelli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@jlebon jlebon merged commit 41133ea into coreos:main Jun 4, 2025
5 checks passed
@jlebon jlebon deleted the pr/add-more-ocp-labels branch June 4, 2025 15:09
jlebon added a commit to coreos/fedora-coreos-pipeline that referenced this pull request Jun 5, 2025
This will fix a small UX issue in the release controller: it will add
working links for our images to the openshift/os commit they were built
from, just like the other images.

See also coreos/coreos-assembler#4124 which this
requires.
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.

2 participants