- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.2k
Migrate VR codebase from Python2 to Python3 #4807
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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
| Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 100 | 
| [S] Trillian test result (tid-124) 
 | 
fix py3 issue - python3: No module named pycodestyle
| [S] Trillian test result (tid-136) 
 | 
| @blueorangutan package | 
| @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. | 
| Packaging result: ✔️ centos7 ✔️ centos8 ✖️ debian. SL-JID 322 | 
| 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. | 
| @GabrielBrascher are you working on this? cc @DaanHoogland | 
| @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. | 
| @GabrielBrascher , I'm on vacation since tomorrow ;) I will help you with testing this when I come back. | 
| Hi @DaanHoogland @GabrielBrascher can you please resolve conflicts? | 
| Ping @GabrielBrascher any update on this should we target this for 4.17 instead? | 
| On second thought and after reviewing the changes, the risks to VR maybe huge so lets target this for 4.17. | 
| @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. | 
| 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). | 
| 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.