Project

General

Profile

Actions

Bug #16580

closed

Remove Python 2 packages from build

Added by Ward Vandewege over 4 years ago. Updated over 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Story points:
-
Release relationship:
Auto

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?

Subtasks 1 (0 open1 closed)

Task #16697: reviewResolvedNico César08/13/2020Actions

Related issues 1 (0 open1 closed)

Blocks Arvados - Idea #15890: [Epic] Eliminate dependency on Python 2ResolvedActions
Actions #1

Updated by Ward Vandewege over 4 years ago

  • Description updated (diff)
  • Subject changed from Python3 loose ends to Python 3 loose ends
Actions #2

Updated by Ward Vandewege over 4 years ago

  • Blocks Idea #15890: [Epic] Eliminate dependency on Python 2 added
Actions #3

Updated by Peter Amstutz over 4 years ago

  • Release set to 25
Actions #4

Updated by Peter Amstutz over 4 years ago

  • Target version set to 2020-08-26 Sprint
Actions #5

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
Actions #6

Updated by Peter Amstutz over 4 years ago

  • Assigned To set to Nico César
Actions #7

Updated by Ward Vandewege over 4 years ago

  • Assigned To changed from Nico César to Ward Vandewege
Actions #8

Updated by Ward Vandewege over 4 years ago

  • Status changed from New to In Progress
Actions #9

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.

Actions #10

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

Actions #11

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.

Actions #12

Updated by Ward Vandewege over 4 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF