Make things more reusable, correct some errors along the SSH key path

Conflicts:
	library/user

if a module returns failed in the results, don't fail unless failed is n...

Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>

Improved lookup plugins

Pretty-print "msg" when task fails

Return original string for non-matched vars

Several modules (e.g., pip) return stdout and stderr together in a "msg" variable (and the module development docs seem to suggest that the standard practice is to return status in this variable).

This change ensures that it is printed out with correct formatting (most importantly, with newlines appearing as newlines instead of \n).

Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>

Use default port if ansible_ssh_port is not set for the delegated node

Also add testcase for it.

Requires that inject be passed to run

This allows using with_fileglob in play context, e.g.
- include: $item
  with_fileglob: tasks/*.yml
as well as prepares for other data to be passed.

Fix service module for varnish initscript in EPEL

It outputs 'dead but pid file exists'.

Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>

Signed-off-by: Brian Coca <briancoca+ansible@gmail.com>

I had made and pushed this change after you already pulled the request.

@dhozac indicated that it would probably be better to use return codes > 255 for anything related to Ansible itself. Which makes sens :)

Make sure ownership is set correctly for user's new ssh keys

- Fixed some typos for the ansible config file in the man page

Add option create= to lineinfile module

fix up remove and fix fixme checking to see if pkg was removed