Update keyserver tests

Use fixed beaker-testmode_switcher gem from git master

More short ID usage to work on older Debian versions

Merge pull request #2 from james-stocks/wip

Use short id (8 chars) when deleting an apt-key

Use short id (8 chars) when deleting an apt-key

See https://bugs.launchpad.net/ubuntu/+source/apt/+bug/1481871 for why the full 40 characters cannot be used on some systems.

Merge pull request #1 from james-stocks/wip

Use UTC for apt_key2 created and expiry

Use UTC for apt_key2 created and expiry

Undo nested context experiment

More acceptance test suite cleanups

Copy subkey fix from original provider

Fix more glitchy tests

Undo the id/name duplication; fixed in Resource API

Handle multiple gpg keys in the content attribute

Pull line.strip into key_list_lines

Refine key validation vs content; port more tests to testmode switcher

Snapshot

* Start converting acceptance tests to beaker-testmode_switcher
* Rename :id to :name attribute to match behaviour expected by puppet
* Update create() method to work in the common cases
* Update temp_key_file to be nicer, removed previous command usage
* Fix doc strings
* start using fedora gpg key for testing, to avoid collisions with existing keys on SUT
* start using puppetlabs_spec_helper fixtures to remove the key CLOBs from the test source
* start `puppet resource` testing
* apply rubocop autofixes

Remove Debian 6 special casing from test suite, as it is not supported anymore

Add basic create/delete handling

This still requires more work to match up to the original implementation,
but suffices for basic acceptance testing.

Change get to not use the Commands API for now

Apply rubocop fixes

Enable tracing on the smoke test to see failure locations

Change Gemfile to use public repo

Cleanup mock setup

Cleanup

Finish `get` method unit testing

Update to puppet-resource_api's rubocop config

Start setting up new-style unit tests; convert get() to new API

Hack acceptance tests to actually run

This is using a local build of the puppet-resource_api gem.
Modify as needed to get it to run in your local environment.

Enable local testing coverage recording

Update to use the puppet-resource_api gem

More logging functions

Run default pdk rubocop rules, update :kind to :behaviour

Fix type mismatch

Logger; fix a few things so that set() works

make the apt_key2 type work read-only

(maint) cut down travis matrix for quicker turnaround

snapshot

Start building out the next phase resource API shim

Merge pull request #665 from tigercomputing/master

Ignore subkeys in apt-key's output

Merge pull request #695 from pmcmaw/remove_debian6

(FM-6273) - Removing Debian 6

(FM-6273) - Removing Debian 6

This is not supported any longer according to the following
documentation: https://docs.puppet.com/pe/latest/sys_req_os.html

Merge pull request #694 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 #690 from tpdownes/master

Allow user to modify loglevel of apt-get update Exec resource

Merge pull request #693 from mauricemeyer/master

Fixed typo in "Configuring Apt from hiera example"

Fixed typo

Allow user to modify loglevel of apt-get update Exec resource so that its
execution goes unlogged

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

(maint) modulesync 915cde70e20

(maint) modulesync 915cde70e20

Merge pull request #640 from ocadotechnology/dpkg_admin_lock

(MODULES-4118) Set dpkg option NoLocking in apt_updates fact

Merge pull request #688 from puppetlabs/msync_modules5187

(MODULES-5187) mysnc puppet 5 and ruby 2.4

Merge pull request #686 from hunner/bump_metadata

(MODULES-5144) Prep for puppet 5

(maint) update sync.yml to not manage NOTICE

(MODULES-5187) mysnc puppet 5 and ruby 2.4

Fix broken markdown of CHANGELOG.md (#687)

* Fix broken markdown of CHANGELOG.md

(MODULES-5144) Prep for puppet 5

Merge pull request #684 from puppetlabs/release

Release 4.1.0 Mergeback

Merge pull request #679 from arioch/fix_typo

Merge pull request #682 from eputnam/fail_message

this failure is related to os.name, not os.release.full

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.

(maint) fix incorrect error message

this failure is related to os.name, not os.release.full

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.

Fix typo in hiera example

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

Ignore subkeys in apt-key's output

The output of `apt-key adv --list-keys --with-colons --fingerprint --fixed-list-mode` is always a `pub` line followed by one of more `fpr` lines (for subkeys, if any). This can cause the wrong pairs of `pub` and `fpr` lines to be hashed, resulting in nonsense.

The ordering `pub`, `fpr`, `pub`, `fpr` is guaranteed by `apt-key`.

Therefore, this commit clears `fpr_line` when a `pub` line is encountered, to reset the grouping. The current code ignores subkeys anyway, so this is no more bad.

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