Also change test name collision.

Devel - fix issue #315

Conflicts:
	library/user

Prevent facter stderr messages from confusing ansible

Don't let facter spew things to stderr and confuse Ansible, which now blends stdout/stderr due to paramiko
pty usage.

(This commit from lcosmin, manually applied due to github crossup on my part -- MPD)

Fixing append when user already in desired groups

This adds some logic when usings groups possibly in combination with append

  if just specifying groups and the current groups do not match the list
     set groups
  if specifying groups with append and any group thats not in the current groups
     set groups with -a

Code was adding '-a' even when the user was already in the desired groups causing usermod to fail since there was no '-G' option. This is duplicate code (3 lines) from the 'group' section, so could be improved, but it works.

fix up service module to make it start processes that are listed to be

Add tests for new advanced inventory features (groups of groups, group variables) in the default INI format file.

Fix playbooks such that they work with external inventory scripts.  We really want to find out why the 'all'
group isn't available, but group vars don't make sense for external inventory anyway.

Next up: YAML format.

format compatible with this still, and add some tests for INI-style groups of groups
and variables.

TODO: logrotate script and append to log instead.

running and to report errors from starting them

in included files can still be templated just like top level playbooks.  Resolves
some issues about statements being evaluated before facts were available

This eliminates the creation of runner and playbook instances and just
invokes the processes ansible and ansible-playbook.

A first stab at a pull-based model for ansible.  This does two things:
    1. Invoke the git module via Runner to set up a git repository on the
       localhost.  It sets up Runner to use transport='local' and forces
       the inventory to just 'localhost'.
    2. Run any playbooks provided.  By default, this wants to run the
       playbook local.yml.  This also sets transport='local' and sets
       the host_list to a list: localhost, fqdn, and hostname.
The reason for setting the host_list and not using override_hosts is
because there may be plays in the playbook that are not meant for a
specific host.  That is, if the git repository is for the entire site
and not host-specific, you don't want to override hosts and apply all
plays to any given host.  This has the downside of potentially running a
play three times if the play is defined for 'hosts: all'.

Playbook will install ansible, create directory where git checkout goes,
and set up a cron job to run ansible-pull.

Define 'inventory_hostname' variable for hosts.