Closed Bug 1247382 Opened 9 years ago Closed 9 years ago

Improve naming of Linux64 debug tasks

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(firefox47 fixed)

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: armenzg, Unassigned)

References

Details

Attachments

(2 files)

Attached patch Improve naming (deleted) — Splinter Review
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
Attachment #8718039 - Flags: review?(jmaher)
Comment on attachment 8718039 [details] [diff] [review] Improve naming Review of attachment 8718039 [details] [diff] [review]: ----------------------------------------------------------------- thank you!
Attachment #8718039 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/99f919028de4105680f223dad86168525d0a7d3b Bug 1247382 - Improve naming of TaskCluster Linux64 debug jobs. DONTBUILD. r=jmaher
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---
Attached patch changes.diff (deleted) — Splinter Review
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}}'
Attachment #8718961 - Flags: feedback?(jmaher)
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 {
Attachment #8718961 - Flags: feedback?(jmaher) → feedback+
> > + name: '[TC] Linux64 mochitest-browser-chrome-e10s-{{chunk}}' > > why is e10s swapped here? All other e10s jobs follow that pattern 'suite-e10s-{{chunk}}'
nice- keeping consistent will be great!
Comment on attachment 8718961 [details] [diff] [review] changes.diff Review of attachment 8718961 [details] [diff] [review]: ----------------------------------------------------------------- I forgot to ask for review!
Attachment #8718961 - Flags: review?(jmaher)
Attachment #8718961 - Flags: review?(jmaher) → review+
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: