Skip to content

Commit ac2ede1

Browse files
authored
Merge pull request kubernetes#2179 from wojtek-t/immutable_secrets_ga
Promote Immutable Secrets/ConfigMaps feature to GA
2 parents bfe36aa + ad350ab commit ac2ede1

File tree

3 files changed

+36
-16
lines changed

3 files changed

+36
-16
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
kep-number: 1412
2+
stable:
3+
approver: "@johnbelamaric"

keps/sig-storage/1412-immutable-secrets-and-configmaps/README.md

Lines changed: 26 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,16 @@ For enhancements that make changes to code or processes/procedures in core Kuber
3737

3838
Check these off as they are completed for the Release Team to track. These checklist items _must_ be updated for the enhancement to be released.
3939

40-
- [ ] kubernetes/enhancements issue in release milestone, which links to KEP (this should be a link to the KEP location in kubernetes/enhancements, not the initial KEP PR)
41-
- [ ] KEP approvers have set the KEP status to `implementable`
42-
- [ ] Design details are appropriately documented
43-
- [ ] Test plan is in place, giving consideration to SIG Architecture and SIG Testing input
44-
- [ ] Graduation criteria is in place
45-
- [ ] "Implementation History" section is up-to-date for milestone
40+
- [x] (R) Enhancement issue in release milestone, which links to KEP dir in [kubernetes/enhancements] (not the initial KEP PR)
41+
- [x] (R) KEP approvers have approved the KEP status as `implementable`
42+
- [x] (R) Design details are appropriately documented
43+
- [x] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input
44+
- [x] (R) Graduation criteria is in place
45+
- [x] (R) Production readiness review completed
46+
- [x] Production readiness review approved
47+
- [x] "Implementation History" section is up-to-date for milestone
4648
- [ ] User-facing documentation has been created in [kubernetes/website], for publication to [kubernetes.io]
47-
- [ ] Supporting documentation e.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
49+
- [x] Supporting documentatione.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
4850

4951
**Note:** Any PRs to move a KEP to `implementable` or significant changes once it is marked `implementable` should be approved by each of the KEP approvers. If any of those approvers is no longer appropriate than changes to that list should be approved by the remaining approvers and/or the owning SIG (or SIG-arch for cross cutting KEPs).
5052

@@ -145,17 +147,21 @@ the new logic behind the feature gate.
145147

146148
### Test Plan
147149

148-
For `Alpha`, we will add e2e tests verifying that contents of Secrets and
149-
ConfigMaps marked as immutable really can't be updated. Additionally, these
150-
will check if the metadata can be modified.
150+
For `Alpha`, e2e tests verifying that:
151+
- data of immutable Secrets and ConfigMaps can't be updated
152+
- metadata of immutable Secrets and ConfigMaps can be update
153+
has been added.
151154

152-
Additionally, unit tests will be added in Kubelet codebase to ensure that
155+
Additionally, necessary unit tests were added in Kubelet codebase to ensure that
153156
the newly added logic to not watch immutable Secrets/ConfigMaps works as
154157
expected.
155158

156-
For `Beta`, we will also extend scalability tests with a number of immutable
157-
`Secrets` and `ConfigMaps` to validate the performance impact (for `Alpha`
158-
only manual scalability tests will be performed).
159+
For `Beta`, scalability tests were extended to exercise this feature. Before this
160+
change 10% of pods were mounting some Secret and some ConfigMap. This change allowed
161+
us to mount some Secret and ConfigMap to every pod (however the remaining 90% of
162+
pods mount immutable ones).
163+
164+
For `GA`, the introduced e2e tests will be promoted to conformance.
159165

160166
### Graduation Criteria
161167

@@ -317,11 +323,17 @@ _This section must be completed when targeting beta graduation to a release._
317323
## Implementation History
318324

319325
2019-11-18: KEP opened
326+
320327
2019-12-09: KEP marked implementable
328+
321329
v1.18: Launched in `Alpha`
330+
322331
2020-04-25: Submitted PR to promote to Beta and enable by default.
332+
323333
2020-04-28: Scalability tests extended to validate this feature
324334

335+
v1.19: Launched to `Beta`
336+
325337
## Alternatives
326338

327339
### Define immutability at VolumeSource level

keps/sig-storage/1412-immutable-secrets-and-configmaps/kep.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,17 @@ reviewers:
1414
- "@msau42"
1515
approvers:
1616
- "@saad-ali"
17+
prr-approvers:
18+
- "@johnbelamaric"
1719
creation-date: 2019-11-17
18-
last-updated: 2019-12-09
20+
last-updated: 2020-12-10
1921
see-also:
2022
replaces:
2123

22-
latest-milestone: "v1.19"
24+
stage: "stable"
25+
26+
latest-milestone: "v1.21"
27+
2328
milestone:
2429
alpha: "v1.18"
2530
beta: "v1.19"

0 commit comments

Comments
 (0)