Skip to content

[OSSM-12848] Fix pilot/gatewayinstance subsuite when sail operator is used#648

Draft
mkralik3 wants to merge 1 commit intoopenshift-service-mesh:release-1.28from
mkralik3:gatewayinstance
Draft

[OSSM-12848] Fix pilot/gatewayinstance subsuite when sail operator is used#648
mkralik3 wants to merge 1 commit intoopenshift-service-mesh:release-1.28from
mkralik3:gatewayinstance

Conversation

@mkralik3
Copy link
Contributor

@mkralik3 mkralik3 commented Mar 9, 2026

Fix pilot/gatewayinstance when sail operator is used as external control plane installer
--istio.test.kube.controlPlaneInstaller=./prow/setup/sail-operator-setup.sh

The gatewayinstance:

  • creates 2 different control planes (mesh and gateway) with different namespace as istio-system, so script must be updated to set correct namespace in sail-iop.yaml
  • creates 2 different control planes (mesh and gateway) with different name as default, so script must be updated to set correct name in sail-iop.yaml
  • mesh should have ingress/egress, gateway should not
    • since there are 2 control planes, the sidecar.istio.io/inject label/annotation must be changed to the istio.io/rev
  • to correctly test ns boundaries, the discovery selector must be set in sail-iop.yaml

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

  • Ambient
  • Configuration Infrastructure
  • Docs
  • Dual Stack
  • Installation
  • Networking
  • Performance and Scalability
  • Extensions and Telemetry
  • Security
  • Test and Release
  • User Experience
  • Developer Infrastructure
  • Upgrade
  • Multi Cluster
  • Virtual Machine
  • Control Plane Revisions

@openshift-ci
Copy link

openshift-ci bot commented Mar 9, 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 size/M label Mar 9, 2026
… used

Signed-off-by: mkralik3 <mkralik@redhat.com>
@mkralik3 mkralik3 requested a review from MaxBab March 10, 2026 07:12
@mkralik3 mkralik3 added the permanent-change OSSM-specific permanent change label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant