Feature #12256
closed
ansible should fallback to sensible defaults
Added by Loïc Dachary almost 9 years ago.
Updated almost 9 years ago.
Description
It would be convenient if the defaults for https://github.com/ceph/ceph-cm-ansible were as described at http://dachary.org/?p=3752. It would allow someone to not do anything when switching from teuthology / chef to teuthology / ansible. These default only help for the testnodes but that's the only thing a custom lab / teuthology installation relies on.
The https://github.com/ceph/ceph-cm-ansible/pull/67 pull request goes in that direction, if I'm not mistaken.
If this feature request is rejected for some reason, there needs to be a clear warning / announcement for people with an existing teuthology installation as it will break things and require them to go thru the steps described at http://dachary.org/?p=3752.
I'm marking that urgent because I think a backward incompatible change that can't easily be resolved should be advertised.
I just merged https://github.com/ceph/ceph-cm-ansible/pull/67 this morning.
The only issue I see with having Teuthology provide defaults like this is that it'd need to use --extra-vars to do so and that has the highest precedence in regards to variables. Meaning, if a user has provided their own inventory with these values set specific to their own lab these defaults would always override them. We can do work on the cm-ansible side though.
I think a good approach here would be to make the testnode role work or otherwise fail gracefully when no inventory with group_vars is defined or provided in a separate secrets repo. I'd like to at least be able to make the vars you need to pass to make this work without an inventory or secrets repo to be very minimal. The only var you might need that I see in that post you linked is 'lab_name'. The rest could most like have sensible defaults like that within the testnodes role.
- Status changed from 12 to Resolved
Also available in: Atom
PDF