Closed
Bug 1647640
Opened 4 years ago
Closed 4 years ago
make perftest Python 3.5 compatible
Categories
(Testing :: mozperftest, defect, P1)
Testing
mozperftest
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: tarek, Assigned: tarek)
References
Details
Attachments
(1 obsolete file)
We have environments with an older version of python 3, like talos generic workers.
Let's avoid f-strings
[task 2020-06-23T08:23:18.152Z] Traceback (most recent call last):
[task 2020-06-23T08:23:18.152Z] File "python/mozperftest/mozperftest/runner.py", line 168, in <module>
[task 2020-06-23T08:23:18.152Z] sys.exit(main())
[task 2020-06-23T08:23:18.152Z] File "python/mozperftest/mozperftest/runner.py", line 152, in main
[task 2020-06-23T08:23:18.152Z] from mozperftest import PerftestArgumentParser
[task 2020-06-23T08:23:18.152Z] File "/home/cltbld/tasks/task_1592896703/builds/worker/checkouts/gecko/python/mozperftest/mozperftest/__init__.py", line 6, in <module>
[task 2020-06-23T08:23:18.152Z] from mozperftest.argparser import PerftestArgumentParser # noqa
[task 2020-06-23T08:23:18.152Z] File "/home/cltbld/tasks/task_1592896703/builds/worker/checkouts/gecko/python/mozperftest/mozperftest/argparser.py", line 19, in <module>
[task 2020-06-23T08:23:18.152Z] from mozperftest.test import get_layers as test_layers # noqa
[task 2020-06-23T08:23:18.152Z] File "/home/cltbld/tasks/task_1592896703/builds/worker/checkouts/gecko/python/mozperftest/mozperftest/test/__init__.py", line 4, in <module>
[task 2020-06-23T08:23:18.152Z] from mozperftest.test.browsertime import BrowsertimeRunner
[task 2020-06-23T08:23:18.152Z] File "/home/cltbld/tasks/task_1592896703/builds/worker/checkouts/gecko/python/mozperftest/mozperftest/test/browsertime/__init__.py", line 5, in <module>
[task 2020-06-23T08:23:18.152Z] from mozperftest.test.browsertime.runner import BrowsertimeRunner # noqa
[task 2020-06-23T08:23:18.152Z] File "/home/cltbld/tasks/task_1592896703/builds/worker/checkouts/gecko/python/mozperftest/mozperftest/test/browsertime/runner.py", line 297
[task 2020-06-23T08:23:18.152Z] result_dir = pathlib.Path(output, f"browsertime-results-{cycle}")
[task 2020-06-23T08:23:18.152Z] ^
[task 2020-06-23T08:23:18.152Z] SyntaxError: invalid syntax
Assignee | ||
Updated•4 years ago
|
Summary: don't use f-strings → make perftest Python 3.5 compatible
Assignee | ||
Comment 1•4 years ago
|
||
This patches uses future-fstrings to make them available
with 3.5, and fix the code and tests for 3.5 compat
Assignee | ||
Comment 2•4 years ago
|
||
We're moving talos boxes to python 3.8
See Bug 1647648
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Updated•4 years ago
|
Attachment #9158886 -
Attachment is obsolete: true
You need to log in
before you can comment on or make changes to this bug.
Description
•