From ad76825e7d64d13bbbecf1bffa2abb2f6151882a Mon Sep 17 00:00:00 2001
From: Peter Palfrader <peter@palfrader.org>
Date: Sun, 15 Sep 2019 21:41:22 +0200
Subject: [PATCH] retire bugsmx role

---
 hieradata/common.yaml                     | 3 ---
 hieradata/nodes/buxtehude.debian.org.yaml | 2 ++
 modules/exim/manifests/init.pp            | 2 +-
 3 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/hieradata/common.yaml b/hieradata/common.yaml
index 7b6ed20f6..48c21b226 100644
--- a/hieradata/common.yaml
+++ b/hieradata/common.yaml
@@ -33,9 +33,6 @@ staticsync::basedir: '/srv/static.debian.org'
 # all of these should be retired in favour of including the class role
 # with the host. weasel, 2019-09
 roles:
-  bugsmx:
-    # XXX - exim config flag only
-    - buxtehude.debian.org
   dns_primary:
     # XXX - used by ferm templates/defs.conf.erb
     - denis.debian.org
diff --git a/hieradata/nodes/buxtehude.debian.org.yaml b/hieradata/nodes/buxtehude.debian.org.yaml
index 8d9fac3df..b93b9648d 100644
--- a/hieradata/nodes/buxtehude.debian.org.yaml
+++ b/hieradata/nodes/buxtehude.debian.org.yaml
@@ -5,3 +5,5 @@ classes:
 apache2::smaller_number_of_threads: true
 # debbugs cgis like to fork and don't deal well with EAGAIN
 apache2::rlimitnproc: 450
+
+exim::is_bugsmx: true
diff --git a/modules/exim/manifests/init.pp b/modules/exim/manifests/init.pp
index 59b87f76a..882a3c2f7 100644
--- a/modules/exim/manifests/init.pp
+++ b/modules/exim/manifests/init.pp
@@ -1,9 +1,9 @@
 class exim (
+  Boolean $is_bugsmx = false,
   Boolean $is_rtmaster = false,
 ) {
 
   $is_mailrelay = has_role('mailrelay')
-  $is_bugsmx = has_role('bugsmx')
   $is_packagesmaster = has_role('packagesmaster')
   $is_packagesqamaster = has_role('packagesqamaster')
 
-- 
2.20.1