Use integration branch sha for treeherder revision_hash

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: kgrandon, Assigned: kgrandon)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Currently it looks like we're posting the same sha for re-triggered jobs, even after a failure/rebase. Instead I suppose we need to use the sha of the integration branch.

If we don't, we can end up with: https://treeherder.allizom.org/ui/#/jobs?repo=gaia-try&revision=e31b4373fef7dc04536df476e7c5650fb826dbd1

Probably need to update this: https://github.com/KevinGrandon/autolander/blob/master/lib/taskgraph.js#L108
Created attachment 8518630 [details] [review]
[PullReq] KevinGrandon:bug_1095250_update_revision_hash to KevinGrandon:master
(Assignee)

Comment 2

4 years ago
Hmm, this might be wrong. When I try this it appears I get no tasks in the treeherder display.
(Assignee)

Updated

4 years ago
Attachment #8518630 - Attachment is obsolete: true
Created attachment 8518665 [details] [review]
[PullReq] KevinGrandon:bug_1095250_alternate to KevinGrandon:master
(Assignee)

Comment 4

4 years ago
Comment on attachment 8518665 [details] [review]
[PullReq] KevinGrandon:bug_1095250_alternate to KevinGrandon:master

I think this is better, but still not 100% right. Will figure out the right thing tomorrow.
Attachment #8518665 - Flags: review+
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Whiteboard: [systemsfe]
You need to log in before you can comment on or make changes to this bug.