Merge pull request #510 from psoloway/readme

Update README per DOC-1503

Update README per DOC-1503

1. Make sure that for each parameter, where applicable, there is a default value.
2. Make sure that for each parameter that's applicable, there is a note if the parameter is optional.
3. Make sure the links in the README work and are accurate.
4. Update the link in the Contributing section to point here: https://docs.puppetlabs.com/forge/contributing.html
5. General copyediting.

* Add a ToC
* Reorganize Usage & Reference sections
* Standardize capitalization of "Apt" (as opposed to "apt", the module)
* Standardize Development section

Apply edits from @jbondpdx

* Re-emphasize the warning about short key names
* Restore the “What apt affects” section
* Re-correct capitalization of “Apt” (where it had reverted to “APT”)
* Clarify OS compatibility in Limitations and elsewhere
* Various edits for clarity

Merge pull request #504 from mhaskel/fix_tests

Update tests to work with rspec-puppet 2.x

Update tests to work with rspec-puppet 2.x

Also enable future parser testing. Need to allow failures with future
parser for now since none of the published gems have the fix for
PUP-4379

Merge pull request #501 from mhaskel/MODULES-1934

MODULES-1934: Iterate through multiple keys

Iterate through multiple keys

Merge pull request #498 from mhaskel/missed_purge_reference

Fogot to remove the purge warning

Fogot to remove the purge warning

Merge pull request #489 from mhaskel/2.0.0-prep

2.0.0 prep

Merge pull request #497 from mhaskel/dont_purge_by_default

Don't purge by default. That seems unnecessarily destructive.

2.0.0 prep

Don't purge by default. That seems unnecessarily destructive.

Merge pull request #496 from puppetlabs/allow-undef-content-conf

apt::conf: Don't require content `ensure=>absent`.

apt::conf: Don't require content `ensure=>absent`.

Merge pull request #493 from mhaskel/remove_mint

Remove default support for Linux Mint and Cumulus Networks

Merge pull request #495 from mhaskel/gepetto

Try to make gepetto happy.

Try to make gepetto happy.

Remove default support for Linux Mint and Cumulus Networks

NOTE: While out-of-the box support is disabled, it is still possible to
get the same configurations, it will just require explicitly setting the
necessary codename-munging.

This should only affect `apt::ppa`

Merge pull request #494 from mhaskel/cleanup-backports

Cleanup backports

Add testing, docs, and examples for backports

backports: Add support back for backports.

This is currently lacking tests.

source: Support complex pin, like key does.

This adds support for passing in a full pin declaration as the pin
parameter on a source declaration.

It keeps the old behaviour intact, you can still simply do `pin => '10'`
and it will pin on origin with a priority of 10.

Should that not be what you want you can now pass in a full pin
declaration instead. We make no assumptions here, whatever you pass in
will be passed through to pin as-is with the exception of the values for
`ensure` and `before` which are always overridden by us to ensure
everything keeps working as designed.

Merge pull request #491 from mhaskel/lint_cleanup

more lint cleanup

more lint cleanup

Merge pull request #490 from mhaskel/lint_cleanup

Lint cleanup

Lint cleanup

Merge pull request #483 from mhaskel/examples

Add missing examples for 'removed' functionality

Example updates

Merge pull request #484 from mhaskel/merge_master_to_next

Merge master to next

Merge remote-tracking branch 'upstream/master' into merge_master_to_next

Merge pull request #482 from leoarnold/master

Updated key fingerprints in README to match v1.8.0 requirements

Example for fancy progress.

Example for disabling keys

Updated key fingerprints in README to match v1.8.0 requirements

Add examples for force.

We've got things in params, let's use those!

Add an example for backports

We got rid of the funny proxy removal

Merge pull request #479 from mhaskel/update_anchors

(MODULES-1156, MODULES-769) Update anchors

(MODULES-1156, MODULES-769) Remove unnecessary anchors

`apt::ppa` and `apt::setting` don't actually include `apt::update` so
anchors are unnecessary. Move `apt` to use contain instead of anchors,
since it wasn't anchoring properly anyways. Update the tests to make
sure it can have settings and ppas depending on each other without
cycles.

Merge pull request #480 from anodelman/master

