(WIP) OCPBUGS-86181 by isabella-janssen · Pull Request #6107 · openshift/machine-config-operator

isabella-janssen · 2026-05-29T20:21:50Z

Closes: OCPBUGS-86181

- What I did
This primarily updates the MCN property and condition transition tests run against the default master MCP instead of a custom MCP when running on a SNO cluster. Some timeouts were increased for the SNO case and the cleanup ordering of the image rebuild required test was updated to decrease the number of image builds.

- How to verify it
All MCN property and transition tests for the ImageModeStatusReporting feature gate should pass in the SNO test suite and continue passing in the default cluster suites.

- Description for the changelog
OCPBUGS-86181: Update ImageModeStatusReporting MCN property and condition transition tests to be resilient on SNO topology

Summary by CodeRabbit

Tests
- Leader-election checks now poll with extended waits for stability.
- Timeouts increased in several wait helpers (node/MCP/garbage-collection) for SNO and slow clusters.
- Image-mode and machine-config transition tests are SNO-aware, tolerate transient connection errors, use adaptive retry logic, and fall back to the master pool when workers are absent.
- Rebootless update flows and more resilient MCN/node revalidation added.
New Files
- Added a test machine configuration that enables the USB Guard extension.

openshift-merge-bot · 2026-05-29T20:21:52Z

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 · 2026-05-29T20:21:54Z

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

coderabbitai · 2026-05-29T20:22:03Z

Caution

Review failed

An error occurred during the review process. Please try again later.

Walkthrough

This PR makes MachineConfig tests SNO-aware, adds a default-MCP test path when worker MCPs lack nodes, hardens machine-count checks against transient connection failures, adds a master MachineConfig fixture, and changes MOSC leader-election log checks to poll until observed.

Changes

SNO and Image Mode Test Reliability

Layer / File(s)	Summary
MOSC leader-election polling & GC timeout `test/extended-priv/mco_ocb.go`	Poll mOSBuilder.Logs() with Eventually for leader-election messages; extend machine-os-builder GC wait to 5m.
Master MachineConfig fixture `test/extended-priv/testdata/files/machineconfigs/master-extension-mc.yaml`	Add `master-extension-mc` template creating `90-master-extension` MachineConfig (ignition 3.2.0) enabling `usbguard`.
Image-mode test helpers and flows `test/extended/image_mode_status_reporting.go`	Branch image-mode tests between default (master) MCP and custom MCP; add `runImageModeMCNTestDefaultMCP`; refactor custom helper to accept `oc`, apply rebootless NDP for non-image updates, use longer MCP-ready waits, and revalidate MCN with Eventually.
Machine-count transition resilience `test/extended/image_mode_status_reporting.go`	Make validateMCPMachineCountTransitions retry counts configurable for layered/SNO updates; change matcher to return `(countsMatch, isConnErr)` and treat certain connection failures as retryable.
SNO-aware MCN transition validation `test/extended/machineconfignode.go`	Refactor ValidateTransitionThroughConditions to accept `oc`, add transient-connection classification (ECONNREFUSED/EOF/timeouts), detect SNO with `extpriv.IsSNOSafe(oc)`, and relax/skip specific transition checks and timeouts when appropriate.
MCP readiness timeout configurability `test/extended/machineconfigpool.go`	Add `timeout time.Duration` parameter to WaitForMCPToBeReady; make DeleteMCAndWaitForMCPUpdate SNO-aware and pass longer timeouts for SNO.
Node current-config wait increase `test/extended/node.go`	Increase WaitForNodeCurrentConfig timeout from 5m to 8m to allow slower annotation propagation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

openshift/machine-config-operator#6110: Related SNO/topology detection changes used by IsSNOSafe/IsSNO.
openshift/machine-config-operator#6089: Related resilience changes to image-mode MCP machine-count transition checks and connection-error handling.

Suggested labels

jira/valid-bug, jira/valid-reference

Suggested reviewers

yuqi-zhang
dkhater-redhat

🚥 Pre-merge checks | ✅ 4 | ❌ 11

