Skip to content

OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE#30074

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
camilamacedo86:remove-olmv1-preflight
Aug 26, 2025
Merged

OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE#30074
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
camilamacedo86:remove-olmv1-preflight

Conversation

@camilamacedo86
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 commented Aug 7, 2025

The tests were migrating in: operator-framework-operator-controller/pull/423

They are in Sippy: https://sippy.dptools.openshift.org/sippy-ng/component_readiness/capability?baseEndTime=2025-06-17%2023%3A59%3A59&baseRelease=4.19&baseStartTime=2025-05-18%2000%3A00%3A00&capability=OCPFeatureGate%3ANewOLMPreflightPermissionChecks&columnGroupBy=Architecture%2CNetwork%2CPlatform%2CTopology&component=OLM&confidence=95&dbGroupBy=Platform%2CArchitecture%2CNetwork%2CTopology%2CFeatureSet%2CUpgrade%2CSuite%2CInstaller&flakeAsFailure=false&ignoreDisruption=true&ignoreMissing=false&includeMultiReleaseAnalysis=true&includeVariant=Architecture%3Aamd64&includeVariant=CGroupMode%3Av2&includeVariant=ContainerRuntime%3Acrun&includeVariant=ContainerRuntime%3Arunc&includeVariant=FeatureSet%3Adefault&includeVariant=FeatureSet%3Atechpreview&includeVariant=Installer%3Ahypershift&includeVariant=Installer%3Aipi&includeVariant=Installer%3Aupi&includeVariant=JobTier%3Ablocking&includeVariant=JobTier%3Ainforming&includeVariant=JobTier%3Astandard&includeVariant=LayeredProduct%3Anone&includeVariant=LayeredProduct%3Avirt&includeVariant=Network%3Aovn&includeVariant=Owner%3Aeng&includeVariant=Owner%3Aservice-delivery&includeVariant=Platform%3Aaws&includeVariant=Platform%3Aazure&includeVariant=Platform%3Agcp&includeVariant=Platform%3Ametal&includeVariant=Platform%3Arosa&includeVariant=Platform%3Avsphere&includeVariant=Topology%3Aexternal&includeVariant=Topology%3Aha&includeVariant=Topology%3Amicroshift&minFail=3&passRateAllTests=0&passRateNewTests=95&pity=5&sampleEndTime=2025-08-21%2023%3A59%3A59&sampleRelease=4.20&sampleStartTime=2025-08-14%2000%3A00%3A00

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

openshift-ci-robot commented Aug 7, 2025

@camilamacedo86: This pull request references OPRUN-4056 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.20.0" version, but no target version was set.

Details

In response to this:

Blocked by: operator-framework-operator-controller/pull/423

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 openshift-ci Bot requested review from jcantrill and joelanford August 7, 2025 18:07
@camilamacedo86 camilamacedo86 changed the title (Blocked by: operator-framework-operator-controller/pull/423) OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE (DO NOT MERGE) - (Blocked by: operator-framework-operator-controller/pull/423) OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE Aug 8, 2025
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/hold

Until: openshift/operator-framework-operator-controller#423 get merged
AND
we have the signal that we can remove

@camilamacedo86 camilamacedo86 changed the title (DO NOT MERGE) - (Blocked by: operator-framework-operator-controller/pull/423) OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE (DO NOT MERGE) OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE Aug 8, 2025
@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 8, 2025
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/hold

@camilamacedo86 camilamacedo86 force-pushed the remove-olmv1-preflight branch 2 times, most recently from a1fbde5 to ba0d415 Compare August 11, 2025 12:43
@openshift-trt
Copy link
Copy Markdown

openshift-trt Bot commented Aug 11, 2025

Job Failure Risk Analysis for sha: ba0d415

Job Name Failure Risk
pull-ci-openshift-origin-main-e2e-gcp-csi IncompleteTests
Tests for this run (18) are below the historical average (1521): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn IncompleteTests
Tests for this run (18) are below the historical average (2841): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn-rt-upgrade IncompleteTests
Tests for this run (18) are below the historical average (1529): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn-techpreview IncompleteTests
Tests for this run (17) are below the historical average (2726): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn-techpreview-serial-1of2 IncompleteTests
Tests for this run (18) are below the historical average (1556): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn-techpreview-serial-2of2 IncompleteTests
Tests for this run (17) are below the historical average (1506): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)
pull-ci-openshift-origin-main-e2e-gcp-ovn-upgrade IncompleteTests
Tests for this run (17) are below the historical average (1627): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)

