CNTRLPLANE-2522: features: add ExternalOIDCExternalClaimsSourcing feature gate#2743
Conversation
Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Hello @everettraven! Some important instructions when contributing to openshift/api: |
📝 WalkthroughWalkthroughThis pull request introduces a new feature gate called ExternalOIDCExternalClaimsSourcing. The feature gate is declared in the Go source code with configuration specifying Jira reporting, contact information, product scope, and an enhancement PR reference. The feature is enabled for inDevPreviewNoUpgrade status. Documentation is updated to reflect this new gate in the features table. Multiple payload manifest files are updated to configure this gate across different cluster deployment types (Hypershift and SelfManagedHA) and preview tiers, setting it as enabled for DevPreviewNoUpgrade configurations and disabled for Default, OKD, and TechPreviewNoUpgrade configurations. 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Review Summary by QodoAdd ExternalOIDCExternalClaimsSourcing feature gate for OIDC claims sourcing
WalkthroughsDescription• Add new ExternalOIDCExternalClaimsSourcing feature gate for authentication • Configure gate as dev preview with authentication component ownership • Update all feature gate manifest files across deployment profiles • Register feature gate in documentation and configuration Diagramflowchart LR
A["Feature Gate Definition"] -->|"registered in"| B["features.go"]
B -->|"documented in"| C["features.md"]
B -->|"deployed via"| D["Manifest Files"]
D -->|"includes"| E["Hypershift & SelfManagedHA"]
E -->|"profiles"| F["Default, DevPreview, OKD, TechPreview"]
File Changes1. features/features.go
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewⓘ The new review experience is currently in Beta. Learn more |
|
@everettraven: This pull request references CNTRLPLANE-2522 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
@everettraven: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/lgtm |
|
@JoelSpeed: This PR has been marked as verified by DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Scheduling tests matching the |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JoelSpeed The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Adds a feature gate for the work to be done as part of openshift/enhancements#1907