(BKR-147) add Gemfile setting for BEAKER_VERSION for puppet...

(BKR-147) add Gemfile setting for BEAKER_VERSION for puppet...

puppetdb, etc

- support for BEAKER_VERSION and BEAKER_RSPEC_VERSION in gemfile

Merge pull request #478 from GhostLyrics/update-readme-unattended-upgrades

Extend docs for unattended_upgrades

Extend docs for unattended_upgrades

documented more common parameters

README: update['frequency'] is not a boolean

README: Add highlighting for code blocks.

Fix typo

Merge pull request #477 from mhaskel/doc_updates

Doc updates

README updates for 2.x rewrite

All classes, defines, parameters, and examples should be up to date.

Merge pull request #472 from mhaskel/acceptance_test_fixes

Acceptance test fixes

Merge pull request #473 from mhaskel/freq_cleanup

Remove `update['always'] = true` support

Merge pull request #474 from mhaskel/containers_ftw

Linux open-source builds are regularly maxed out.

Linux open-source builds are regularly maxed out.

Container-based are not. Let's switch!

Remove `update['always'] = true` support

We don't really need both of `update['always'] = true` and
`update['frequency'] = 'always'`.

Make the acceptance tests work

Actually make it possible to use apt_key

The provider wasn't updated for a parameter rename.

Merge pull request #471 from mhaskel/merge_1.8.x_to_master

Merge 1.8.x to master

Merge pull request #469 from mhaskel/test_fixes

Update all the unit tests to look for full fingerprints

Update all the unit tests to look for full fingerprints

Merged #466 too quickly

Merge pull request #468 from mhaskel/1.8.0-prep

1.8.0 prep

Merge pull request #466 from paroga/master

Fix gpg key checking warings after f588f26

1.8.0 prep

Fix gpg key checking warings after f588f26

Use the full fingerprint for all keys to silence the warning.

Merge pull request #465 from bootc/fixes/gpg2_date_parsing

apt_key: fix parsing invalid dates when using GnuPG 2.x

apt_key: fix some whitespace issues

apt_key: fix parsing invalid dates when using GnuPG 2.x

If one should happen to have redirected /usr/bin/gpg to run GnuPG 2.x
rather than the more usual GnuPG 1.x, the apt_key provider fails with
the following error:

Could not prefetch apt_key provider 'apt_key': invalid date

This is because the output of "--with-colons" defaults to using
"fixed-list-mode" in 2.x but did not do so for 1.x. This new format
gives much more information about keys and also uses timestamps in
seconds from 1970-01-01 (UNIX epoch) rather than dates in the format
YYYY-MM-DD.

This patch adds "--fixed-list-mode" when calling apt-key, and adjusts
the code to parse the timestamps instead. This actually has several
advantages:

- Works the same with GnuPG 1.x and 2.x.
- More accurate expiry time tracking, not just entire days.
- No need to require 'date' any longer.
- Will allow the provider to expose more key information in future.

Tested on:
- Debian Wheezy (Puppet 2.7.23, Ruby 1.8.7p358)
- Debian Jessie (Puppet 3.7.2, Ruby 2.1.5p273)

Merge pull request #464 from mhaskel/README_cleanup

Update docs for unattended_upgrades

Update docs and test for $legacy_origin

Merge pull request #463 from mhaskel/toggle-legacy-origin

Allow changing legacy_origin

Update docs and test for $legacy_origin

unattended_upgrades: Allow changing legacy_origin

This enables using Origins-Pattern in Ubuntu.

Merge pull request #459 from tphoney/modules-1675_check_gpg_version

initial commit for apt_key checking

Merge pull request #456 from johanfleury/cleaning/template/unattended-upgrades

Cleaning 50unattended-upgrades.erb

Merge pull request #462 from mhaskel/optional_software_properties

Make installation of software-properties optional

Make installation of software-properties optional

This is cherry-picked from the PPA cleanup happening for the 2.0.0 release.

Conflicts:
manifests/params.pp
manifests/ppa.pp

initial commit for gpg key checking

better attempt at gpg version checking

adding in key length warning

removing version check, adding key check

adding tests

clean up the code

small changes

use commands

documentation updates

Merge pull request #461 from mhaskel/boo_inheritance

Inheritance of apt::params means it can't be private

Inheritance of apt::params means it can't be private

Otherwise, if another module has `class { 'apt': }` in it everything
fails, as `$caller_module_name` will be the other module name.

Cleaning 50unattended-upgrades.erb

Merge pull request #453 from puppetlabs/daenney/location-required

apt::source: Make location required.

apt::source: Make location mostly required.

In what universe does it make sense to create a `sources.list.d` entry
for a repository **without** specifying where this repository is?
:confounded:

:disappointed: :weary: :anguished: :scream:

Only when removing the resource should a location not be required.

Merge pull request #454 from LeslieCarr/MODULES-1827-add-cumulus-linux

MODULES-1827 adding Cumulus Linux detection

MODULES-1827 adding Cumulus Linux detection

the apt module did not correctly detect Cumulus Linux with lsbdistid.
This change adds several lines in params.pp to detect Cumulus Linux and
set $distid and $distcodename

Merge pull request #452 from puppetlabs/daenney/rename-trusted-source

apt::source: Rename `trusted_source`.

apt::source: Rename `trusted_source`.

It is weird that `trusted_source` would default to `false` as that would
imply that we normally don't trust our sources. This is opposite to the
truth, by default we trust them but only if the Releases file can be
verified (meaning it is signed by a GPG key known to apt).

What we were telling apt is that it should trust this source even if the
Releases file and the repository is unsigned. This is better captured
with `allow_unsigned` and better highlights the danger of what you're
doing, installing packages from a source we cannot authenticate.

Merge pull request #451 from puppetlabs/daenney/consolidate-include

apt::source: Merge `include_*` options into hash.

apt::source: Merge `include_*` options into hash.

This makes its behaviour similar to the `update`, `proxy` and `purge`
hashes on the main classes bringing its API more in line with the rest
of the module.

Merge pull request #450 from puppetlabs/daenney/guard-lsb-facts

apt::params: Complete $xfacts.

apt::params: Complete $xfacts.

`$xfacts` now contains and guards all `lsb*` facts. Looking up any
`lsb*`-related fact should now always be done through `$::apt::xfacts`
to ensure that the values are always set to either the value of the fact
or undef. This avoids all sorts of kerfuffles with strict variables.

Merge pull request #449 from puppetlabs/daenney/fix-strict-vars

apt: Fix all strict variable cases.

apt: Fix all strict variable cases.

A few of these fixes are absolutely horrendous but we have no choice as
we need to stay current- and future-parser compatible for now.

Once we can go Puppet 4 only we can use the `$facts` hash lookup instead
which will return undef/nil for things that aren't set instead of them
not being defined at all.

Merge pull request #448 from puppetlabs/daenney/kill-file-perms

apt::setting: Remove file_perms.

apt::setting: Remove file_perms.

This was a great idea but is pretty pointless. It's also not being used
by anything and not exposed as a switch on the main class so it would
almost never affect any behaviour.

Merge pull request #447 from puppetlabs/daenney/hulk-smash-2

It's been a long time - how have you been

apt::key: Be explicit about the keyserver.

The behaviour of passing down undef through multiple layers gets fuzzy
so for now be explicit about the keyserver.

Once Puppet 4 is out and this behaviour has been crystallised and tested
we can revisit it.

apt::key: Rename $key to $id to match apt_key.

apt::source: Allow passing in a complex key.

Turn `$key` into something that accepts a string or a hash of four keys
representing the different options that can be passed on to `apt::key`.

apt(::key|_key) Remove the `key.*_` prefixes.

Remove a lot of the redundant `key_` prefixes on `apt::key` and the
`keyserver_` prefix on `apt_key`.

apt: Add settings, keys and ppas.

* Allow any configuration of apt to be done through data bindings by
  passing in hashes representing the resources.
* Switch apt::ppa to use `distid` as set in `apt::params. This makes
  `apt::ppa` also work for LinuxMint.

apt: Change how update is managed.

* Instead of having 4 options controlling purging we now have a single
  hash with four possible keys.
* Include `apt::update` only _after_ we've assembled the `$_update`
  hash.