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

Change image tags with digests in bundle metadata #1246

Merged
merged 17 commits into from
Dec 2, 2024
Merged

Conversation

jankalinic
Copy link
Contributor

@jankalinic jankalinic commented Nov 26, 2024

This PR introduces the use of SHA digests for built images in operator environment variables. Instead of relying on image tags, which can be unreliable for snapshot and latest tagged images, the operator now explicitly references UI and API images using their SHA digests.
The API and UI images will now always pull the correct version, ensuring consistency and avoiding issues caused by tag mismatches.

This change was necessary because previously, using snapshot images with tags could result in the wrong image being pulled due to the console deployment's IfNotPresent image pull policy.

@jankalinic jankalinic marked this pull request as ready for review November 28, 2024 15:15
@jankalinic jankalinic requested a review from MikeEdgar November 28, 2024 15:15
@jankalinic jankalinic changed the title [WIP] Change tags with digests in bundle metadata Change image tags with digests in bundle metadata Nov 28, 2024
@jankalinic jankalinic added this to the 0.5.0 milestone Nov 28, 2024
@jankalinic jankalinic self-assigned this Nov 28, 2024
@jankalinic jankalinic added the enhancement New feature or request label Nov 28, 2024
Copy link
Member

@MikeEdgar MikeEdgar left a comment

Choose a reason for hiding this comment

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

Just one comment that I think needs an update for this PR. I also noticed an existing issue (not from this change) where the operator deployment and labels are named streamshub-console-operator-v, without the version. We can fix that in a separate PR.

operator/bin/modify-bundle-metadata.sh Show resolved Hide resolved
@MikeEdgar MikeEdgar force-pushed the instance-use-digests branch 3 times, most recently from d129872 to 27dd17a Compare December 2, 2024 22:29
jankalinic and others added 17 commits December 2, 2024 18:15
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
@MikeEdgar MikeEdgar force-pushed the instance-use-digests branch from 0986ab3 to 4a6abb0 Compare December 2, 2024 23:16
Copy link

sonarqubecloud bot commented Dec 2, 2024

@MikeEdgar MikeEdgar merged commit 484b73a into main Dec 2, 2024
9 checks passed
@MikeEdgar MikeEdgar deleted the instance-use-digests branch December 2, 2024 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants