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

[Feat] Develop ansible module for Any #489

Open
xerxist opened this issue Jul 25, 2024 · 4 comments
Open

[Feat] Develop ansible module for Any #489

xerxist opened this issue Jul 25, 2024 · 4 comments
Assignees
Labels

Comments

@xerxist
Copy link

xerxist commented Jul 25, 2024

Requirements:

Can you please start using Ansible standards instead of making your own flavor.

Like for instance a VM name to be absent instead of looking up the UUID via a another module fist.
It makes things very complicated overcoded.

@bhati-pradeep
Copy link
Collaborator

@xerxist Can you elaborate your request, thanks !

@xerxist
Copy link
Author

xerxist commented Aug 2, 2024

As an example

Instead of this.

  • name: delete VM
    ntnx_vms:
    state: absent
    vm_uuid: "{{output.vm_uuid}}"

To this.

  • name: delete VM
    ntnx_vms:
    state: absent
    vm_name: {{ vmname }}

Otherwise you have get the output of the uiid first which makes things overcoded and complicated.

@bhati-pradeep
Copy link
Collaborator

bhati-pradeep commented Aug 5, 2024

Hi @xerxist, Prism Central supports multiple VM with same names. So thats why we have kept vm uuid as unique identifier for update or delete.
You can refer : #284 where discussions are done on this.

@xerxist
Copy link
Author

xerxist commented Aug 5, 2024

Im not sure what the benefit is.

As I would have to list the vm from the ntnx_vms_info module register the result and if a machine pops up in there twice it would do the same thing.

Only thing is you can do is some debug on that output.

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

No branches or pull requests

5 participants