Skip to content

Commit 2d3d1d7

Browse files
authored
Merge pull request kubernetes#3772 from nabokihms/3325-self-subject-review-beta
KEP-3325: SelfSubjectReview API Beta graduation
2 parents b8170b3 + 90f4c9b commit 2d3d1d7

File tree

3 files changed

+26
-18
lines changed

3 files changed

+26
-18
lines changed
+2
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
kep-number: 3325
22
alpha:
33
approver: "@deads2k"
4+
beta:
5+
approver: "@deads2k"

keps/sig-auth/3325-self-subject-attributes-review-api/README.md

+20-16
Original file line numberDiff line numberDiff line change
@@ -243,16 +243,33 @@ We expect no non-infra related flakes in the last month as a GA graduation crite
243243

244244
#### Alpha
245245

246+
- `SelfSubjectReview` endpoint is introduced in `authentication.k8s.io/v1alpha1` API
246247
- Feature implemented behind a feature flag
247248
- Initial unit and integration tests completed and enabled
249+
- Corresponding kubectl command implemented: `kubectl alpha auth whoami`
248250

249251
#### Beta
250252

251253
- Gather feedback from users
254+
- `SelfSubjectReview` is promoted to `authentication.k8s.io/v1beta1` API (Beta APIs are not enabled by default, [see](https://github.com/kubernetes/enhancements/blob/master/keps/sig-architecture/3136-beta-apis-off-by-default/README.md)).
255+
- Promote feature gate to Beta and make it enabled by default
256+
- Unit tests coverage improved
257+
- `kubectl alpha auth whoami` command uses `authentication.k8s.io/v1beta1` API, falls back to `authentication.k8s.io/v1alpha1` API
258+
- Fix [documentation](https://kubernetes.io/docs/reference/access-authn-authz/authentication/#self-subject-review):
259+
- Change API version
260+
- Rewrite conditions to enable the feature
252261

253262
#### GA
254263

255-
- Corresponding kubectl command implemented
264+
- `SelfSubjectReview` is promoted to `authentication.k8s.io/v1` API and enable by default
265+
- Promote feature gate to Stable
266+
- `kubectl alpha auth whoami` replaced with `kubectl auth whoami`
267+
- `kubectl auth whoami` command prefers `authentication.k8s.io/v1` API over `authentication.k8s.io/v1beta1` and `authentication.k8s.io/v1alpha1`
268+
- More integration and e2e tests cases
269+
- Fix [documentation](https://kubernetes.io/docs/reference/access-authn-authz/authentication/#self-subject-review):
270+
- Change API version
271+
- Rewrite conditions to enable the feature
272+
- Change kubectl command
256273

257274
NOTE: Should not be a part of [conformance tests](https://git.k8s.io/community/contributors/devel/sig-architecture/conformance-tests.md).
258275
The fact that a user possesses a token does not necessarily imply the power to know to whom that token belongs.
@@ -263,22 +280,9 @@ The fact that a user possesses a token does not necessarily imply the power to k
263280

264281
###### How can this feature be enabled / disabled in a live cluster?
265282

266-
<!--
267-
Pick one of these and delete the rest.
268-
-->
269-
270-
- Feature gate
283+
- [X] Feature gate (also fill in values in `kep.yaml`)
271284
- Feature gate name: `APISelfSubjectReview`
272-
- Components depending on the feature gate:
273-
- kube-apiserver
274-
275-
```go
276-
FeatureSpec{
277-
Default: false,
278-
LockToDefault: false,
279-
PreRelease: featuregate.Alpha,
280-
}
281-
```
285+
- Components depending on the feature gate: `kube-apiserver`
282286

283287
###### Does enabling the feature change any default behavior?
284288

keps/sig-auth/3325-self-subject-attributes-review-api/kep.yaml

+4-2
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,13 @@ reviewers:
1010
- "@enj"
1111
- "@deads2k"
1212
- "@mikedanese"
13+
- "@liggitt"
1314
approvers:
14-
- TBD
15+
- "@deads2k"
16+
- "@liggitt"
1517
creation-date: "2022-05-30"
1618
status: implementable
17-
stage: alpha
19+
stage: beta
1820
latest-milestone: "v1.26"
1921
milestone:
2022
alpha: "v1.26"

0 commit comments

Comments
 (0)