1585252 - [python3] Manually test Pontoon app and sync for regressions

Status: RESOLVED FIXED

(Webtools Graveyard :: Pontoon, task, P2)

Description

[Matjaz Horvat [:mathjazz]]

Once all the tests will pass in the python3 env, we should perform extensive manual test of the app, with special focus on sync.

Comment 1

[Jarek Śmiejczak [:jotes]]

Hey,

I'll create a pull request to remove all TypeError: object of type 'map' has no len() errors and we'll see how many tests it will fix.

Comment 2

[Matjaz Horvat [:mathjazz]]

Thanks! Please link the PR to bug 1585251.

Comment 3

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1410

Comment 4

[[DEACTIVATED] Adrian Gaudebert]

To be clear, this work will happen on our stage instance.

This work includes the following steps:

1. Deploy a patch that updates the runtime to Python 3.7 to our stage Heroku environment.
2. Trigger a sync job without commit for all jobs, verify that no errors happen and data is correctly modified or created in Pontoon.
3. Trigger a sync job with commit for a single, test repo, verify that changes are correctly synchronized to that repo.
4. Browse the web app and verify that no functionalities are broken (login, dashboards, translate, review… ).

The command to trigger a sync job is: ./manage.py sync_projects --projects=$SLUG --no-commit

Comment 5

[[DEACTIVATED] Adrian Gaudebert]

With help form Matjaz, I was able to deploy Python 3 to stage and run a sync job. I found a few problems along the way, that I transformed into bugs:

1. mercurial needs to be updated: bug 1597732
2. celery needs to be updated (with its dependencies): bug 1597733
3. There's a TypeError while running sync: bug 1597731

The command I used to run sync is: heroku run -a mozilla-pontoon-staging ./manage.py sync_projects --no-commit

Comment 6

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1509

Comment 7

[[DEACTIVATED] Adrian Gaudebert]

adngdb merged PR #1509: "Bug 1585252 - Fix string encoding errors for sync with Python 3." in 9b79186.

Comment 8

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1538

Comment 9

[[DEACTIVATED] Adrian Gaudebert]

adngdb merged PR #1538: "Bug 1585252 - Upgrade dependencies for Python 3 support" in 3605523.

Comment 10

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/1540

Comment 11

[[DEACTIVATED] Adrian Gaudebert]

adngdb merged PR #1540: "Bug 1585252 - Update Python runtime to version 3." in 8d709f4.