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.

Sign up for GitHub

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

Jump to bottom

Composite variables support for Proxmox nodes in dynamic inventory plugin #6640

Closed
1 task done
levytskyip opened this issue Jun 7, 2023 · 4 comments
Closed
1 task done

Composite variables support for Proxmox nodes in dynamic inventory plugin #6640

levytskyip opened this issue Jun 7, 2023 · 4 comments
Labels
feature This issue/PR relates to a feature request has_pr plugins plugin (any type)

Comments

@levytskyip
Copy link
Contributor

levytskyip commented Jun 7, 2023 •
edited by felixfontein
Loading

Summary

In my environment Proxmox nodes are assigned an IP address in management VLAN and DNS name in a subdomain of my local domain. Currently if I try to configure the ansible_host variable in compose: section it will only work for hosts on the Proxmox nodes but not for the nodes themselves.

Issue Type

Feature Idea

Component Name

plugins/inventory/proxmox.py

Additional Information

plugin: community.general.proxmox
url: https://pve.domain.local:8006
want_facts: false
want_proxmox_nodes_ansible_host: false
compose:
   ansible_host: inventory_hostname + '.domain. Local'

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot

This comment was marked as outdated.

Sign in to view
@ansibullbot
Copy link
Collaborator

cc @Ajpantuso @Thulium-Drake @UnderGreen @joshainglis @karmab @tleguern
click here for bot help

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module plugins plugin (any type) labels Jun 7, 2023
@levytskyip levytskyip mentioned this issue Jun 7, 2023
Merged
@ansibullbot
Copy link
Collaborator

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot
Copy link
Collaborator

cc @ilijamt
click here for bot help

@ansibullbot ansibullbot added has_pr and removed module module labels Jun 8, 2023
patchback bot pushed a commit that referenced this issue Jun 12, 2023
@levytskyip @patchback
Proxmox inventory plugin: support composite variables (#6641)
999c685 
* Added composite var support for proxmox inventory plugin

* Composite variables support for Proxmox nodes in dynamic inventory plugin
Fixes #6640

* Composite variables support for Proxmox nodes in dynamic inventory plugin
Fixes #6640

* Added composite var support for proxmox inventory plugin

* Added composite var support for proxmox inventory plugin

* Update changelogs/fragments/6640-proxmox-composite-variables-support.yml

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 1f6d404)
@patchback patchback bot mentioned this issue Jun 12, 2023
Merged
felixfontein pushed a commit that referenced this issue Jun 12, 2023
@patchback @levytskyip
[PR #6641/1f6d404d backport][stable-7] Proxmox inventory plugin: supp…
3b24363 
…ort composite variables (#6684)

Proxmox inventory plugin: support composite variables (#6641)

* Added composite var support for proxmox inventory plugin

* Composite variables support for Proxmox nodes in dynamic inventory plugin
Fixes #6640

* Composite variables support for Proxmox nodes in dynamic inventory plugin
Fixes #6640

* Added composite var support for proxmox inventory plugin

* Added composite var support for proxmox inventory plugin

* Update changelogs/fragments/6640-proxmox-composite-variables-support.yml

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 1f6d404)

Co-authored-by: Paul Levytskyi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature This issue/PR relates to a feature request has_pr plugins plugin (any type)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@levytskyip @ansibullbot