-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[proxmox_vm_info] Re-use cluster resources API to use module without requiring node param #6976
[proxmox_vm_info] Re-use cluster resources API to use module without requiring node param #6976
Conversation
@felixfontein let me know please what type of change is it and should we mention somewhere that |
@UnderGreen I would say this is a new feature, so it's a |
3dbf7bd
to
8533e31
Compare
I'll merge in a couple of days if nobody objects. |
Backport to stable-7: 💚 backport PR created✅ Backport PR branch: Backported as #6993 🤖 @patchback |
@UnderGreen thanks for this one as well! |
…er resources API to use module without requiring node param (#6993) [proxmox_vm_info] Re-use cluster resources API to use module without requiring node param (#6976) * [proxmox_vm_info] Re-use cluster resources API to use module without requiring node param * More consife if * Fix use case when requesting all vms from specific node * Add changelog fragment (cherry picked from commit d7c1a81) Co-authored-by: Sergei Antipov <greendayonfire@gmail.com>
…oxmox_vm_info] Re-use cluster resources API to use module without requiring node param (ansible-collections#6993)" This reverts commit fb3768a.
* Revert "[PR #7020/b46d5d81 backport][stable-7] redfish_utils: Add support for "nextLink" property tag pagination (#7026)" This reverts commit 1dad953. * Revert "[PR #6914/17b4219b backport][stable-7] proxmox_kvm: enable 'force' restart of vm (as documented) (#6997)" This reverts commit 7d68af5. * Revert "[PR #6976/d7c1a814 backport][stable-7] [proxmox_vm_info] Re-use cluster resources API to use module without requiring node param (#6993)" This reverts commit fb3768a.
SUMMARY
As requested in comment module can re-use cluster/resources API to get first information about VMs. Then that info is enriched from lxc/qemu API responses.
In that case we can retrieve VM(s) info without requiring node parameter.
ISSUE TYPE
COMPONENT NAME
proxmox_vm_info
ADDITIONAL INFORMATION