android twinopen is failing due to a bug in talos configuration step

RESOLVED FIXED

Status

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

({intermittent-failure})

Trunk
ARM
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mobile_unittests][android_tier_1])

Attachments

(1 attachment)

In refactoring talos to allow for --develop and a in tool webserver, we broke twinopen.  This showed up when we updated talos on the foopies.  This is an easy and I have tested it locally.
Assignee

Comment 1

8 years ago
wlach, can we get a review on this today, this is fixing a red on all branches for android twinopen.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #576489 - Flags: review?(wlachance)
Comment on attachment 576489 [details] [diff] [review]
fix handling of remote line conversion (1.0)

The comment is a bit outside of its immediate context, which makes this a bit confusing. I think it would be a bit clearer if it said something like:

# Convert URLs containing .html files (and manifest files containing URLs) to 
# point at local webserver. .xul files should always be opened locally (for the 
# case that talos is driving a remote device, we copy over the .xul file in 
# remotePerfConfigurator.py)

A bit verbose but at least it's unambiguous. :)
Attachment #576489 - Flags: review?(wlachance) → review+
Assignee

Comment 3

8 years ago
landed on talos:
http://hg.mozilla.org/build/talos/rev/7eb0f8040d4c
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [orange][mobile_unittests][android_tier_1] → [mobile_unittests][android_tier_1]
You need to log in before you can comment on or make changes to this bug.