Closed Bug 1194801 Opened 4 years ago Closed 4 years ago

Autophone not submitting results to Treeherder

Categories

(Testing :: Autophone, defect, P1, critical)

defect

Tracking

(firefox43 affected)

RESOLVED FIXED
Tracking Status
firefox43 --- affected

People

(Reporter: bc, Assigned: bc)

References

Details

Autophone gets a build's revision from the corresponding txt file, e.g.:

http://stage.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/firefox-43.0a1.en-US.linux-i686.txt

which is now a 40 char revision. This causes Treeherder's resultset api to fail to retrieve the result set:

https://treeherder.mozilla.org/api/project/mozilla-central/resultset/?revision=4e883591bb5dff021c108d3e30198a99547eed1e

I need to only use the first 12 characters in order to be able to pull up the revision hash for the build:

https://treeherder.mozilla.org/api/project/mozilla-central/resultset/?revision=4e883591bb5d

camd: maybe treeherder could just ignore everything past the 12 character revision.
From :gps in https://bugzilla.mozilla.org/show_bug.cgi?id=1175655#c0, we should be using the 40 character revision everywhere. Would it be possible to use the 40 character rev here instead of truncating it to 12?
camd: I'm going to switch to truncating the revision I submit to 12 characters. If we need to change Treeherder, lets do this compatibly. First allow both 12 and 40 characters, then announce the change and the need to update clients before we drop the 12 character support. Do you want to file the Treeherder bug?
Flags: needinfo?(cdawson)
https://github.com/mozilla/autophone/commit/0ee96a97a82b67639a8df0ae35119ddb07511323
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Hey Bob-- Sure thing.  I entered bug 1194908 for this.  There are possibly more implications than just a one-line fix in the code of that endpoint -- aren't there always?  :)

What's your urgency on this?  Just trying to prioritize.
Depends on: 1194908
Flags: needinfo?(cdawson)
camd: No urgency now. I have a work around where I trim the revision before submitting to the resultset api. My email alerts still have the full revision in their links to Treeherder but I rather than scatter this change around my code, I'll just manually edit those. This will of course break when Treeherder moves to the 40 char revision, so I'll have to revert the change when you deploy.
You need to log in before you can comment on or make changes to this bug.