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

fix type issue with VirtualPCIPassthrough #1318

Conversation

ckhordiasma
Copy link
Contributor

@ckhordiasma ckhordiasma commented May 9, 2022

Depends-On: ansible/ansible-zuul-jobs#1580
Depends-On: ansible/ansible-zuul-jobs#1582

SUMMARY

Received an error when trying to set up SRIOV:

AttributeError: type object 'vim.vm.device.VirtualPCIPassthrough' has no attribute 'DeviceBacking'

According to the docs as well, there is no attribute on VirtualPCIPassthrough named DeviceBacking https://vdc-repo.vmware.com/vmwb-repository/dcr-public/3325c370-b58c-4799-99ff-58ae3baac1bd/45789cc5-aba1-48bc-a320-5e35142b50af/doc/vim.vm.device.VirtualDevice.html#backing

I believe the intent was to compare the nic backing to the VirtualPCIPassthrough device itself. After making this change, the module seems to work correctly for me

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/modules/vmware_guest_network.py

ADDITIONAL INFORMATION

Received an error when trying to set up SRIOV: 

`AttributeError: type object 'vim.vm.device.VirtualPCIPassthrough' has no attribute 'DeviceBacking'`

According to the docs as well, there is no attribute on VirtualPCIPassthrough named DeviceBacking https://vdc-repo.vmware.com/vmwb-repository/dcr-public/3325c370-b58c-4799-99ff-58ae3baac1bd/45789cc5-aba1-48bc-a320-5e35142b50af/doc/vim.vm.device.VirtualDevice.html#backing 

I believe the intent was to compare the nic backing to the VirtualPCIPassthrough device itself. After making this change, the module seems to work correctly for me
@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.

Thanks for this PR @ckhordiasma!

Could you please add a changelog fragment like this? Something like:

bugfixes:
  - vmware_guest_network - Fix an bug when configuring SR-IOV NICs (https://github.com/ansible-collections/community.vmware/pull/1318).

@mariolenz
Copy link
Collaborator

recheck

@mariolenz
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator

recheck

@sky-joker
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator

recheck

@mariolenz
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz mariolenz closed this Jun 25, 2022
@mariolenz mariolenz reopened this Jun 25, 2022
@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator

recheck

@mariolenz
Copy link
Collaborator

recheck

@mariolenz
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@sky-joker
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator

recheck

@softwarefactory-project-zuul
Copy link

@mariolenz
Copy link
Collaborator

recheck

@goneri goneri reopened this Jun 27, 2022
@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

@softwarefactory-project-zuul
Copy link

@goneri
Copy link
Member

goneri commented Jun 28, 2022

recheck

@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

@goneri goneri closed this Jun 28, 2022
@goneri goneri reopened this Jun 28, 2022
@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

@goneri
Copy link
Member

goneri commented Jun 28, 2022

recheck

@softwarefactory-project-zuul
Copy link

@goneri
Copy link
Member

goneri commented Jun 28, 2022

recheck

@goneri goneri closed this Jun 28, 2022
@goneri goneri reopened this Jun 28, 2022
@goneri
Copy link
Member

goneri commented Jun 28, 2022

recheck

@softwarefactory-project-zuul
Copy link

This change depends on a change that failed to merge.

@softwarefactory-project-zuul
Copy link

@goneri
Copy link
Member

goneri commented Jun 28, 2022

Greeeeeeeeeeeeeeeeen 🥳 💚

@goneri
Copy link
Member

goneri commented Jun 28, 2022

Thank you for your patience :-).

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.

LGTM

I'll add a changelog fragment myself later.

@mariolenz
Copy link
Collaborator

Thank you for your patience :-).

No problem, you're welcome. Thanks for fixing the CI 👍

@softwarefactory-project-zuul
Copy link

Build succeeded (gate pipeline).

✔️ ansible-tox-linters SUCCESS in 4m 08s
✔️ build-ansible-collection SUCCESS in 3m 27s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 09s
✔️ ansible-test-units-community-vmware-python38 SUCCESS in 5m 18s
✔️ ansible-galaxy-importer SUCCESS in 4m 16s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit cfbf63f into ansible-collections:main Jun 28, 2022
mariolenz added a commit to mariolenz/community.vmware that referenced this pull request Jun 28, 2022
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 28, 2022
Changelog fragment for PR #1318

SUMMARY
Add a changelog fragment for PR #1318.
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
vmware_guest_network
ADDITIONAL INFORMATION
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.

4 participants