Closed
Bug 1488468
Opened 6 years ago
Closed 6 years ago
Resolve CI test failures for the telemetry-tests-client suite
Categories
(Toolkit :: Telemetry, enhancement, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: raphael, Assigned: raphael)
References
Details
Attachments
(11 files, 5 obsolete files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details |
Currently the telemetry-tests-client suite fails on CI due to an issue with installing Python dependencies.
See try build https://treeherder.mozilla.org/#/jobs?repo=try&revision=ba593ba8f3c602ae79e1b0d410218a515a541d61&filter-tier=1&filter-tier=2&filter-tier=3&selectedJob=197340728
Assignee | ||
Comment 1•6 years ago
|
||
Resolved an issue with installing "mozterm" from telemetry_tests_requirements.txt by updating "../tools/mozterm" to "../../python/mozterm".
Assignee | ||
Comment 2•6 years ago
|
||
It appears marionette/harness/marionette_harness/runner/httpd.py is coded against "WebTestHttpd" of wptserve 2.0, however the environment has 1.4 installed which is the latest release on PyPI and Mozilla's index server. The path to wptserve in config/marionette_requirements.txt is "../tools/wptserve", but probably needs to be "../web-platform/tests/tools/wptserve".
Comment 3•6 years ago
|
||
Setting as P1 (which is how we track bugs that are actively worked on).
Priority: -- → P1
Assignee | ||
Comment 4•6 years ago
|
||
Currently the telemetry-tests-client suite fails on CI. This patch addresses a series of problems with Python dependencies and the test implementation.
Comment 5•6 years ago
|
||
Comment on attachment 9011804 [details]
[telemetry] Resolve CI test failures in telemetry-tests-client suite (Bug 1488468).
Dave Hunt [:davehunt] ⌚️UTC+1 has approved the revision.
Attachment #9011804 -
Flags: review+
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
Depends on D7773
Assignee | ||
Comment 8•6 years ago
|
||
Depends on D7774
Assignee | ||
Comment 9•6 years ago
|
||
Depends on D7775
Assignee | ||
Comment 10•6 years ago
|
||
Depends on D7776
Assignee | ||
Comment 11•6 years ago
|
||
Depends on D7777
Assignee | ||
Comment 12•6 years ago
|
||
Depends on D7774
Assignee | ||
Comment 13•6 years ago
|
||
Depends on D7776
Assignee | ||
Comment 14•6 years ago
|
||
Assignee | ||
Comment 15•6 years ago
|
||
Depends on D7902
Assignee | ||
Comment 16•6 years ago
|
||
Depends on D7903
Assignee | ||
Comment 17•6 years ago
|
||
Depends on D7904
Assignee | ||
Comment 18•6 years ago
|
||
Depends on D7905
Assignee | ||
Comment 19•6 years ago
|
||
Try build for telemetry-tests-client
https://treeherder.mozilla.org/#/jobs?repo=try&revision=93f829cf058a833bfe8356a8a3cd3d41ba9c0156
QA Contact: gfritzsche
Updated•6 years ago
|
QA Contact: gfritzsche
Comment 20•6 years ago
|
||
Pushed by dhunt@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/246160f26d96
Bump wptserve in marionette_requirements.txt to 2.0.0; r=davehunt
https://hg.mozilla.org/integration/autoland/rev/af05b1808f36
Do not share ping_list of TelemetryTestCase between instances; r=davehunt
https://hg.mozilla.org/integration/autoland/rev/5084bccfd9ac
Update telemetry_client.py to expect three return values for evaluate_parser(); r=davehunt
https://hg.mozilla.org/integration/autoland/rev/b6f68f56cc7f
Remove wait.until calls and improve formatting in test_main_tab_scalar; r=davehunt
https://hg.mozilla.org/integration/autoland/rev/b2a9499e2c3e
Update requirements files for telemetry tests; r=davehunt
Updated•6 years ago
|
Attachment #9014841 -
Attachment is obsolete: true
Updated•6 years ago
|
Attachment #9014530 -
Attachment is obsolete: true
Updated•6 years ago
|
Attachment #9014537 -
Attachment is obsolete: true
Updated•6 years ago
|
Attachment #9014539 -
Attachment is obsolete: true
Updated•6 years ago
|
Attachment #9011804 -
Attachment is obsolete: true
Comment 21•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/246160f26d96
https://hg.mozilla.org/mozilla-central/rev/af05b1808f36
https://hg.mozilla.org/mozilla-central/rev/5084bccfd9ac
https://hg.mozilla.org/mozilla-central/rev/b6f68f56cc7f
https://hg.mozilla.org/mozilla-central/rev/b2a9499e2c3e
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Assignee | ||
Comment 22•6 years ago
|
||
Assignee | ||
Comment 23•6 years ago
|
||
Depends on D8531
Comment 24•6 years ago
|
||
:raphael as this bug is closed, we typically take care of follow ups by opening new bugs. I reviewed the patches before realising they're attached to a closed bug. Could you open a new bug and submit your patches again, and you can carry across my r+ or request it again and I'll approve/land them.
Updated•6 years ago
|
Flags: needinfo?(rpierzina)
Comment 25•6 years ago
|
||
Reopening the bug this time for the follow-ups. Next time please really create a new bug for that.
Status: RESOLVED → REOPENED
Flags: needinfo?(rpierzina)
Resolution: FIXED → ---
Comment 26•6 years ago
|
||
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bf88501c1f8a
Filter ping_list in Wait function of TelemetryTestCase; r=davehunt
https://hg.mozilla.org/integration/autoland/rev/51b87be56bca
Move HTTP POST request to wait_for_ping in TestPingServer; r=davehunt
Comment 27•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bf88501c1f8a
https://hg.mozilla.org/mozilla-central/rev/51b87be56bca
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•