plugins/inventory/cobbler: Collect IP addresses for hosts and add opt…

[opoplawski]

…ion to collect all DNS name to IP address mappings

SUMMARY

It can be very helpful to have easy access to a host's primary IP address and various DNS name to IP address mappings defined in cobbler.

ISSUE TYPE

• Feature Pull Request

COMPONENT NAME

plugins/inventory/cobbler.py

ADDITIONAL INFORMATION

- debug:
    msg: "My IP address is {{ ip_address }}"
- debug:
    msg: "The IP address of server.example.com is {{ hostvars['server.example.com']['ip_address'] }}"
- debug:
    mesg "The IP address of server-interface.example.com is {{ cobbler_ip_addresses['server-interface.example.com'] }}"

[opoplawski]

One question I have is whether or not it is okay to simply add the ip_address and ipv6_address variables to the host variables. Not sure if there is any standard among inventory plugins.

[felixfontein]

One problem with such generic names is that you can likely interfere with existing uses of these names in existing playbooks, roles, or inventories. Usually it's better to prefix them with something that's specific to this inventory plugin; for example, the community.docker.docker_containers inventory plugin prefixes everything with docker_. (This of course can still cause conflicts, but it is a little bit less likely :) )

[opoplawski]

Yeah, I'm going to think about it for a little bit, but I think that a cobbler_ prefix probably is in order.

[opoplawski]

Though I think cobbler_ip_address might be confused with the IP address of the cobbler server.

[felixfontein]

If nobody objects, I'll merge this by tomorrow morning.

[patchback]

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/24f27a0bdf3a2e72a96072a222845235f1e64e88/pr-6711

Backported as #6727

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[felixfontein]

@opoplawski thanks for your contribution!