Merge ~juliank/britney/+git/britney2-ubuntu:dont-group-ppc64el-rebuilds into ~ubuntu-release/britney/+git/britney2-ubuntu:master

[Julian Andres Klode]

Untested, but syntax parsed :D

[Julian Andres Klode (juliank)]

This is a hotfix so we get the ppc64el rebuilds moving again, but really should be configurable via hints long term.

[Andy Whitcroft (apw)]

That is a lot of a horror show ... that syntax is nasty, nasty, nasty. (Python I hate you.)

As far as I can see this would work as a temporary work-around. Someone needs to rip it out once it has done its thing.

review: Approve

[Łukasz Zemczak (sil2100)]

Let's do this then.

review: Approve

[Julian Andres Klode (juliank)]

@apw Normally I'd write

    if any(team in sourceppa for team in EXCLUDE)

but was following the line below :)

diff --git a/britney2/policies/sourceppa.py b/britney2/policies/sourceppa.py
index ea01e76..b203fb8 100644
--- a/britney2/policies/sourceppa.py
+++ b/britney2/policies/sourceppa.py
@@ -15,6 +15,7 @@ from britney2.policies.policy import BasePolicy, PolicyVerdict
 LAUNCHPAD_URL = "https://api.launchpad.net/1.0/"
 PRIMARY = LAUNCHPAD_URL + "ubuntu/+archive/primary"
 INCLUDE = ["~bileto-ppa-service/", "~ci-train-ppa-service/"]
+EXCLUDE = ["~ci-train-ppa-service/+archive/ubuntu/4810", "~ci-train-ppa-service/+archive/ubuntu/4815", "~ci-train-ppa-service/+archive/ubuntu/4816"]
 
 
 class SourcePPAPolicy(BasePolicy, Rest):
@@ -105,6 +106,8 @@ class SourcePPAPolicy(BasePolicy, Rest):
         sourceppa = self.lp_get_source_ppa(source_name, version) or ""
         verdict = excuse.policy_verdict
         self.source_ppas_by_pkg[source_name][version] = sourceppa
+        if [team for team in EXCLUDE if team in sourceppa]:
+            return PolicyVerdict.PASS
         if not [team for team in INCLUDE if team in sourceppa]:
             return PolicyVerdict.PASS