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

latest image tags used in the release code #262

Open
tumberino opened this issue Sep 29, 2023 · 2 comments
Open

latest image tags used in the release code #262

tumberino opened this issue Sep 29, 2023 · 2 comments

Comments

@tumberino
Copy link

When using a tagged version of the code the image tag specified for the manager is always latest, but the payload images are tagged to specific digests. Therefore if I try to use version v0.7.0 it doesn't work because of the changes in the way that the runtimeclass gets created in recent updates.

https://github.com/confidential-containers/operator/blob/v0.7.0/config/manager/kustomization.yaml#L13-L16

images:
- name: controller
  newName: quay.io/confidential-containers/operator
  newTag: latest

It would be benefical it the newTag was v0.7.0 in this case. But I understand that that is not a simple fix as the tag gets created as part of the release process.

@stevenhorsman
Copy link
Member

Yeah, we have a bit of a cycle here - IIUC the release image of the operator is created by the release process after the tag is done, so if we updated the kustomize to point to this tag before this is cut then I assume that all the tests will fail. I agree that pointing to latest isn't ideal though.

/cc @bpradipt @jensfr @fidencio who might have thoughts/ideas on this

@wainersm
Copy link
Member

On peer pods we have a similar issue. There we point to the tag (commit SHA) of the latest built cloud-api-adaptor image before the commit that bump the version. It doesn't point to the version tagged image but at least it will be pinned rather than following latest.

To implement that approach on the operator, we would need to change https://github.com/confidential-containers/operator/blob/main/.github/workflows/docker-publish-latest-on-merge.yml to tag the image with the commit SHA; currently that workflow only publishes to latest.

@fitzthum you might be interested on this issue...

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

No branches or pull requests

3 participants