-
Notifications
You must be signed in to change notification settings - Fork 54
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
Updating kubernetes-client version #184
base: master
Are you sure you want to change the base?
Conversation
@@ -38,6 +38,7 @@ | |||
|
|||
import com.github.zafarkhaja.semver.Version; | |||
|
|||
import io.fabric8.kubernetes.api.model.*; |
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.
Better not replace classes with pounds...
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.
Fixed, thanks!
The main problem is the CRD handling. The 3.x client libraries need hacks that may not work with 4.x. Away from computer right now but will take a look after my day job. |
BTW I use compiled jar file in my project and it works fine ;) |
Any update ? We need to upgrade the kubernetes client to 4.x version but the helm does not support it. |
Not ignoring you folks; just need to get to this on my open source development schedule (read: nights and weekends!). |
Working on this now. Probably won't pull in this PR exactly but I will incorporate its contents and credit you—and I thank you greatly for your work. |
…g spirit of #195. Signed-off-by: Laird Nelson <[email protected]>
@d-kononov @fidget83 I have deployed |
hi @ljnelson , @fidget83 , @d-kononov , |
No description provided.