-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
CRD versioning design proposal [No Conversion] #2054
CRD versioning design proposal [No Conversion] #2054
Conversation
c19312a
to
c256d36
Compare
|
no other comments from me |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* `Spec.Version` field exists in `Spec.Versions` field. | ||
* The version defined in `Spec.Version` field should point to a `Served` Version in `Spec.Versions` list except when we do not serve any version (i.e. all versions in `Spec.Versions` field are disabled by `Served` set to `False`). This is for backward compatibility. An old controller expect that version to be served but only the whole CRD is served. CRD Registration controller should unregister a CRD with no serving version. | ||
* None of the `Status.StoredVersion` can be removed from `Spec.Versions` list. | ||
* Only one of the versions in `spec.Versions` can flag as `Storage` version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly one...
Lgtm. |
CRD versioning design proposal [No Conversion]
CRD versioning proposal is discussed in detail here:
https://docs.google.com/document/d/1Ucf7JwyHpy7QlgHIN2Rst_q6yT0eeN9euzUV6kte6aY/edit
This is the main no conversion part of the design agreed on by the community/API machinery SIG.
@sttts @deads2k @lavalamp