Closed
Bug 1254804
Opened 8 years ago
Closed 8 years ago
mozregression doesn't properly visit inbound builds.
Categories
(Testing :: mozregression, defect)
Testing
mozregression
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jrmuizel, Assigned: parkouss)
References
Details
Attachments
(1 file)
In bug 1251868 we ran into this weird log: https://bug1251868.bmoattachments.org/attachment.cgi?id=8727934 which caused us to not properly bisect the inbound builds.
Assignee | ||
Comment 1•8 years ago
|
||
From what I can see, we ends up with the following last pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c1e0d1890cfee9d86c8d566b0490053f21e0afc6&tochange=97cf677ee66802809808a3e61a0ccb89542ca54e And this looks like a merge, but without information about it - so mozregressio does not know where these commits comes from and just stop. Indeed, they are on inbound: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?tochange=97cf677ee668&fromchange=9ecc9682c3a3 So you can manually ask mozregression to bisect that: > mozregression --good 9ecc9682c3a3 --bad 97cf677ee668 --repo inbound Eventually, we could make mozregression smarter, trying to bisect on mozilla-inbound automatically if we don't find any merge info and that we have multiple commits with different authors. (So it should apply the above logic automatically)
Assignee | ||
Comment 2•8 years ago
|
||
This should be good - it works for the current example at least.
Comment 3•8 years ago
|
||
Comment on attachment 8728324 [details] [review] auto switch on m-i Thanks!! One comment in the PR.
Attachment #8728324 -
Flags: review?(wlachance) → review+
Assignee | ||
Comment 4•8 years ago
|
||
Thanks Will - and sorry for the dely here. :) I fixed the message as you suggested. I'm going to create a patch release with this fix now. landed in https://github.com/mozilla/mozregression/commit/2a886a5a79f2afcef2c073ca340006f7a8fe27c6.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•