Bug #16580

Remove Python 2 packages from build

Added by Ward Vandewege over 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
08/13/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
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

Task #16697: reviewResolvedNico César


Related issues

Blocks Arvados - Story #15890: [Epic] Eliminate dependency on Python 2Resolved

Associated revisions

Revision 99e8567b
Added by Ward Vandewege about 1 year ago

Merge branch '16580-remove-python2-packages'

closes #16580

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

Revision 5c181101 (diff)
Added by Ward Vandewege about 1 year ago

Stop building the python pam package, that code was removed.

refs #16580

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

Revision c186f7ec (diff)
Added by Ward Vandewege about 1 year ago

The python pam and node manager packages have been removed, do not try
to upload them.

refs #16580

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

Revision b1a9b18e (diff)
Added by Ward Vandewege about 1 year ago

bugfix: remove unnecessarily dependency on python2 sdk package in our
arvados/jobs Docker image.

refs #16580

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

History

#1 Updated by Ward Vandewege over 1 year ago

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

#2 Updated by Ward Vandewege over 1 year ago

  • Blocks Story #15890: [Epic] Eliminate dependency on Python 2 added

#3 Updated by Peter Amstutz about 1 year ago

  • Release set to 25

#4 Updated by Peter Amstutz about 1 year ago

  • Target version set to 2020-08-26 Sprint

#5 Updated by Peter Amstutz about 1 year ago

  • Description updated (diff)
  • Subject changed from Python 3 loose ends to Remove Python 2 packages from build

#6 Updated by Peter Amstutz about 1 year ago

  • Assigned To set to Nico César

#7 Updated by Ward Vandewege about 1 year ago

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

#8 Updated by Ward Vandewege about 1 year ago

  • Status changed from New to In Progress

#9 Updated by Ward Vandewege about 1 year ago

Ready for review in afb2e4f88d30ac60fda89136fd73fecfb74ba5eb on branch 16580-remove-python2-packages.

Tests are running at https://ci.arvados.org/view/Developer/job/developer-run-tests/2001/. The developer-run-tests-sdk-python-ruby test failed for transient reasons, re-run passed at https://ci.arvados.org/job/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.

#10 Updated by Nico César about 1 year 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

#11 Updated by Ward Vandewege about 1 year 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 https://ci.arvados.org/view/Developer/job/developer-run-tests/2010/. The failed wb integration test was re-run at https://ci.arvados.org/view/Developer/job/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.

#12 Updated by Ward Vandewege about 1 year ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF