Skip to content

[release-4.19] WIP: NE-2480: Promote GatewayAPIWithoutOLM feature gate to Default#2870

Draft
gcs278 wants to merge 2 commits into
openshift:release-4.19from
gcs278:backport-GatewayAPIWithoutOLM-4.19
Draft

[release-4.19] WIP: NE-2480: Promote GatewayAPIWithoutOLM feature gate to Default#2870
gcs278 wants to merge 2 commits into
openshift:release-4.19from
gcs278:backport-GatewayAPIWithoutOLM-4.19

Conversation

@gcs278
Copy link
Copy Markdown
Contributor

@gcs278 gcs278 commented Jun 3, 2026

Summary

Promote the GatewayAPIWithoutOLM feature gate to Default (enabled) on release-4.19. This activates the Sail Library installation path, replacing OLM-based Istio management for Gateway API.

This is a WIP — do not merge until the CIO noOLM backport and E2E tests are in place.

Implementation: openshift/cluster-ingress-operator#1460

Why

See the noOLM backport PR for full context. The primary driver is OCPBUGS-86778 which blocks all OSSM z-stream upgrades via OLM on 4.19-4.21, preventing CVE fixes from shipping.

🤖 Generated with Claude Code

gcs278 and others added 2 commits June 2, 2026 20:23
Backport of the GatewayAPIWithoutOLM feature gate definition to
release-4.19 with all feature sets disabled. This provides the
gate definition so that the CIO noOLM backport can compile against
it. The gate is OFF in all profiles — the Sail Library code path
is present but dormant.

Cherry-picked from: a47ac12
Original PR: openshift#2191

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Enable the GatewayAPIWithoutOLM feature gate by default on release-4.19.
This activates the Sail Library installation path, replacing OLM-based
Istio management for Gateway API.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-merge-bot
Copy link
Copy Markdown
Contributor

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: LGTM mode

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 3, 2026
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Jun 3, 2026

@gcs278: This pull request references NE-2480 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.19.z" version, but no target version was set.

Details

In response to this:

Summary

Promote the GatewayAPIWithoutOLM feature gate to Default (enabled) on release-4.19. This activates the Sail Library installation path, replacing OLM-based Istio management for Gateway API.

This is a WIP — do not merge until the CIO noOLM backport and E2E tests are in place.

Implementation: https://github.com/openshift/cluster-ingress-operator/pull/TBD

Why

See the noOLM backport PR for full context. The primary driver is OCPBUGS-86778 which blocks all OSSM z-stream upgrades via OLM on 4.19-4.21, preventing CVE fixes from shipping.

🤖 Generated with Claude Code

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.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 3, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 3, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 3, 2026

Hello @gcs278! Some important instructions when contributing to openshift/api:
API design plays an important part in the user experience of OpenShift and as such API PRs are subject to a high level of scrutiny to ensure they follow our best practices. If you haven't already done so, please review the OpenShift API Conventions and ensure that your proposed changes are compliant. Following these conventions will help expedite the api review process for your PR.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: fc25a332-b741-4ead-ac46-ae72dc6f3529

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci openshift-ci Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 3, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 3, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign joelspeed for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants