-
Notifications
You must be signed in to change notification settings - Fork 80
Track Kubernetes Channels for latest versions #351
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
base: main
Are you sure you want to change the base?
Conversation
This look great! It would be great to mention this new feature to the README.md |
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 think you've got some great progress here!! Just a few things to consider.
|
||
if err := c.SetupWithManager(mgr); err != nil { | ||
kubeController := controller.NewKubeReconciler( |
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 was thinking, if it's worth checking ig opts.KubeChannel
is not empty, and not starting the kubernetes channel version Reconciler... Maybe that's the SetupWithManager's responsibility? wdyt?
This change introduces a kubernetes version checking ability, it uses the official k8s channels .txt file and defaults to "stable" for its release channel, this can be overriden with a cli flag
Things outstanding:
Stretch goals and more longer term idea's:
gke
detected within the version string