Skip to content

Commit 69312be

Browse files
committed
KEP-3221: Update PRR responses
1 parent 15553a3 commit 69312be

File tree

1 file changed

+8
-36
lines changed
  • keps/sig-auth/3221-structured-authorization-configuration

1 file changed

+8
-36
lines changed

keps/sig-auth/3221-structured-authorization-configuration/README.md

Lines changed: 8 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -624,17 +624,15 @@ the scenarios.
624624

625625
### Upgrade / Downgrade Strategy
626626

627-
While the feature is in Alpha, there is no change if cluster administrators want to
628-
keep on using command line flags.
627+
There is no change if cluster administrators want to keep on using command line flags.
629628

630-
When the feature goes to Beta/GA or the cluster administrators want to configure
631-
authorizers using a config file, they need to make sure the config file exists before
632-
upgrading the cluster. Similarly when downgrading clusters, they would need to add
633-
the flags back to their bootstrap mechanism.
629+
If the cluster administrators wants to configure authorizers using a config file,
630+
they need to make sure the config file exists before upgrading the cluster.
631+
When downgrading clusters, they would need to switch their invocation back to use flags.
634632

635633
### Version Skew Strategy
636634

637-
Not applicable.
635+
Not applicable, authorizers are configured per API server.
638636

639637
## Production Readiness Review Questionnaire
640638

@@ -646,6 +644,8 @@ Not applicable.
646644
- Feature gate name: `StructuredAuthorizationConfiguration`
647645
- Components depending on the feature gate:
648646
- kube-apiserver
647+
- [x] Other
648+
- `kube-apiserver` command-line flag: `--authorization-config`
649649

650650
###### Does enabling the feature change any default behavior?
651651

@@ -673,8 +673,6 @@ command line flags should return an error.
673673

674674
### Rollout, Upgrade and Rollback Planning
675675

676-
> Note: This section is required when targeting Beta to a release.
677-
678676
###### How can a rollout or rollback fail? Can it impact already running workloads?
679677

680678
A rollout can fail when the authorization configuration file being passed doesn't
@@ -701,16 +699,6 @@ No.
701699

702700
### Monitoring Requirements
703701

704-
<!--
705-
This section must be completed when targeting beta to a release.
706-
707-
For GA, this section is required: approvers should be able to confirm the
708-
previous answers based on experience in the field.
709-
-->
710-
711-
> Note: To be elaborated more during Beta graduation since this section
712-
must be completed when targeting beta to a release.
713-
714702
###### How can an operator determine if the feature is in use by workloads?
715703

716704
The cluster administrators can check the flags passed to the `kube-apiserver` if
@@ -773,8 +761,6 @@ None
773761

774762
### Scalability
775763

776-
> Note: This section is good-to-have for Alpha.
777-
778764
###### Will enabling / using this feature result in any new API calls?
779765

780766
No. No additional calls will be made to the Kubernetes API Server.
@@ -798,7 +784,7 @@ cluster administrator defines multiple webhooks.
798784

799785
**Note**: This is a result of the intended feature.
800786
If multiple webhooks are defined and one or more of them are unreachable, the
801-
request latency would get a hit but this is upto the configuration made by the
787+
request latency would get a hit but this is up to the configuration made by the
802788
user. The feature implementation itself doesn't introduce any change to the
803789
existing SLIs/SLOs.
804790

@@ -826,20 +812,6 @@ number of network calls.
826812

827813
### Troubleshooting
828814

829-
<!--
830-
This section must be completed when targeting beta to a release.
831-
832-
For GA, this section is required: approvers should be able to confirm the
833-
previous answers based on experience in the field.
834-
835-
The Troubleshooting section currently serves the `Playbook` role. We may consider
836-
splitting it into a dedicated `Playbook` document (potentially with some monitoring
837-
details). For now, we leave it here.
838-
-->
839-
840-
> Note: To be elaborated more during Beta graduation since this section
841-
must be completed when targeting beta to a release.
842-
843815
###### How does this feature react if the API server and/or etcd is unavailable?
844816

845817
No effect.

0 commit comments

Comments
 (0)