❌ Failed checks (1 warning, 10 inconclusive)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title '(WIP) OCPBUGS-86181' only contains a work-in-progress marker and a Jira ticket reference without describing the actual changes made to the codebase.	Replace with a descriptive title that summarizes the main changes, such as 'Add SNO-aware transition validation and image mode test improvements for machine config updates' or similar.
Stable And Deterministic Test Names	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Test Structure And Quality	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Microshift Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Single Node Openshift (Sno) Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Topology-Aware Scheduling Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Ote Binary Stdout Contract	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Ipv6 And Disconnected Network Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
No-Weak-Crypto	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Container-Privileges	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
No-Sensitive-Data-In-Logs	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)

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

openshift-ci · 2026-05-29T20:22:05Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: isabella-janssen

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:

~~OWNERS~~ [isabella-janssen]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

isabella-janssen · 2026-05-29T20:22:18Z

/payload-aggregate periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview 5

openshift-ci · 2026-05-29T20:22:22Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/178b0a90-5b9c-11f1-8b57-67f8e83fcc97-0

coderabbitai

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@test/extended-priv/mco_ocb.go`:
- Around line 314-317: The callback passed to o.Eventually currently discards
the error from mOSBuilder.Logs(); change it to capture the error (logs, err :=
mOSBuilder.Logs()) and handle it instead of using the blank identifier: if err
!= nil, return a descriptive string containing the error (or otherwise surface
the error to the test/failer) so the Eventually assertion sees failures to fetch
logs; update the callback around mOSBuilder.Logs() used inside o.Eventually(...)
to propagate or report the error rather than ignoring it.

In `@test/extended/image_mode_status_reporting.go`:
- Around line 394-409: In validateMCPMachineCountTransitions, stop ignoring the
error from extpriv.IsSNOSafe(oc): capture the returned error, log or return it
(or set a safe default for loopCount) and decide fallback behavior when
IsSNOSafe fails; also ensure the SNO retry-budget bump (loopCount = 10) is
applied where intended—either move the IsSNOSafe check out of the layered-only
block or duplicate its handling for the non-layered path so SNO clusters get the
higher retry count in both cases if that’s the desired behavior.

In `@test/extended/machineconfignode.go`:
- Around line 302-315: The code currently skips asserting UpdateExecuted=True
unconditionally; restore a failing assertion for non-SNO/non-connection-error
paths: after calling waitForMCNConditionStatus (function
waitForMCNConditionStatus, vars conditionMet and err, checking isSNO and
errors.Is(err, syscall.ECONNREFUSED)), keep the special-case that logs and
tolerates the ECONNREFUSED for SNO, but in the else branch ensure you both
assert no error (o.Expect(err).NotTo(HaveOccurred())) and assert conditionMet is
true (o.Expect(conditionMet).To(o.BeTrue(), fmt.Sprintf("Error, could not detect
%s on node %s: %v", mcfgv1.MachineConfigNodeUpdateExecuted, updatingNodeName,
err))). This guarantees non-SNO and non-conn-error runs fail if UpdateExecuted
was not observed.
- Around line 208-216: The call to extpriv.IsSNOSafe(oc) currently discards its
error (isSNO, _ := extpriv.IsSNOSafe(oc)); change it to capture the error and
handle it explicitly (e.g., isSNO, err := extpriv.IsSNOSafe(oc); if err != nil {
/* fail the test or log and return */ }) so that SNO detection failures do not
silently fall back to non-SNO timing; update the surrounding logic that uses
isSNO, and ensure any failure uses the test framework's failure/logging
mechanism (e.g., t.Fatalf, framework.Failf or klog/Logf as appropriate) instead
of ignoring the error while still setting updatingWaitTime and
updatingWaitInterval.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: 08da7797-b463-4018-8e76-ae6f432da12b

📥 Commits

Reviewing files that changed from the base of the PR and between d72b715 and 5614069.

📒 Files selected for processing (4)

test/extended-priv/mco_ocb.go
test/extended-priv/testdata/files/machineconfigs/master-extension-mc.yaml
test/extended/image_mode_status_reporting.go
test/extended/machineconfignode.go

coderabbitai

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@test/extended/machineconfignode.go`:
- Around line 292-301: The SNO branch current logic swallows non-ECONNREFUSED
errors (variables: isSNO, err, syscall.ECONNREFUSED) and only logs on
ECONNREFUSED or missing condition, causing inconsistent behavior with the
non-SNO path; update the isSNO handling in the function that waits for
AppliedFiles=True so that if err != nil and not errors.Is(err,
syscall.ECONNREFUSED) you call the same assertion used in the non-SNO path
(o.Expect(err).NotTo(o.HaveOccurred(), fmt.Sprintf(...))) so non-ECONNREFUSED
errors fail the test, while preserving the existing special-case log for
ECONNREFUSED and the conditionMet check (logger.Infof and
o.Expect(conditionMet).To(o.BeTrue())).
- Around line 310-319: The SNO branch in the UpdateExecuted check currently
swallows non-ECONNREFUSED errors; update the isSNO handling in the same block
that checks err and conditionMet so that if err is non-nil and not
syscall.ECONNREFUSED you log the error and/or fail the test similarly to the
non-SNO branch (use logger.Errorf or o.Expect to surface the error), and if
conditionMet is false ensure you log or fail (use logger.Infof + o.Expect or
o.Expect(conditionMet).To(o.BeTrue(...))) so behavior matches the non-SNO path;
reference variables/function names: isSNO, err, syscall.ECONNREFUSED,
conditionMet, logger, and o.Expect/BeTrue.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: b795a6e6-b331-41eb-898e-3110a36678bb

