-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding sushy tools to the ocp-on-libvirt solution #9
Conversation
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
cac091b
to
068dfa1
Compare
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
ERROR unable to rebase PR 9 from redhatci/ansible-collection-redhatci-ocp branch main |
1 similar comment
ERROR unable to rebase PR 9 from redhatci/ansible-collection-redhatci-ocp branch main |
068dfa1
to
e8210ef
Compare
ERROR unable to rebase PR 9 from redhatci/ansible-collection-redhatci-ocp branch main |
e8210ef
to
c470ee4
Compare
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
PR squashed and rebased as requested
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
Starting dci-check-change job. |
|
recheck |
Build succeeded. ✔️ dci-rpm-build-el8 SUCCESS in 3m 33s |
ee2c5b8
to
cacb0e4
Compare
Build succeeded. ✔️ dci-rpm-build-el8 SUCCESS in 3m 48s |
Starting dci-check-change job. |
|
roles/ocp_on_libvirt/README.md
Outdated
ocp_on_libvirt_cert_locality | No | Westford | see *cert_country* | ||
ocp_on_libvirt_cert_organization | No | DCI | see *cert_country* | ||
ocp_on_libvirt_cert_organizational_unit | No | Lab | see *cert_country* | ||
sushy_ignore_boot_device | No | false | When deploying the sushy-tools redfish emulator, set to "true" to instruct the libvirt driver to ignore any instructions to set the boot device. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we name this one with ocp_on_libvirt_
prefix? The reasoning is because this is the same variable used in setup_sushy_tools
role. See my other comment when using that role
Currently ocp-on-libvirt installs the vBMC service along with the VMs to allow for IPMI remote management of the hosts, which in turn allows for PXE image boot up. In order to support ACM managed deployments, virtual media image boot up is needed, and to manage it remotely, the target server BMCs must suport the Redfish protocol. Sushy tools provides support for the Redfish protocol for libvirt VMs. This change installs sushy tools, as implemented for assisted-installer based deployments, when running the ocp-on-libvirt role.
cacb0e4
to
f618b7a
Compare
Build succeeded. ✔️ dci-rpm-build-el8 SUCCESS in 3m 16s |
Starting dci-check-change job. |
1 similar comment
Starting dci-check-change job. |
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for all the effort and the work put in this!
Starting dci-check-change job. |
|
Currently ocp-on-libvirt installs the conserver service along with the VMs to allow for IPMI remote management of the hosts, which in turn allows for PXE image boot up.
In order to support ZTP deployments, virtual media image boot up is needed, and to manage it remotely, the target server BMCs must suport the redfish protocol.
Sushy tools provides support for the redfish protocol for libvirt VMs.
This change installs sushy tools, as implemented in the assisted-installer ansible project, when running the ocp-on-libvirt role.
Build-Depends: 28766
Build-Depends: https://github.com/dci-labs/bos2-ci-config/pull/93
Test-Args-Hints: -e enable_redfish=true -e enable_virtualmedia=true