redhat-cop / aap_configuration_template

Ansible Automation Platform Configuration as Code examples template
MIT License
55 stars 33 forks source link

Clarify Inventory config /w comments #29

Closed cjnovak98 closed 1 year ago

cjnovak98 commented 1 year ago

I was talking with @sean-m-sullivan the other day and while solving some other problems I had in my configuration, he had clarified a few things, which I believe if included as comments in the inventory files, would make it easier for newbies like myself to understand how to properly configure and use the template the team has put together.

This is my first shot at improving it's clarity and understanding. I can edit the remaining inventory files to read similarly, but wanted to start a discussion first. Let me know what you think!

Also obligatory this is my first time committing to a community developed repo.

cjnovak98 commented 1 year ago

Applied the changes you requested, applied them to all of the inventory files, with exception to the inventory_ci.yml as I don't fully understand the purpose of that one. I assume it's purpose is for pipeline/testing.

djdanielsson commented 1 year ago

yes the ci is for my pipeline testing but that is fine to update just in case. I should prob try and hide it honestly