-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add scripts for testing and development #26
Conversation
Using network workload requires some infrastructure in place. Add few scripts to make it easier, plus one "proto" integration test to verify the dynamic configuration is generating some connections.
The script for the smoke test is a good idea, but I am having difficulty getting it to run and pass. A docker image that could run it would be nice. Here is what I have tried so far. I modified
Ran the following commands.
Within the docker container
|
I got this to pass. I changed
It also seems flaky, since it did not pass on the first try. |
Make the user option to expect an argument Co-authored-by: Jouko Virtanen <[email protected]>
Wording Co-authored-by: Jouko Virtanen <[email protected]>
Wording Co-authored-by: Jouko Virtanen <[email protected]>
Braces for the command invokation Co-authored-by: Jouko Virtanen <[email protected]>
Set shell options Co-authored-by: Jouko Virtanen <[email protected]>
Set shell options Co-authored-by: Jouko Virtanen <[email protected]>
Along the way fix the issue about searching for a wrong text in the tcpaccept test.
I have tested this after the latest changes and it is working. |
Using network workload requires some infrastructure in place. Add few scripts to make it easier, plus one "proto" integration test to verify the dynamic configuration is generating some connections.