The default bug view has changed. See this FAQ.

Autophone - update requirements and deploy thclient 1.6

RESOLVED FIXED

Status

Testing
Autophone
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox42 affected)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

2 years ago
Created attachment 8637228 [details] [diff] [review]
bug-1186420-v1.patch
Attachment #8637228 - Flags: review?(gbrown)
(Assignee)

Comment 2

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

thclient 1.6 is not backwards compatible.
Attachment #8637228 - Flags: review?(gbrown)
(Assignee)

Updated

2 years ago
Attachment #8637228 - Attachment is obsolete: true
(Assignee)

Updated

2 years ago
Depends on: 1177257
(Assignee)

Comment 3

2 years ago
Created attachment 8637933 [details] [diff] [review]
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
Attachment #8637933 - Flags: review?(gbrown)
(Assignee)

Comment 4

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

gbrown is away longer than I thought. Mark, can you take a look?
Attachment #8637933 - Flags: review?(gbrown) → review?(mcote)

Comment 5

2 years ago
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.
Attachment #8637933 - Flags: review?(mcote) → review+
(Assignee)

Comment 6

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

https://github.com/mozilla/autophone/commit/91eac8f41de0284560509fabd3297828531a70c9
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
(Assignee)

Updated

2 years ago
Blocks: 1187183
You need to log in before you can comment on or make changes to this bug.