Skip to content
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

vmware_host_passthrough: Fix to be able to specify PCI id in the devices parameter #1365

Merged

Conversation

sky-joker
Copy link
Collaborator

@sky-joker sky-joker commented Jun 23, 2022

Depends-On: ansible/ansible-zuul-jobs#1579

SUMMARY

If ESXi has multiple PCI devices and they are the same name, and you want to enable the passthrough, you need to specify the PCI id.
But the module hasn't been able to specify the PCI id of the PCI device.
This PR is to be able to specify the id in the devices parameter the module has.

fixes: #1364

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/vmware_host_passthrough.py

ADDITIONAL INFORMATION

tested in vCenter/ESXi 7.0.0

@sky-joker sky-joker changed the title [WIP] vmware_host_passthrough: Fix the devices parameter also to specify PCI id vmware_host_passthrough: Fix the devices parameter also to specify PCI id Jun 23, 2022
@sky-joker sky-joker changed the title vmware_host_passthrough: Fix the devices parameter also to specify PCI id vmware_host_passthrough: Fix to be able to specify PCI id in the devices parameter Jun 23, 2022
@sky-joker
Copy link
Collaborator Author

recheck

@sky-joker
Copy link
Collaborator Author

recheck

@sky-joker
Copy link
Collaborator Author

recheck

@goneri
Copy link
Member

goneri commented Jun 24, 2022

Hi @sky-joker, I think the last failure of the vcenter+2esxi integration test was cause by a recent change. This PR reverts it ansible/ansible-zuul-jobs#1579

@softwarefactory-project-zuul
Copy link

@sky-joker
Copy link
Collaborator Author

recheck

@softwarefactory-project-zuul
Copy link

Copy link
Collaborator

@mariolenz mariolenz left a comment

Choose a reason for hiding this comment

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

I can't test this because we don't have any passthrough devices, but the code LGTM.

@softwarefactory-project-zuul
Copy link

Build succeeded (gate pipeline).

✔️ ansible-tox-linters SUCCESS in 4m 37s
✔️ build-ansible-collection SUCCESS in 3m 41s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 20s
✔️ ansible-test-units-community-vmware-python38 SUCCESS in 5m 43s
✔️ ansible-galaxy-importer SUCCESS in 3m 55s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 1af3a72 into ansible-collections:main Jul 7, 2022
@sky-joker
Copy link
Collaborator Author

Thanks @mariolenz

@fs30000
Copy link

fs30000 commented Nov 20, 2024

So, after enabling passthrought for a device, how can it be mapped to a guest with ansible? On my particular case, a NIC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vmware_host_passthrough by id
4 participants