@camilamacedo86 camilamacedo86 force-pushed the remove-olmv1-preflight branch from ba0d415 to 549904b Compare August 20, 2025 11:10
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Aug 21, 2025

@camilamacedo86: This pull request references OPRUN-4056 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.20.0" version, but no target version was set.

Details

In response to this:

The tests were migrating in: operator-framework-operator-controller/pull/423
They are in Sippy: https://sippy.dptools.openshift.org/sippy-ng/component_readiness/capability?baseEndTime=2025-06-17%2023%3A59%3A59&baseRelease=4.19&baseStartTime=2025-05-18%2000%3A00%3A00&capability=OCPFeatureGate%3ANewOLMPreflightPermissionChecks&columnGroupBy=Architecture%2CNetwork%2CPlatform%2CTopology&component=OLM&confidence=95&dbGroupBy=Platform%2CArchitecture%2CNetwork%2CTopology%2CFeatureSet%2CUpgrade%2CSuite%2CInstaller&flakeAsFailure=false&ignoreDisruption=true&ignoreMissing=false&includeMultiReleaseAnalysis=true&includeVariant=Architecture%3Aamd64&includeVariant=CGroupMode%3Av2&includeVariant=ContainerRuntime%3Acrun&includeVariant=ContainerRuntime%3Arunc&includeVariant=FeatureSet%3Adefault&includeVariant=FeatureSet%3Atechpreview&includeVariant=Installer%3Ahypershift&includeVariant=Installer%3Aipi&includeVariant=Installer%3Aupi&includeVariant=JobTier%3Ablocking&includeVariant=JobTier%3Ainforming&includeVariant=JobTier%3Astandard&includeVariant=LayeredProduct%3Anone&includeVariant=LayeredProduct%3Avirt&includeVariant=Network%3Aovn&includeVariant=Owner%3Aeng&includeVariant=Owner%3Aservice-delivery&includeVariant=Platform%3Aaws&includeVariant=Platform%3Aazure&includeVariant=Platform%3Agcp&includeVariant=Platform%3Ametal&includeVariant=Platform%3Arosa&includeVariant=Platform%3Avsphere&includeVariant=Topology%3Aexternal&includeVariant=Topology%3Aha&includeVariant=Topology%3Amicroshift&minFail=3&passRateAllTests=0&passRateNewTests=95&pity=5&sampleEndTime=2025-08-21%2023%3A59%3A59&sampleRelease=4.20&sampleStartTime=2025-08-14%2000%3A00%3A00

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.

@camilamacedo86 camilamacedo86 changed the title (DO NOT MERGE) OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE OPRUN-4056: [OLMv1] Remove Preflight checks since those were migrated to OTE Aug 21, 2025
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/retest-required

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/hold cancel

@openshift-ci openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 21, 2025
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Aug 21, 2025

@camilamacedo86: This pull request references OPRUN-4056 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.20.0" version, but no target version was set.

Details

In response to this:

The tests were migrating in: operator-framework-operator-controller/pull/423

They are in Sippy: https://sippy.dptools.openshift.org/sippy-ng/component_readiness/capability?baseEndTime=2025-06-17%2023%3A59%3A59&baseRelease=4.19&baseStartTime=2025-05-18%2000%3A00%3A00&capability=OCPFeatureGate%3ANewOLMPreflightPermissionChecks&columnGroupBy=Architecture%2CNetwork%2CPlatform%2CTopology&component=OLM&confidence=95&dbGroupBy=Platform%2CArchitecture%2CNetwork%2CTopology%2CFeatureSet%2CUpgrade%2CSuite%2CInstaller&flakeAsFailure=false&ignoreDisruption=true&ignoreMissing=false&includeMultiReleaseAnalysis=true&includeVariant=Architecture%3Aamd64&includeVariant=CGroupMode%3Av2&includeVariant=ContainerRuntime%3Acrun&includeVariant=ContainerRuntime%3Arunc&includeVariant=FeatureSet%3Adefault&includeVariant=FeatureSet%3Atechpreview&includeVariant=Installer%3Ahypershift&includeVariant=Installer%3Aipi&includeVariant=Installer%3Aupi&includeVariant=JobTier%3Ablocking&includeVariant=JobTier%3Ainforming&includeVariant=JobTier%3Astandard&includeVariant=LayeredProduct%3Anone&includeVariant=LayeredProduct%3Avirt&includeVariant=Network%3Aovn&includeVariant=Owner%3Aeng&includeVariant=Owner%3Aservice-delivery&includeVariant=Platform%3Aaws&includeVariant=Platform%3Aazure&includeVariant=Platform%3Agcp&includeVariant=Platform%3Ametal&includeVariant=Platform%3Arosa&includeVariant=Platform%3Avsphere&includeVariant=Topology%3Aexternal&includeVariant=Topology%3Aha&includeVariant=Topology%3Amicroshift&minFail=3&passRateAllTests=0&passRateNewTests=95&pity=5&sampleEndTime=2025-08-21%2023%3A59%3A59&sampleRelease=4.20&sampleStartTime=2025-08-14%2000%3A00%3A00

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.

