Skip to content
This repository was archived by the owner on Nov 22, 2024. It is now read-only.

Drop v1beta1 crds for k8s 122 compliance#1212

Merged
franciscolopezsancho merged 9 commits into
lightbend:mainfrom
thomasschoeftner:drop-v1beta1-crds-for-k8s-122-compliance
Aug 8, 2022
Merged

Drop v1beta1 crds for k8s 122 compliance#1212
franciscolopezsancho merged 9 commits into
lightbend:mainfrom
thomasschoeftner:drop-v1beta1-crds-for-k8s-122-compliance

Conversation

@thomasschoeftner

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

I've tried to get rid of v1beta1 CRD in the Cloudflow code base.

Why are the changes needed?

With Kubernetes 1.22 support for v1beta1 CRDs is removed and Cloudflow will not be deployable anymore.
The issue had already been raised by another user: #1199

Does this PR introduce any user-facing change?

no

How was this patch tested?

I do not have an appropriate environment at the moment to verify the change.

Comment thread core/cloudflow-crd/src/main/scala/akka/datap/crd/App.scala
Comment thread core/cloudflow-crd/src/main/scala/akka/datap/crd/App.scala
Comment thread core/cloudflow-operator/src/main/scala/cloudflow/operator/Main.scala Outdated
@thomasschoeftner

Copy link
Copy Markdown
Contributor Author

@RayRoestenburg - any chance you could take a look, pls.

@franciscolopezsancho

Copy link
Copy Markdown
Contributor

I'll have a look at it @thomasschoeftner

@thomasschoeftner

thomasschoeftner commented Jul 5, 2022

Copy link
Copy Markdown
Contributor Author

I'll have a look at it @thomasschoeftner

Thank you, @franciscolopezsancho ! - I'm not sure whether the changes in core/cloudflow-crd/src/main/scala/akka/datap/crd/App.scala are too excessive, but IMO they make sense...

@thomasschoeftner thomasschoeftner marked this pull request as ready for review July 5, 2022 15:37
@thomasschoeftner

Copy link
Copy Markdown
Contributor Author

@franciscolopezsancho Do you have any updates on this?

Comment thread core/cloudflow-crd/src/main/scala/akka/datap/crd/App.scala

@franciscolopezsancho franciscolopezsancho left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@franciscolopezsancho franciscolopezsancho merged commit 6187fbf into lightbend:main Aug 8, 2022
@thomasschoeftner

thomasschoeftner commented Aug 9, 2022

Copy link
Copy Markdown
Contributor Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants