Skip to content

✨ Promote Single Own Feature Gate AND Config spec in the CR to GA - #2428

Merged
openshift-merge-bot[bot] merged 1 commit into
operator-framework:mainfrom
camilamacedo86:promote-single-ns
Jan 6, 2026
Merged

✨ Promote Single Own Feature Gate AND Config spec in the CR to GA#2428
openshift-merge-bot[bot] merged 1 commit into
operator-framework:mainfrom
camilamacedo86:promote-single-ns

Conversation

@camilamacedo86

Copy link
Copy Markdown
Contributor

No description provided.

@camilamacedo86
camilamacedo86 requested a review from a team as a code ownerJanuary 6, 2026 13:12
CopilotAI review requested due to automatic review settings January 6, 2026 13:12
@netlify

netlifyBot commented Jan 6, 2026

Copy link
Copy Markdown

Deploy Preview for olmv1 ready!

NameLink
🔨 Latest commitbda781b
🔍 Latest deploy loghttps://app.netlify.com/projects/olmv1/deploys/695d0cb478d8bf0008ca4ec0
😎 Deploy Previewhttps://deploy-preview-2428--olmv1.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@camilamacedo86

Copy link
Copy Markdown
ContributorAuthor

@perdasilva@tmshort@anik120

I think we are in time to promote those to GA.
Could you please help us out?

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR promotes the SingleOwnNamespaceInstallSupport feature gate and the config field in the ClusterExtension spec from experimental/alpha to General Availability (GA). The feature enables installing operators in SingleNamespace or OwnNamespace modes and allows bundle-specific configuration via the config field.

Key Changes:

  • Promoted SingleOwnNamespaceInstallSupport feature gate from Alpha (default: false) to GA (default: true)
  • Added the config spec to standard CRDs (previously only available in experimental CRDs)
  • Removed feature gate flags from experimental manifests and helm configurations since it's now enabled by default

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
internal/operator-controller/features/features.goChanged feature gate from Alpha/default:false to GA/default:true
test/e2e/steps/hooks.goUpdated E2E test configuration to enable the feature gate by default and cleaned up import statement
manifests/standard.yamlAdded config spec to ClusterExtension CRD
manifests/standard-e2e.yamlAdded config spec to ClusterExtension CRD
manifests/experimental.yamlRemoved feature gate flag from deployment args and updated config description
manifests/experimental-e2e.yamlRemoved feature gate flag from deployment args and updated config description
helm/olmv1/base/operator-controller/crd/standard/olm.operatorframework.io_clusterextensions.yamlAdded config spec to standard CRD
helm/olmv1/base/operator-controller/crd/experimental/olm.operatorframework.io_clusterextensions.yamlUpdated config field description to match standard behavior
helm/tilt.yamlRemoved SingleOwnNamespaceInstallSupport from enabled features list
helm/experimental.yamlRemoved SingleOwnNamespaceInstallSupport from enabled features list
hack/demo/single-namespace-demo-script.shSimplified to use standard manifests and removed feature gate enablement steps
hack/demo/own-namespace-demo-script.shSimplified to use standard manifests and removed feature gate enablement steps
docs/tutorials/explore-available-content.mdRemoved note about requiring the feature gate for webhook support
docs/project/olmv1_limitations.mdUpdated to reflect support for SingleNamespace and OwnNamespace install modes
docs/draft/tutorials/explore-available-content-metas-endpoint.mdRemoved note about requiring the feature gate for webhook support
docs/draft/howto/single-ownnamespace-install.mdUpdated to reflect feature gate is enabled by default and removed enablement instructions
docs/api-reference/olmv1-api-reference.mdUpdated config field description to remove experimental marker
api/v1/clusterextension_types.goRemoved experimental marker from config field comments

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecovBot commented Jan 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.00%. Comparing base (dba48b9) to head (bda781b).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #2428 +/- ##
==========================================
+ Coverage 68.82% 73.00% +4.17% 
==========================================
Files 100 100 Lines 7641 7641 ==========================================
+ Hits 5259 5578 +319 + Misses 1947 1625 -322 - Partials 435 438 +3 
FlagCoverage Δ
e2e46.76% <ø> (+2.94%)⬆️
experimental-e2e48.68% <ø> (+35.00%)⬆️
unit57.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pedjakpedjak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ciopenshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Jan 6, 2026
@tmshort

Copy link
Copy Markdown
Contributor

/approve

@openshift-ci

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tmshort

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ciopenshift-ciBot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 6, 2026
@openshift-merge-bot
openshift-merge-botBot merged commit bcb3d07 into operator-framework:mainJan 6, 2026
30 checks passed
@camilamacedo86
camilamacedo86 deleted the promote-single-ns branch January 6, 2026 20:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by an approver from all required OWNERS files.lgtmIndicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@camilamacedo86@tmshort@pedjak