diff options
author | varac <varacanero@zeromail.org> | 2016-02-01 11:54:33 +0100 |
---|---|---|
committer | varac <varacanero@zeromail.org> | 2016-02-02 09:47:45 +0100 |
commit | 1e2cac92bbb667ab7e84e121f125f38976f34d3a (patch) | |
tree | 2d23a8f7a782754ac15ba2528f0846e2c4f22379 /puppet/modules/site_apt | |
parent | e244e724c40c5f4770a43721e4c7441dec242e4f (diff) |
[refactor] Don't declare dependencies for apt resources
The apt module now takes care of all the dependencies removed
from `site_apt`.
Also, the dependency to install the `lsb` package after
`refresh_apt` is unnesseccary because lsb facts won't work
anyway on the first run if `lsb` is not installed before, so
we can safely remove it.
Diffstat (limited to 'puppet/modules/site_apt')
-rw-r--r-- | puppet/modules/site_apt/manifests/init.pp | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/puppet/modules/site_apt/manifests/init.pp b/puppet/modules/site_apt/manifests/init.pp index c809a837..fea4b8e7 100644 --- a/puppet/modules/site_apt/manifests/init.pp +++ b/puppet/modules/site_apt/manifests/init.pp @@ -52,13 +52,10 @@ class site_apt { pin => 'origin "deb.leap.se"' } - # All packages should be installed _after_ refresh_apt is called, - # which does an apt-get update. - # There is one exception: - # The creation of sources.list depends on the lsb package +# All packages should be installed _after_ refresh_apt is called, +# which does an apt-get update. + + Exec['refresh_apt'] -> + Package <||> - File['/etc/apt/preferences'] -> - Apt::Preferences_snippet <| |> -> - Exec['refresh_apt'] -> - Package <| ( title != 'lsb' ) |> } |