Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Kubernetes v1.22+ compatibility #385

Open
Subetov opened this issue Dec 24, 2021 · 5 comments
Open

Kubernetes v1.22+ compatibility #385

Subetov opened this issue Dec 24, 2021 · 5 comments

Comments

@Subetov
Copy link

Subetov commented Dec 24, 2021

Feature Request

Is your feature request related to a problem? Please describe.
Helm charts compatibility with kubernetes v1.22+

Describe the solution you'd like to see
Need to fix manifest with API versions:

  • authorization.k8s.io/v1beta1
  • apiextensions.k8s.io/v1beta1
  • rbac.authorization.k8s.io/v1beta1

Additional context
https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22

@cscetbon
Copy link
Contributor

@Subetov there is a pending PR that upgrades the framework used for the operator. I suppose it'll solve this issue. @AKamyshnikova can you confirm ? If not, it'll be better to wait after #376 is approved and merged.

@AKamyshnikova
Copy link
Contributor

AKamyshnikova commented Dec 27, 2021

@cscetbon @Subetov Yes, upgrading operator-sdk unlocks upgrade/compatibility with kuberentes versions [1]. At this moment PR #376 updates client-go to v0.22.0 version [2]
[1] - https://sdk.operatorframework.io/docs/overview/#kubernetes-version-compatibility
[2] - https://github.com/kubernetes/client-go#compatibility-matrix

@kzzalews
Copy link

As #376 was merged and version 2.1.0 of CassKop operator was released two days ago, do we know which versions of K8s it supports right now? Am I right that we've just gained support of K8s 1.22? Is migration of old CassandraCluster objects necessary? For sure, I would like to test it, but getting such info from author @AKamyshnikova will be much appreciated :)

@AKamyshnikova
Copy link
Contributor

@kzzalews Sorry, in #376 I was not able to update client-go to v.0.22.0 in the end https://github.com/Orange-OpenSource/casskop/blob/master/go.mod#L103 - the was an issue with unit test.
So current issue is still actual and required separate pull request.
No additional steps are required for an update.

@cscetbon
Copy link
Contributor

Also fyi be careful that the release i did failed on releasing the helm chart. @fdehay is fixing it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants