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

Drop v1beta1 crds for k8s 122 compliance #1212

Conversation

thomasschoeftner
Copy link
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.

@thomasschoeftner
Copy link
Contributor Author

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

@franciscolopezsancho
Copy link
Collaborator

I'll have a look at it @thomasschoeftner

@thomasschoeftner
Copy link
Contributor Author

thomasschoeftner commented Jul 5, 2022

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
Contributor Author

@franciscolopezsancho Do you have any updates on this?

Copy link
Collaborator

@franciscolopezsancho franciscolopezsancho left a comment

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
Copy link
Contributor Author

thomasschoeftner commented Aug 9, 2022

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