Change deprecated v1beta1 CRD to v1#1258
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
|
@googlebot I signed it! |
liyinan926
left a comment
There was a problem hiding this comment.
LGTM. Can we bump the chart verson?
|
@orfeas0 could you please bump up the chart version in this PR? Thanks! |
Bumped version, but not appVersion. Is this ok? |
|
Can you resolve the conflict? |
|
Looks like the chart integration test failed with the following message: |
|
I don't get the test fail, is it because I'm missing permissions? |
@orfeas0 @liyinan926 Seems you need to have a PR in kubernetes GitHub then once you have approval you add this annotation: I think this is only due to the fact the sparkoperator team used a protected internal URL spec for the CRD spec with: sparkoperator.k8s.io If it was just: sparkoperator.spark.io then you'd need no such annotation. So I suppose best idea is to submit a PR or an issue on kubernetes repo and ask them how to proceed. I'll be rooting for you because I need this, but I am not part of the project so not my place to make such a request externally. |
|
I think this is superceded by #1298 and can be closed now |
As per helm3's warning:
apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition