Make consistent the use of absolute path to apt-get.

Factorize.

apt::reboot_required_notify: take into account the random sleep time.

Merge remote branch 'riseup/master'

switch default frontend to 'mail' instead of pager, as most things are done non-interactively

fixing alias bug

Additionally use version number in Lenny default pinning.

Lenny's APT does not support pinning like this:

  Pin: release o=Debian,n=<%= codename %>

We therefore switched (in commit ef2ebdffd) to:

  Pin: release o=Debian,a=<%= release %>

With such a pinning setup, when Squeeze is released, systems using this module
with $apt_use_next_release set to true would immediately switch to prefer
packages from Squeeze. If an automated upgrade process is setup, they would be
automatically upgraded to Squeeze.

This does not sound safe to me, so let's use the release version number as an
additional selection criterion to prevent upgrades to Squeeze to happen behind
our back:

  Pin: release o=Debian,a=<%= release %>,v=<%= release_version %>*

Note that the trailing '*' is intentional and necessary to match stable
point-releases.

Bypass default APT proxy to access bugs.debian.org.

This is needed when using apt-listbugs with an APT proxy that only {wants,can}
proxy "normal" APT traffic.

Allow running cron-apt more than once a day.

New class: apt::reboot_required_notify

This class installs a daily cronjob that checks if a package upgrade
requires the system to be rebooted; if so, cron sends a notification
email to root.

Fix formatting.

Merge remote branch 'riseup/master'

Update comment: Squeeze's unattended-upgrades 0.62 still does not support wildcards.

removed bit from README that is no longer true

fix compile syntax error in ubuntu sources.list, closes #2639

Merge remote branch 'lelutin/master'

fix syntax error

fix template missing <%- end -%>

fix confusion over lsbrelease / lsbdistcodename

add apt-listchanges support. this is a nice compliment to using the upgrade_package functionality, because you get an email when the package has been upgraded.

add apticron support.

Why apticron, when we have cron-apt already? Some people have different preferences, we use apticron along with the upgrade_package functionality in this module. I know someone who uses cron-apt to run the upgrades, but apticron for notifications, because apticron's notifications are much nicer (cron-apt just gives you the output of apt-get upgrade)

provide an "03clean" alias for the 03clean_vserver to satisfy dependencies

fix typo

multiple sources for templates are not allowed (see #1818), so revert it to a single source referenced by the README, and clarify the README to indicate how you can pass the preseed contents directly

change the name of the custom_keys exec to be "custom_keys" instead of using the alias metaparameter.

the reason for this change was because the Exec override was not able to find the exec through its alias

update apt::preseeded_package template sources to use $lsbdistcodename instead of $debian_release, also expand it to allow for site-apt sources

switch to archive name for lenny default preferences

the templates/Debian/preferences_lenny.erb file checked in with
e2f80db7b76171e5945127e2fd42fb35043990fb contains pinning based on codename,
which is not supported in lenny (see #433624 - if you look at the version graph,
you see, "Fixed in version 0.7.21", and lenny has 0.7.20.2+lenny1.)

README update: minor typo fix

README update: clarify that the debian-archive-keyring is managed already

Just so people are clear that they do not need to specify a $custom_key_dir to manage the debian archive keyring, I've added some clarifying text so you know that this is not necessary

README update: clarify the site-apt preferences options

The README described a few things that were not true relating to the
apt/preferences file.

First of all it said you could ship a 'file', but preferences.pp very clearly
uses the 'content => $custom_preferences' parameter, which will not take file
sources, only templates.

Secondly, it seemed to imply that you could just drop the custom preferences
into your site-apt and it would work. But you actually need to set the
$custom_preferences to indicate the content source.

Lastly, it said that you could specify a host-specific file in the site-apt
module, but there is no facility for this (nor can you use files).

Perhaps this is where this module is going eventually, once we have a
preferences.d possibility? Until then, it makes more sense to have it reflect
the current situation.

format standardization

switch the default $debian_url to use the Debian CDN mirror address.

The CDN checks your IP and uses the Max-Mind geoIP DB to determine your location and then uses your local country Debian Mirror, or if your local country doesn't have a debian mirror, it uses the local continent. It automatically checks and prunes dead mirrors. the technology for the Debian CDN setup is still improving, I believe that ASN-number associations are the next step to attempt to make more informed, finer-grained choices.

This is more useful than using ftp.debian.org because that is just a round-robin DNS entry that could get you any mirror anywhere.

Set dselect DSelect::Clean to 'pre-auto' by default for hosts that are vservers.

Before you only had the choice of setting a 03clean apt configuration for either
all hosts, or every single host. Setting it to have the recommended settings for
vservers for all hosts meant that you were setting it for non-vservers as well
as vservers. The other option you had was to set it per host. This was a bit
annoying if you have any more than one vserver because you would need to create
a 03clean for every single vserver guest.

This change auto-detects if the node is a vserver, and if it is it automatically
installs the 03clean_vserver file, with the recommended DSelect::Clean settings,
and allows you to override this for all of your vservers, or for specific hosts.

apt_conf: change mode to 644

Current mode (600) makes it impossible to use apt-cache to search for
package names and info, since it tries to read all configuration files
in /etc/apt/apt.conf.d before executing.

Change configuration file mode to 644 so that search tasks can be made
without root priviledge.

minor README fixups

Bugfix

Fix typo and rephrase.

Merge remote branch 'immerda/master'.

Conflicts:
README
files/preferences
templates/Debian/sources.list.deb-src.erb
templates/Debian/sources.list.volatile.erb
templates/Ubuntu/sources.list.backports.erb
templates/Ubuntu/sources.list.deb-src.erb

Merge commit '92aa5a51e95d74604a40318558aa98f70fcf7720'.

A bunch of painlessly merged commits.

Conflicts:
manifests/init.pp

Merge commit 'e2578c3b9820e9ff17f8a8881b8b22b5e241ee21'.

My branch already has support for defining custom APT mirrors location.
See $apt_debian_url, $apt_security_url, $apt_backports_url, $apt_volatile_url.
=> Discarding the addition of $main_apt_source.

Merge commit '3c88b19dd33b4da1f37e2c08c981ccbcbe5a45a8'.

Conflicts:
manifests/init.pp

Merge commit '27d5808299045ebd6f428e7d8131d11d9ab37712'.

One more painless merge commit.

Conflicts:
manifests/init.pp

Merge commit '1b0bc415977c1c8b49390e0cdb1361769e0fdbae'.

Merging one more commit.

Conflicts:
files/preferences
templates/Debian/sources.list.deb-src.erb
templates/Debian/sources.list.volatile.erb
templates/Ubuntu/sources.list.backports.erb
templates/Ubuntu/sources.list.deb-src.erb

Merge commit 'd97a49b7b2c020c2c43df6524236f50a421789ee'.

Going on merging immerda changes.

Incompatible change: I do prefer my own generic apt/preferences template to
shipping a static file that only supports Lenny. My own template supports
security, backports, volatile, next release etc.

Conflicts:
files/preferences
manifests/init.pp
manifests/preferences.pp

Merge commit 'ac76a5d52df78aec919f08334ca5b140902a9298'

(Starting to merge changes from immerda.)

Conflicts:
manifests/default_sources_list.pp
manifests/init.pp

Do not refresh APT lists on every run.

Let's set refreshonly on the 'update_apt' Exec (aliased to 'apt_updated')
instead of removing it for backward compatibility: it may be in use at some
places to trigger updates.

Merge remote branch 'nadir/master'

Conflicts:
README
manifests/custom_sources.pp
manifests/default_preferences.pp
manifests/init.pp
manifests/unattended_upgrades.pp
templates/Debian/sources.list.volatile.erb

unfortunately no comments are allowed in that file

fixing comments

apt preferences actually want // to comment, while sources are fine
to be commented with # ...

Revert "remove comments as lenny apt doesn't like them"

This reverts commit 5de6431b89b696ce17874952dceb5968108449a9.

Actually the comments are just the wrong way...

remove comments as lenny apt doesn't like them

I got errors like:

E: Syntax error /etc/apt/apt.conf.d/03clean:4: Extra junk after value

Removing these comments fixed these problems.

put the file at the right place & fix typos

Correct some errors in the README

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Change "brought to you" into "managed" in headers

File headers are there to indicate that the files should not be touched
directly on the server. By changing the first sentence to "This file is
managed by Puppet", we reinforce the idea that it is already taken care
of by something else.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Update the README

Include new classes and defines and move things around for a little bit
of consistency.
Also remove the now unused variables.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Ensure presence of sources.list.d and watch it

Add a file resource on the apt/sources.list.d directory to ensure that
it is created as a directory and watch it for modifications to
automatically refresh the apt cache.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Remove $apt_unattended_upgrades

Unattended upgrades are not always necessary. Including the
apt::unattended_upgrades class through the main apt class is not
appropriate. It is better to make nodes include the
apt::unattended_upgrades class explicitly.

Remove the $apt_unattended_upgrades variable along with the include in
the 'apt' class.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Fix apt::preferences::absent

Currently, setting the $custom_preferences variable to false fails
because of the /etc/apt/preferences concatenated_file being re-defined.

Fix this by setting dependencies on the resource only when we expect to
create the /etc/apt/preferences file. Also, since there is no "ensure"
parameter to concatenated_file, use a simple file resource to ensure
that it is removed.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Include the protocol in the proxy URL

The current proxy_client template makes it possible to communicate with
the proxy only via HTTP. Add the protocol to the variable so that it's
possible to change the protocol.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Add $main_apt_source

Currently, it's either we use the default source that's hardcoded in the
sources.list template or we redefine entirely this template.

Make it easier to just change the URL of the apt source while using the
rest of the default template by adding a $main_apt_source variable.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Remove the now unused $apt_clean variable

This variable was deprecated in favor of using a list of file sources.
to change the value of the 'clean' property, you can either define a
file '03clean' in a site-apt module, or a per-host '$fqdn/03clean' file
in the same site-apt module.

Also, move the moduledir stuff below nearer to where it's actually used.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

rename things for easier understanding

remove the use of _snippet in names (except for preferences_snippet) so
that they represent directly a resource name.

rename custom_sources_template to sources_list.

modify sources_list to make it more flexible (gives the opportunity to
provide sources or content). this changes its behaviour in that the name
is now the name of the file in sources.list.d

rename proxy-client to proxy_client to use the same standard for its
name as the other classes.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Add file headers for awareness of puppet presence

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

refactor: simplify and remove inline content

Simplifications: make apt_conf_snippet repeat less code, make code that
generates sources.list more concise.

Remove all inline content in favor of templates and static files.
The ability to include sources for static files was needed for
the main 'preferences' file, so it was added to the preferences_snippet
define.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Add apt_conf_snippet and use it where possible

With the new define, it's easy to add an apt.conf snippet in apt.conf.d
It accepts either 'sources' to get a static file or 'content' to define
content inline or with the help of a template.

Put it to use where we create files in apt.conf.d

Finally, fix the dependancy to the apt_config file (however, I don't see
the need for this dependancy)

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Move default_sources_list into apt class

The contents of the file default_sources_list.pp is used in only one
place. Remove the file and move its contents inside the apt class in
order to have one less useless depth level.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

notify => Exec['refresh_apt']

merged with immerda/master

Merge remote branch 'lelutin/master'

Integrate no custom preference into our new
way to manage the preferences.

Conflicts:
README
manifests/default_preferences.pp
manifests/init.pp

Make it possible to ensure preferences is absent

The current code makes it mandatory to have a file /etc/apt/preferences
present. In the event that this file is empty or contains a space,
apt-get update cannot execute.

Add a case with the special value "false" that ensures the file does not
exist.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Remove unnecessary exec

When $custom_key_dir has not been assigned any value, a useless exec of
/bin/true is run. Remove this by using 'if' instead of 'case'.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Add a proxy-client class

The module currently doesn't have a handy way of making a server get its
packages via a proxy. Add a "proxy-client" class that makes this easy.

$apt_proxy declares the proxy server's fqdn
$apt_proxy_port declares the port used for connection

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Make $apt_clean effective in all cases

The file "/etc/apt/apt.conf.d/from_puppet" is currently only defined
when $custom_sources_list is not used (it is declared in
default_preferences).

Bring this file up in the apt class so that using custom sources doesn't
remove its addition.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Merge remote branch 'riseup/master'

Conflicts:
manifests/init.pp

Ensure apt-listbugs is absent in dist-upgrade mode.

Else it will crash automatic upgrades.

Need to update APT lists after apt.conf.d changes.

No need to update APT lists when pinning changes (bis).

Fix one more relationship.

No need to refresh APT lists after changes to pinning preferences.

Fix indentation.

Fix relationships.

minor improvements, especially do not refresh apt every run

improve example

empty line between snippets

fix header file

fix classname

fix modules directory stuff

improve prefrences snippet

introduce preferences snippet

Now, we have the possibility to externally add snippes, so that
we can preferences for packages that are for example only in backports
or unstable.

Support choosing a custom Ubuntu mirror.

Use append_if_no_such_line instead of line.

Migrate config file to line-editing.

This allows other modules to add lines there too.

Backward compatibility: ensure old renamed file is absent.

Bring /etc/apt/apt.conf.d/99from_puppet to init.

APT configuration has (unfortunately) nothing to do with APT preferences.

Support $apt_repos for Ubuntu as well.

Merge remote branch 'nadir/master' into wip

Conflicts:
manifests/default_preferences.pp
manifests/default_sources_list.pp
manifests/init.pp
templates/Debian/sources.list.deb-src.erb
templates/Debian/sources.list.erb
templates/Debian/sources.list.volatile.erb
templates/Ubuntu/sources.list.erb

Bring in a bit of Ubuntu support from Nadir.

Rename $apt_use_volatile to $apt_volatile_enabled.

This will help merging with Nadir's changes.

Manage a /etc/apt/apt.conf.d/99-puppet file.

Syntax fix