launchpad-reviewers team mailing list archive

[Andrey Fedoseev <mp+428395@xxxxxxxxxxxxxxxxxx>]

Andrey Fedoseev has proposed merging ~andrey-fedoseev/launchpad:ci-build-email into launchpad:master.

Commit message:
Allow notifications for failed CI builds

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~andrey-fedoseev/launchpad/+git/launchpad/+merge/428395

One line change to allow the buildd manager to send notifications for the failed CI builds
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~andrey-fedoseev/launchpad:ci-build-email into launchpad:master.

diff --git a/lib/lp/code/model/cibuildbehaviour.py b/lib/lp/code/model/cibuildbehaviour.py
index 7655ff8..726bb80 100644
--- a/lib/lp/code/model/cibuildbehaviour.py
+++ b/lib/lp/code/model/cibuildbehaviour.py
@@ -108,7 +108,7 @@ class CIBuildBehaviour(BuilderProxyMixin, BuildFarmJobBehaviourBase):
     builder_type = "ci"
     image_types = [BuildBaseImageType.LXD, BuildBaseImageType.CHROOT]
 
-    ALLOWED_STATUS_NOTIFICATIONS = []
+    ALLOWED_STATUS_NOTIFICATIONS = ["PACKAGEFAIL"]
 
     def redactXmlrpcArguments(self, args):
         # we do not want to have secrets in logs