📥 Commits

Reviewing files that changed from the base of the PR and between de9e297 and a1d9689.

📒 Files selected for processing (3)

test/extended-priv/mco_ocb.go
test/extended/image_mode_status_reporting.go
test/extended/machineconfignode.go

🚧 Files skipped from review as they are similar to previous changes (2)

test/extended-priv/mco_ocb.go
test/extended/image_mode_status_reporting.go

isabella-janssen · 2026-06-04T17:39:15Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-04T17:39:19Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/4f0ef2c0-603c-11f1-97d4-770ff5b6a596-0

isabella-janssen · 2026-06-04T20:30:53Z

/payload-aggregate periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview 5

openshift-ci · 2026-06-04T20:31:01Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/48ff8260-6054-11f1-931e-e338040069af-0

isabella-janssen · 2026-06-05T15:31:08Z

/payload-aggregate periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview 5

openshift-ci · 2026-06-05T15:31:13Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/93f06240-60f3-11f1-9f6f-88cb747357c8-0

isabella-janssen · 2026-06-05T17:37:06Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-05T17:37:09Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/2c4f6ed0-6105-11f1-820a-cab7dccb558e-0

isabella-janssen · 2026-06-05T17:54:37Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-05T17:54:41Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/9f119130-6107-11f1-9c1a-f7f378e83c7c-0

isabella-janssen · 2026-06-08T19:59:50Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-1of3 periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-2of3 periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-3of3

openshift-ci · 2026-06-08T20:00:02Z

@isabella-janssen: trigger 4 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview
periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-1of3
periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-2of3
periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-gcp-mco-disruptive-techpreview-3of3

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/9bf24ca0-6374-11f1-8445-7c974cd28ef6-0

isabella-janssen · 2026-06-09T02:26:49Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T02:26:52Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/abfd4b50-63aa-11f1-8d8f-0706f6c7906b-0

isabella-janssen · 2026-06-09T02:32:05Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T02:32:08Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/681e7520-63ab-11f1-886c-4a22a2927691-0

isabella-janssen · 2026-06-09T13:02:29Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T13:02:34Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/795ec710-6403-11f1-90e3-6a5ae3549b16-0

isabella-janssen · 2026-06-09T13:16:36Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T13:16:42Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/71bad600-6405-11f1-852f-7d5259abdd84-0

…ition tests resilient on SNO

isabella-janssen · 2026-06-09T13:41:30Z

@coderabbitai resume

