Support python 3 in mozbase.

NEW
Unassigned

Status

Testing
Mozbase
3 years ago
a month ago

People

(Reporter: Ms2ger, Unassigned)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
#ateam seemed generally supportive of supporting python 3 alongside python 2.7, to avoid forcing new consumers to stick to python 2.
(Reporter)

Comment 1

3 years ago
Created attachment 8516081 [details] [diff] [review]
WIP

This allows us to at least run most mozlog tests under python 3; a bunch fail because of the expected byte/string issues.

Actual patches should probably end up in dependent bugs.

Comment 2

5 months ago
Python 2.7 will be EOL in 2020 (https://www.python.org/dev/peps/pep-0373/#update), and by not supporting Python 3 we are preventing other packages from adding mozbase packages as dependencies if they want to support Python 3. Should we turn this into a tracking bug, and raise blocking bugs for each mozbase package?

We should probably start with at least running the mozbase tests against Python 3, so that once we fix this, we avoid regressions in the future.
CC'ing Gregory to this bug given that he knows best about the build system and what we could do for Python 3 support.
(In reply to Henrik Skupin (:whimboo) from comment #3)
> CC'ing Gregory to this bug given that he knows best about the build system
> and what we could do for Python 3 support.

It might help to also do it.

Comment 5

5 months ago
While I personally like a lot of the advancements in Python 3, there is really not yet a compelling reason for us to prioritize supporting Python 3 in the build system. Python 2.7 works just fine and spending the time to support Python 3 for little to no end-user benefit is difficult to justify when we could be spending time improving the build system in other ways.

That being said, most of the Python code I've written in the past few years strives to be compatible with 2.7 and 3.5+. I encourage others to take this approach so an eventual Python 3 port is easier. Also, I encourage others to not care about anything less than Python 3.5, as IMO that is the first Python 3 release that is reasonable to target for writing code that works on both 2.7 and 3.5 (a lot of that is due to 3.5 reintroducing % formatting on bytes types).

Finally, the build system relies on so many other components that it will likely be the last thing to be ported to Python 3. So I think it is fine for projects like mozbase to cross the Python 3 bridge earlier.
Also what I have seen when working on mozbuild changes, we are using __future__ imports a lot which already helps with a lot of those cross-version incompatibilities. Maybe we should get started with those to ensure everything is still tested by unittests.

Comment 7

3 months ago
I've been watching the Mercurial project make pretty good work porting from Python 2.6/2.7 to dual support with 3.5+ and think their process is worth following:

1) Adopt `from __future__ import absolute_import`
2) Make every file AST parse cleanly in Python 3 (does `ast.parse()` not raise)
3) Make every file import cleanly in Python (importlib.import_module() doesn't raise)
4) Start modifying code to actually run properly on Python 3. This involves using six (or rolling your own), using b'' literals when appropriate, etc.

#4 is definitely the time consuming part. But you can generally divide and conquer once all the modules import cleanly. Once 1-3 happened with Mercurial, the patches for #4 started flowing very rapidly.
The last time I looked into this I stumbled upon modernize: http://python-modernize.readthedocs.io/en/latest/index.html

It's basically a layer on top of 2to3 that rewrites your code to use six instead of just rewriting it to be Python 3-compatible. This seems like it might be a viable approach to tackling the problem.

Now that we have the mozbase tests running in a standalone job (bug 1003417) we should be able to stand up another job that runs the same tests in Python 3. We could add a mozinfo key for Python 3, and skip/fail tests as necessary to get a green subset, and then enable more as we do the work to fix them.

Updated

2 months ago
Blocks: 1355537

Comment 9

2 months ago
Of the 200 most widely used Python packages, only 13 have not been converted to Python 3, according to: https://python3wos.appspot.com/

Nine of those are moz packages. That means, given the proportions, that almost all the most important python packages that have not been converted to Python 3 are moz packages.

The other non-converts are httplib2 (renamed to http.client in Python 3), supervisor, carbon, graphite web and..., no that's it.

It makes Mozilla look luddite.

It makes Mozilla look like a stubborn old man holed up in a shanty shack on a little grassy roundabout around which a whole city and its busy roads are whirring on regardless. Hurrumph!

That's just how it looks.

Five years ago it was already obvious that anyone starting afresh should start with Python 3. They continue to do so: building around Mozilla?

Comment 10

2 months ago
The following Moz packages are all in the top 500 most downloaded PyPI packages of all time.  If they can not be ported to Python 3 then their setup.py trove classifiers should be updated to add 'Programming Language :: Python :: 2 :: Only'.  Similar changes should be made to their README docs which currently often say 'Python 2.7 or later' which is ambiguous because Python 3.6.1 is later than 2.7 but is not supported.  These simple updates would save a lot of end user aggravation.


