ansible-collection-icinga
ansible-collection-icinga copied to clipboard
Misleading variable names in Icinga 2 API feature
As already discussed 'in real life' there are some variables with misleading names.
The variable ca_host inside the API feature for example is incorrect from an Icinga 2 technical point of view, as the "parent host" must be specified here in a distributed system with satellites. Naming to parent_host or similar would be more suitable.
Other examples are ca_host_port and icinga2_delegate_to.
As @mkayontour said, we need to make sure to have a transition phase for that. We could deprecate them and remove them in future versions.
The way this feature works, ca_host is the correct name to go with, in my opinion, as the collection will indeed default to delegating the certificate creation to the ca_host.