1247382 - Improve naming of Linux64 debug tasks

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Armen [:armenzg]]

Attachment: Improve naming

jmaher has a script to analyze runs of jobs between TaskCluster and Buildbot.
Unfortunately, he had to do some hacking of the naming to match the jobs [1]

[1] https://github.com/jmaher/randomtools/blob/master/newplatform/timeint.py#L50

Comment 1

[Armen [:armenzg]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=10951b451570

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8718039 [details] [diff] [review]
Improve naming

Review of attachment 8718039 [details] [diff] [review]:
-----------------------------------------------------------------

thank you!

Comment 3

[Armen [:armenzg]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/99f919028de4105680f223dad86168525d0a7d3b
Bug 1247382 - Improve naming of TaskCluster Linux64 debug jobs. DONTBUILD. r=jmaher

Comment 4

[Armen [:armenzg]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=9a627e1c4085

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/99f919028de4

Comment 6

[Armen [:armenzg]]

Attachment: changes.diff

For the tool I'm building, I want to make sure that every job has chunk defined in the name.

In the near future, I would like to add a check for the decision task to ensure this.

I've tried to normalize a bit more the names with this patch.

These are the actual differences (from grep -h "name: '\[TC" fx_linux64_* | sort):
@@ -6 +6 @@
-    name: '[TC] Linux64 jittest'
+    name: '[TC] Linux64 jittest-{{chunk}}'
@@ -12,6 +12,6 @@
-    name: '[TC] Linux64 mochitest-browser-chrome'
-    name: '[TC] Linux64 mochitest-chrome {{chunk}}'
-    name: '[TC] Linux64 mochitest {{chunk}}'
-    name: '[TC] Linux64 mochitest-devtools-chrome'
-    name: '[TC] Linux64 mochitest-devtools-chrome e10s M(dt{{chunk}})'
-    name: '[TC] Linux64 mochitest-e10s-browser-chrome'
+    name: '[TC] Linux64 mochitest-browser-chrome-{{chunk}}'
+    name: '[TC] Linux64 mochitest-browser-chrome-e10s-{{chunk}}'
+    name: '[TC] Linux64 mochitest-chrome -{chunk}}'
+    name: '[TC] Linux64 mochitest-{{chunk}}'
+    name: '[TC] Linux64 mochitest-devtools-chrome-{{chunk}}'
+    name: '[TC] Linux64 mochitest-devtools-chrome-e10s-{{chunk}}'
@@ -20 +20 @@
-    name: '[TC] Linux64 mochitest-plain e10s {{chunk}}'
+    name: '[TC] Linux64 mochitest-plain-e10s-{{chunk}}'
@@ -22,5 +22,5 @@
-    name: '[TC] Linux64 reftest {{chunk}}'
-    name: '[TC] Linux64 reftest {{chunk}}'
-    name: '[TC] Linux64 reftest e10s {{chunk}}'
-    name: '[TC] Linux64 web-platform-tests {{chunk}}'
-    name: '[TC] Linux64 web-platform-tests-e10s {{chunk}}'
+    name: '[TC] Linux64 reftest-{{chunk}}'
+    name: '[TC] Linux64 reftest-{{chunk}}'
+    name: '[TC] Linux64 reftest-e10s-{{chunk}}'
+    name: '[TC] Linux64 web-platform-tests-{{chunk}}'
+    name: '[TC] Linux64 web-platform-tests-e10s-{{chunk}}'
@@ -29 +29 @@
-    name: '[TC] Linux64 xpcshell {{chunk}}'
+    name: '[TC] Linux64 xpcshell-{{chunk}}'

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8718961 [details] [diff] [review]
changes.diff

Review of attachment 8718961 [details] [diff] [review]:
-----------------------------------------------------------------

::: testing/taskcluster/tasks/tests/fx_linux64_mochitest_bc_e10s.yml
@@ +12,5 @@
>    scopes:
>      - 'docker-worker:capability:device:loopbackVideo'
>      - 'docker-worker:capability:device:loopbackAudio'
>    metadata:
> +    name: '[TC] Linux64 mochitest-browser-chrome-e10s-{{chunk}}'

why is e10s swapped here?

::: testing/taskcluster/tasks/tests/fx_linux64_mochitest_chrome.yml
@@ +4,5 @@
>    scopes:
>      - 'docker-worker:capability:device:loopbackVideo'
>      - 'docker-worker:capability:device:loopbackAudio'
>    metadata:
> +    name: '[TC] Linux64 mochitest-chrome -{chunk}}'

please cleanup this:
* space before the -
* need two {

Comment 8

[Armen [:armenzg]]

> > +    name: '[TC] Linux64 mochitest-browser-chrome-e10s-{{chunk}}'
> 
> why is e10s swapped here?

All other e10s jobs follow that pattern 'suite-e10s-{{chunk}}'

Comment 9

[Joel Maher ( :jmaher ) (UTC -8)]

nice- keeping consistent will be great!

Comment 10

[Armen [:armenzg]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=20aaa816a3f4

Comment 11

[Armen [:armenzg]]

Comment on attachment 8718961 [details] [diff] [review]
changes.diff

Review of attachment 8718961 [details] [diff] [review]:
-----------------------------------------------------------------

I forgot to ask for review!

Comment 12

[Armen [:armenzg]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e898fed67a0

Comment 13

[Armen [:armenzg]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5d65efb34cd5ebdde5f6ff09de25fcb4165820d0
Bug 1247382 - Improve TaskCluster Linux64 debug test names. DONTBUILD. r=jmaher

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5d65efb34cd5

Comment 15

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.