Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hosts.example contains invalid characters in group name #253

Open
sourcejedi opened this issue Jun 11, 2019 · 0 comments
Open

hosts.example contains invalid characters in group name #253

sourcejedi opened this issue Jun 11, 2019 · 0 comments

Comments

@sourcejedi
Copy link

I looked at wordpress-nginx_rhel7/hosts.example:

[wordpress-server]
webserver2

See ansible/ansible-documentation#89 ("TRANSFORM_INVALID_GROUP_CHARS doesn't document valid group patterns").

(I would rather Ansible re-think this new rule! But if they do not, they need to make sure they get all the docs & associated examples updated, to avoid confusing people).

sourcejedi added a commit to sourcejedi/ansible-site that referenced this issue Jun 27, 2019
https://github.com/ansible/ansible/issues/56930#issuecomment-498933773
https://github.com/ansible/ansible/issues/56930#issuecomment-499781095
https://github.com/ansible/ansible/issues/56930#issuecomment-496521613

force_valid_group_names=false does not work: it breaks some of the links
between groups in my inventory.  (I.e. it is not consistent enough in its
conversions).

The ansible change does not seem very useful to me.  It seems aimed at
people who aren't already familiar with this from programming in common
languages.  I'd rather they worked out how to generate warnings when
`groups.my-group` or even `groups.mygroup` is used.

As noted in the last linked comment, Ansible's *EC2 inventory* uses group
names with hyphens in them.

The same is true of ansible-examples:
ansible/ansible-examples#253
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant