-
Notifications
You must be signed in to change notification settings - Fork 13
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
Reenable bf-python CLI. #29
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: fruffy <[email protected]>
@jafingerhut I assuming for you enable bf-python would still cause issues with tabulate, right? |
I can try it out after my current large-VM-experiment with enabling Tofino2 is finished. |
OK, another point of confusion for me: Why isn't the part of the CI test runs that runs bf_switchd and a PTF test on one P4 program also failing to find the What is it doing differently than I would on my personal VM? I build using batch-install.sh, which runs most install as root, then I try running the model, bf_switch, and the PTF test as a normal non-root user. Is that the difference? |
For Ubuntu 22.04 the default Python version is 3.10 so all packages are in the same folder. For Ubuntu 20.04 the default Python version is 3.8, unclear to me why it works. It is possible that both 3.8 and 3.10 are included in PYTHONPATH. |
Manual test results with this version:
Started from freshly installed Ubuntu 20.04 x86_64 VM.
Build went OK with no errors. I ran these commands, and the last one gave an error about not being able to find the tabulate Python package. There were some python3.8 packages, and some python3.10 packages, installed beneath open-p4studio/install/lib after the install was done.
All Python packages were installed in python3.10 packages on an Ubuntu 22.04 VM. I tried 3 times, but the run_switchd.sh command above never generated a command prompt, which seems to be the signal that it is ready to receive an incoming connection request from the run_p4_tests.sh command. |
Fixes #23.