1186420 - Autophone - update requirements and deploy thclient 1.6

Status: RESOLVED FIXED

(Testing Graveyard :: Autophone, defect)

Comment 1

[Bob Clary [:bc] (inactive)]

Attachment: bug-1186420-v1.patch

Comment 2

[Bob Clary [:bc] (inactive)]

Comment on attachment 8637228 [details] [diff] [review]
bug-1186420-v1.patch

thclient 1.6 is not backwards compatible.

Comment 3

[Bob Clary [:bc] (inactive)]

Attachment: bug-1186420-v2.patch

smoketest, s1s2test, webappstartuptest:

https://treeherder.allizom.org/#/jobs?filter-searchStr=autophone&exclusion_profile=false&repo=mozilla-inbound&fromchange=f6d29009ae0a&tochange=9be3d57c2e15

Mochitest-media:

https://treeherder.allizom.org/#/jobs?repo=mozilla-inbound&revision=609926a39887&exclusion_profile=false&filter-searchStr=autophone

Comment 4

[Bob Clary [:bc] (inactive)]

Comment on attachment 8637933 [details] [diff] [review]
bug-1186420-v2.patch

gbrown is away longer than I thought. Mark, can you take a look?

Comment 5

[Mark Côté [:mcote]]

Comment on attachment 8637933 [details] [diff] [review]
bug-1186420-v2.patch

Review of attachment 8637933 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm.  In the future, perhaps you can rope jmaher in for reviews, since he's been doing Autophone stuff recently. :)

::: autophonetreeherder.py
@@ +383,5 @@
> +                # Job Info panel, it is important to only specify one log
> +                # file to prevent the multiple log buttons from hiding the
> +                # retrigger button. If the test is a Unit Test, its log
> +                # will marked as the log file. Otherwise, the Autophone
> +                # log will be marked as the log file.

Heh kind of gross that we have to hack around UI problems by limiting what we send.  Can you ensure there's a Treeherder bug on file and mention it here, so that some day we could send them all?  This assumes it's useful to specify multiple log files; if not, please modify the comment appropriately.

Comment 6

[Bob Clary [:bc] (inactive)]

I'm still breaking him in. You haven't gotten rid of me and Autophone quite yet. ;-)

https://github.com/mozilla/autophone/commit/91eac8f41de0284560509fabd3297828531a70c9