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

Upgrade VPA CRD version from apiextensions.k8s.io/v1beta1 to apiextensions.k8s.io/v1 #3508

Closed
Ladicle opened this issue Sep 14, 2020 · 3 comments
Labels
area/vertical-pod-autoscaler kind/feature Categorizes issue or PR as related to a new feature.

Comments

@Ladicle
Copy link
Contributor

Ladicle commented Sep 14, 2020

CRD apiextensions.k8s.io/v1beta1 has already been deprecated in v1.16. And now that 3 versions have passed, I'd like to migrate it to v1.

@bskiba
Copy link
Member

bskiba commented Oct 2, 2020

Sounds reasonable.

  1. Do you know if we need an upgrade story when we switch? i.e. what will happen with existing objects?
  2. Are you interested in contributing? :)

@bskiba bskiba added kind/feature Categorizes issue or PR as related to a new feature. area/vertical-pod-autoscaler labels Oct 2, 2020
@Ladicle
Copy link
Contributor Author

Ladicle commented Oct 5, 2020

  1. Do you know if we need an upgrade story when we switch? i.e. what will happen with existing objects?

CRD v1beta1 will be removed v1.22+, so we need to support v1 before that.
For upgrades, applying the rewritten v1 manifests does not affect an existing object. (at this time preseveUnknownFields is true.) This is because of k8s support API conversion. Of course, it's better to keep the v1beta1 manifest for backwards compatibility.

However, preseveUnknownFields became false by default at GA, so we need to set true or define an openAPIV3Scheme. Considering the future, the latter is recommended. In that case, if someone misses specifying the field, the value will be prune. I think it's better to generate openAPIV3Schema from types.go using controller-tools.

  1. Are you interested in contributing? :)

Yes! I'd like to contribute. :)

@bskiba
Copy link
Member

bskiba commented Oct 7, 2020

If you're interested, we welcome contributions :) Let me know if you need help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/vertical-pod-autoscaler kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants