Originally, we used a CSV file as a way to abstract the administrators from Ansible inventory file, especially since at the time (ansible 2.2) only INI files were supported. (I hate INI files).
Now, absent the original usage case and with the vastly superior YAML host syntax, it is time to deprecate the CSV. The ansible.cfg will now have two files: odie-hosts.yml and "inventory" (which is the OCP INI file that is generated from the ocp_inventory_generator role).
Doing a quick ag, this change breaks roles/rsyslog_configuration/files/create-rsyslog-subdirs.sh as it directly parses hosts.csv (tsk tsk). Also numberous references to the docs which grow further and further out of date.
mike4263
commented
5 years ago
Originally, we used a CSV file as a way to abstract the administrators from Ansible inventory file, especially since at the time (ansible 2.2) only INI files were supported. (I hate INI files).
Now, absent the original usage case and with the vastly superior YAML host syntax, it is time to deprecate the CSV. The ansible.cfg will now have two files: odie-hosts.yml and "inventory" (which is the OCP INI file that is generated from the ocp_inventory_generator role).
Doing a quick
ag, this change breaksroles/rsyslog_configuration/files/create-rsyslog-subdirs.shas it directly parses hosts.csv (tsk tsk). Also numberous references to the docs which grow further and further out of date.