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

fix(kube-client): upgrade fabric8io/kubernetes-client to 4.11.1 #35

Closed

Conversation

patrickleet
Copy link
Contributor

#34 is a kubernetes client error, so maybe just updating to the latest kubernetes client version is all that's needed to fix it.

@patrickleet
Copy link
Contributor Author

it'd probably be good to set up renovate, that way updates like these would have PRs created automatically - the pipelines seem robust enough that it'd be a good addition

@patrickleet patrickleet force-pushed the feat/upgrade-kube-client branch 2 times, most recently from 40f359c to 6b34b7a Compare September 10, 2020 19:05
@patrickleet
Copy link
Contributor Author

well, as you can see, that did not work

all the examples I can find are still using withVersion, so idk what's up with that

@monotek monotek assigned monotek and rpahli and unassigned monotek Sep 10, 2020
fix(kube-client): https://github.com/fabric8io/kubernetes-client/releases/tag/v4.11.0 CustomResourceDefinition class has moved

fix(kube-client): https://github.com/fabric8io/kubernetes-client/releases/tag/v4.11.0 CustomResourceDefinition class has moved

fix(kube-client): version is in package - withVersion function is gone

fix(kube-client): version is in package - getVersion function is gone

fix(crds): use crd v1

fix(crds): kiwigrid is still v1beta1

fix(crds): withApiVersion

fix(crds): revert to withVersion

fix(crds): revert to withVersion

fix(crds): order?

fix(crds): follow example from https://github.com/fabric8io/kubernetes-client/blob/master/doc/CHEATSHEET.md\#customresource-typeless-api in RealmResource

fix(crds): metadata name

fix(crds): try v1beta1 ???

fix(crds): more v1beta1s

fix(crds): revert changes
@patrickleet patrickleet force-pushed the feat/upgrade-kube-client branch from 8095317 to be9d28a Compare September 15, 2020 17:12
@patrickleet
Copy link
Contributor Author

got mvn build working, but now getting errors from test keycloak-controller

Also, I noticed the examples all rely on a realm 'food-realm' that isn't actually created?

@patrickleet
Copy link
Contributor Author

in working on this it became pretty clear that the tests need improvement - doing that in #37

will come back to these upgrades once we have better tests

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.

3 participants