Skip to content

Conversation

@rashmigottipati
Copy link
Member

Description of the change:
If subscription doesn't match any existing package name, then return error instead of panic.

Motivation for the change:
Improve error handling for run bundle-upgrade command

Checklist

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

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-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 29, 2021
@rashmigottipati rashmigottipati merged commit fbce7e7 into operator-framework:master Jan 29, 2021
@rashmigottipati
Copy link
Member Author

/cherry-pick v1.3.x

@openshift-cherrypick-robot

@rashmigottipati: new pull request created: #4454

Details

In response to this:

/cherry-pick v1.3.x

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 4, 2021
…ackage name (operator-framework#4452)

Signed-off-by: rashmigottipati <[email protected]>
Signed-off-by: Rein van 't Veer <[email protected]>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 4, 2021
…ackage name (operator-framework#4452)

Signed-off-by: rashmigottipati <[email protected]>
Signed-off-by: Rein van 't Veer <[email protected]>
reinvantveer pushed a commit to reinvantveer/operator-sdk that referenced this pull request Feb 5, 2021
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.

4 participants