stackhpc / ansible-role-openhpc

Ansible role for OpenHPC
Apache License 2.0
46 stars 15 forks source link

Fix typo in README and note for clarity. #30

Closed sjpb closed 4 years ago

sjpb commented 4 years ago

29 unfortunately introduced a typo when trying to clarify the required inventory group names, saying it was {cluster_name}-{group_name} when actually it's {cluster_name}_{group_name}.

This PR fixes that, adds some {} to make it clearer what is variable and what's not, and adds a note specifically pointing out the difference because it's easy to miss.

Relevant lines in slurm.conf: inventory group @115, instance name at @108 and @129.