-
Notifications
You must be signed in to change notification settings - Fork 108
Move to perftool-incubator/bench-trafficgen
#2722
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
Move to perftool-incubator/bench-trafficgen
#2722
Conversation
The old TrafficGen repo has moved: * From: https://github.com/atheurer/trafficgen.git * To: https://github.com/perftool-incubator/bench-trafficgen.git The new installed location is now `/opt/bench-trafficgen/trafficgen`.
FYI @pradiptapks. |
I have verified the latest Environment:
Kernel and Tuned Profile setting
pbench-agent installed version
DPDK Version:
Set Alias for DPDK Devbind:
Install JSON schema Applying pbench-trafficgen patch:
pbench-trafficgen command: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @pradiptapks!
For us Pbench folks,
Set Alias for DPDK Devbind
Unfortunately, I don't think that DPDK gets installed implicitly or automatically...but, however it gets installed, there should probably be a symlink set up for dpdk-devbind
so that the script doesn't have to be modified (although, I suppose we could augment the script to make it smart enough to figure out how to find the .py
file if the naked command isn't there). I'm surprised that the DPDK RPM doesn't do this already...it seems like changing the name of the command would be an incompatible change.
dnf install python3-jsonschema
I've seen what happens if this is missing, but I haven't learned where the dependency for this is. Is this something that the user just has to do (like installing DPDK), or should this be a dependency of the Pbench Agent RPM? (Given that, apparently, nothing but the trafficgen support uses it, I would prefer that it not be added as a general dependency, but....) If it's not going to be installed implicitly, we need to figure out how to fail more gracefully when it is missing!
The old TrafficGen repo has moved:
The new installed location is now
/opt/bench-trafficgen/trafficgen
.