IBM-Security / isam-ansible-collection

Ansible Collection for providing ISAM Modules, Roles and Playbooks.
Apache License 2.0
12 stars 23 forks source link

Remove inventory_dir in all roles #175

Closed tombosmansibm closed 11 months ago

tombosmansibm commented 1 year ago

Files like certificates and html files and mapping rules should NEVER be stored in the inventory directory (or even suggested to put it there). There's a bit of potential ambiguity - inventory_dir is not a full path. The default would then be a directory in the current dir , with the name of the inventory. This could be your actual inventory directory (and that is really wrong).

Unfortunately, a number of roles in this collection do just that.

The current workaround is to override the inventory_dir variable.

I will remove the dependency on inventory_dir in the near future, because it's just a wrong default value.