Skip to content

[libvirt_manager] Allow for virtualized pci passthrough#3755

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
jamepark4:enable_iommu_dev
Mar 17, 2026
Merged

[libvirt_manager] Allow for virtualized pci passthrough#3755
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
jamepark4:enable_iommu_dev

Conversation

@jamepark4
Copy link
Contributor

@jamepark4 jamepark4 commented Mar 10, 2026

To allow for a virtualized compute to passthrough its devices to the emulated guests the compute needs to have iommu device present. If pci_passthrough_enabled is present and true add device element to VM XML.

Signed-off-by: James Parker jparker@redhat.com

@softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/a352c6f36d924b17b1a9153944be578b

openstack-k8s-operators-content-provider FAILURE in 7m 44s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 5m 18s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 15s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 04s
cifmw-molecule-libvirt_manager FAILURE in 10m 53s
cifmw-molecule-reproducer FAILURE in 10m 21s

danpawlik
danpawlik previously approved these changes Mar 12, 2026
@danpawlik danpawlik requested a review from a team March 13, 2026 09:16
amartyasinha
amartyasinha previously approved these changes Mar 13, 2026
Copy link
Contributor

@amartyasinha amartyasinha left a comment

Choose a reason for hiding this comment

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

/lgtm

To allow for a virtualized compute to passthrough its devices to the
emulated guests the compute needs to have iommu device present. If
pci_passthrough_enabled is present and true add device element to VM
XML.

Signed-off-by: James Parker jparker@redhat.com
@softwarefactory-project-zuul
Copy link

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/044df2fc89ca4e8283511a6c4fadf2c6

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 11m 12s
podified-multinode-edpm-deployment-crc FAILURE in 21m 39s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 21m 35s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 58m 17s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 37s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 22m 03s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 59s
✔️ cifmw-molecule-libvirt_manager SUCCESS in 40m 24s
✔️ cifmw-molecule-reproducer SUCCESS in 14m 31s

@jamepark4
Copy link
Contributor Author

recheck

Copy link
Contributor

@danpawlik danpawlik left a comment

Choose a reason for hiding this comment

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

thank you

@danpawlik danpawlik requested review from a team and amartyasinha March 17, 2026 07:17
Copy link
Contributor

@amartyasinha amartyasinha left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Mar 17, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 17, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amartyasinha

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-merge-bot openshift-merge-bot bot merged commit cee62cd into openstack-k8s-operators:main Mar 17, 2026
12 checks passed
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.

3 participants