launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #30148
[Merge] ~cjwatson/launchpad:clarify-domination-messages into launchpad:master
Colin Watson has proposed merging ~cjwatson/launchpad:clarify-domination-messages into launchpad:master.
Commit message:
Clarify domination logging a bit
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/445188
I wrote these log messages, and even I don't quite immediately associate "supersession" with "the act of superseding" (though that's the correct dictionary form). Adjust these messages to be slightly more accessible.
--
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:clarify-domination-messages into launchpad:master.
diff --git a/lib/lp/archivepublisher/domination.py b/lib/lp/archivepublisher/domination.py
index d705f81..466d1ba 100644
--- a/lib/lp/archivepublisher/domination.py
+++ b/lib/lp/archivepublisher/domination.py
@@ -708,7 +708,7 @@ class Dominator:
def execute_plan():
if supersede:
- self.logger.info("Applying supersessions...")
+ self.logger.info("Superseding binaries...")
for pub, dominant in supersede:
pub.supersede(dominant, logger=self.logger)
IStore(pub).flush()
@@ -723,7 +723,7 @@ class Dominator:
dominated.supersede(dominant, logger=self.logger)
IStore(dominated).flush()
if delete:
- self.logger.info("Applying deletions...")
+ self.logger.info("Deleting binaries...")
for pub in delete:
pub.requestDeletion(None)
IStore(pub).flush()