Skip to content

Conversation

@helderjs
Copy link
Collaborator

@helderjs helderjs commented Jan 3, 2023

All Submissions:

  • Have you signed our CLA?

Comment on lines 87 to 89
value="${IMG_SHA}" yq e -i '.spec.install.spec.deployments[0].spec.template.spec.containers[0].image = env(value)' \
"${VERSION}"/manifests/mongodb-atlas-kubernetes.clusterserviceversion.yaml

Copy link
Collaborator

Choose a reason for hiding this comment

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

There is one more place to replace the image tag with image SHA: .metadata.annotations.containerImage

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, for certified operators please make sure that images have quay.io/ prefix

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!
Also, as we now support multi-arch, we don't use the manifest digest(sha256) as the version anymore.
We just refer to the actual version of the release

Copy link
Contributor

@Sugar-pack Sugar-pack left a comment

Choose a reason for hiding this comment

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

LGTM

@helderjs helderjs merged commit 3f2a47a into main Jan 13, 2023
@helderjs helderjs deleted the CLOUDP-143633-automate-opnshift-release branch January 13, 2023 18:22
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.

4 participants