Backed out changeset 8db96096643b (bug 1494333) for breaking nigthlies a=backout
authorAndreea Pavel <apavel@mozilla.com>
Thu, 11 Oct 2018 02:51:53 +0300
changeset 488923 2f85c2b55cd2f1b241f27b7ad8b9dfe42dfc139c
parent 488916 9ba60bd4482561db2a86f7ebcee1f0b66b32746b
child 488995 b89a744deccb5be6113036d95c5c208e1ae2b59f
push id246
push userfmarier@mozilla.com
push dateSat, 13 Oct 2018 00:15:40 +0000
reviewersbackout
bugs1494333
milestone64.0a1
backs out8db96096643ba13f2c5e5db526de96f1cf55dddc
Backed out changeset 8db96096643b (bug 1494333) for breaking nigthlies a=backout
.taskcluster.yml
taskcluster/taskgraph/actions/util.py
--- a/.taskcluster.yml
+++ b/.taskcluster.yml
@@ -88,18 +88,16 @@ tasks:
               else:
                 $if: 'tasks_for == "action"'
                 then:
                 - "notify.email.taskcluster-notifications+action-task@mozilla.com.on-failed"
                 - "notify.email.taskcluster-notifications+action-task@mozilla.com.on-exception"
                 - "index.gecko.v2.${repository.project}.pushlog-id.${_pushId}.actions.${ownTaskId}"
                 else:  # cron
                 - "index.gecko.v2.${repository.project}.latest.taskgraph.decision-${cron.job_name}"
-                # list each cron task on this revision, so actions can find them
-                - "index.gecko.v2.${repository.project}.revision.${push.revision}.cron.${ownTaskId}"
                 # These are the old index routes for the decision task.
                 - "index.gecko.v2.${repository.project}.latest.firefox.decision-${cron.job_name}"
 
         scopes:
           $if: 'tasks_for == "hg-push"'
           then:
             - 'assume:repo:${repoUrl[8:]}:branch:default'
             - 'queue:route:notify.email.${ownerEmail}.*'
--- a/taskcluster/taskgraph/actions/util.py
+++ b/taskcluster/taskgraph/actions/util.py
@@ -82,37 +82,22 @@ def fetch_graph_and_labels(parameters, g
     label_to_taskid = get_artifact(decision_task_id, "public/label-to-taskid.json")
 
     # Now fetch any modifications made by action tasks and swap out new tasks
     # for old ones
     namespace = '{}.v2.{}.pushlog-id.{}.actions'.format(
         graph_config['trust-domain'],
         parameters['project'],
         parameters['pushlog_id'])
-    for task_id in list_tasks(namespace):
-        logger.info('fetching label-to-taskid.json for action task {}'.format(task_id))
+    for action in list_tasks(namespace):
         try:
-            run_label_to_id = get_artifact(task_id, "public/label-to-taskid.json")
+            run_label_to_id = get_artifact(action, "public/label-to-taskid.json")
             label_to_taskid.update(run_label_to_id)
         except HTTPError as e:
-            logger.debug('No label-to-taskid.json found for {}: {}'.format(task_id, e))
-            continue
-
-    # Similarly for cron tasks..
-    namespace = '{}.v2.{}.revision.{}.cron'.format(
-        graph_config['trust-domain'],
-        parameters['project'],
-        parameters['head_rev'])
-    for task_id in list_tasks(namespace):
-        logger.info('fetching label-to-taskid.json for cron task {}'.format(task_id))
-        try:
-            run_label_to_id = get_artifact(task_id, "public/label-to-taskid.json")
-            label_to_taskid.update(run_label_to_id)
-        except HTTPError as e:
-            logger.debug('No label-to-taskid.json found for {}: {}'.format(task_id, e))
+            logger.info('Skipping {} due to missing artifact! Error: {}'.format(action, e))
             continue
 
     return (decision_task_id, full_task_graph, label_to_taskid)
 
 
 def create_task_from_def(task_id, task_def, level):
     """Create a new task from a definition rather than from a label
     that is already in the full-task-graph. The task definition will