Closed Bug 1647640 Opened 4 years ago Closed 4 years ago

make perftest Python 3.5 compatible

Categories

(Testing :: mozperftest, defect, P1)

defect

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
Depends on: 1647648
Summary: don't use f-strings → make perftest Python 3.5 compatible

This patches uses future-fstrings to make them available
with 3.5, and fix the code and tests for 3.5 compat

Blocks: 1644778

We're moving talos boxes to python 3.8

See Bug 1647648

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Attachment #9158886 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: