Closed Bug 1639024 Opened 5 years ago Closed 4 years ago

Convert 'mach mozharness' to run with Python 3

Categories

(Release Engineering :: Applications: MozharnessCore, task)

Tracking

(firefox88 fixed)

RESOLVED FIXED
Tracking Status
firefox88 --- fixed

People

(Reporter: KrisWright, Assigned: ahal)

References

Details

Attachments

(1 file)

mach mozharness still runs with python 2 [1]. We want to convert all mach commands to python 3.

To do this, any dependencies of the command may need to be converted as well as the command itself.

[1] https://searchfox.org/mozilla-central/rev/9193635dca8cfdcb68f114306194ffc860456044/testing/mozharness/mach_commands.py

Hi -- is there a status update on this work?

Flags: needinfo?(aki)

I think this is dependent on the test harnesses being py3 compatible. I haven't been directly involved in the porting, but I do know some patches have landed. #python may know more.

mhentges:crab: 12:40
Very good question, I'm unsure. All (regular?) CI jobs doing ./mach python mozharness ... should by py3-compatible, but I'm not sure about the rest of it

Flags: needinfo?(aki)
Depends on: 1532427

Since ./mach mozharness is never run (IIRC), perhaps we can just rip that command out?

For context this command was only added to help developers replicate a mozharness invocation locally. It's not used in CI.

I was talking to James about this earlier.. We think it could definitely provide value with a little bit of work. That said I just tried to run it and discovered it's busted due to bug 1697919. I think removing it or fixing it up and making it useful are both viable options (I'm slightly inclined to the latter).

Either way, let's not block the py3 migration on this.. I'll just make it run with Python 3 and we can decide what to do with this later.

Assignee: nobody → ahal
Status: NEW → ASSIGNED

Sounds good to me 👍

Pushed by ahalberstadt@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/68f9e3957c4d Run |mach mozharness| with Python 3, r=releng-reviewers,aki
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: