-
Notifications
You must be signed in to change notification settings - Fork 103
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
Operator Install from URL #489
Conversation
huh?
|
f615d29
to
3b02c14
Compare
looking for #479 to land before working on this more... it will collide |
I was wondering if we could break this out into a separate interface where we treat each install target as a KUDO package repository. For example, we can implement the interface for the local directory that treats it as a repository. This would be useful for me in the test harness as I want to be able to easily install all operator versions from a repository, whether that repository is GCS or on disk or etc. |
962ff2b
to
a761358
Compare
This PR lgtm. While discussing it here we probably should use it as the foundation of multiple PRs that explore what has been discussed. In particular around ability to search for Operators ( see #332 ) or caching. |
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 see also my latest comment
cb4e724
to
7b345e7
Compare
// - a operator name in the remote repository | ||
// in that order. Should there exist a local folder e.g. `cassandra` it will take precedence | ||
// over the remote repository package with the same name. | ||
func getPackageCRDs(name string, options *Options, repository repo.Repository) (*repo.PackageCRDs, error) { | ||
func getPackageCRDs(name string, options *Options, repository repo.Repository) (*bundle.PackageCRDs, error) { |
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.
Could we make this public?
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.
I've been reducing all things to be private until needed to be public.. is there are reason to make it public?
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 is great, thanks!
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.
Definitely missing some tests here, otherwise looks good :)
mostly taken the day off... but I will add some tests and hope to merge today or tomorrow. thanks team! |
@fabianbaier I love the search options in #332 . If you are ok with it... I would like to grab that in another PR. |
What type of PR is this?
/component kudoctl
/kind enhancement
What this PR does / why we need it:
Adds the ability to install operator from a URL.
This enables install from:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: