1217906 - investigate to see if we can find builds using buildid

Status: RESOLVED FIXED

(Testing :: mozregression, defect)

Description

[Julien Pagès (:parkouss)]

So this comes from bug 1194941 c17. It would be a great feature to allow to find builds using buildid!

I think mozdownload is doing this, maybe we can discuss about how you do that :whimboo :) That would help for the nighly (archive.m.o) side of mozregression.

Our inbound side is full taskcluster. So we would need routes indexed uppon build id here - CC'ing mshal for this, see what we can do.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 10/03 - 10/13)]

We have a kinda nice trick here in mozdownload. Given that the build id is encoded in the directory name we simply reformat it: https://github.com/mozilla/mozdownload/blob/master/mozdownload/scraper.py#L376

If you want we can have a discussion too.

Comment 2

[Julien Pagès (:parkouss)]

(In reply to Henrik Skupin (:whimboo) from comment #1)
> We have a kinda nice trick here in mozdownload. Given that the build id is
> encoded in the directory name we simply reformat it:
> https://github.com/mozilla/mozdownload/blob/master/mozdownload/scraper.
> py#L376
> 
> If you want we can have a discussion too.

Thanks Whimbo! This is interesting. So at least we could do that for our nightlies (e.g. coming from archives.m.o). Looking.

Comment 3

[Julien Pagès (:parkouss)]

Attachment: allow to use build id to find builds

I implemented what :whimboo said, it seems to work very well!

So, this path would allow us to use build ids in --good/--bad/--launch. Example:

> mozregression --launch 20150511122605

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2] (away 10/03 - 10/13)]

Talked with Julien on IRC and we might want to have another discussion about using mozdownload for mozregression. We have such a big overlap meanwhile that I don't see why we should develop two things in parallel. Julien mentioned that we could talk about it in Orlando.

Comment 5

[William Lachance (:wlach)]

Comment on attachment 8687843 [details] [review]
allow to use build id to find builds

Some nits but overall looks good.

Comment 6

[William Lachance (:wlach)]

(In reply to Henrik Skupin (:whimboo) from comment #4)
> Talked with Julien on IRC and we might want to have another discussion about
> using mozdownload for mozregression. We have such a big overlap meanwhile
> that I don't see why we should develop two things in parallel. Julien
> mentioned that we could talk about it in Orlando.

We talked about this a few years ago. At the time it didn't really make sense to me, as it didn't seem like there was really that much code to share. I guess we can see if that view holds up in a post-taskcluster world.

Comment 7

[William Lachance (:wlach)]

(In reply to William Lachance (:wlach) from comment #6)
> (In reply to Henrik Skupin (:whimboo) from comment #4)
> > Talked with Julien on IRC and we might want to have another discussion about
> > using mozdownload for mozregression. We have such a big overlap meanwhile
> > that I don't see why we should develop two things in parallel. Julien
> > mentioned that we could talk about it in Orlando.
> 
> We talked about this a few years ago. At the time it didn't really make
> sense to me, as it didn't seem like there was really that much code to
> share. I guess we can see if that view holds up in a post-taskcluster world.

Some discussion of this in bug 997390.

Comment 8

[Julien Pagès (:parkouss)]

Thanks Will for the review. Fixed and merged: https://github.com/mozilla/mozregression/commit/fbfa55af69b7eb14a192c4743e833365dcf45bb2

I think I will do a minor release soon.