Add ConfidentialCluster feature gate#2779
Add ConfidentialCluster feature gate#2779fangge1212 wants to merge 1 commit intoopenshift:masterfrom
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Hello @fangge1212! Some important instructions when contributing to openshift/api: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
✅ Files skipped from review due to trivial changes (7)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughA new feature gate, ConfidentialCluster, was added and registered in Go as ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 golangci-lint (2.11.4)Error: build linters: unable to load custom analyzer "kubeapilinter": tools/_output/bin/kube-api-linter.so, plugin: not implemented Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
features/features.go
Outdated
| contactPerson("fjin"). | ||
| productScope(ocpSpecific). | ||
| enhancementPR("https://github.com/openshift/enhancements/pull/1962"). | ||
| enable(inDevPreviewNoUpgrade(), inTechPreviewNoUpgrade()). |
There was a problem hiding this comment.
When adding a feature into TechPreviewNoUpgrade, the expectation is that the enhancement proposal has been merged - signaling that stakeholders have agreed upon a plan for a tech preview implementation.
If you'd like to have a feature gate for earlier experimentation, you can add this in DevPreviewNoUpgrade only without having a merged EP.
That being said, it does help to have some kind of general EP information written that explains the motivations of the feature.
There was a problem hiding this comment.
Thank you. I will restrict it to DevPreviewNoUpgrade for now
eb558a3 to
58f0a5d
Compare
Signed-off-by: Fangge Jin <fjin@redhat.com>
58f0a5d to
13f1241
Compare
|
@fangge1212: 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. |
No description provided.