* manifestparser
* marionette_client
* marionette_driver
* marionette-transport
* mozcrash
* mozdevice
* mozfile
* mozhttpd
* mozinfo
* mozlog
* moznetwork
* mozprocess
* mozprofile
* mozrunner
* mozsystemmonitor
* moztest
* mozversion

PyPI Trove Classifiers: https://pypi.python.org/pypi?%3Aaction=list_classifiers
PyPI Top 500: https://python3wos.mybluemix.net

Comment 11

2 months ago
@Mark: thanks for raising this up.

New projects are to use Python 3.

From my understanding those moz packages are actually not that used. I believe the high stats was because during a while some of the automation would grab python packages directly from pypi instead of internal alternatives, thus, inflating the metrics. I think we fixed that a while back.

If there was a way to reset the metrics that would be super! From looking at the table I'm at least happy to see that all those packages are down to below 100 most used. I remember the day when they showed up very very high on the able.

@cclaus: that seems like a good suggestion. I will pass it along.

Comment 12

2 months ago
I agree that there's no current significant benefit to the build system itself supporting Python 3. However for mozbase specifically, lack of Python 3 support will soon have an impact, since it's consumed by others. 

For example, Treeherder uses Django whose next major release (2.0, due December 2017) drops support for Python 2. Treeherder also uses mozlog (during the ingestion of structured logs from test runs), so if by the end of the year mozlog doesn't support Python 3, it will prevent Treeherder from being able to use the latest version of Django (the complexities of Treeherder making itself Python 3 compatible aside).

Comment 13

2 months ago
Armen - I'm sorry to hear usage is low after all. Let's then dismiss popularity from our reckonings. Their use will decline further anyway, as long as they are not updated. I meant only to say high popularity increased the justification for converting these tools to python3. You imply low popularity suggests no need to upgrade the tools. Yet this argument would not hold.

If the tools are intended to be used, then their utility depends on their being updated. It seems then Mozilla intends to let the tools die, though your reply also implies alternatives might be developed. In either case, clear statements up front in *user documentation* are essential. The documentation I saw was actually good, but still neglected to give these most crucial details: it won't work on python3 and Mozilla either intends to let them die a natural death or to replace them at some likely time in the future. If Mozilla has a non-resuscitate order on the moz tools, it should put a non-resuscitate notice on them. It's like a central bank making it easy for people to buy bonds, but issuing bonds in such a way that, say, only babyboomers can buy them; and then not telling anybody about the provisos, so half the next generation waste time trying to learn about bonds and trying to buy bonds they can't buy; while the bank issues only vague and enigmatic explanations when people try to find out why they can't. In a market of misleading information everyone trundles around bumping into things, getting lost, retracing their steps, studying maps, stepping out and getting lost again, in circles and snarl-ups. Investors, as they say, lose confidence in the future. The economy evaporates as any fuel does in a traffic jam.
Mark: if you're signing up to do the work then that's great! If not, writing long rants on bugzilla is not going to change things.
(In reply to Mark from comment #13)
> Armen - I'm sorry to hear usage is low after all. Let's then dismiss
> popularity from our reckonings. Their use will decline further anyway, as
> long as they are not updated. I meant only to say high popularity increased
> the justification for converting these tools to python3. You imply low
> popularity suggests no need to upgrade the tools. Yet this argument would
> not hold.

Hey Mark, mozbase is almost entirely used internally by us to build and test Firefox (although there might be external users of the marionette packages). It's an essential part of our internal toolchain, it won't be going away any time soon. For our own reasons (some of which are outlined above), I think it is pretty much inevitable that we will make mozbase work with python 3 *at some point*, perhaps even sooner than later -- it really just depends on finding motivated person(s) to do the work.

As always, the people best qualified/placed to do this are tasked with other stuff which is higher priority, but there's nothing stopping someone with a bit of determination from making this happen. As RMS once said when asked when asked if there was a release date for some FSF package: "No. It will be done sooner if you help." :)
For the record, I don’t encourage non-Mozilla projects to rely on the Marionette Python packages.  If you can, please use geckodriver (https://github.com/mozilla/geckodriver) and a W3C WebDriver-conforming client binding such as Selenium (https://github.com/seleniumhq/selenium) wdclient (https://github.com/w3c/wpt-tools/tree/master/webdriver).

Comment 17

a month ago
Any progress on adding 'Programming Language :: Python :: 2 :: Only' to the PyPI trove classifiers of the the above modules?
I like gps' suggested approach from comment 7. The only thing I'd add is that getting tox to work with the python unittests so we can run them on multiple versions should be a prerequisite for this. Until there's a way to do this, the python 3 support would likely regress quickly.
You need to log in before you can comment on or make changes to this bug.