Skip to content

Commit 37fdc1a

Browse files
Update Release Signoff Checklist for KEP-3751 before GA
1 parent b11e3cb commit 37fdc1a

File tree

3 files changed

+37
-15
lines changed

3 files changed

+37
-15
lines changed

keps/prod-readiness/sig-storage/3751.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,5 @@ alpha:
66
approver: "@johnbelamaric"
77
beta:
88
approver: "@johnbelamaric"
9+
stable:
10+
approver: "@johnbelamaric"

keps/sig-storage/3751-volume-attributes-class/README.md

+30-10
Original file line numberDiff line numberDiff line change
@@ -82,15 +82,15 @@
8282
Items marked with (R) are required *prior to targeting to a milestone / release*.
8383

8484
- [X] (R) Enhancement issue in release milestone, which links to KEP dir in [kubernetes/enhancements] (not the initial KEP PR)
85-
- [ ] (R) KEP approvers have approved the KEP status as `implementable`
85+
- [X] (R) KEP approvers have approved the KEP status as `implementable`
8686
- [X] (R) Design details are appropriately documented
87-
- [ ] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input (including test refactors)
88-
- [ ] e2e Tests for all Beta API Operations (endpoints)
89-
- [ ] (R) Ensure GA e2e tests for meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
87+
- [X] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input (including test refactors)
88+
- [X] e2e Tests for all Beta API Operations (endpoints) - [dashboard](https://testgrid.k8s.io/sig-storage-kubernetes#kind-storage-alpha-beta-features&include-filter-by-regex=%5BFeature%3AVolumeAttributesClass%5D&include-filter-by-regex=%5BFeature%3AVolumeAttributesClass%5D&include-filter-by-regex=%5C%5BFeature%3AVolumeAttributesClass%5C%5D)
89+
- [X] (R) Ensure GA e2e tests for meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
9090
- [ ] (R) Minimum Two Week Window for GA e2e tests to prove flake free
9191
- [ ] (R) Graduation criteria is in place
9292
- [ ] (R) [all GA Endpoints](https://github.com/kubernetes/community/pull/1806) must be hit by [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
93-
- [ ] (R) Production readiness review completed
93+
- [X] (R) Production readiness review completed
9494
- [ ] (R) Production readiness review approved
9595
- [X] "Implementation History" section is up-to-date for milestone
9696
- [X] User-facing documentation has been created in [kubernetes/website], for publication to [kubernetes.io]
@@ -694,10 +694,7 @@ VolumeAttributesClass parameters can be considered as best-effort parameters, th
694694

695695
* Basic unit tests for performance and quota system.
696696
* API conformance tests
697-
* E2E tests with happy tests in the [K8s storage framework](https://github.com/kubernetes/kubernetes/tree/master/test/e2e/storage/testsuites) for different drivers testing
698-
* E2E tests using mock driver to cause failure on create, update and recovering cases
699-
* [K8s storage framework](https://github.com/kubernetes/kubernetes/tree/master/test/e2e/storage/testsuites)
700-
* [csi-tes](https://github.com/kubernetes-csi/csi-test)
697+
* E2E tests: https://github.com/kubernetes/kubernetes/blob/master/test/e2e/storage/volumeattributesclass.go
701698
* Test coverage of quota usage with ResourceQuota and LimitRange
702699
* Measure latency impact to CreateVolume during beta and provide feedback to operators
703700
* Upgrade and rollback test when the feature gate changes to beta
@@ -843,7 +840,30 @@ A metric `controller_modify_volume_errors_total` will indicate a problem with th
843840

844841
###### Were upgrade and rollback tested? Was the upgrade->downgrade->upgrade path tested?
845842

846-
TODO Upgrade and rollback will be tested when the feature gate will change to beta.
843+
Tested in Beta:
844+
845+
1. Enable both feature flag and beta API in api-server, create PVC with VAC1, and then modify to VAC2
846+
2. Turn off the feature flag first, and then try to modify PVC back to VAC1, got error:
847+
848+
```
849+
The PersistentVolumeClaim "test-pvc" is invalid: spec.volumeAttributesClassName: Forbidden: update
850+
is forbidden when the VolumeAttributesClass feature gate is disabled
851+
```
852+
853+
The pod and volume are both up and running.
854+
855+
3. Turn off the beta API, this time ``kubectl get vac`` got error:
856+
```
857+
Error from server (NotFound): Unable to list "storage.k8s.io/v1beta1, Resource=volumeattributesclasses":
858+
the server could not find the requested resource
859+
```
860+
861+
The pod and volume are both up and running.
862+
863+
4. Turn on both feature flag and beta API in api-server again. ``kubectl get vac`` shows the VACs again.
864+
865+
Change PVC back to VAC1, modify is applied.
866+
847867

848868
###### Is the rollout accompanied by any deprecations and/or removals of features, APIs, fields of API types, flags, etc.?
849869

keps/sig-storage/3751-volume-attributes-class/kep.yaml

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
title: Kubernetes Volume Provisioned IO
1+
title: Kubernetes VolumeAttributesClass and ModifyVolume
22
kep-number: 3751
33
authors:
44
- "@mattcarry"
55
- "@sunnylovestiramisu"
66
owning-sig: sig-storage
77
participating-sigs:
88
status: implementable
9-
creation-date: 2023-01-24
9+
creation-date: 2024-01-22
1010
reviewers:
1111
- "@msau42"
1212
- "@gnufied"
@@ -18,18 +18,18 @@ see-also:
1818
replaces:
1919

2020
# The target maturity stage in the current dev cycle for this KEP.
21-
stage: beta
21+
stage: stable
2222

2323
# The most recent milestone for which work toward delivery of this KEP has been
2424
# done. This can be the current (upcoming) milestone, if it is being actively
2525
# worked on.
26-
latest-milestone: "1.31"
26+
latest-milestone: "1.33"
2727

2828
# The milestone at which this feature was, or is targeted to be, at each stage.
2929
milestone:
3030
alpha: "v1.29"
3131
beta: "v1.31"
32-
stable:
32+
stable: "v1.33"
3333

3434
# The following PRR answers are required at alpha release
3535
# List the feature gate name and the components for which it must be enabled

0 commit comments

Comments
 (0)