summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorvarac <varacanero@zeromail.org>2016-02-01 11:54:33 +0100
committervarac <varacanero@zeromail.org>2016-02-02 09:47:45 +0100
commit1e2cac92bbb667ab7e84e121f125f38976f34d3a (patch)
tree2d23a8f7a782754ac15ba2528f0846e2c4f22379
parente244e724c40c5f4770a43721e4c7441dec242e4f (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.
-rw-r--r--puppet/modules/site_apt/manifests/init.pp13
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' ) |>
}