(maint) Adds vcloud pooling to the win*all beaker node definitions

Closes PR #2100
Merge branch 'maint/stable/use_vcloud_pooling_api' into stable

* maint/stable/use_vcloud_pooling_api:
  (maint) Adds vcloud pooling to the win*all beaker node definitions
Maint: Change "inactive" to "stopped" in reinventory action's description

"Inactive" is ambiguous, and could describe a stopped master, a master which
is running but happens to be idle, a master which has no agents attempting to
reach it, a master which has been removed from the load balancer, or any number
of other things. Instead, we should use the unambiguous term "stopped."
Merge remote-tracking branch 'upstream/stable'
Maint: Adjust formatting of "priority" setting description

The \n" + ..." trick isn't really necessary, since we can strip whitespace and
handle first-line underhang.
(maint) Adds vcloud pooling to the remaining windows beaker node definitions
allow configuration of Bundler source
(#18931) File source_permissions setting applies to local files

The new source_permissions setting was original excluded from effecting
files with a local source (where the file is a path on the agent).  This
leaves a problem on Windows, because attempting to copy mode, even from
a local file, still produces insufficient acls.  This patch allows
source_permissions to affect local files, so that mode can be ignored,
and deprecation warnings appear.  We also added a debug message to the
resource to make it possible to pinpoint which File resources are using
this deprecated method.
Merge remote-tracking branch 'jpartlow/issue/18931/source_permissions_apply_to_local'

* jpartlow/issue/18931/source_permissions_apply_to_local:
  (#18931) File source_permissions setting applies to local files
Use the pooling api by default
Merge pull request #2106 from nfagerlund/priority_formatting

Maint: Adjust formatting of "priority" setting description
Merge pull request #2104 from nfagerlund/docs_inactive_stopped

Maint: Change "inactive" to "stopped" in reinventory action's descriptio...
Merge pull request #2109 from justinstoller/maint/master/pooling

Use the pooling api by default
Maint: Copyedit description of source_permissions attribute for clarity
(Maint) Be more explicit about when the SYSTEM ACE is overwritten

Previously, when the specs were not run as SYSTEM, the mode property
would be considered out-of-sync, due to the S_IEXTRA bit being set. So
puppet would rebuild the DACL and remove the inherited SYSTEM ACEs as
expected in the test.

When the specs were run as SYSTEM, the mode was not out-of-sync. This
is because the default DACL for the SYSTEM accounts sets the owner to
Administrators, group to SYSTEM, and an effective mode of 0770. In
other words, there were only two ACEs and they could be mapped into
our POSIX mode.

Commit 5a2d7b89d fixed this by explicitly setting the group to 'None'
thereby forcing the resource to be out-of-sync. However, this means
that the resource was out-of-sync in the SYSTEM case due to the group
property, and out-of-sync in the non-SYSTEM case due to the mode
property.

This commit makes the specs more explicit about when the resource is
insync and not. The former is pending, the latter explicitly sets the
owner and group to 'None', something that will never happen by
default.
(Maint) Refactor expectations when system granted full access explicitly

Refactor code that asserts every ACE for SYSTEM in the DACL grants full
control and is not inherited.
(Maint) Refactor expectations for at least one inherited system ace

Refactor specs that assert the DACL contains at least one inherited ACE
granting SYSTEM full control.
(Maint) Refactor method names in file_spec

 - Existing method names were misleading since the contained 'system'
  but the SYSTEM SID was passed as a parameter
Merge branch 'maint/master/fixup-source-tests'

* maint/master/fixup-source-tests:
  (Maint) Refactor method names in file_spec
  (Maint) Refactor expectations for at least one inherited system ace
  (Maint) Refactor expectations when system granted full access explicitly
  (Maint) Be more explicit about when the SYSTEM ACE is overwritten
(Maint) Eliminate Windows network lookups

When using a UNC style file path, the windows file provider invokes
File.expand_path('//server/...'), which results in a failed Windows
network lookup. Executing these two tests was previously taking 15.4
seconds.

This commit instead provides a `localhost` server to short-circuit the
lookup. The tests now only take 1.7 seconds.