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
instancetype: Label ControllerRevisions with stored object metadata #9932
instancetype: Label ControllerRevisions with stored object metadata #9932
Conversation
Skipping CI for Draft Pull Request. |
a7fb79f
to
c8117ce
Compare
f79dd12
to
ae8abbd
Compare
/retest-required |
/cc @0xFelix |
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.
Thanks!
Looks good to me. But this only helps converting newly created ControllerRevisions, right? We still need to figure out how to convert older existing ones?
I also like that his allows for deduplication of ControllerRevisions!
I assume functests are not needed at this point?
/lgtm
Yes I think any conversion is going to assume that these labels aren't present to be begin with.
Yeah hopefully allows us to look things up quickly without relying on the names etc.
I didn't think so at this point but happy to add something if required. I'm still not sure about the names tbh, I think these could easily conflict in their present form so I might make them a little more specific to the ControllerRevisions this morning. |
ControllerRevisions are used to store point in time copies of instance type and preference objects for each VirtualMachine making use of the feature, see the following user-guide docs for more on this: Instance types and preferences - Versioning https://kubevirt.io/user-guide/virtual_machines/instancetypes/#versioning This change adds labels to these ControllerRevision, exposing metadata of the stored objects. With this in place it becomes easier to track the objects stored within these ControllerRevisions and in the future orchestrate the migration of these objects to newer versions of the API and CRDs as these are introduced. Signed-off-by: Lee Yarwood <lyarwood@redhat.com>
ae8abbd
to
e0ee835
Compare
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.
/lgtm
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: xpivarc 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 |
/retest-required |
/cherry-pick release-1.0 |
@lyarwood: once the present PR merges, I will cherry-pick it on top of release-1.0 in a new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@lyarwood: new pull request created: #10153 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/area instancetype
What this PR does / why we need it:
ControllerRevisions are used to store point in time copies of instance
type and preference objects for each VirtualMachine making use of the
feature, see the following user-guide docs for more on this:
Instance types and preferences - Versioning
https://kubevirt.io/user-guide/virtual_machines/instancetypes/#versioning
This change adds labels to these ControllerRevision, exposing metadata
of the stored objects. With this in place it becomes easier to track the
objects stored within these ControllerRevisions and in the future
orchestrate the migration of these objects to newer versions of the API
and CRDs as these are introduced.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #9925
Special notes for your reviewer:
Release note: