Skip to content

[release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR"#150

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.21from
jacob-anders:981089-4.21-backport
Apr 15, 2026
Merged

[release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR"#150
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.21from
jacob-anders:981089-4.21-backport

Conversation

@jacob-anders
Copy link
Copy Markdown

This reverts commit 595ffbc.

Newer firmware on the Supermicro ARS-111GL-NHR has changed the virtual media device string back to 'UsbCd', consistent with other Supermicro servers. The model-specific exception that skipped the CD-to-UsbCd override for this model is no longer correct and causes boot from virtual media to fail on machines running updated firmware.

Revert the workaround to restore the unconditional CD-to-UsbCd boot source override for all Supermicro machines. Users running older ARS-111GL-NHR firmware where 'Cd' was the correct device will need to update their BMC firmware.

Change-Id: I188e7fb3a60868cfa1ec9428dc974528d2db8e7b

Assisted-By: Claude Opus 4.6
(cherry picked from commit 950648a)

This reverts commit 595ffbc.

Newer firmware on the Supermicro ARS-111GL-NHR has changed the
virtual media device string back to 'UsbCd', consistent with other
Supermicro servers. The model-specific exception that skipped the
CD-to-UsbCd override for this model is no longer correct and causes
boot from virtual media to fail on machines running updated firmware.

Revert the workaround to restore the unconditional CD-to-UsbCd boot
source override for all Supermicro machines. Users running older
ARS-111GL-NHR firmware where 'Cd' was the correct device will need
to update their BMC firmware.

Change-Id: I188e7fb3a60868cfa1ec9428dc974528d2db8e7b
Signed-off-by: Jacob Anders <jacob-anders-dev@proton.me>
Assisted-By: Claude Opus 4.6
(cherry picked from commit 950648a)
@openshift-ci openshift-ci Bot requested review from elfosardo and zaneb April 9, 2026 11:50
@jacob-anders jacob-anders changed the title Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" [release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" Apr 9, 2026
@jacob-anders jacob-anders changed the title [release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" OCPBUGS-82298 [release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" Apr 9, 2026
@jacob-anders jacob-anders changed the title OCPBUGS-82298 [release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" [release-4.21] Bug OCPBUGS-82298 Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" Apr 9, 2026
@jacob-anders
Copy link
Copy Markdown
Author

/hold
#152 needs to merge first to preserve commit ordering

@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 Apr 10, 2026
@jacob-anders
Copy link
Copy Markdown
Author

/retest

@MahnoorAsghar
Copy link
Copy Markdown

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 10, 2026
@jacob-anders jacob-anders changed the title [release-4.21] Bug OCPBUGS-82298 Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" [release-4.21] Revert "Skipping UsbCd workaround on Supermicro ARS-111GL-NHR" Apr 15, 2026
@jacob-anders
Copy link
Copy Markdown
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 Apr 15, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 15, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: iurygregory, jacob-anders

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 Apr 15, 2026
@openshift-merge-bot
Copy link
Copy Markdown

/retest-required

Remaining retests: 0 against base HEAD 6f4c2dc and 2 for PR HEAD 9cb1996 in total

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 15, 2026

@jacob-anders: all tests passed!

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-merge-bot openshift-merge-bot Bot merged commit e5fdbeb into openshift:release-4.21 Apr 15, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants