Closed JackKuo-tw closed 3 years ago
In response to PR creation
Aborting, need an authorized user to run CI
@JackKuo-tw thanks for looking into this. I thought we got rid of our Python 2 dependencies, do you remember, or have screenshots of where it was necessary? I think it would be better to remove those instances if possible, rather than adding the python 2 package.
@kevindweb Please refer to the screenshot below, it looks like dpdk-devbind.py
needs python (#! /usr/bin/env python3
in the first line)
But I can use Python3 to run it correctly, so I think the problem is I need to link pyhton to python3 in my env.
Although, in this transition period (py2 -> py3), I think add a comment to remind the user is the best way.
After I change the doc, I'll notify you again, thanks.
@kevindweb done, please have a look.
Summary:
This PR is tested on clean Vagrant Box
bento/ubuntu-20.04
, add some missing packages needed by ONVMThere is one thing interesting, the original installation doc said ONVM needs Python3, but in my test it also needs Python2, so I added it.
Usage:
Merging notes:
TODO before merging :
Test Plan:
Review:
Anyone.