Copy link
Copy Markdown
Contributor

@anik120 anik120 left a comment

Choose a reason for hiding this comment

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

/lgtm

I see openshift/operator-framework-operator-controller#423 merged 19 hours ago.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 21, 2025
@tmshort
Copy link
Copy Markdown
Contributor

tmshort commented Aug 21, 2025

/lgtm
/approve

@smg247
Copy link
Copy Markdown
Member

smg247 commented Aug 21, 2025

/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 21, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anik120, camilamacedo86, smg247, 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-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 21, 2025
@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD b53c4b6 and 2 for PR HEAD 549904b in total

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD b53c4b6 and 2 for PR HEAD 549904b in total

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD 0dfa8e5 and 1 for PR HEAD 549904b in total

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-gcp-ovn

1 similar comment
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-gcp-ovn

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD cc8ae10 and 2 for PR HEAD 549904b in total

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-gcp-ovn

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/retest-required

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-metal-ipi-ovn-ipv6

@openshift-ci-robot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD cc8ae10 and 2 for PR HEAD 549904b in total

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-metal-ipi-ovn-ipv6

1 similar comment
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/test e2e-metal-ipi-ovn-ipv6

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 26, 2025

@camilamacedo86: 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/e2e-gcp-ovn-rt-upgrade 549904b link false /test e2e-gcp-ovn-rt-upgrade
ci/prow/e2e-gcp-ovn-techpreview 549904b link false /test e2e-gcp-ovn-techpreview
ci/prow/e2e-aws-ovn-single-node-upgrade 549904b link false /test e2e-aws-ovn-single-node-upgrade
ci/prow/e2e-aws-ovn-single-node-serial 549904b link false /test e2e-aws-ovn-single-node-serial
ci/prow/e2e-aws-disruptive 549904b link false /test e2e-aws-disruptive
ci/prow/e2e-openstack-ovn 549904b link false /test e2e-openstack-ovn
ci/prow/e2e-metal-ipi-virtualmedia 549904b link false /test e2e-metal-ipi-virtualmedia

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.

@openshift-trt
Copy link
Copy Markdown

openshift-trt Bot commented Aug 26, 2025

Job Failure Risk Analysis for sha: 549904b

Job Name Failure Risk
pull-ci-openshift-origin-main-e2e-aws-disruptive Medium
[sig-node] static pods should start after being created
Potential external regression detected for High Risk Test analysis
---
[sig-arch] events should not repeat pathologically for ns/openshift-kube-apiserver-operator
Potential external regression detected for High Risk Test analysis
pull-ci-openshift-origin-main-e2e-metal-ipi-ovn-ipv6 IncompleteTests
Tests for this run (104) are below the historical average (2490): IncompleteTests (not enough tests ran to make a reasonable risk analysis; this could be due to infra, installation, or upgrade problems)

@smg247
Copy link
Copy Markdown
Member

smg247 commented Aug 26, 2025

/override ci/prow/e2e-metal-ipi-ovn-ipv6

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 26, 2025

@smg247: Overrode contexts on behalf of smg247: ci/prow/e2e-metal-ipi-ovn-ipv6

Details

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-ipv6

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.

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/acknowledge-critical-fixes-only

While this isn’t a critical fix from the user’s perspective, keeping the same tests in both OTE and ocp/origin makes debugging and troubleshooting harder in the long run. Now that everything has been migrated, we should clean up to ensure consistency for 4.20. All tests are already in OTE.

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/acknowledge-critical-fixes-only

1 similar comment
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/acknowledge-critical-fixes-only

@camilamacedo86
Copy link
Copy Markdown
Contributor Author

/label acknowledge-critical-fixes-only

@openshift-ci openshift-ci Bot added the acknowledge-critical-fixes-only Indicates if the issuer of the label is OK with the policy. label Aug 26, 2025
@openshift-merge-bot openshift-merge-bot Bot merged commit e7b4f7e into openshift:main Aug 26, 2025
40 of 47 checks passed
@camilamacedo86 camilamacedo86 deleted the remove-olmv1-preflight branch August 26, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

acknowledge-critical-fixes-only Indicates if the issuer of the label is OK with the policy. approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants