(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.

Merge pull request #612 from puppetlabs/modulesync

{maint} modulesync 0794b2c

{maint} modulesync 0794b2c

Merge pull request #611 from mattkenn4545/master

Fix TYPO

Fix TYPO

Ubuntu not Ubunto

Merge pull request #610 from puppetlabs/unused-lsb

apt/params: Remove unused LSB facts

Merge pull request #609 from hfm/fix-regexp

Fix regexp for $ensure params

apt/params: Remove unused LSB facts

Fix regexp for $ensure params

/\Aabsent|present\Z/ match wrong values like 'absentaaa' or 'aaapresent'.

And add tests to the context of 'invalid ensure'.

Merge pull request #608 from DavidS/use-more-pool

Update tests to use the pool servers instead of ubuntu's

Update tests to use the pool servers instead of ubuntu's

Using the ubuntu keyserver causes stability issues during CI runs,
hopefully the pool servers are less fragile.

Merge pull request #606 from DavidS/use-key-server-pool

Use hkps.pool.sks-keyservers.net instead of pgp.mit.edu

Remove useless, failing test; cleanup whitespace

Use hkps.pool.sks-keyservers.net instead of pgp.mit.edu

The MIT server is sometimes unreliable during tests. The pool should
provide a much stabler service.

Merge pull request #607 from HelenCampbell/XenialTestFix

Removal of a breaking test for Xenial

Removal of a breaking test for Xenial

Merge pull request #605 from imphil/master

Install software-properties-common for xenial

Install software-properties-common for xenial

Instead of adding every newer release of Ubuntu, let's assume that the
package name that is used today will be used in the future, until the
next change is necessary.

Merge pull request #604 from tdb/fix_version_check

Fix version check on 16.04.

Fix version check on 16.04.

The old test obviously broke on 16.04. Switch to using versioncmp
since it does the right thing for us.

Merge pull request #602 from DG-i/master

apt::setting expects priority to be an integer, set defaults accordingly

apt::setting expects priority to be an integer, set defaults accordingly

Signed-off-by: Mathias Merscher <Mathias.Merscher@dg-i.net>

Merge pull request #600 from puppetlabs/update-msync

Update to newest modulesync_configs [9ca280f]

Update to newest modulesync_configs [9ca280f]

Merge pull request #596 from danielhoherd/master

Expose notify_update to apt::source

Merge pull request #599 from DavidS/fix-strict-var-testing

Fix STRICT_VARIABLE testing

(maint) clean up key_compat_specs

(maint) Update ke_compat_spec to run under STRICT_VARIABLES

Merge pull request #581 from robinelfrink/master

Make apt_updates facts use /usr/bin/apt-get.

Exposing notify_update to apt::source

Merge pull request #595 from danielhoherd/master

Typo: missing colon

Typo: missing colon

Merge pull request #593 from puppetlabs/2.2.x

Mergeback 2.2.x

Merge pull request #592 from HelenCampbell/2.2.2

Prep for 2.2.2 Release

Prep for 2.2.2 Release

Merge pull request #590 from DavidS/debian-8-support

Update metadata to note Debian 8 support

Update metadata to note Debian 8 support

Merge pull request #589 from DavidS/fm-4046-update-msync

(FM-4046) Update to current msync configs [006831f]

(FM-4046) Update to current msync configs [006831f]

This moves all copyright statements to the NOTICE file in accordance with the ASFs guidelines on applying the Apache-2.0 license.

Make apt_updates facts use /usr/bin/apt-get.

/usr/lib/update-notifier/apt-check is not available on all systems,
but /usr/bin/apt-get is.

Merge pull request #582 from imphil/master

Look for correct sources.list.d file for apt::ppa

Look for correct sources.list.d file for apt::ppa

In Ubuntu 15.10 the path of the apt sources file, which is generated by
apt-add-repository, changed to include the distid. This breaks apt::ppa
idempotency, since it does not recognize the repository is already
added.

Reported on puppet-users as well:
https://groups.google.com/forum/#!topic/puppet-users/YzeMyZYUo98

Ensure PPAs in tests have valid form

"man apt-add-repository" notes: "REPOSITORY can be either a line that
can be added directly to sources.list(5), in the form ppa:<user>/<ppa-name>
for adding Personal  Package  Archives".

Fix the tests to always use the format ppa:<user>/<ppa-name> when adding
PPAs.

Merge pull request #588 from werekraken/modules2873

MODULES-2873 - Avoid multiple package resource declarations

MODULES-2873 - Avoid duplicate package resources when package_manage => true

Merge pull request #587 from jbondpdx/master

MODULES-2941: add example to README

MODULES-2941: added more info about replacing sources.list

MODULES-2941: add example to README

Merge pull request #585 from DavidS/fm-4049-update-msync

(FM-4049) update to modulesync_configs

(FM-4049) Update to current msync configs [2c99161]

Merge pull request #583 from tdb/fix_ppa_pluses

Handle PPA names that contain a plus character.

Handle PPA names that contain a plus character.

It looks like add-apt-repository changes pluses to underscores when
creating the sources.list.d file. So the name it creates doesn't match
what Puppet expects, and the resource keeps applying on every Puppet
run.

This works around that problem.

Merge pull request #572 from twc-openstack/master-tempfile-race-condition

Fix apt_key tempfile race condition

Merge pull request #578 from phervieux/15.10

Add ubuntu 15.10 support

Merge pull request #577 from syseleven/pp-2587

fix whitespace in source.list

MODULES-2889 - remove unneeded whitespace in source.list template

Merge pull request #575 from ryayon/master

MODULES-2861: run stages limitation added to the documentation.

MODULES-2861: run stages limitation added to the documentation

Merge pull request #576 from puppetlabs/2.2.x

Mergeback 2.2.x

Merge pull request #573 from HelenCampbell/release2.2.1

Release 2.2.1

Release 2.2.1

Fix apt_key tempfile race condition

The Ruby Tempfile class has a finalizer that removes the file when the
GC runs.  It's not predictible when the GC will run, so you have to
ensure that the instance of the class stays in scope for as long as you
need it.

Unfortunately the tempfile method is returning just the filename of the
temporary file, which means it goes out of scope when that method
returns.  This allows the GC to reap it at any time after return.

In both CI and production environments we've seen this race fail,
causing apt-key add to fail a small (2-3%) amount of the time.

This changes the tempfile and source_to_file methods to return the
underlying Tempfile object, pushing it up into the caller's scope.  Both
of the callers immediately use the object to get its filename and then
open the file, eliminating the race.

Tested this by adding 'GC.start; sleep(1)' immediately before the
command is run, to give the GC plenty of time to remove the tempfile if
it was going to.

Add 15.10 support

Merge pull request #569 from DavidS/improve-readme

(MAINT) improve hiera description

(MAINT) improve hiera description

Merge pull request #567 from puppetlabs/2.2.x

Mergeback 2.2.x to master

Merge pull request #566 from bmjen/2.2.0-prep

Release prep for 2.2.0

Release prep for 2.2.0

Merge pull request #564 from rfdrake/apt_pin

Add support for creating pins from main class

Add support for creating pins from main class

This lets you create apt::pin resources as an apt param hash.  It also
supplies appropriate tests and documentation.

Merge pull request #562 from RewardGateway/preference_file_name

Corrected regression with preference files name

Corrected regression with preference files name

Signed-off-by: Vincent Deygas <Vincent.Deygas@rewardgateway.com>

Merge pull request #560 from mhaskel/MODULES-2446

MODULES-2446 - Fix pinning for backports

MODULES-2446 - Fix pinning for backports

The 2.x series added a changed behavior for backport pinning to pin to
origin instead of release. Pinning to release is the correct behavior
for backports though.

Merge pull request #551 from bdellegrazie/MODULES-2269

Expose notify_update to apt::conf

Merge pull request #557 from fbarbeira/patch-1

Fix path to 'preferences' and 'preferences.d'.

Fix path to 'preferences' and 'preferences.d'.

If you follow the documentacion, the hiera file does not work.

MODULES-2269: Expose notify_update setting

* Expose the underlying notify_update setting of the apt::settings resource
This is because not all configuration file changes should trigger an apt::update notification
* apt::pin also shouldn't result in an apt-update call
Adding a pin configuration should apply to the next apt-get update call it shouldn't trigger one itself.
* Added documentation
* Add tests for apt::conf notify_update

Merge pull request #556 from callahm3/master

Proxy ensure parameter.

Added an ensure parameter for user control of proxy presence. Defaults to undef to resist breaking any current setup.

added spec entry

Added validation

fixed ensure hash

Merge pull request #553 from bmjen/2.1.x_mergeback

2.1.x mergeback

Merge pull request #552 from jbondpdx/2.1.x

light edits to apt README

light edits to apt README

Merge pull request #545 from raoulbhatia/master

Use Debian's new official mirrors redirector

Merge pull request #550 from mhaskel/2.1.1-prep

2.1.1 prep

2.1.1 prep

Merge pull request #549 from mhaskel/merge_master_to_2.1.x

Merge master to 2.1.x

Merge remote-tracking branch 'upstream/master' into merge_master_to_2.1.x

Merge pull request #547 from mhaskel/MODULES-2190

Fix anchor issues

Add limitations note

MODULES-2190 - Fix anchor issues

Merge pull request #548 from leoarnold/README

Corrected documentation warning about `purge` parameters to fit v2.1.0 syntax

Corrected documentation warning about `purge` parameters to fit v2.1.0 syntax

Use Debian's new official mirrors redirector

See https://www.debian.org/News/weekly/2015/05/#httpredir

Merge pull request #543 from puppetlabs/revert-542-fix_apt_params

Revert "Fix use of $::apt::params::backports and $::apt::params::xfac…

Revert "Fix use of $::apt::params::backports and $::apt::params::xfacts."

Merge pull request #542 from infopro-digital/fix_apt_params

Fix use of $::apt::params::backports and $::apt::params::xfacts.