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

Add related images to bundle generation #5519

Conversation

ryantking
Copy link
Contributor

Signed-off-by: Ryan King ryking@redhat.com

Description of the change:

During bundle generation (operator-sdk generate bundle or make bundle), operator-sdk will look through the environment of the controller manager for related images, named RELATED_IMAGE_<IMAGE_NAME>, and add them to the ClusterServiceVersion spec.relatedImages field.

Motivation for the change:

Checklist

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

Signed-off-by: Ryan King <ryking@redhat.com>
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

minor typo.

Copy link
Member

@theishshah theishshah left a comment

Choose a reason for hiding this comment

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

looks good, given nit & typo fix

Signed-off-by: Ryan King <ryking@redhat.com>
@ryantking ryantking changed the base branch from master to feature/image-digests January 27, 2022 16:02
Copy link
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2022
Signed-off-by: Ryan King <ryking@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2022
@jmrodri
Copy link
Member

jmrodri commented Jan 27, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2022
Signed-off-by: Ryan King <ryking@redhat.com>
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2022
@jmrodri
Copy link
Member

jmrodri commented Jan 27, 2022

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 27, 2022
@ryantking ryantking merged commit 5385577 into operator-framework:feature/image-digests Jan 27, 2022
@ryantking ryantking deleted the feature/related-images branch January 27, 2022 20:16
ryantking added a commit that referenced this pull request Feb 23, 2022
* add related images to bundle generation

Signed-off-by: Ryan King <ryking@redhat.com>

* regenerate and fix typos

Signed-off-by: Ryan King <ryking@redhat.com>

* no related images will be null, not an empty slice

Signed-off-by: Ryan King <ryking@redhat.com>

* regenerate to fix sanity

Signed-off-by: Ryan King <ryking@redhat.com>
jmrodri pushed a commit that referenced this pull request Feb 25, 2022
* add related images to bundle generation

Signed-off-by: Ryan King <ryking@redhat.com>

* regenerate and fix typos

Signed-off-by: Ryan King <ryking@redhat.com>

* no related images will be null, not an empty slice

Signed-off-by: Ryan King <ryking@redhat.com>

* regenerate to fix sanity

Signed-off-by: Ryan King <ryking@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants