Merge pull request #52 from nanliu/tb/14308

(#14308) Add ensure=>absent for define resource.

(#14308) Add ensure=>absent for define resource.

Several apt::* define resource type does not support ensurable. This
update allows ensure=>absent to support the removal of these
configuration files.

* apt::conf
* apt::pin
* apt::source

Merge pull request #51 from nanliu/tb/14299

(#14299) Resolve Modulefile name mismatch.

(#14299) Resolve Modulefile name mismatch.

The modulefile name is puppet-apt, but the module is released to forge
under puppetlabs. This causes a dependency confusion for puppet module
tool, which has been fixed by this patch.

Previous:

    $ puppet module list
    ~/.puppet/modules
    ├── puppet-apt (v0.0.2)
    ├── puppetlabs-mongrodb (v0.0.1)
    └── puppetlabs-stdlib (v2.3.1)

Now:

    $ puppet module list
    ~/.puppet/modules
    ├── puppetlabs-apt (v0.0.2)
    ├── puppetlabs-mongrodb (v0.0.1)
    └── puppetlabs-stdlib (v2.3.1)

Merge pull request #48 from nanliu/tb/lint

Cleanup apt-module puppet-lint related issues.

Cleanup apt-module puppet-lint related issues.

Merge pull request #47 from nanliu/master

(#14138) Fix spec test for aptitude changes.

(#14138) Fix spec test for aptitude changes.

Fix spec test related to apttitude update -> apt-get update change,

Merge pull request #41 from tbroyer/install-proxy-first

Make sure we configure the proxy before doing apt-get update.

Merge pull request #44 from relud/patch-1

(#14138) Modify apt::ppa's update-apt exec to use the ${apt::params::provider} parameter.

Merge pull request #42 from ody/sync_with_pl_ops

Sync with pl ops

Merge pull request #46 from nanliu/master

(#14287) Fix apt::pin resource for rspec-puppet.

(#14287) Fix apt::pin resource for rspec-puppet.

The shorthand syntax cause rspec-puppet failure for external modules
depending on the puppet-apt module. This patch uses the require
metaparameter to avoid this issue.

(#14138) Modify apt::ppa's update-apt exec to use the ${apt::params::provider} parameter rather than explicitly calling aptitude.
Previously the update-apt exec would always use /usr/bin/aptitude, which is not necessarily present. This change makes it use ${apt::params::provider} which defaults to /usr/bin/apt-get. This also adds some consistency so that ${apt::params::provider} is used everywhere.

Tests to validate apt::{conf,backports}

  This patch adds the appropriate spec tests to validate the changes
  introduced by e5f2dfe.  As a bonus it includes fixes to the manifests
  that were discovered while writing the tests.

Adds apt::{conf,backports} and variable style.

  With the addition of this patch two new defines will be added; one to
  manage APT configuration files (apt::conf) and one that abstracts out the
  requirements needed to turn on backport repositories (apt::backports).

  In addition, the patch takes the opportunity to clean up variable
  definitions so they follow a consistent pattern of setting local
  variables to the fully qualified value stored in the apt::params
  class.  Previously all variable used within a class directly addressed
  the apt::params namespace when ever the variable was used.  In the
  pattern they now adhere to we can more easily switch the namespace
  data lives in or externalize it even more using hiera.

Make sure we configure the proxy before doing apt-get update.

Merge pull request #39 from rcoleman/0.0.2_release

Updated CHANGELOG and Modulefile for 0.0.2

Updated CHANGELOG and Modulefile for 0.0.2

This commit updates both the Modulefile and CHANGELOG for the 0.0.2
Puppet Forge release.

Merge pull request #36 from rcoleman/tickets/13261/improve_smoke_tests

Tickets/13261/improve smoke tests

(#13261) Add real examples to smoke tests.

This commit modifies some smoke tests with real-world usage examples
instead of providing things like 'foo' that the module user must
replace in order to conduct a smoke test or try out example functionality.

(#13261) Add key.pp smoke test

This commit adds a smoke test for key.pp, adding a key declaration
for the puppetlabs apt source.

(#13261) Replace foo source with puppetlabs source

Prior to this commit, the source.pp test included an apt source
named foo which broke the apt-get update exec when applied on a
target system. This commit removes it in favor of the puppetlabs
apt source which is valid on all target platforms.

Ignore pkg directory.

This commit modifies .gitingore to ignore changes to the pkg directory
used when building Forge releases.

Merge pull request #37 from kbarber/ticket/master/13289

Ticket/master/13289 Style violations & rspec cleanup

(#13289) Fix some more style violations

(#13289) Change test scaffolding to use a module & manifest dir fixture path

This removes the need for depending on a global site.pp, and depending on the
checked out apt module to be called 'apt' as well.

It pulls in stdlib via sub-modules as I don't have a better way to deal with
dependencies yet. It has pinned the revision for stdlib to 2.2.1 which is the
minimum required version for apt to work.

(#13289) Clean up style violations and fix corresponding tests

(#13289) Add puppet lint tests to Rakefile

Merge pull request #34 from pdxcat/apt_key_should_be_case_insensitive

(#13125) Apt keys should be case insensitive

(#13125) Apt keys should be case insensitive

Previously lowercase keys would be installed every
puppet run because apt-key list returns an uppercase
key. This commit makes the comparison case insensitive.

Merge pull request #32 from pdxcat/convert_apt__key_to_use_anchors

Convert apt::key to use anchors

Merge pull request #33 from rcoleman/tickets/13016/create_CHANGELOG

(#13016) Add a CHANGELOG since first commit
Skipping the merge review rules on this harmless commit of a changelog.

(#13016) Add a CHANGELOG since first commit

Convert apt::key to use anchors

Previously, apt::key used a noop exec hack to do exactly what anchors were
intended to be used for. This commit removes the exec hack and achieves the
same end using Anchor resources from the puppetlabs/stdlib module.

Merge pull request #31 from pdxcat/fix_release_parameter_test

Modify apt::source release verification to work correctly with puppet-rspec

Modify apt::source release parameter test

This commit modifies the release parameter test in apt::source to work
correctly within puppet-rspec for edge-case resource definitions. Previously,
the test for the $release parameter was written as

`if ! $release { fail() }`

This commit updates the test to be written as

`if $release == undef { fail() }`

Additionally, the tests for correct behavior in the presence or absence of a
$release parameter have been beefed up.

The reason for making this change relates to examples such as the following
resource definition:

apt::source { "jenkins":
  location    => "http://pkg.jenkins-ci.org/debian",
  release     => "",
  repos       => "binary/",
  key         => "D50582E6",
  key_source  => "http://pkg.jenkins-ci.org/debian/jenkins-ci.org.key",
  include_src => false,
}

Note that the $release parameter is given as the empty string. In practice,
this is perfectly valid and everything will work great. However, it seems that
the empty string gets interpreted by something in puppet-rspec as something
equivalent to "False", and thus when testing, the above resource definition
would fail with "Puppet::Error: lsbdistcodename fact not available: release
parameter required" even though the $release parameter has been explicitely
specified (as the empty string).

See also: https://github.com/rtyler/puppet-jenkins/issues/9

Merge pull request #30 from rcoleman/tickets/12917/add_contributors_to_readme

(#12917) Add contributors to README

(#12917) Add contributors to README

Prior to this commit, this modules README had no list of contributors.
This commit adds such a list and corrects a typo I overlooked on the
last pull request.

Merge pull request #26 from pdxcat/ticket_12823_aptkey_defined_type

(#12823) Add apt::key defined type and modify apt::source to use it

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

(#12823) Add apt::key defined type and modify apt::source to use it

Adding this defined type allows puppet to add keys to the apt keystore without
needing to add a corresponding source; it also adds the "key_source" parameter
for wget'ing keys from arbitrary URLs, and allows for keys to be explicity
removed.

apt::key allows a key to be ensured present multiple times to account for
apt::source resources that all reference the same key. However, this means
that it is possible for a given key to be defined multiple times with
differing source parameters. e.g.

apt::key { "Add key: 4BD6EC30 from Apt::Source bunny":
  key        => "4BD6EC30",
  key_server => "pgp.mit.edu",
}

apt::key { "Add key: 4BD6EC30 from Apt::Source rabbit":
  key        => "4BD6EC30",
  key_server => "keyserver.ubuntu.com",
}

The defined type will accept both definitions and will create multiple exec
resources. This was deemed preferable to the alternative (creating only one
exec resource) in that one broken definition won't hose an entire catalog. If
one definition fails to install the key because of a bad "key_server", the
next apt::key that uses the key will get it done.

Merge pull request #27 from blkperl/ticket_12809_refactor_release

(#12809) $release should use $lsbdistcodename and fall back to manual in...

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

(#12809) $release should use $lsbdistcodename and fall back to manual input

This commit changes $release to default to Facter's $lsbdistcodename
and fall back to a Parse Error if $release is not set and $lsbdistcodename
does not exist. Previously $release was hardcoded to karmic.

This commit also modifies apt::ppa to use $release and sets the
files to be ensured so that they are not purged when purge_sources_list_d
is set to true.

Merge pull request #23 from haus/python_properties

Check if python-software-properties is defined before attempting to define it.

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #22 from haus/unique_required_packages

Allow duplicate required_packages between sources

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #24 from haus/fix_source_key_tests

Fix source specs to test all key permutations

Reviewed and tested by Ryan Coleman (ryan@puppetlabs.com)

Merge pull request #25 from rcoleman/ticket_12522_purge_option_take_2

(#12522) Split purge option to spare sources.list

(#12522) Adjust spec test for splitting purge

Now that the purge parameter is split into two, the spec test needs
adjusted. This commit makes that adjustment.

(#12522) Split purge option to spare sources.list

Prior to this commit, when using the purge option, unmanaged entries
in both /etc/apt/sources.list and sources.list.d would be purged.

This commit splits purge into purge_sources_list and purge_sources_list_d
which handle the purging of those items separately. Brief documentation on
each added to class documentation.

Fix source specs to test all key permutations

Previously only one should in each block was being evaluated. This moves each
should to its own block so that all values are tested. It also adds another set
of params so that all useful permutations of key, key_server, and key_content
are generated. It also replaces the previous ternary assignment for param_set
with a hash merge.

Add test for python-software-properties package

This test adds the precondition that the python-software-packages package be
installed before the apt class is synced. If the defined function were not
called around the package resource, this test would fail with a duplicate
package resource error.

Check if python-software-properties is defined before attempting to define it.

Add tests for required_packages change

This updates the tests for the apt::source defined type to use the new name for
the exec resource and the new parameter for the same exec.

Allow duplicate $required_packages

Previously, if more than one apt::source required the same packages to be
installed it would fail with a duplicate exec resource. This adds the name of
the source resource to the exec and gives the exec a name, moving it to a
command parameter for the exec.

Merge branch 'allow-same-key'

(#12430) Add tests for changes to apt module

This update reflects the changes to the apt module to allow duplicate keys. It
mostly involves tests for changes to the resource names to make them unique
between defines.

Test two sources with the same key

Merge pull request #17 from blkperl/ticket_12526_apt_disable_keys

(#12526) Add ability to reverse apt { disable_keys => true }

Reviewed by Ryan Coleman (ryan@puppetlabs.com)

(#12526) Add ability to reverse apt { disable_keys => true }

The setting `disable_keys => true` parameter in the apt module creates
/etc/apt/apt.conf.d/99unauth with the contents
"APT::Get::AllowUnauthenticated 1;". Changing `disable_keys`
does not remove this file. This patch makes it so that
`disable_keys => false` will remove /etc/apt/apt.conf.d/99unauth.

Merge pull request #20 from blkperl/ticket_12522_purge_option

(#12522) Adding purge option to apt class

Reviewed by Ryan Coleman

Merge pull request #1 from rcoleman/ticket_12522_purge_option

Add Modulefile to puppet-apt

Add Modulefile to puppet-apt

Now that puppet-apt depends on puppetlabs-stdlib, an explicit
dependency is required. Modulefile was entirely missing. This commit
adds on.

Allow the use of the same key in multiple sources

Allow the use of the same key in multiple sources by including the name
of the source in the declaration for the exec which adds the key.

(#12522) Adding purge option to apt class

Adds a purge option to the apt class to remove repositories
that are not managed by apt::source

Merge pull request #18 from bodepd/apt_proxy_tests

(#12529) Add unit test for apt proxy settings

(#12529) Add unit test for apt proxy settings

This commit adds unit tests to validate that the
apt proxy is configured as expected when the class
parameter proxy_host is set as a class parameter for
the apt class.

Merge pull request #15 from bodepd/proxysupport

(#12529) Add parameter to support setting a proxy for apt

(#12529) Add parameter to support setting a proxy for apt

This commit adds two class parameter to apt that can be used to
specify a proxy to use with apt.

  - proxy_host
  - proxy_port

Merge pull request #14 from haus/ticket/12094_add_spec_tests

Ticket/12094 add spec tests

(#12094) Replace chained .with_* with a hash

The hash passing to the with method is cleaner and closer to puppet code, so
all of the with_$param have been replaced with with($hash). This also
includes two minor whitspace changes to unstable.pp and source.pp.
This also replaces the ternary switch on param_set with a hash merge,
which is cleaner and will support more use cases.

(#12094) Remove deprecated spec.opts file

Rspec no longer uses the spec.opts file, and this particular file also uses
options for rspec that are no longer honored or available, like loadby. So this
commit removes the file. Users that want to customize rspec behavior should use
a .rspec file in the repository root or in their home directory.

(#12094) Add rspec-puppet tests for apt

This commit adds full coverage for the apt module as it currently exists. It
adds rspec-puppet tests for the defines (apt::builddep, apt::force, apt::pin,
apt::ppa, apt::source) and classes (apt, debian::testing, debian::unstable,
apt::params, apt::release).

(#12094) Replace name with path in file resources

When the name is used with a file resource, it becomes difficult to test the
resource using rspec-puppet, as the name parameter gets aliased to path. So to
maintain consistency between tests and manifests, this replaces all name
parameters in file resources with the equivalent path parameter.

Merge pull request #4 from webfactory/pin_before_source

Suggested fix for #10449

Merge pull request #12 from haus/11953_fix_apt_force_version

(#11953) Apt::force passes $version to aptitude

(#11953) Apt::force passes $version to aptitude

Previously, even if $version were passed to apt::force, aptitude would just
install the default version of the package available. This updates the aptitude
call to use the version string if it exists. If no version is passed, or if it
is false, no version is added to the aptitude call.  This also updates the
rspec-puppet tests for the define, to reflect the changes to the exec.

Merge pull request #11 from bodepd/11413_apt_force_query_fix

11413 apt force query fix

Merge pull request #8 from cwarden/source-example-improvement

include_src parameter in example; Whitespace cleanup

(#11413) Add spec test for apt::force to verify changes to unless

In the Previous commits, the query being done by the
apt::force install command was not strict enough and
could lead to false positives.

These queries represented by the unless parameter have been
resolved in another commit. This commit accompanies that commit
and adds basic unit tests to correspond to the changes.

(#11413) Update dpkg query used by apt::force

This patch fixes the query used by apt::force to
determine rather or not the package is installed.

Previously, the expression was not specific enough and
could not lead to false positives in cases where a package
name is contained within another package name (puppet could be
incorrectly determined as being installed if puppet-common is
installed)

This commit resolves that by improving the query expression.

Merge pull request #9 from bodepd/10451_fix_apt_update_refresh

10451 fix apt update refresh

(#10451) Add test coverage to apt::ppa

This commit adds test coverage for apt::ppa.

This test coverage is suficient to verify the
code changes that resolve the issue with
aptitude update not being called when ppas were
added (#10451).

include_src parameter in example; Whitespace cleanup

In the apt::source example, include the include_src parameter.  Clean-up
whitespace in the example to be consistent with the style guide.

Merge pull request #7 from cwarden/patch-1

fix typos in "repositories"

fix typos in "repositories"

Fix (#10451) - apt::ppa fails to "apt-get update" when new PPA source is added

Set the pin priority before adding the source (Fix #10449)

Merge pull request #3 from rnavarro/master

Fixed the infinite ppa add bug.

Refactored as per mlitteken

* https://github.com/rnavarro/puppet-apt/commit/1af9a13c40c33e1ab8fb56f33cb9aa5e16cead0e#commitcomment-539287

Added some crazy bash madness to check if the ppa is installed already. Otherwise the manifest tries to add it on every run!

(#8720) Replace Apt::Ppa with Apt::Builddep

Probably a copy & paste error

Reviewed-by: Matthaus Litteken <matthaus@puppetlabs.com>
Signed-off-by: Spencer Krum <spencer@puppetlabs.com>

added builddep command.

added the ability to specify the content of a key

Allows us to pass key_content when we specify a source.

Fixes ppa test.

reformatted whitespace to match recommended style
of 2 space indentation.

ignore swap files.

added smoke tests for module.

reformatted apt::ppa according to recommended
style.

removed require apt in favor of marking a
dependency.

converted release define into a class since
the resources are singletons.

Created a params class to hold global data.

- Removes coupling between global data and
  resources from apt class.
- Makes it easier to organize things into stages.

Added two params for apt class

both of these params facilitate options that
ease the management of apt repos in dev
environments

1. disable_keys - allows repos without properly
signed keys
2. always_apt_update - refreshes via apt update
every time that puppet runs.