Skip to content

Commit 8a8a3c2

Browse files
committed
address comments kubernetes#2, remove trailing whitespaces
1 parent 39d4ecf commit 8a8a3c2

File tree

2 files changed

+17
-12
lines changed

2 files changed

+17
-12
lines changed

keps/sig-auth/3926-handling-undecryptable-resources/README.md

+14-8
Original file line numberDiff line numberDiff line change
@@ -136,10 +136,10 @@ Items marked with (R) are required *prior to targeting to a milestone / release*
136136
- [ ] (R) Design details are appropriately documented
137137
- [ ] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input (including test refactors)
138138
- [ ] e2e Tests for all Beta API Operations (endpoints)
139-
- [ ] (R) Ensure GA e2e tests meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
139+
- [ ] (R) Ensure GA e2e tests meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
140140
- [ ] (R) Minimum Two Week Window for GA e2e tests to prove flake free
141141
- [ ] (R) Graduation criteria is in place
142-
- [ ] (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)
142+
- [ ] (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)
143143
- [ ] (R) Production readiness review completed
144144
- [ ] (R) Production readiness review approved
145145
- [ ] "Implementation History" section is up-to-date for milestone
@@ -175,8 +175,8 @@ updates.
175175
176176
[documentation style guide]: https://github.com/kubernetes/community/blob/master/contributors/guide/style-guide.md
177177
-->
178-
Encryption at rest has been a stable Kubernetes for a long time. Every now and
179-
then there had been cases where, be it by improper handling or external system
178+
Encryption at rest for API resources has been a stable part of Kubernetes for a long time.
179+
Every now and then there had been cases where, be it by improper handling or external system
180180
failures, the cluster encryption got into a broken state.
181181

182182
If a single object of a resource type cannot be decrypted, listing resources of that
@@ -254,6 +254,12 @@ with it. I know that these will either be recreated by a controller, or I can
254254
manually recreate them. I would like a simple way to figure out which resources
255255
fail decryption and I would like a way to remove them via Kubernetes API.
256256

257+
#### Story 2
258+
259+
I would like to remove a namespace I no longer need. However, some of the resources
260+
inside of the namespace were encrypted before the encryption at
261+
rest configuration broke, which blocks a successful namespace delete.
262+
257263
### Notes/Constraints/Caveats (Optional)
258264

259265
<!--
@@ -323,7 +329,7 @@ a new status should be introduced - `StatusReasonStoreReadError`.
323329
// Details:
324330
// "kind" string - the kind attribute of the resource being acted on.
325331
// "name" string - the prefix where the reading error(s) occurred
326-
// "causes" []StatusCause
332+
// "causes" []StatusCause
327333
// - (optional):
328334
// - "type" CauseType - CauseTypeUnexpectedServerResponse
329335
// - "message" string - the error message from the store backend
@@ -710,10 +716,10 @@ Recall that end users cannot usually observe component logs or access metrics.
710716
-->
711717

712718
- [ ] Events
713-
- Event Reason:
719+
- Event Reason:
714720
- [ ] API .status
715-
- Condition name:
716-
- Other field:
721+
- Condition name:
722+
- Other field:
717723
- [ ] Other (treat as last resort)
718724
- Details:
719725

keps/sig-auth/3926-handling-undecryptable-resources/kep.yaml

+3-4
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,18 @@
11
title: Handling undecryptable resources
22
kep-number: 3926
33
authors:
4-
- "@jane.doe"
4+
- "@stlaz"
55
owning-sig: sig-auth
66
participating-sigs:
77
- sig-auth
88
- sig-api
99
status: provisional
1010
creation-date: 2023-03-27
1111
reviewers:
12-
- TBD
13-
- "@alice.doe"
12+
- "@liggit"
13+
- "@deads2k"
1414
approvers:
1515
- TBD
16-
- "@oscar.doe"
1716

1817
see-also:
1918
-

0 commit comments

Comments
 (0)