this one is harder than it should be. _normalize_users ends up fixing the '-' to '_' in all those before 'distro.create_user' gets called. So either actually works. Documentation and implementation in cloud-init have unfortunately been inconsistent with '-' versus '_'.
We should absolutely strive to be consistent both in implementation and in documentation. So... * I'm 98% certain that 'ssh-import-id' in the diff context and 'ssh-authorized-keys' support either - or _. Lets just change everything to document '_' for consistency. Please fix those also. * lets also update the in-module documentation in cloudinit/config/cc_users_groups.py to match (that is what ends up being documented at http://cloudinit.readthedocs.io/en/latest/topics/modules.html#users-and-groups -- https://code.launchpad.net/~d-info-e/cloud-init/+git/cloud-init/+merge/342251 Your team cloud-init commiters is requested to review the proposed merge of ~d-info-e/cloud-init:fix-user-group-doc into cloud-init:master. _______________________________________________ Mailing list: https://launchpad.net/~cloud-init-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~cloud-init-dev More help : https://help.launchpad.net/ListHelp
