691090 - Fennec Talos doesn't need to TinderboxPrint moz: and mobile: revs anymore

Status: RESOLVED FIXED

(Release Engineering :: General, defect, P2)

Description

[Phil Ringnalda (:philor)]

I'm not sure whether any part of the block in http://mxr.mozilla.org/build/source/buildbotcustom/process/factory.py#7204 is still doing something useful, but the part that TinderboxPrints the moz: and mobile: revisions certainly isn't, since they are the same revision, a revision which will later be TinderboxPrinted as rev:, a way of putting it which tbpl knows how to ignore because it doesn't actually want to be told the revision via TinderboxPrint at all, much less three times over with three different labels :)

Comment 2

[Chris Cooper [:coop] (he/him)]

Aki: do we need anything that's in the fennec-specific block that begins at https://hg.mozilla.org/build/buildbotcustom/file/36c8c6de903f/process/factory.py#l7182 ? 

I'd rather rip out the whole thing if possible rather than just the output step.

Comment 3

[Aki Sasaki (not active)]

(In reply to Chris Cooper [:coop] from comment #2)
> Aki: do we need anything that's in the fennec-specific block that begins at
> https://hg.mozilla.org/build/buildbotcustom/file/36c8c6de903f/process/
> factory.py#l7182 ? 
> 
> I'd rather rip out the whole thing if possible rather than just the output
> step.

Nuke it! :)
Thanks Coop!

Comment 4

[Chris Cooper [:coop] (he/him)]

Attachment: Stop printing mobile revisions we don't use/need

Comment 5

[Chris Cooper [:coop] (he/him)]

Comment on attachment 575572 [details] [diff] [review]
Stop printing mobile revisions we don't use/need

https://hg.mozilla.org/build/buildbotcustom/rev/cb12f1cf88d6