-
Notifications
You must be signed in to change notification settings - Fork 56
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
Make framework configurable to run hierarchical suite of workloads #20
Comments
added -c option which takes folder/sub-folder that contains test names, implementing folders in testnames (-t) option will not have clear identifcation of path/testnames, this change doesnt breaks old way of running.. some tests are given as follows ##to run list of tests in custom path ##if custom path is not provided , default custom path ./config dir is taken (original behavior) Valid test names: all || all_control_plane || all_data_plane || ##original behavior of the tests,works original behavior, run all control plane testsndn@issubuntu01:~/k-bench$ ./run.sh -t all_control_plane original behavior, run all control plane testsndn@issubuntu01:~/k-bench$ ./run.sh -t all_data_plane |
Hi Nagesh, thanks for the change. I understand why you think we should have two separate options for test name and config directory. But, in the case of an ask where the user has multiple tests he wants to run in a particular directory, the user still have to use the top level directory as the test name. My suggestion would be to remove test names altogether and just use one directory name. If the directory has a config.json, it will be run. If not, we recursively look at the subdirectories for config.json files and execute them one by one. |
The user should be able to provide a folder, subfolder or a test name to run.
Example:
./run.sh -t dataplane
./run.sh -t dataplane/ai/classification/resnet50
./run.sh -t dataplane/ai
The text was updated successfully, but these errors were encountered: