Drop apt::cron::download, and turn $apt_cron_hours into a class parameter for ap…
…t::cron::dist_upgrade.
I've introduced apt::cron::download, am not using it anymore, and have
never seen anyone else submit a bug or a merge request for it. Thus,
I feel pretty confident in dropping it.
And this allows me to trivially get rid of the $apt_cron_hours global
variable noticed by LeLutin in #13.
(I've mistakenly marked !44 as merged)
See merge request !47
I've introduced apt::cron::download, am not using it anymore, and have
never seen anyone else submit a bug or a merge request for it. Thus,
I feel pretty confident in dropping it.
And this allows me to trivially get rid of the $apt_cron_hours global
variable noticed by LeLutin in #13.
Update the README to use markdown
This module's documentation is pretty good, but god plain text is awful when a document gets this long.
I _really_ dislike Gitlab's markdown flavor, but IMHO bad markdown is better than no markdown.
You can see the result here: https://gitlab.com/sogeecom/apt/blob/master/README.md
--
pollo
See merge request !41
The logic in specifying the default value for apt::repos is in the wrong
place: it should be in apt::params since this is exactly what this
latter class is for.
There's no special case that can involve making default the value follow
the value of another parameter so there's no point in having that logic
in the main class.
backports_url was already set to the same value than debian_url by
default, but it was set to the default value of debian_url. this meant
that when giving a url to the debian_url parameter to the apt class, the
backports_url would not follow and would still be using the default
value for debian_url.
with this change, when backports_url is not specified, but debian_url
is, then both of them take on the value given to the debian_url. of
course if backports_url is also given a value, then it keeps the value
given by the user.
Remove loglevel directive from Exec[update_apt]
When using puppet apply (v 3.7), loglevel 'info'
won't show the output on error. This is bad for
debugging.
micah, assigning to you because this has already bitten me while debugging a failed `apt-get update` for the leap_platform.
See merge request !39
