Make the functions only get generated once instead of every pillar
Keep the functions if they are passed in
Merge pull request #9595 from jacksontj/develop

Keep the functions if they are passed in
Break out the function loading into a seperate if block
Merge pull request #9596 from jacksontj/develop

Break out the function loading into a seperate if block
Attempt at fixing some of the pillar platform tests
Merge pull request #9597 from UtahDave/develop

slight doc reformat on windows install doc page
Do! if, not elif
the jobid section CLEARLY does not work, stubbing out for now

I also made the first arguments name and made backdoors for limit
and job arguments
Make sure we load ufnctions in ALL situations
Merge pull request #9598 from jacksontj/develop

Fix for pillar loading and comment update
Fix ownership/perms on salt.utils.copyfile()

When salt.utils.copyfile() uses shutil.move() to move the target file to
the destination, the ownership/permissions from the original file that
existed at the destination path are not preserved. For security reasons,
files cached locally and compared to existing files in file.managed
states are created with ownership of root:root (or user:primarygroup, if
the master is running as non-root), with a mode of 600.

Normally, this would not be an issue since
salt.modules.file.check_perms() is invoked to correct the ownership and
permissions. However, recent modifications to that function have
resulted in the ownership/permissions not being modified if these
parameters are passed to check_perms() as None.

Since the user, group, and mode default to None in file.managed states,
if they are not explicitly defined in a state, this results in is the
newly-created file overwriting the old file, and the
ownership/permissions being left untouched.

This commit modifies salt.utils.copyfile() to restore the old file's
ownership and permissions after shutil.move is invoked to replace the
old file with the new one.

Fixes #9527.
Some "DRY" cleanup in the file module

The get_{uid,gid,user,group,mode} functions duplicate a lot of work done
in file.stats. this commit changes file.stats slightly to support
getting uid/gid of broken symlinks (code that was in get_{uid,gid}), and
then uses file.stats to supply the uid, gid, etc. for the get_*
functions.

Additionally, the "follow_symlinks" behavior in file.stats is opposite
from everywhere else in the file module, so the default value for this
parameter has been changed to True, and refs to this function from the
file state has been manually passed as False to preserve backwards
compatibility.
Fix the following crash in a minion when master executes
state.highstate on it.

    2014-01-07 01:45:43,315 [salt.minion      ][INFO    ] User sudo_ubuntu Executing command state.highstate with jid 20140107014542797484
    2014-01-07 01:45:43,332 [salt.minion      ][WARNING ] The minion function caused an exception: Traceback (most recent call last):
      File "/usr/local/lib/python2.7/dist-packages/salt/minion.py", line 626, in _thread_return
        '''
      File "/usr/local/lib/python2.7/dist-packages/salt/modules/state.py", line 237, in highstate
        conflict = running()
      File "/usr/local/lib/python2.7/dist-packages/salt/modules/state.py", line 91, in running
        active = __salt__['saltutil.is_running']('state.*')
      File "/usr/local/lib/python2.7/dist-packages/salt/modules/saltutil.py", line 365, in is_running
        '''
      File "/usr/local/lib/python2.7/dist-packages/salt/modules/saltutil.py", line 396, in running
        continue
    AttributeError: 'module' object has no attribute 'os_is_running'

    2014-01-07 01:45:43,333 [salt.minion      ][INFO    ] Returning information for job: 20140107014542797484
Clarify that execution modules and state modules are not the same thing
Merge pull request #9602 from getlantern/develop

Fix the following crash in a minion when master executes
Merge pull request #9603 from techhat/jcl

Clarify that execution modules and state modules are not the same thing
Merge pull request #9599 from terminalmage/issue9527

Fix ownership/perms on salt.utils.copyfile()
Merge pull request #9600 from terminalmage/file-module-cleanup

Some "DRY" cleanup in the file module
fix for windows masterless read-only highstate.cache.p file