Skip to content
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

select version by priority rather than picking first #27

Merged
merged 3 commits into from
Jan 16, 2022

Conversation

clux
Copy link
Member

@clux clux commented Dec 27, 2021

uses master version of kube for this with the newly merged kube::core::Version add (will be avialable in 0.66.0)

only did the basics here of linking up the feature we did in kube-rs/kube#764

we still need some way of testing this - and haven't gotten a crd to test out multi version atm.

replaces #24 for #23 - but needs some way of testing it. @imp if you are free - some review / future help here would be great.

if not, i'll pick this up at some point over the new year - and enjoy the end of your year :-)

uses master version of kube for this with the newly merged
kube::core::Version add (will be avialable in 0.66.0)

Signed-off-by: clux <sszynrae@gmail.com>
@imp
Copy link
Collaborator

imp commented Jan 5, 2022

Looks good. Nice touch sorting all_versions - if there happened to be a lot of them - the highest priority shows up first.

clux added 2 commits January 16, 2022 15:33
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux linked an issue Jan 16, 2022 that may be closed by this pull request
@clux clux marked this pull request as ready for review January 16, 2022 16:01
@clux clux merged commit fe14b5b into main Jan 16, 2022
@clux clux deleted the version-priority-select branch January 16, 2022 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default version need to follow official Version Priority
2 participants