Closed
Bug 1068653
Opened 10 years ago
Closed 10 years ago
Mirror try flag functionality in taskcluster decision task.
Categories
(Taskcluster :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlal, Assigned: jlal)
References
Details
Attachments
(1 file)
Eventually we need to support the same try flags that we have elsewhere but run them inside of taskcluster (in a decision task). The goal here is for people not to even notice that taskcluster is running their b2g jobs it should report identically to whatever we are doing now.
Comment 1•10 years ago
|
||
We should probably have a folder in-tree called test-tasks/ (or something), with YAML task templates. If we're really smart some declarative flag notation could be used to decide which tasks should be added to the graph based on try flags.
Comment 2•10 years ago
|
||
A while back I suggested the following format for in-tree config: https://gist.github.com/jonasfj/d16a2d6edd6dc75f1599 Note the trigger part that triggers tasks using trigger.name and trigger.dependsOn should probably be replaced with requires as suggested in the comment on the gist...
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jlal
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
Hopefully reviewing this on GH is not too awkward... I want to make sure we get this close to right while not waiting to long to land something so lets make sure the base is sane and continue work from my branch until we are fully ready to deploy the server side part.
Attachment #8507644 -
Flags: review?(wcosta)
Attachment #8507644 -
Flags: review?(mdas)
Attachment #8507644 -
Flags: review?(garndt)
Attachment #8507644 -
Flags: feedback?(jhford)
Updated•10 years ago
|
Attachment #8507644 -
Flags: review?(wcosta) → review+
Comment 4•10 years ago
|
||
Comment on attachment 8507644 [details] [review] https://github.com/lightsofapollo/gecko-dev/pull/2 looks good, comments in PR
Attachment #8507644 -
Flags: review?(mdas) → review+
Comment 5•10 years ago
|
||
Comment on attachment 8507644 [details] [review] https://github.com/lightsofapollo/gecko-dev/pull/2 Looks good. r+ with my comment about adding Z and treating the datetime as UTC.
Attachment #8507644 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 6•10 years ago
|
||
Landed in my staging branch https://github.com/lightsofapollo/gecko-dev/commit/21f6ef8b5c270a87980b03469c2c6640a0e424f6
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5edda558c655 https://hg.mozilla.org/mozilla-central/rev/f6a7135ba012
Updated•9 years ago
|
Component: TaskCluster → General
Product: Testing → Taskcluster
Target Milestone: --- → mozilla41
Version: unspecified → Trunk
Comment 8•9 years ago
|
||
Resetting Version and Target Milestone that accidentally got changed...
Target Milestone: mozilla41 → ---
Version: Trunk → unspecified
Updated•9 years ago
|
Attachment #8507644 -
Flags: feedback?(jhford)
You need to log in
before you can comment on or make changes to this bug.
Description
•