ArielGlenn has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/397790 )

Change subject: fix up for script that lists last good dumps for mirrors
......................................................................


fix up for script that lists last good dumps for mirrors

Did not handle properly the case where a wiki had never been dumped

Change-Id: Id03a41d384926220209c1e779ad6c6dcf8654d8f
---
M modules/dumps/files/web/list-last-n-good-dumps.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  ArielGlenn: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/dumps/files/web/list-last-n-good-dumps.py 
b/modules/dumps/files/web/list-last-n-good-dumps.py
index 6d7e80e..46070e5 100755
--- a/modules/dumps/files/web/list-last-n-good-dumps.py
+++ b/modules/dumps/files/web/list-last-n-good-dumps.py
@@ -206,7 +206,7 @@
         dir_first = get_first_dir(dirs, dir_to_check)
         if not dir_first:
             # never dumped
-            return dirs_reported
+            return dirs_reported, None
 
         text = get_dir_status(os.path.join(dir_to_check, dir_first))
         if "in-progress" in text:

-- 
To view, visit https://gerrit.wikimedia.org/r/397790
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id03a41d384926220209c1e779ad6c6dcf8654d8f
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: ArielGlenn <ar...@wikimedia.org>
Gerrit-Reviewer: ArielGlenn <ar...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to