Bug #16580
closedRemove Python 2 packages from build
Description
We still have some work to do to complete the migration to python 3:
- we should remove our python 2 packages for the next major release. For 2.1?
Updated by Ward Vandewege over 4 years ago
- Description updated (diff)
- Subject changed from Python3 loose ends to Python 3 loose ends
Updated by Ward Vandewege over 4 years ago
- Blocks Idea #15890: [Epic] Eliminate dependency on Python 2 added
Updated by Peter Amstutz over 4 years ago
- Target version set to 2020-08-26 Sprint
Updated by Peter Amstutz over 4 years ago
- Description updated (diff)
- Subject changed from Python 3 loose ends to Remove Python 2 packages from build
Updated by Ward Vandewege over 4 years ago
- Assigned To changed from Nico César to Ward Vandewege
Updated by Ward Vandewege over 4 years ago
- Status changed from New to In Progress
Updated by Ward Vandewege over 4 years ago
Ready for review in afb2e4f88d30ac60fda89136fd73fecfb74ba5eb on branch 16580-remove-python2-packages.
Tests are running at developer-run-tests: #2001 . The developer-run-tests-sdk-python-ruby test failed for transient reasons, re-run passed at developer-run-tests-sdk-python-ruby: #1301 .
TODO before merging: disable the nodemanager test on run-tests (Jenkins). I already disabled it on developer-run-tests.
Updated by Nico César over 4 years ago
review at afb2e4f88d30ac60fda89136fd73fecfb74ba5eb
2 small details about the configuration:
lib/config/generated_config.go still has NodeManager reference.
lib/config/config.default.yml still has NodeManager reference.
(probably those 2 files need some extra cleanup+comments, but not critical for this bug)
Test passed on jenkins and the diff looks good to me.
Ready to merge
Updated by Ward Vandewege over 4 years ago
Nico César wrote:
review at afb2e4f88d30ac60fda89136fd73fecfb74ba5eb
2 small details about the configuration:
lib/config/generated_config.go still has NodeManager reference.
lib/config/config.default.yml still has NodeManager reference.
Ah, yes, and there were a bunch more. I've cleaned all those up in 4b4431eb7fbf575789be9c5798b6a3bf43a07396 on branch 16580-remove-python2-packages, tests are at developer-run-tests: #2010 . The failed wb integration test was re-run at developer-run-tests-apps-workbench-integration: #2130 and passed.
Test passed on jenkins and the diff looks good to me.
Ready to merge
Thanks will do.
Updated by Ward Vandewege over 4 years ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|99e8567b4130771caabcf9b735bfbdb3585bd018.