Merge pull request #683 from HelenCampbell/rpreppy

4.1.0 Release Prep

4.1.0 Release Prep

Merge pull request #677 from syseleven/fix-4913

https://tickets.puppetlabs.com/browse/MODULES-4913

Prior to this commit, apt_key always sent auth basic header when source
came from http(s). This might fail for some webservers (e.g. Amazon S3)
when empty auth basic is sent.

Now apt_key only sends the auth basic header when userinfo can
be parsed from the URL.

Please backport this to 2.4.x, since this is the release before incompatibility changes are made.

Merge pull request #681 from HelenCampbell/emptystringing

Ensure release allows empty strings

Test to ensure empty string allowed for $release in apt::source

Test added to make sure when release is set to an empty string the
source file is rendered correctly.

Merge pull request #680 from eputnam/remove_dim

this module is not able to find a value for `$keyserver` when it is
included in postgresql, as stated in the ticket. this changes the module
back to using the params.pp model. we can revisit data in modules once
LTS moves up from 4.7.0. for now, we need the compatibility.

(MODULES-4973) rip out data in modules

this module is not able to find a value for `$keyserver` when it is
included in postgresql, as stated in the ticket. this changes the module
back to using the params.pp model. we can revisit data in modules once
LTS moves up from 4.7.0. for now, we need the compatibility.

Merge pull request #678 from DavidS/revert-attribution-removal

Revert removal of Evolving Web's attribution in NOTICE file

Revert removal of Evolving Web's attribution in NOTICE file

This reverts commit part of 26608d1ae2d2a3668a49357cd00dad97ce01d7cf.

