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

docs: Image Digest Pinning #6870

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adambkaplan
Copy link

Description of the change:

Add an "advanced" article to describe how relatedImages and image digest pinning works. This includes the use of RELATED_IMAGE_* environment variables to digest pin operands deployed by the operator, which is not currently documented.

Motivation for the change:

I was very confused by the behavior of relatedImages and digest pinning. For example - when attempting to set relatedImages in the base CSV manifest, I discovered that operator-sdk generate bundle would always overwrite the relatedImages field in tree. It wasn't until I started iterating on a pull request (#6869) that I discovered there is an undocumented environment variable that lets operator authors define and wire in image digests for their operands.

Checklist

If the pull request includes user-facing changes, extra documentation is required:

Add an "advanced" article to describe how `relatedImages` and image digest
pinning works. This includes the use of `RELATED_IMAGE_*` environment variables
to digest pin operands deployed by the operator, which is not
currently documented.

Signed-off-by: Adam Kaplan <[email protected]>
@openshift-ci openshift-ci bot requested review from fabianvf and grokspawn November 20, 2024 17:31
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.

1 participant