Bug 1289122 - bugfix to chain together the elements to be removed. r=dustin draft
authorMihai Tabara <mtabara@mozilla.com>
Tue, 26 Jul 2016 00:11:40 +0100
changeset 392651 6106107c6001c0781421cd7bda3356e9e42a0f33
parent 392337 a0c7e7df8772ca95f38ef12a41baeb5bf9ffd531
child 526374 c0deec8d487299367efe9b86c587d069fd4f390a
push id24077
push userbmo:mtabara@mozilla.com
push dateMon, 25 Jul 2016 23:12:28 +0000
reviewersdustin
bugs1289122
milestone50.0a1
Bug 1289122 - bugfix to chain together the elements to be removed. r=dustin MozReview-Commit-ID: 3XhHZgmq6e3
taskcluster/taskgraph/task/legacy.py
--- a/taskcluster/taskgraph/task/legacy.py
+++ b/taskcluster/taskgraph/task/legacy.py
@@ -174,21 +174,22 @@ def remove_coalescing_from_task(task):
     :param task: task definition.
     """
     patterns = [
         re.compile("^coalesce.v1.builds.*pgo$"),
     ]
 
     try:
         payload = task["task"]["payload"]
-        payload.pop("supersederUrl")
-
         routes = task["task"]["routes"]
         removable_routes = [route for route in list(routes)
                             if any([p.match(route) for p in patterns])]
+        if removable_routes:
+            # we remove supersederUrl only when we have also routes to remove
+            payload.pop("supersederUrl")
 
         for route in removable_routes:
             routes.remove(route)
     except KeyError:
         pass
 
 
 def query_vcs_info(repository, revision):