From aeb8090d64c250e3b7473ce21cfc9eaff088cb19 Mon Sep 17 00:00:00 2001 From: Dale Wahl Date: Tue, 6 Feb 2024 13:38:56 +0100 Subject: [PATCH] do not exit migrate.py after successfully removing git user in Docker This fix may have never worked. I don't understand as I know we tested extensively. Perhaps it updated and then migrate was run again by another process after the user was removed. --- helper-scripts/migrate.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/helper-scripts/migrate.py b/helper-scripts/migrate.py index 7ba04971d..005863ea9 100644 --- a/helper-scripts/migrate.py +++ b/helper-scripts/migrate.py @@ -254,10 +254,10 @@ def finish(args, logger): logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.") logger.info(fetch.stderr) exit(1) - - logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.") - logger.info(fetch.stderr) - exit(1) + else: + logger.info("Error while fetching latest tags with git. Check that the repository URL is correct.") + logger.info(fetch.stderr) + exit(1) if args.branch: logger.info(f" ...checking out branch '{args.branch}'")