From 45fc5c36dfd6e692fe098c63d998933d3bcafc73 Mon Sep 17 00:00:00 2001
From: Alex Vandiver <alexmv@zulip.com>
Date: Tue, 4 Nov 2025 15:30:16 +0000
Subject: [PATCH] puppet: Add a postfix.uninstall flag to allow skipping that
 step.

---
 puppet/zulip/manifests/local_mailserver.pp | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/puppet/zulip/manifests/local_mailserver.pp b/puppet/zulip/manifests/local_mailserver.pp
index e797ecfe57..a8ce3a264e 100644
--- a/puppet/zulip/manifests/local_mailserver.pp
+++ b/puppet/zulip/manifests/local_mailserver.pp
@@ -1,18 +1,20 @@
 class zulip::local_mailserver {
   include zulip::snakeoil
 
-  package { 'postfix':
-    # TODO/compatibility: We can remove this when upgrading directly
-    # from 10.x is no longer possible.  We do not use "purged" here,
-    # since that would remove config files, which users may have had
-    # installed.
-    ensure => absent,
+  if zulipconf('postfix', 'uninstall', true) {
+    package { 'postfix':
+      # TODO/compatibility: We can remove this when upgrading directly
+      # from 10.x is no longer possible.  We do not use "purged" here,
+      # since that would remove config files, which users may have had
+      # installed.
+      ensure => absent,
+      before => Service[$zulip::common::supervisor_service],
+    }
   }
   file { "${zulip::common::supervisor_conf_dir}/email-mirror.conf":
     ensure  => file,
     require => [
       Package[supervisor],
-      Package[postfix],
     ],
     owner   => 'root',
     group   => 'root',