(#4913) apt_key now does not always send auth basic

Prior to this commit, apt_key always sent auth basic header when source
came from http(s). This might fail for some webservers (e.g. Amazon S3)
when empty auth basic is sent.

Now apt_key only sends the auth basic header when userinfo can
be parsed from the URL.

Merge pull request #676 from puppetlabs/release

4.0.0 Mergeback

Merge branch 'master' into release

Merge pull request #672 from domcleal/architecture-fact

Fix architecture fact overriding unset `architecture` source option

Merge pull request #674 from hunner/fix_specs

(maint) Fix for different versions

(maint) Fix for different versions

Merge pull request #673 from hunner/secondRelease

Rebase of #668

(maint) bump version to 4.0.0

(maint) Unit test fixes

Removal of deprecated functionality

Removal of compat types and validate_legacy calls

Fix architecture fact overriding unset `architecture` source option

When the `architecture` parameter to apt::source wasn't set, the
`architecture` fact would be used instead, causing the source to always
be limited by the architecture of the host.

The EPP variable has been renamed to avoid clashing with the fact name.

Merge pull request #671 from puppetlabs/release

Release Mergeback

(maint) Clear facter before to avoid parallel pollution

(maint) Stub fact exists call with argument passed

I can't replicate what's going on, but I wonder if the stub is being
bypassed and the actual underlying debian filesystem is being hit. The
timestamps returned by the unit test are from just before the unit test
ran.

(maint) Move stubs for apt update fact

I can't reproduce this, so just a guess.

(maint) Two missed failures due to parallel specs

Merge pull request #670 from hunner/fix_specs

(PUP-6856) Always define facts

(PUP-6856) Always define facts

If facts are not defined for hiera lookups, the lookup will fail instead
of falling back to common on puppet < 4.9.0

Merge pull request #667 from HelenCampbell/firstRelease

First Release prep for Puppet 4 language update

Puppet 4 language update

Merge pull request #669 from puppetlabs/release

Release Mergeback

Merge pull request #666 from HelenCampbell/release

2.4.0 Release Prep

2.4.0 Release Prep

Merge pull request #662 from puppetlabs/modules-4506

[msync] 786266 Implement puppet-module-gems, a45803 Remove metadata.json from locales config

[maint] Fix puppet-lint warnings introduced with 2.2.0 puppet-lint release

[msync] 786266 Implement puppet-module-gems, a45803 Remove metadata.json from locales config

Merge pull request #660 from puppetlabs/modules-4528

[MODULES-4528] Replace Puppet.version.to_f version comparison from spec_helper.rb

[MODULES-4528] Replace Puppet.version.to_f version comparison from spec_helper.rb

Merge pull request #658 from puppetlabs/MODULES-4528

[MODULES-4528] Replace Puppet.version.to_f with Puppet::Util::Package.versioncmp

[MODULES-4528] Replace Puppet.version.to_f with Puppet::Util::Package.versioncmp

Merge pull request #653 from Cosaquee/remove-comma-from-example-in-readme

remove unnecessary comma in example

Merge pull request #656 from WhatsARanjit/master

apt::key is a defined type, not a class

apt::key is a defined type, not a class

Merge pull request #654 from puppetlabs/hunner_msync

(MODULES-4098) Sync the rest of the files

(MODULES-4098) Sync the rest of the files

remove unnecessary comma in example

Merge pull request #650 from puppetlabs/hunner_msync

(MODULES-4097) Sync travis.yml

Merge pull request #652 from puppetlabs/implement-beaker-module_install_helper

[MODULES-4224] Implement beaker-module_install_helper

(MODULES-4097) Sync travis.yml

[MODULES-4224] Implement beaker-module_install_helper

Merge pull request #646 from puppetlabs/modsync

(FM-5972) gettext and spec.opts

gettext and spec.opts

Merge pull request #641 from DavidS/use-stdlib-deprecation

Use stdlib deprecation

Fix for tests against PR-641

Merge pull request #644 from eputnam/spec_opts

(maint) updates spec.opts in prep for parallel_spec

removes spec.opts

Merge pull request #643 from puppetlabs/hunner_msync

(MODULES-3631) msync Gemfile for 1.9 frozen strings

(MODULES-3631) msync Gemfile for 1.9 frozen strings

Merge pull request #638 from lrrb/master

remove port number from hostname

Use the stdlib deprecation() function

Update to require using stdlib 4.13 for deprecation function

remove port number from hostname

Solve the issue reported in https://tickets.puppetlabs.com/browse/MODULES-4104 :  remove the port number from repository location in order to get the host name of the repository.

Merge pull request #636 from puppetlabs/hunner_msync

(MODULES-3704) Update gemfile template to be identical

(MODULES-3704) Update gemfile template to be identical

Merge pull request #635 from lkoranda/fix_string_comparison_error

Avoid string comparison error

Avoiding string comparison error

Signed-off-by: Lukas Koranda <lkoranda@redhat.com>

mocha version update for test hanging issue

Merge pull request #633 from puppetlabs/Paula_modulesync

(MODULES-3983) Update parallel_tests for ruby 2.0.0

Merge pull request #631 from wilson208/key-server-timeout-fix

[MODULES-3562] Implement retry for tests which require modules to pull key from keyserver

(MODULES-3983) Update parallel_tests for ruby 2.0.0

Implement retry on tests which pull key from a key server which sometimes times out (transient error)

Fix tests to check for idempotency correctly.

Merge pull request #629 from puppetlabs/modulesync

Update modulesync_config [51f469d]

Merge pull request #628 from cpavanrun/master

Undef default for $notify_update in source.pp results in problem with Puppet 3.7.2

Update modulesync_config [51f469d]

Merge pull request #624 from seffenberg-naspers/fix_apt_key_with_userinfo

apt_key now really works with userinfo for http(s)

On Puppet 3.7.2 the undef default for $notify_update will overwrite the default boolean setting for $notify_update in apt::settings resulting in an error 400: "" is not a boolean

Merge pull request #627 from puppetlabs/release

Release Mergeback

Merge pull request #626 from underscorgan/maint/master/update-gpg-key

(maint) Update the gpg key

(maint) Update the gpg key

Merge pull request #625 from HelenCampbell/rprep

Release prep 2.3.0

Release prep

apt_key now really works with userinfo for http(s)

now http and https sources for apt_key can take a userinfo
as it should be already since #384

```
apt_key { '...':
  source => 'https://username:password@myhost.com/mypath/key.gpg'
  ..
}
```

Merge pull request #616 from ghoneycutt/add_confs

Add ability to specify a hash of apt::conf defines

Add ability to specify a hash of apt::conf defines

Merge pull request #622 from puppetlabs/modulesync

Update modulesync_config [a3fe424]

Update modulesync_config [a3fe424]

Merge pull request #621 from puppetlabs/modulesync

Update modulesync_config [0d59329]

Update modulesync_config [0d59329]

Merge pull request #618 from puppetlabs/update-modulesync

(MAINT) Update for modulesync_config 72d19f184

(MAINT) Update for modulesync_config 72d19f184

Merge pull request #619 from DavidS/fix

Fix syntax error

Fix syntax error

Merge pull request #617 from hunner/strict_error

(MODULES-3414) Limit non-strict parsing to pre-3.5.0 only

(MODULES-3414) Limit non-strict parsing to pre-3.5.0 only

Puppet 3.5.0 introduced strict variables and the module handles strict
variables by using the defined() function. This does not work on prior
versions of puppet so we must gate based on that version.

Puppet 4 series has a new setting `strict` that may be set to enforce
strict variables while `strict_variables` remains unset (see PUP-6358)
which causes this conditional to erroniously use non-strict 3.5-era
parsing and fail.

The new conditional corrects the cases such that strict variable
behavior happens on versions 3.5.0 and later.

Merge pull request #615 from DavidS/modulesync

(MODULES-3581) modulesync [067d08a]

(MODULES-3581) modulesync [067d08a]

Merge pull request #613 from nicobn/master

Fixed "unless" test condition for ppa repository

Replaced `-s` with `-f` in ppa rspec tests

See 38263ac768349bf0eab56194aa70f4a78fb664af. The previous commit makes the tests fail.

After the repository is added, the ${::apt::sources_list_d}/${sources_list_d_filename} file is created as an empty file.
The unless condition of Exec["add-apt-repository-${name}"] calls test -s, which returns 1 if the file is empty. Because
the file is empty, the unless condition is never true and the repository is added on every execution. This commit
replaces the -s test condition with -f, which is true if the file exists or false otherwise.