Closed Bug 1284911 Opened 5 years ago Closed 5 years ago
Enable Treeherder's ability to add Task
Cluster jobs to try pushes
47 bytes, text/x-github-pull-request
|Details | Review|
In bug 1254325 we added TaskCluster jobs to Treeherder's runnable API. In this bug we will enable the feature of adding new TaskCluster jobs through Treeherder's UI. This depends on being able to schedule actions tasks via pulse_actions (bug 1281062).
Once this has been landed, the following test cases need to be evaluated :- 1. Only a Build is scheduled 2. Only a Test is scheduled 3. Both a build and test are scheduled 4. Test depending on a running build is scheduled We also need to follow up by further optimizing action tasks to utilize old action tasks in same push.
Comment on attachment 8771000 [details] [review] [treeherder] martiansideofthemoon:thefinalcountdown > mozilla:master This Pull Request should be merged once the Pulse Actions credentials have been updated. That should happen later today right @Armen?
martianwars tested these case scenarions and I verified it. This push only requested a L64 debug test with a reftest job. https://treeherder.mozilla.org/#/jobs?repo=try&revision=20af57a728fb We used that push for cases 1, 2 & 4. For case #3 we used this push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=31ed08cd2d7d69c7f12222597124dfaea8b285f7 #1 - Only a build is scheduled martianwars requested an Android build. You can see the aftermath in this screenshot  The build started running after the action task completed. You can see the task group for it   http://people.mozilla.org/~armenzg/sattap/0bd7cead.png  https://tools.taskcluster.net/task-group-inspector/#FEWCgomXRMKCzZ98cP1sDw/ #2 - Only a test is scheduled I have requested an Android debug test. Since the test depends on a build that's not running, a build has been scheduled. Here is the task group for this . A snapshot of the result can be seen in . * android debug build scheduled . Now complete * android debug test scheduled . Now it's complete.  https://tools.taskcluster.net/task-group-inspector/#Ho0l-41aR1yBfhqgWP7bDw/  http://picpaste.com/Screenshot_from_2016-07-14_21_51_21-sBKp8gvh.png  https://tools.taskcluster.net/task-inspector/#HUHyR2_XQd-X745gDTYZGA/0  https://tools.taskcluster.net/task-inspector/#RQ-sEUU6TAmbPDJHQFyZrA/0 #3 - Both a build and test are scheduled We can see both test and build scheduled in the group inspector: https://tools.taskcluster.net/task-group-inspector/#GtWiBT5jRPO79oAziMws3g/ #4 - Test depending on a running build is scheduled In this case, only an action task is scheduled  since the build is still running. From looking at the output I can see that a task has been scheduled  If I open the group inspector I only see that task , however, in the task inspector I can see that it depends on: * desktop test docker imager builder  (completed) * Linux 64 debug job  (running) (now complete) * An e10s mochitest job is now running . Now it's complete.  https://tools.taskcluster.net/task-inspector/#G_ICo2v9TJy5KQf1vYfFsw/0  https://tools.taskcluster.net/task-inspector/#dxBXPA9qTy-DpvLjxDYydg/  https://tools.taskcluster.net/task-group-inspector/#a2ybAUuES82lln9dlFMZuA/  https://tools.taskcluster.net/task-inspector/#SzWsEsVwRJihXcPvY7uwjA/  https://tools.taskcluster.net/task-inspector/#HprF83hvRNmYD58wOYTsAA/  https://tools.taskcluster.net/task-inspector/#dxBXPA9qTy-DpvLjxDYydg/0
I'm hoping garndt or dustin can get to update the credentials either today or tomorrow. I think we can go ahead and enable it on Treeherder since it will take a bit before we make a production push for Treeherder.
Comment on attachment 8771000 [details] [review] [treeherder] martiansideofthemoon:thefinalcountdown > mozilla:master Left a comment on the PR
Comment on attachment 8771000 [details] [review] [treeherder] martiansideofthemoon:thefinalcountdown > mozilla:master I hope it's better now
What did we decide last week on IRC? Can we enable it this week?
Commits pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/9a30c530ee1a2d49ad91bb12fd9334b40ccd9ef5 Bug 1284911 - Enable Treeherder's ability to add TaskCluster jobs to try pushes. https://github.com/mozilla/treeherder/commit/915c962eb3d010e290bf3369e7ad43e95c96f81b Bug 1284911 - Adding check to avoid confusion with old action tasks. Action Tasks are tasks which will be generated when the "Add New Jobs" button is pressed, and TaskCluster jobs are scheduled. Action Tasks will contain all the relevant logs. An orange Action Task would mean "Add New Jobs" has failed for Action Tasks. Action Tasks are placed under the gecko-decision group. This check improves the filter which earlier assumed that there was just a single job under gecko-decision. Hence when we've scheduled an Action Task on that push in the past, this filter will be necessary to avoid internal server errors.
Seems to work well! https://treeherder.mozilla.org/#/jobs?repo=try&revision=b649a1cda39222950c94ab6c3fe4c6d61a3ff149
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.