From 382b96856d6c214d79ae17d7f3b919ed7ae99bf8 Mon Sep 17 00:00:00 2001
From: Stephen Gran <steve@lobefin.net>
Date: Fri, 27 Apr 2012 20:15:31 +0100
Subject: [PATCH] try to order these a bit

Signed-off-by: Stephen Gran <steve@lobefin.net>
---
 manifests/site.pp                    |  3 ++-
 modules/buildd/manifests/init.pp     |  7 +++++--
 modules/debian-org/manifests/init.pp |  1 +
 modules/site/manifests/aptrepo.pp    | 21 ++++++++++++---------
 4 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/manifests/site.pp b/manifests/site.pp
index 3aa59909..9c714a5c 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -1,5 +1,6 @@
 Package {
-	require => File['/etc/apt/apt.conf.d/local-recommends']
+	require => File['/etc/apt/apt.conf.d/local-recommends'],
+	stage   => deploy_infra
 }
 
 File {
diff --git a/modules/buildd/manifests/init.pp b/modules/buildd/manifests/init.pp
index 1de4f337..6bd381e1 100644
--- a/modules/buildd/manifests/init.pp
+++ b/modules/buildd/manifests/init.pp
@@ -7,13 +7,16 @@ class buildd {
 		package { 'schroot': ensure => held }
 	}
 
+	package { 'apt-transport-https':
+		ensure => installed,
+		stage  => setup,
+	}
 	package { [
 			'sbuild',
-			'apt-transport-https',
 			'debootstrap',
 			'dupload'
 		]:
-			ensure => installed
+		ensure => installed
 	}
 
 	site::linux_module { 'dm_snapshot': }
diff --git a/modules/debian-org/manifests/init.pp b/modules/debian-org/manifests/init.pp
index 912cfd61..c3488e50 100644
--- a/modules/debian-org/manifests/init.pp
+++ b/modules/debian-org/manifests/init.pp
@@ -179,6 +179,7 @@ class debian-org {
 
 	exec { 'apt-get update':
 		path        => '/usr/bin:/usr/sbin:/bin:/sbin',
+		stage       => setup_infra,
 		refreshonly => true,
 	}
 
diff --git a/modules/site/manifests/aptrepo.pp b/modules/site/manifests/aptrepo.pp
index d0e34eca..b71c5fa2 100644
--- a/modules/site/manifests/aptrepo.pp
+++ b/modules/site/manifests/aptrepo.pp
@@ -1,17 +1,18 @@
-define site::aptrepo ($key = undef, $keyid = undef, $template = undef, $config = undef, $ensure = present, $require = []) {
-
+define site::aptrepo ($key = undef, $keyid = undef, $template = undef, $config = undef, $ensure = present) {
 
 	case $ensure {
 		present: {
 			if $key {
 				exec { "apt-key-update-${name}":
 					command     => "apt-key add /etc/apt/trusted-keys.d/${name}.asc",
+					stage       => setup,
 					refreshonly => true,
 				}
 
 				file { "/etc/apt/trusted-keys.d/${name}.asc":
 					source => $key,
 					mode   => '0664',
+					stage  => setup,
 					notify => Exec["apt-key-update-${name}"]
 				}
 			}
@@ -20,19 +21,23 @@ define site::aptrepo ($key = undef, $keyid = undef, $template = undef, $config =
 			if ($keyid) and ($key) {
 				file { "/etc/apt/trusted-keys.d/${name}.asc":
 					ensure => absent,
+					stage  => setup,
 					notify => Exec["apt-key-del-${keyid}"]
 				}
 				exec { "apt-key-del-${keyid}":
 					command     => "apt-key del ${keyid}",
+					stage       => setup,
 					refreshonly => true,
 				}
 			} elsif $key {
 				file { "/etc/apt/trusted-keys.d/${name}.asc":
 					ensure => absent,
+					stage  => setup,
 				}
 			} elsif $keyid {
 				exec { "apt-key-del-${keyid}":
-					command     => "apt-key del ${keyid}",
+					command => "apt-key del ${keyid}",
+					stage   => setup,
 				}
 			}
 		}
@@ -49,15 +54,13 @@ define site::aptrepo ($key = undef, $keyid = undef, $template = undef, $config =
 		file { "/etc/apt/sources.list.d/${name}.list":
 			ensure  => $ensure,
 			content => template($template),
-			notify => Exec['apt-get update'],
-			require => $require,
+			notify  => Exec['apt-get update'],
 		}
 	} else {
 		file { "/etc/apt/sources.list.d/${name}.list":
-			ensure => $ensure,
-			source => $config,
-			notify => Exec['apt-get update'],
-			require => $require,
+			ensure  => $ensure,
+			source  => $config,
+			notify  => Exec['apt-get update'],
 		}
 	}
 }
-- 
2.39.2