Closed
Bug 1251734
Opened 8 years ago
Closed 8 years ago
Fix jetpack definitions
Categories
(Taskcluster :: Services, defect)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla46
People
(Reporter: armenzg, Assigned: armenzg)
References
Details
Attachments
(3 files, 1 obsolete file)
5.87 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
652 bytes,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
1.54 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
I've noticed some inconsistencies with the jetpack definitions.
Assignee | ||
Comment 1•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=2e7ab49dc5c4
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8724732 -
Flags: review?(jmaher)
Updated•8 years ago
|
Attachment #8724732 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f282f0355f9e5b5c6f806b02405412bd2ee11690 Bug 1251734 - Fix TC jetpack definition inconsistencies. r=jmaher
Assignee | ||
Comment 4•8 years ago
|
||
I've noticed that it is one of the offenders for the 3600 seconds time out bug.
Attachment #8724852 -
Flags: review?(jmaher)
Comment 5•8 years ago
|
||
Comment on attachment 8724852 [details] [diff] [review] increase timeout Review of attachment 8724852 [details] [diff] [review]: ----------------------------------------------------------------- good find
Attachment #8724852 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 6•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/95703fdc2175444a69dfb154f7d1bad74f19cf79 Bug 1251734 - Increase TC Linux64 jetpack timeout. DONTBUILD. r=jmaher
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f282f0355f9e https://hg.mozilla.org/mozilla-central/rev/95703fdc2175
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment 8•8 years ago
|
||
Moving closed bugs across to new Bugzilla product "TaskCluster".
status-firefox47:
fixed → ---
Component: TaskCluster → Integration
Product: Testing → Taskcluster
Target Milestone: mozilla47 → mozilla46
Assignee | ||
Comment 9•8 years ago
|
||
I totally thought I had landed this patch but it seems I didn't! This allows for this syntax: * -u jetpack * -u mochitest-jetpack In general, jetpack would already trigger with this: * -u all * -u mochitest
Attachment #8726714 -
Flags: review?(jmaher)
Assignee | ||
Comment 10•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b99156a09ba2
Comment 11•8 years ago
|
||
Comment on attachment 8726714 [details] [diff] [review] include two aliases for jetpack Review of attachment 8726714 [details] [diff] [review]: ----------------------------------------------------------------- is there a chance that we would run two jetpacks on a push? Do we really need to have this support 'jetpack' and 'mochitest-jetpack'?
Assignee | ||
Comment 12•8 years ago
|
||
It won't schedule it twice [1] I don't need the 'jetpack' alias. In my mind, I keep using 'jetpack' even though it is wrong syntax. I'm happy to drop it. What do you say? [1] armenzg@armenzg-thinkpad:~/repos/mozilla-central$ ./mach taskcluster-graph --pushlog-id=112616 --project=try '--message=try: -b d -p linux64 -u jetpack,mochitest -t none' --owner=armenzg@mozilla.com --level=1 --revision-hash=f016aa05b5c6f7864e97d638b71eba088f8ef899 --extend-graph | grep "jetpack" | grep "TC" cannot query vcs info because vcs info not provided "name": "[TC] Linux64 mochitest-jetpack",
Flags: needinfo?(jmaher)
Assignee | ||
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 13•8 years ago
|
||
lets drop jetpack, if it is mochitest-jetpack, lets make it more specific.
Flags: needinfo?(jmaher)
Assignee | ||
Comment 14•8 years ago
|
||
Attachment #8726714 -
Attachment is obsolete: true
Attachment #8726714 -
Flags: review?(jmaher)
Attachment #8726898 -
Flags: review?(jmaher)
Comment 15•8 years ago
|
||
Comment on attachment 8726898 [details] [diff] [review] include try syntax alias for Jetpack Review of attachment 8726898 [details] [diff] [review]: ----------------------------------------------------------------- thanks!
Attachment #8726898 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 16•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb913717b2f5e9cb21e1dbb256776afdeaac4712 Bug 1251734 - Add try syntax alias for jetpack. DONTBUILD. CLOSED TREE. r=jmaher
Assignee | ||
Comment 17•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=789b44958727
Comment 18•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fb913717b2f5
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Component: Integration → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•