|
82 | 82 | Items marked with (R) are required *prior to targeting to a milestone / release*.
|
83 | 83 |
|
84 | 84 | - [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` |
86 | 86 | - [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) |
90 | 90 | - [ ] (R) Minimum Two Week Window for GA e2e tests to prove flake free
|
91 | 91 | - [ ] (R) Graduation criteria is in place
|
92 | 92 | - [ ] (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 |
94 | 94 | - [ ] (R) Production readiness review approved
|
95 | 95 | - [X] "Implementation History" section is up-to-date for milestone
|
96 | 96 | - [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
|
694 | 694 |
|
695 | 695 | * Basic unit tests for performance and quota system.
|
696 | 696 | * 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 |
701 | 698 | * Test coverage of quota usage with ResourceQuota and LimitRange
|
702 | 699 | * Measure latency impact to CreateVolume during beta and provide feedback to operators
|
703 | 700 | * Upgrade and rollback test when the feature gate changes to beta
|
@@ -843,7 +840,28 @@ A metric `controller_modify_volume_errors_total` will indicate a problem with th
|
843 | 840 |
|
844 | 841 | ###### Were upgrade and rollback tested? Was the upgrade->downgrade->upgrade path tested?
|
845 | 842 |
|
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. Change PVC back to VAC1, modify is applied. |
| 864 | + |
847 | 865 |
|
848 | 866 | ###### Is the rollout accompanied by any deprecations and/or removals of features, APIs, fields of API types, flags, etc.?
|
849 | 867 |
|
|
0 commit comments