Skip to content

(bugfix): update operator-sdk run bundle(-upgrade) to ignore channels in the bundle and use a default one#6042

Merged
everettraven merged 9 commits into
operator-framework:masterfrom
everettraven:bugfix/rbu-channel
Sep 28, 2022
Merged

(bugfix): update operator-sdk run bundle(-upgrade) to ignore channels in the bundle and use a default one#6042
everettraven merged 9 commits into
operator-framework:masterfrom
everettraven:bugfix/rbu-channel

Conversation

@everettraven

Copy link
Copy Markdown
Contributor

Description of the change:

  • Makes operator-sdk run bundle(-upgrade) ignore channel information from a bundle and instead either use a default value of operator-sdk-run-bundle or the one specified in an existing Subscription CR.

Motivation for the change:

Checklist

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

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:32 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:32 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:33 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:34 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:35 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:36 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 17:37 Inactive
@jmrodri

jmrodri commented Sep 26, 2022

Copy link
Copy Markdown
Member

Retitle the bug to be something meaningful

@everettraven everettraven changed the title Bugfix/rbu channel (bugfix): update operator-sdk run bundle(-upgrade) to ignore channels in the bundle and use a default one Sep 26, 2022
@everettraven

Copy link
Copy Markdown
Contributor Author

Retitle the bug to be something meaningful

Updated. meant to do that before opening the PR but must've forgot :)

@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive
@everettraven everettraven temporarily deployed to deploy September 26, 2022 19:11 Inactive

@jmrodri jmrodri left a comment

Copy link
Copy Markdown
Member

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 Sep 28, 2022
@everettraven everettraven merged commit b97d84f into operator-framework:master Sep 28, 2022
@everettraven everettraven deleted the bugfix/rbu-channel branch September 28, 2022 19:41
dacamposol pushed a commit to dacamposol/operator-sdk that referenced this pull request Oct 6, 2022
…ls in the bundle and use a default one (operator-framework#6042)

* fix run bundle(-upgrade) to no longer stall

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* add changelog

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* add err check and log warning for linter

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* update run bundle(-upgrade) to use a default channel

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* update channel name checks

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* add changelog

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

* update changelog

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
Signed-off-by: Daniel Campos Olivares <daniel.campos.olivares@sap.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.

Run bundle and upgrade bundle does not work when the bundles is not added to default channel

2 participants