From 01d569e67380677e59f96a9dfe1221f76daac2ff Mon Sep 17 00:00:00 2001 From: Stefano Belforte Date: Fri, 1 Nov 2024 16:05:54 +0100 Subject: [PATCH] recover should succeed if all lumis are pub. Fix #5344 (#5345) --- src/python/CRABClient/Commands/recover.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/python/CRABClient/Commands/recover.py b/src/python/CRABClient/Commands/recover.py index 22d76cfc..2f616775 100644 --- a/src/python/CRABClient/Commands/recover.py +++ b/src/python/CRABClient/Commands/recover.py @@ -510,7 +510,10 @@ def stepReport(self): if notPublishedLumis: publishedAllLumis = False if publishedAllLumis: - self.logger.info("stepReport() - all lumis have been published in the output dataset. crab recover will exit") + msg = "stepReport() - all lumis have been published in the output dataset. crab recover will exit" + self.logger.info(msg) + retval.update({"commandStatus": "NothingToDo", "msg": msg}) + return retval else: if failingTaskPublish == "T" and self.options.__dict__["strategy"] == "notFinished": self.logger.warning("%sWarning%s: You are recovering a task with publication enabled with notFinished strategy, this will likely cause to have DUPLICATE LUMIS in the output dataset." % (colors.RED, colors.NORMAL))