Skip to content

Fix security QUIC Sail Operator config #553

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
MaxBab wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix security QUIC Sail Operator config #553

MaxBab wants to merge 1 commit into from
+7 −1

Conversation

@MaxBab
Copy link
Contributor

@MaxBab MaxBab commented Jan 15, 2026

Please provide a description of this PR:
Add missing configuration for security QUIC based Istio integration tests.

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

Please check any characteristics that apply to this pull request.

  • Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@MaxBab
Fix security QUIC Sail Operator config
31cba8d 
Add missing configuration for security QUIC based Istio integration
tests.

Signed-off-by: Maxim Babushkin <[email protected]>
@openshift-ci openshift-ci bot added the size/XS label Jan 15, 2026
@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 15, 2026

/retest

6 similar comments
@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 18, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 19, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 19, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 19, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 19, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 20, 2026

/retest

fjglira
fjglira approved these changes Jan 20, 2026
View reviewed changes
Copy link
Contributor

@fjglira fjglira left a comment

Choose a reason for hiding this comment

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

LGTM the fix, but I see that there are some tests on pilot and others getting failures, are this a flaky test?

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 20, 2026

/retest

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 20, 2026

LGTM the fix, but I see that there are some tests on pilot and others getting failures, are this a flaky test?

Yes.
I'm looking into it.

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 20, 2026

/retest

1 similar comment
@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 20, 2026

/retest

@openshift-ci
Copy link

openshift-ci bot commented Jan 20, 2026

@MaxBab: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/istio-integration-sail-pilot 31cba8d link true /test istio-integration-sail-pilot
ci/prow/istio-integration-sail-ambient 31cba8d link true /test istio-integration-sail-ambient
ci/prow/istio-integration-sail-telemetry 31cba8d link true /test istio-integration-sail-telemetry
ci/prow/istio-integration-sail-security 31cba8d link true /test istio-integration-sail-security

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

@MaxBab
Copy link
Contributor Author

MaxBab commented Jan 21, 2026

/retest

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

Reviewers

@fjglira fjglira fjglira approved these changes

@ctartici ctartici Awaiting requested review from ctartici

@FilipB FilipB Awaiting requested review from FilipB

@mkralik3 mkralik3 Awaiting requested review from mkralik3

@pbajjuri20 pbajjuri20 Awaiting requested review from pbajjuri20

Assignees

No one assigned

Labels

size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@MaxBab @fjglira