Skip to content

Adding check_cluster_operators for OCP clusters#1238

Merged
istio-testing merged 1 commit intoistio-ecosystem:mainfrom
fjglira:add-checkopearator-before-testrun
Sep 29, 2025
Merged

Adding check_cluster_operators for OCP clusters#1238
istio-testing merged 1 commit intoistio-ecosystem:mainfrom
fjglira:add-checkopearator-before-testrun

Conversation

@fjglira
Copy link
Copy Markdown
Contributor

@fjglira fjglira commented Sep 26, 2025

Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

What type of PR is this?

  • Enhancement / New Feature
  • Bug Fix
  • Refactor
  • Optimization
  • Test
  • Documentation Update

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Related Issue/PR #

Additional information:

Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
@fjglira fjglira requested a review from a team as a code owner September 26, 2025 14:51
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.39%. Comparing base (2826822) to head (d80167a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1238      +/-   ##
==========================================
+ Coverage   77.18%   77.39%   +0.21%     
==========================================
  Files          44       44              
  Lines        2823     2823              
==========================================
+ Hits         2179     2185       +6     
+ Misses        533      529       -4     
+ Partials      111      109       -2     

☔ 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.

@istio-testing istio-testing merged commit 02818d8 into istio-ecosystem:main Sep 29, 2025
17 checks passed
@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: #1238 failed to apply on top of branch "release-1.27":

Applying: Adding check_cluster_operators for OCP clusters
Using index info to reconstruct a base tree...
M	tests/e2e/common-operator-integ-suite.sh
Falling back to patching base and 3-way merge...
Auto-merging tests/e2e/common-operator-integ-suite.sh
CONFLICT (content): Merge conflict in tests/e2e/common-operator-integ-suite.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Adding check_cluster_operators for OCP clusters

@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: new issue created for failed cherrypick: #1239

@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: #1238 failed to apply on top of branch "release-1.26":

Applying: Adding check_cluster_operators for OCP clusters
Using index info to reconstruct a base tree...
M	tests/e2e/common-operator-integ-suite.sh
Falling back to patching base and 3-way merge...
Auto-merging tests/e2e/common-operator-integ-suite.sh
CONFLICT (content): Merge conflict in tests/e2e/common-operator-integ-suite.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Adding check_cluster_operators for OCP clusters

@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: new issue created for failed cherrypick: #1240

@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: #1238 failed to apply on top of branch "release-1.27":

Applying: Adding check_cluster_operators for OCP clusters
Using index info to reconstruct a base tree...
M	tests/e2e/common-operator-integ-suite.sh
Falling back to patching base and 3-way merge...
Auto-merging tests/e2e/common-operator-integ-suite.sh
CONFLICT (content): Merge conflict in tests/e2e/common-operator-integ-suite.sh
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Adding check_cluster_operators for OCP clusters

@istio-testing
Copy link
Copy Markdown
Collaborator

In response to a cherrypick label: new issue created for failed cherrypick: #1241

fjglira added a commit to fjglira/sail-operator that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
fjglira added a commit to fjglira/sail-operator that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
fjglira added a commit to fjglira/sail-operator that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
fjglira added a commit to fjglira/sail-operator that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
istio-testing pushed a commit that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
istio-testing pushed a commit that referenced this pull request Sep 29, 2025
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Sep 29, 2025
* upstream/release-1.26:
  Adding check_cluster_operators for OCP clusters (istio-ecosystem#1238) (istio-ecosystem#1243)
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Sep 29, 2025
* upstream/release-1.27:
  Adding check_cluster_operators for OCP clusters (istio-ecosystem#1238) (istio-ecosystem#1244)
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Sep 29, 2025
* upstream/release-1.27:
  Adding check_cluster_operators for OCP clusters (istio-ecosystem#1238) (istio-ecosystem#1244)
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Sep 29, 2025
* upstream/main:
  Adding check_cluster_operators for OCP clusters (istio-ecosystem#1238)
openshift-merge-bot bot added a commit to openshift-service-mesh/sail-operator that referenced this pull request Oct 21, 2025
* Cherry-pick: Add OCP-specific flag to CreateRemoteSecret, Allow set multicluster flag with env to release-1.26 (istio-ecosystem#1214)

* Add OCP-specific flag to CreateRemoteSecret (istio-ecosystem#1213)

Use the '--create-service-account=false' flag when creating remote secrets on OpenShift.

Signed-off-by: Mikhail Abramov <mabramov@redhat.com>

* Allow set multicluster flag with env (istio-ecosystem#1212)

This change allows setting the multicluster flag via environment variables.

Signed-off-by: Mikhail Abramov <mabramov@redhat.com>

---------

Signed-off-by: Mikhail Abramov <mabramov@redhat.com>

* [release-1.26] Fix junit report generation (istio-ecosystem#1219)

* Fix junit report generation

Fix the junit report generation for every test run for our e2e test execution

Signed-off-by: Francisco Herrera <fjglira@gmail.com>

* Fix lint

Fixing lint error

Signed-off-by: Francisco Herrera <fjglira@gmail.com>

---------

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
Co-authored-by: Francisco Herrera <fjglira@gmail.com>

* [release-3.1] Bump version and update GA images (#516)

* Fix new lint errors (istio-ecosystem#1234)

Fix lint error that are happening in the PR: 1231

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
Co-authored-by: Francisco Herrera <fjglira@gmail.com>

* Adding check_cluster_operators for OCP clusters (istio-ecosystem#1238) (istio-ecosystem#1243)

Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>

* [release-3.1] Automator: Update Istio module (#522)

Co-authored-by: openshift-service-mesh-bot <null>

* [3.1.3] Prepare placeholders for OSSM 3.1.3 release without istio changes (#540)

---------

Signed-off-by: Mikhail Abramov <mabramov@redhat.com>
Signed-off-by: Francisco Herrera <fjglira@gmail.com>
Co-authored-by: Mikhail Abramov <mabramov@redhat.com>
Co-authored-by: openshift-service-mesh-bot <null>
Co-authored-by: openshift-merge-bot[bot] <148852131+openshift-merge-bot[bot]@users.noreply.github.com>
Co-authored-by: Istio Automation <istio-testing-bot@google.com>
Co-authored-by: Francisco Herrera <fjglira@gmail.com>
Co-authored-by: openshift-service-mesh-bot <165402251+openshift-service-mesh-bot@users.noreply.github.com>
Co-authored-by: Cansin Tartici <ctartici@redhat.com>
dgn pushed a commit to dgn/sail-operator that referenced this pull request Mar 17, 2026
Adding check_cluster_operators will help to avoid test run issues while executing the test over OCP clusters

Signed-off-by: Francisco Herrera <fjglira@gmail.com>
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
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.

3 participants