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
Install InstallPlan #102
Install InstallPlan #102
Conversation
operators/catalog/operator.go
Outdated
continue | ||
} | ||
|
||
case "ClusterServiceVersion-v1": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you use the constants for these from the types.go?
operators/catalog/operator.go
Outdated
} | ||
|
||
default: | ||
panic("attempted to install an unknown resource Kind") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will just kill the operator right? we don't want that?
operators/catalog/operator.go
Outdated
} | ||
|
||
default: | ||
panic("attempted to install step in a plan with an undefined status") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
followup for tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
followup for tests
…allplan Install InstallPlan
No description provided.