-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
replace usage of v1beta1 VolumeAttachments with v1 #74606
replace usage of v1beta1 VolumeAttachments with v1 #74606
Conversation
4e17d2d
to
1501e90
Compare
/priority important-longterm |
/test pull-kubernetes-e2e-gce-alpha-features |
For |
I don't think we should remove the API versions, but is the test defaulting to v1beta? |
Ooh I see now, I'll update that, thanks! |
2f6e2ca
to
e061430
Compare
@@ -54,7 +54,7 @@ func NewStorageFactoryConfig() *StorageFactoryConfig { | |||
DefaultResourceEncoding: serverstorage.NewDefaultResourceEncodingConfig(legacyscheme.Scheme), | |||
ResourceEncodingOverrides: []schema.GroupVersionResource{ | |||
batch.Resource("cronjobs").WithVersion("v1beta1"), | |||
apisstorage.Resource("volumeattachments").WithVersion("v1beta1"), | |||
apisstorage.Resource("volumeattachments").WithVersion("v1"), |
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.
Curious if this is still needed
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.
v1 is the preferred version, you can drop this line completely
e061430
to
274c696
Compare
}, | ||
// -- | ||
|
||
// k8s.io/kubernetes/pkg/apis/storage/v1beta1 | ||
gvr("storage.k8s.io", "v1beta1", "volumeattachments"): { | ||
Stub: `{"metadata": {"name": "va2"}, "spec": {"attacher": "gce", "nodeName": "localhost", "source": {"persistentVolumeName": "pv2"}}}`, | ||
ExpectedEtcdPath: "/registry/volumeattachments/va2", | ||
ExpectedGVK: gvkP("storage.k8s.io", "v1", "VolumeAttachment"), |
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.
this is what verifies the data hitting etcd is in the version we expect, and this looks good
117f5f1
to
e0440f8
Compare
/milestone v1.14 |
/test pull-kubernetes-e2e-gce-alpha-features |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewsykim, liggitt, msau42 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 |
e0440f8
to
01933b0
Compare
PTAL again, had to resolve a conflict |
/lgtm |
/retest Review the full test history for this PR. Silence the bot with an |
cc @cheftako |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
As part of CSI post-GA clean up
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
/assign @msau42 @vladimirvivien