In the case of mass-retriggers, having to click the button repeatedly gets old in a hurry. While BuildAPI is great for buildbot jobs, Taskcluster doesn't have any sort of similar functionality that I'm aware of, so that limits us to what the Treeherder interface gives us. So I'd like to have a workflow of using the arrow keys to move between jobs and a hotkey I can hit quickly to retrigger the jobs.
So I have this pretty much done. As discussed in channel, we are considering the following change: 'r' - retrigger job 'b' - add related bug (** was 'r') This means folks who are used to 'r' need to be made aware, so adding other sheriffs for agreement. If there's another shortcut key we'd prefer to use instead for retrigger which preserves 'r' for add-related-bug, let me know. The first day may be a bit painful during the transition to production, but ultimately if we think it's the right key we might as well do it before it becomes even more entrenched, I think.
I am +1 to r :)
Jmaher is fine with the change also.
Created attachment 8578741 [details] [review] treeherder-ui-PR#425 Please see above PR for status and review.
Comment on attachment 8578741 [details] [review] treeherder-ui-PR#425 r+ with the comment addressed :-)
I've made the change, it's totally the right thing to do. I re-tested and it appears to be fine. I'll check in with Wes if he's around, before a merge.
Commits pushed to master at https://github.com/mozilla/treeherder-ui https://github.com/mozilla/treeherder-ui/commit/99bedc1543a044356149752a808bf1f83fc62f46 Bug 1143666 - Add a shortcut for retrigger job https://github.com/mozilla/treeherder-ui/commit/ca0c4e8a00da169558f9f07cca6d9b5ddd25fed1 Merge pull request #425 from tojonmz/retrigger-shortcut Bug 1143666 - Add a shortcut for retrigger job
I mailed Wes to let him know of the change. Marking fixed per above merge. Will verify on the next push to stage/prod.
Verified fixed on stage and prod.
Commits pushed to master at https://github.com/mozilla/treeherder https://github.com/mozilla/treeherder/commit/eea9be46a32c4e61257c714689c8496cca04b4b4 Bug 1143666 - Add a shortcut for retrigger job https://github.com/mozilla/treeherder/commit/8cc0f7d4629db49dd6e8a5f53b7285ecf2a5f50d Merge pull request #425 from tojonmz/retrigger-shortcut Bug 1143666 - Add a shortcut for retrigger job