Merge remote-tracking branch 'origin/master' into leap_master

Merge pull request #820 from alexharv074/maint_clarify_docs

(maint) Clarify docs and add new tests

(maint) Clarify docs and add new tests

Based on a Stack Overflow question, it was noted that the documentation
for `file_line` isn't complete and the underlying behaviour somewhat
confusing.

https://stackoverflow.com/questions/46468922/how-to-change-the-contents-of-a-file-by-using-puppet/46473458

In this patch I add additional unit tests and better examples and
complete documentation.

Merge pull request #811 from kofrezo/master

Allow root as valid UNIX path

Merge pull request #818 from tphoney/remove_defunct_test

removing duplicate test absolute_path test

removing duplicate test absolute_path test

Merge pull request #817 from puppetlabs/msync_18sep17_892c4cf

(maint) modulesync 892c4cf

Merge pull request #777 from alexjfisher/fix_test_filenames

Fix filenames of two function spec tests

fix quoting style in validate_legacy example (#816)

I used the example code from this README file but i got downvotes from CI / jenkins / puppet-lint because it doesn't conform to Puppet style guides to use double quotes here. It produces "double quoted string containing no variables" warnings.

I wanted to fix the example locally but it was pointed out to me we should fix it upstream stdlib instead.

Allow root as valid UNIX path

Since the purpose of this validation type is to check if the given parameter is a valid UNIX path it should also cover the root path "slash" since it might be possible that it is specified as a target directory.

(maint) modulesync 892c4cf

Merge pull request #815 from puppetlabs/release

version 4.20.0 mergeback

Updating translations for readmes/README_ja_JP.md

Updating translations for readmes/README_ja_JP.md

Merge pull request #814 from eputnam/release

(maint) re-send push action to transifex

(maint) re-send push action to transifex

I rebased release on to master, so changes in master to the readme were not interpretted as changes on release in the payload sent to transifex. This commit should mark the README as modified in Transifex and hopefully they will be able to see the changes, e.g. to_json, to_yaml docs.

Merge pull request #812 from eputnam/release_prep

(MODULES-5508) release prep for 4.20.0

(MODULES-5508) release prep for 4.20.0

Updating translations for readmes/README_ja_JP.md

Merge pull request #810 from eputnam/pw_hash_fix

(MODULES-5546) add check for pw_hash

Merge pull request #809 from WhatsARanjit/to_json_yaml

Added to_json, to_json_pretty, and to_yaml functions

(MODULES-5546) add check for pw_hash

Older versions of Puppet end up choking on the sensitive data type check in the pw_hash function added in 42d4ea7af9197f77a3062727eb166c719ca6296a. this adds a check for the Sensitive class and then the type of the arg, so that if the class has not been declared, it will just move on.

Added to_json function with tests and README

Merge pull request #808 from puppetlabs/release

release 4.19.0 mergeback

Merge pull request #807 from tphoney/4.19.0_prep

4.19.0 prep

MODULES-5298 4.19.0 release prep

Merge pull request #806 from pmcmaw/rm_ubuntu

(MODULES-5501) - Remove unsupported Ubuntu

(MODULES-5501) - Remove unsupported Ubuntu

Removing older version of Ubuntu that are not supported by the module.

Merge pull request #805 from puppetlabs/release

Release mergeback 4.18.0

Merge branch 'master' into release

Merge pull request #804 from eputnam/release_prep

(MODULES-5436) release prep for 4.18.0

(MODULES-5436) release prep for 4.18.0

This isn't actually another release, we decided to do 4.18.0 instead of 4.17.2

Merge pull request #803 from tphoney/amend_puppet_dependency

MODULES-5440 fix upper bound for puppet

MODULES-5440 fix upper bound for puppet

Merge pull request #802 from eputnam/release_prep

(MODULES-5436) release prep for 4.17.2

(MODULES-5436) release prep for 4.17.2

(maint) adjust puppet version requirement

Merge pull request #801 from eputnam/revert_compat

(maint) revert puppet version requirement

(maint) revert puppet version requirement

puppet version requirement lower bound was moved to 4.7.0 without a major bump. this returns the lower bound to 2.7 but also pushes the upper bound to include puppet 5.

Merge pull request #800 from tphoney/MODULES-5382

MODULES-5382 Add documentation for email functions

MODULES-5382 Add documentation for email functions

Updating translations for locales/ja/puppetlabs-stdlib.po

Merge pull request #753 from frapex/master

Add validate_domain_name function

Merge pull request #798 from eputnam/round_func

(maint) move/rewrite round() as ruby function

Merge pull request #799 from puppetlabs/maint-modsync-915cde70e20

(maint) modulesync 915cde70e20

(maint) modulesync 915cde70e20

(maint) move/rewrite round() as ruby function

Merge pull request #796 from bastelfreak/mac

add type for MAC address

add documentation

Merge pull request #717 from ripclawffb/filelineoption

Add new file_line option append_on_no_match

Merge pull request #797 from reidmv/fact_function

Update README for fact() function

Update README for fact() function

Merge pull request #787 from reidmv/fact_function

(FACT-932) Add new function, fact()

Merge pull request #791 from eputnam/MODULES-4908

(MODULES-4908) adds support for sensitive data type to pw_hash

Merge pull request #748 from npwalker/add_round_function

Add a round function to complement ceiling and floor

Add a round function to complement ceiling and floor

Add new file_line option append_on_no_match

(MODULES-4908) adds support for sensitive data type to pw_hash

Also includes a small fix to integer_spec so tests pass.

add type for MAC address

Merge pull request #794 from tkishel/MODULES-5003-file_line_does_not_change_multiple_lines_when_one_matches

(MODULES-5003) file_line does not change multiple lines when one matches

Merge pull request #793 from tphoney/FM-6239

(FM-6239) rewrite of test following std patterns

FM-6239 clean up for puppet 5.tests/deprecation

(MODULES-5003) file_line_does_not_change_multiple_lines_when_one_matches

The exists? method is called to determine the need to call the create and
destroy methods. When the ensure, match, and multiple attributes are defined,
the exists? method needs to return false unless all the lines that match the
match attribute equal the line attribute.

The first commit is minimal, and implements this change.
The second commit normalizes the code, which I needed for comprehension.

(MODULES-5003) file_line_does_not_change_multiple_lines_when_one_matches

The exists? method is called to determine the need to call the create and
destroy methods. When the ensure, match, and multiple attributes are defined,
the exists? method needs to return false unless all the lines that match the
match attribute equal the line attribute.

The first commit is minimal, and implements this change.
The second commit normalizes the code, which I needed for comprehension.

(FM-6239) rewrite of test following std patterns

Merge pull request #790 from puppetlabs/msync_modules5187

(MODULES-5187) mysnc puppet 5 and ruby 2.4

(MODULES-5187) fix spec tests for puppet 5 changes

Fix filenames of two function spec tests

The tests weren't being run.  Total tests increase from 2742 to 2769.

Also fix 'when using a class extending String' test.
It had been failing with...
```
RuntimeError:
       can't modify frozen AlsoString
```

Add validate_domain_name function

(MODULES-5187) mysnc puppet 5 and ruby 2.4

Merge pull request #716 from EmersonPrado/MODULES-4377

(Modules 4377) Causes ensure_packages to accept concurrent declarations with ensure => 'present' and 'installed'

Merge pull request #788 from tphoney/MODULES-5003

(MODULES-5003) file_line fix all broken lines

Merge pull request #756 from hunner/newline_integers

(maint) Stdlib::Compat::Integer accepts numbers with newlines apparently

Merge pull request #786 from reidmv/MODULES-5113

(MODULES-5113) Make line support Sensitive

Merge pull request #789 from tphoney/MODULES-5186

(MODULES-5186) - do not run file_line unit tests on windows.

(MODULES-5186) dont run this test on windows

(MODULES-5186) move test to correct location

(MODULES-5186) - removing whitespace from EOL

Revert "Allow use of fact() on other hashes"

This reverts commit 409a974095a3f5b637e091494b5d14b451c5de78.

After thinking about this, use case, and function naming, I think it's
probably best for now to keep things simple and let `fact()` be
single-purpose for looking up facts with no potentially confusing
extensions. The only use case where the name makes sense is where it's
being used on the `$facts` hash, and I think we'd rather in that
circumstance promote the raw use of `fact()`.

(MODULES-5003) file_line fix all broken lines

Merge pull request #783 from davewongillies/fix_changelog_headers

Fix headers in CHANGELOG.md so that headers render correctly

(FACT-932) Allow use of fact() on other hashes

Because sometimes people want to use an alternative data set, but treat
it like it's a set of facts.

(FACT-932) Add new function, fact()

The fact() function allows dot-notation reference to facts. It is an
alternative to using $facts directly with array-indexing. Array-indexing
is often onerous to use since it doesn't align with how structured facts
are accessed elsewhere in the ecosystem and if any element in a
multi-step path doesn't exist, array indexing can cause a compilation
failure.

Example usage:

    fact('os.family')

(MODULES-5113) Make line support Sensitive

Do this by making it a property. The idea is to get rid of the warning
that appears when a Sensitive value is passed to the line attribute.

Merge pull request #784 from hunner/bump_metadata

(MODULES-5144) Prep for puppet 5

(MODULES-5144) Prep for puppet 5

Fix headers in CHANGELOG.md so that headers render correctly

Merge pull request #782 from puppetlabs/release

4.17.1 Release Mergeback

Merge pull request #781 from HelenCampbell/rpreppo

Release prep for 4.17.1

Release prep for 4.17.1

Merge pull request #780 from thallgren/issue/modules-5095/workaround-pup-7650

(MODULES-5095) Workaround for PUP-7650

(MODULES-5095) Workaround for PUP-7650

This commit adds a simple workaround for the problem described in
PUP-7650. The workaround is harmless and can remain in place regardless
of if the fix for PUP-7650 is in place or not.

Merge pull request #778 from marsmensch/master

MODULES-4821 puppetlabs-stdlib: Update the version compatibility to >= 4.7.0 < 5.0.0

Merge pull request #779 from jbondpdx/master

(FM-6197) formatting fixes for file_line resource

(FM-6197) formatting fixes for file_line resource

MODULES-4821 puppetlabs-stdlib: Update the version compatibility to >= 4.7.0 < 5.0.0

Merge pull request #776 from puppetlabs/release

Merge back 4.17.0

add tests

Fix issue where the following causes obscure catalog compilation errors:

```
file { '/tmp/somefile':
  ensure => 'file',
}

File['/tmp/somefile'] -> Package <| |>

ensure_packages($somearray)
```

If $somearray is undefined or one of the elements contains an empty
string, an error like the following is thrown:

Could not find resource 'Package[]' for relationship from
'File[/tmp/somefile]' on node $::fqdn

(MODULES-4706) release prep 4.17.0