Uh oh!
There was an error while loading. Please reload this page.
Migrate VR codebase from Python2 to Python3 - #4807
Conversation
Also: port cloud-grab-dependent-library-versions to python3
Conflicts were found & fixed: modified: systemvm/debian/opt/cloud/bin/cs/CsNetfilter.py modified: systemvm/debian/root/health_checks/router_version_check.py modified: systemvm/test/runtests.sh
blueorangutan
commented
Mar 12, 2021
Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 100 |
blueorangutan
commented
Mar 13, 2021
[S] Trillian test result (tid-124)
|
fix py3 issue - python3: No module named pycodestyle
blueorangutan
commented
Mar 15, 2021
[S] Trillian test result (tid-136)
|
yadvr
commented
Apr 5, 2021
@blueorangutan package |
blueorangutan
commented
Apr 5, 2021
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
blueorangutan
commented
Apr 5, 2021
Packaging result: ✔️ centos7 ✔️ centos8 ✖️ debian. SL-JID 322 |
DaanHoogland
commented
Apr 6, 2021
ATM this is a mix of python2 and python3 during the travis run and though I tried to implement it according to their spec, I had to put a question to them concerning a python not being found. |
yadvr
commented
May 10, 2021
@GabrielBrascher are you working on this? cc @DaanHoogland |
GabrielBrascher
commented
May 10, 2021
@rhtyd I am doing some tests but it still need to cover more networks. I would appreciate any help testing it on environment with advanced networks + VPCs. I am testing at first with advacned networks + SG. |
DaanHoogland
commented
May 11, 2021
@GabrielBrascher , I'm on vacation since tomorrow ;) I will help you with testing this when I come back. |
nvazquez
commented
Jun 23, 2021
Hi @DaanHoogland@GabrielBrascher can you please resolve conflicts? |
yadvr
commented
Sep 8, 2021
Ping @GabrielBrascher any update on this should we target this for 4.17 instead? |
yadvr
commented
Sep 8, 2021
On second thought and after reviewing the changes, the risks to VR maybe huge so lets target this for 4.17. |
GabrielBrascher
commented
Sep 8, 2021
@rhtyd sorry for the delay here. We might need to postpone it to 4.17 indeed. One of the biggest issues I am having is the testing, it can work great with my test environment (Advanced + SG, or basic network) but someone using a specific advanced network + VPC/VPN, etc could see issues. I would appreciate it if anyone could check it on advanced without SG, I am mostly able to reproduce on a (decent) test environment with advanced + security groups. |
GabrielBrascher
commented
Sep 8, 2021
This is also one of the reasons I made this PR in an The main concerns are related to the fact that such implementation is quite complex (too many cases for testing and evaluating) and critical (a small python porting issue could mess with the VRs scripts). |
nvazquez
commented
Feb 5, 2022
Hi @GabrielBrascher should this be included on 4.17? Can you please fix the conflicts? |
Hi @${author}, your pull request has merge conflicts. Can you fix the conflicts and sync your branch with the base branch? |
Description
Considering the Python2 end of life we must migrate the VR's codebase to Python3.
This PR covers partially issue #3195. Additionally, this PR is a joint of the work done in #3730 and #4727.