launchpad-reviewers team mailing list archive

[Guruprasad <mp+491121@xxxxxxxxxxxxxxxxxx>]

Guruprasad has proposed merging ~lgp171188/launchpad-bastion-scripts:more-staging-restore-script-fixes into launchpad-bastion-scripts:main.

Commit message:
Remove the staging restore commands to sync mailman from production

This has not worked since the PS5 migration and atemoya doesn't exist any more


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~lgp171188/launchpad-bastion-scripts/+git/launchpad-bastion-scripts/+merge/491121
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad-bastion-scripts:more-staging-restore-script-fixes into launchpad-bastion-scripts:main.

diff --git a/staging-restore/staging_restore.sh b/staging-restore/staging_restore.sh
index 4762882..1b70334 100755
--- a/staging-restore/staging_restore.sh
+++ b/staging-restore/staging_restore.sh
@@ -206,11 +206,6 @@ then
     echo "$(date) Syncing branches from production"
     in-model staging juju run-action --wait launchpad-codehosting/leader sync-branches branches='bzr bzrtools bzrtools/0.12 bzrtools/2.3 drizzle' >> "$LOGFILE" 2>&1
     echo "$(date) Synced branches from production"
-
-    # Sync staging mailman to production
-    echo $(date) "About to sync mailman to production" >> $LOGFILE
-    ssh launchpad@xxxxxxxxxxxxxxxxxxxxx /srv/lists.staging.launchpad.net/scripts/list_sync.sh >> $LOGFILE 2>&1
-    echo $(date) "Mailman synced to production" >> $LOGFILE
 fi
 
 # Testing URLs