From 039705f0607d33be574f6b53bb66ced21e38faa1 Mon Sep 17 00:00:00 2001 From: Oksana Naumov Date: Mon, 6 Jun 2022 18:36:48 -0700 Subject: [PATCH] Promote Portworx CSI migration to Beta in v1.25 --- .../2589-csi-migration-portworx/README.md | 35 ++++++++++++++++++- .../2589-csi-migration-portworx/kep.yaml | 4 +-- 2 files changed, 36 insertions(+), 3 deletions(-) diff --git a/keps/sig-storage/2589-csi-migration-portworx/README.md b/keps/sig-storage/2589-csi-migration-portworx/README.md index a92d3f8fadf..d6ee15ce0ba 100644 --- a/keps/sig-storage/2589-csi-migration-portworx/README.md +++ b/keps/sig-storage/2589-csi-migration-portworx/README.md @@ -7,6 +7,12 @@ - [New Feature Gates](#new-feature-gates) - [Production Readiness Review Questionnaire](#production-readiness-review-questionnaire) - [Implementation History](#implementation-history) +- [Design details](#design-details) + - [Test Plan](#test-plan) + - [Prerequisite testing updates](#prerequisite-testing-updates) + - [Unit tests](#unit-tests) + - [Integration tests](#integration-tests) + - [e2e tests](#e2e-tests) @@ -50,6 +56,33 @@ Major milestones for Portworx in-tree plugin CSI migration: - 1.23 - Portworx CSI migration to Alpha -- 1.24 +- 1.25 - Portworx CSI migration to Beta, off by default +## Design details + +### Test Plan + + I/we understand the owners of the involved components may require updates to +existing tests to make this code solid enough prior to committing the changes necessary +to implement this enhancement. + +##### Prerequisite testing updates + +No additional tests are needed, rather the issue is orchestrating CSI driver +deployment for prow jobs. This has been complicated by the storage provider +extraction work, which no longer permits storage provider specific orchestration +in the k/k repository. This means that it is not possible to run any test for +portworx-volume in k/k. + +##### Unit tests + +See tests in the [https://github.com/kubernetes/csi-translation-lib/blob/master/plugins/portworx_test.go](https://github.com/kubernetes/csi-translation-lib/blob/master/plugins/portworx_test.go) + +##### Integration tests + +N/A + +##### e2e tests + +To ensure the implementation correctness, I/we have manually run the e2e tests, [located in the main k8s repository](https://github.com/kubernetes/kubernetes/blob/master/test/e2e/storage/drivers/in_tree.go). Test results are attached to the pull requests diff --git a/keps/sig-storage/2589-csi-migration-portworx/kep.yaml b/keps/sig-storage/2589-csi-migration-portworx/kep.yaml index ed7ed1d1cfb..cbdd904a8f9 100644 --- a/keps/sig-storage/2589-csi-migration-portworx/kep.yaml +++ b/keps/sig-storage/2589-csi-migration-portworx/kep.yaml @@ -27,12 +27,12 @@ stage: beta # The most recent milestone for which work toward delivery of this KEP has been # done. This can be the current (upcoming) milestone, if it is being actively # worked on. -latest-milestone: "v1.24" +latest-milestone: "v1.25" # The milestone at which this feature was, or is targeted to be, at each stage. milestone: alpha: "v1.23" - beta: "v1.24" + beta: "v1.25" # The following PRR answers are required at alpha release # List the feature gate name and the components for which it must be enabled