Skip to content

Bm sno idrac10 and extra agent config support#4008

Draft
bogdando wants to merge 5 commits into
openstack-k8s-operators:mainfrom
bogdando:bm_sno_idrac10
Draft

Bm sno idrac10 and extra agent config support#4008
bogdando wants to merge 5 commits into
openstack-k8s-operators:mainfrom
bogdando:bm_sno_idrac10

Conversation

@bogdando

Copy link
Copy Markdown
Contributor
  • BM SNO: partial support iDRAC10

    • Use the standard bm_discover_vmedia_target path for iDRAC9.
    • Add a specific bm_discover_vmedia_target path for iDRAC10.
  • BM SNO: add more config options for agent

    • Support tagged interface config for agent
    • Support additional NTP servers

The InsertMedia and EjectMedia action URLs were hardcoded as
VirtualMedia/CD, which is only valid on older iDRAC firmware.

Newer firmware versions expose the virtual optical drive under
a different member name (e.g. RemovableDisk, 1, 2), causing a
404 ResourceNotFound error on the InsertMedia call.

Add bm_discover_vmedia_member.yml that GETs the VirtualMedia
collection, selects the first member whose MediaTypes includes
CD or DVD (fallback: first member with an InsertMedia action),
and exposes _vmedia_member_uri, _vmedia_insert_action, and
_vmedia_eject_action for use by other tasks.

Update main.yml to run discovery before the eject+insert
sequence and use the discovered action URLs.

Update bm_eject_vmedia.yml to trigger lazy discovery when
the action variable is not already set.

Delete stale iDRAC sessions as well.

Fix bm_discover_vmedia_target.yml to verify the inserted ISO
against the discovered member URI rather than the hardcoded
VirtualMedia/CD path.

Add cifmw_bm_agent_iso_server_ip to override the IP the iDRAC uses
to fetch the agent ISO. Auto-detection picks the default-route
address, which is unreachable by iDRAC when the controller runs
over VPN.

Generated-By: cursor-agent (claude-4.6-sonnet-medium)
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@openshift-ci

openshift-ci Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

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

@openshift-ci

openshift-ci Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign dasm for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

bogdando added 2 commits June 19, 2026 10:21
When the iDRAC cannot fetch the agent ISO over HTTP (network
routing, firewall), the only viable path is to upload the ISO
through the iDRAC web UI as a local file.

The role previously had no way to skip ISO generation and
VirtualMedia insertion on a second run

Introduce cifmw_bm_agent_reuse_vmedia (default: false). When
true, the role skips: stale-state cleanup, ISO generation, live-
debug ISO patching, podman HTTP server start/stop, VirtualMedia
eject before insert, ISO insertion, and VirtualMedia eject after
install. All remaining steps run unchanged: USB boot check, power-
off, SSH key and config generation, openshift-install acquisition,
LVMS MachineConfig generation, UEFI boot-override discovery,
power-on, and install wait.

The intended two-run workflow (which rather works the best for
wrappers using the bm_sno role instead of CI jobs):
- Run 1 (reuse_vmedia=false) generates the ISO and auth artifacts;
- the operator uploads the ISO via the iDRAC UI and confirms it
  is Connected;
- run 2 (reuse_vmedia=true) powers the host off, sets the one-time
  UEFI boot override, powers it back on, and waits for install-complete

Assisted-By: claude-4.6-sonnet-medium
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
Use the standard bm_discover_vmedia_target path for iDRAC9.
Add a specific bm_discover_vmedia_target path for iDRAC10.

Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/dff8e249198b47bbb0852b49244967c5

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 38m 01s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 34m 33s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 42m 21s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 2h 24m 07s
✔️ cifmw-pod-zuul-files SUCCESS in 5m 38s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 55s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 46s
cifmw-molecule-bm_sno FAILURE in 4m 23s

bogdando added 2 commits June 19, 2026 17:15
Support tagged interface config for agent
Support additional NTP servers

Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/05db6fb9d60a437a84b0c5cd6609e8a9

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 21m 29s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 27m 59s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 09s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 08m 07s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 56s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 56s
✔️ cifmw-pod-pre-commit SUCCESS in 8m 05s
cifmw-molecule-bm_sno FAILURE in 3m 54s

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.

1 participant