coderabbitai · 2026-06-09T13:41:38Z

✅ Action performed

Reviews resumed.

isabella-janssen · 2026-06-09T14:08:32Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T14:08:43Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b38a5a90-640c-11f1-9fd3-a7781afd28e8-0

isabella-janssen · 2026-06-09T15:16:33Z

/payload-job periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

openshift-ci · 2026-06-09T15:16:37Z

@isabella-janssen: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

periodic-ci-openshift-machine-config-operator-release-5.0-periodics-e2e-aws-mco-single-node-disruptive-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/33f1aa90-6416-11f1-8421-b9f8cf1f51ed-0

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 May 29, 2026

openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 29, 2026

coderabbitai Bot reviewed May 29, 2026

View reviewed changes

Comment thread test/extended-priv/mco_ocb.go Outdated

Comment thread test/extended/image_mode_status_reporting.go

Comment thread test/extended/machineconfignode.go Outdated

Comment thread test/extended/machineconfignode.go Outdated

isabella-janssen force-pushed the ocpbugs-86181 branch from de9e297 to a1d9689 Compare June 2, 2026 13:43

coderabbitai Bot reviewed Jun 2, 2026

View reviewed changes

Comment thread test/extended/machineconfignode.go Outdated

Comment thread test/extended/machineconfignode.go Outdated

isabella-janssen force-pushed the ocpbugs-86181 branch from e79ee19 to acdf2e4 Compare June 2, 2026 14:11

openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 2, 2026

isabella-janssen force-pushed the ocpbugs-86181 branch from 31e8864 to b1c6eff Compare June 4, 2026 17:37

openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 4, 2026

isabella-janssen force-pushed the ocpbugs-86181 branch 3 times, most recently from 3eaa05f to bc777ae Compare June 5, 2026 15:17

tests: make ImageModeStatusReporting MCN property and condition trans…

04b9eb4

…ition tests resilient on SNO

isabella-janssen force-pushed the ocpbugs-86181 branch from 5b251b8 to 04b9eb4 Compare June 9, 2026 13:40

Conversation

isabella-janssen commented May 29, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

openshift-merge-bot Bot commented May 29, 2026

Uh oh!

openshift-ci Bot commented May 29, 2026

Uh oh!

coderabbitai Bot commented May 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

❌ Failed checks (1 warning, 10 inconclusive)

Uh oh!

openshift-ci Bot commented May 29, 2026

Uh oh!

isabella-janssen commented May 29, 2026

Uh oh!

openshift-ci Bot commented May 29, 2026

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

isabella-janssen commented Jun 4, 2026

Uh oh!

openshift-ci Bot commented Jun 4, 2026

Uh oh!

isabella-janssen commented Jun 4, 2026

Uh oh!

openshift-ci Bot commented Jun 4, 2026

Uh oh!

isabella-janssen commented Jun 5, 2026

Uh oh!

openshift-ci Bot commented Jun 5, 2026

Uh oh!

isabella-janssen commented Jun 5, 2026

Uh oh!

openshift-ci Bot commented Jun 5, 2026

Uh oh!

isabella-janssen commented Jun 5, 2026

Uh oh!

openshift-ci Bot commented Jun 5, 2026

Uh oh!

isabella-janssen commented Jun 8, 2026

Uh oh!

openshift-ci Bot commented Jun 8, 2026

Uh oh!

isabella-janssen commented Jun 9, 2026

Uh oh!

openshift-ci Bot commented Jun 9, 2026

Uh oh!

isabella-janssen commented Jun 9, 2026

Uh oh!

openshift-ci Bot commented Jun 9, 2026

Uh oh!

isabella-janssen commented Jun 9, 2026

Uh oh!

openshift-ci Bot commented Jun 9, 2026

Uh oh!

isabella-janssen commented Jun 9, 2026

Uh oh!

openshift-ci Bot commented Jun 9, 2026

Uh oh!

isabella-janssen commented Jun 9, 2026

Uh oh!

isabella-janssen commented May 29, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented May 29, 2026 •

edited

Loading