Fix typo in create_resources function's description string

This documentation used a hash rocket in a variable assignment instead of the
required equals sign. This commit fixes the typo.
Merge pull request #1058 from nfagerlund/typo/2.7.x/create_resources

Fix typo in create_resources function's description string
Update CHANGELOG, conf/redhat/puppet.spec for 2.7.19
Merge branch '2.7rc' into 2.7.x
(Maint) Fix Puppet::Util.execute warning

Without this patch applied the end user receives a nasty deprecation
warning when running `puppet agent` with a `puppet.conf` that configured
the node terminus to exec as in the following example:

    [main]
    node_terminus = exec
    external_nodes = $confdir/enc.rb

This a problem because we shouldn't issue warnings to the users about
our own use of our own code.

This patch fixes the problem by proxying the execute method from the
object that is self when the find method is called from the child class
using super.
(#15975) Proxy the request_get block to Net::HTTP

The fix in a40639a is insufficient because it silently drops the block
passed to the request_get block.  This is a problem because it causes
file content to be silently dropped and truncated to zero bytes.

This patch fixes the problem by proxying the get request block along to
the underlying implementation in Net::HTTP.
Merge branch 'fix/3.x/environment_acceptance_tests' into 3.x

* fix/3.x/environment_acceptance_tests:
  (#15975) Proxy the request_get block to Net::HTTP
  (Maint) Fix Puppet::Util.execute warning
(#15797) Change the argument to chkconfig from 'on' to 'reset'

Per the man page, 'reset' is more correct because it uses
the runlevels specified in the chkconfig header line, rather
than always using '2345'. This fixes #15797.
(#15797) Add test coverage for redhat service provider

Without this patch there is no test coverage to validate that the
service provider actually calls chkconfig with the reset argument
instead of the on argument.
(Maint) Whitespace only cleanup

This patch is a simple whitespace only minor cleanup of the redhat
service provider.  I noticed some indentation problems while in the file
so I re-indented the whole thing with ggVG= in vim.
(#15797) Disable RedHat services in all run levels

Without this patch applied the `chkconfig <name> off` command is used to
disable services.  This command operates on run levels 2,3,4 and 5 by
default.  This behavior is a problem because the `chkconfig <name>
reset` method may turn the service on in run levels 0,1 or 6.

This patch fixes the problem by explicitly disabling the service in all
run levels.
Merge branch 'ticket/2.7.x/15797_redhat_chkconfig' into 2.7.x

* ticket/2.7.x/15797_redhat_chkconfig:
  (#15797) Disable RedHat services in all run levels
  (Maint) Whitespace only cleanup
  (#15797) Add test coverage for redhat service provider
  (#15797) Change the argument to chkconfig from 'on' to 'reset'
Merge branch '2.7.x' into 3.x

* 2.7.x:
  (#15797) Disable RedHat services in all run levels
  (Maint) Whitespace only cleanup
  (#15797) Add test coverage for redhat service provider
  (#15797) Change the argument to chkconfig from 'on' to 'reset'
  Update CHANGELOG, conf/redhat/puppet.spec for 2.7.19
  Fix typo in create_resources function's description string

Conflicts:
	CHANGELOG
	conf/redhat/puppet.spec
	spec/unit/provider/service/redhat_spec.rb
(Maint) dont fail on path unless absolutely required.

Previously, ensure=>running was required to specify the path. However,
if the zone is already configured, we dont really need the path.
This patch gets rid of that requirement.
- adds spec tests for start.
(Maint) - simplify methods in ZoneMultiConfigProperty (adds wrap).

This patch extracts the arrification into a single method wrap,
and shortens the method definitions by removing redundant code.
(Maint) Type Zone : fix formatting, comments

Add comment explaining why ensure:retrieve is necessary, added the output
of zone info to explain the regex, Fix format space, refactored global
retrieve to make it simple and succinct, simplify autorequire(:zfs),
setstatus, updated the regex in getconfig so that it matches the output
better. Updated the test case for getconfig
(#15886) Refactor state machine

The zone type has an internal statemachine that ensures that the state
transition is linear from configured -> installed -> running.
Previously, the state machine was intertwined with the code of zone
type. This commit extracts the statemachine so that thestatemachine is
in its own class

In order to remove future confusion, we also rename newvalue to seqvalue
and ensure that the newvalue is called from within the seqvalue. The
seqvalue is so named to indicate that the order of declaration of the
seqvalues determine the order of progression of states in the state machine.
Since we call parent newvalue from inside the seqvalue, we can now avoid using
munge which was added for this reason.

- Added test cases for fsm
(Maint) simplify line2hash

Updated line2hash so that the method is succinct and simple, and added test cases for it.
(#15886) remove ZoneConfig and ZoneMultiConfig