introducing the foreman ext_pillar module
Cleanup for #13160

This commit makes the changes from #13160 conform to the Salt way by
kreturning booleans for enable/disable. In addition, it makes the
following changes:

1. Adds versionadded directives

2. Fixes leaked file handles by using with statements (and
   salt.utils.fopen).

3. Raises a CommandExecutionError in ares where the failure of a
   function reaches an error case.
correct erroneous os.kill() statement
add missing reference to error message
Allow arbitrary absolute path. Error on relative path without user argument.
Return error on relative path without user.
Fixed moved uinfo.
Merge pull request #7 from mfournier/os.kill-fix

fix mistake mentioned in #13162
Lint
Merge branch 'puppet-housekeeping' of https://github.com/terminalmage/salt into terminalmage-puppet-housekeeping

Conflicts:
	salt/modules/puppet.py
Merge branch 'terminalmage-puppet-housekeeping' into develop
Merge pull request #13157 from bechtoldt/foreman_pillar

implementing a foreman ext_pillar module
move conditionsl logic and remove parens per lint #13157
Merge pull request #13165 from nicksloan/absolute_known_hosts

Allow arbitrary absolute path. Error on relative path without user argument.
don't import stdlib funcs into state modules
Add versionadded for varnish module

Also did some cleanup and made it conform to the way Salt does things
Add varnish module to docs
Remove redundant function