-
Notifications
You must be signed in to change notification settings - Fork 115
Add dataplane performance automated test #1356
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
Conversation
✅ Deploy Preview for nginx-gateway-fabric ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
caa0831
to
571ba40
Compare
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.
I'd like to be able to fully remove the dataplane-performance
directory. We can migrate the past results into the new results folder.
I removed it in the first commit, I think you can possibly just see the files being renamed? |
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.
I ran it on GCP. worked as expected
fff33ff
to
dbb92be
Compare
bc4faa2
to
797ee69
Compare
797ee69
to
64d8b9d
Compare
f0893d0
to
3255ed7
Compare
ba8ccbc
to
899f7ad
Compare
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.
🚀 🚀 🚀
I tested this on GCP and it worked like a charm!
899f7ad
to
e0ed759
Compare
e0ed759
to
90fd874
Compare
* Add dataplane performance automated test
Proposed changes
Problem: Currently, all our NFR tests need to be run manually. This is time consuming every release.
Solution: Use the framework to automate one test to start off.
Testing: Ran the tests against a local kind cluster, locally against a GKE cluster, and on a GCP VM against a GKE cluster
Closes #1286
Checklist
Before creating a PR, run through this checklist and mark each as complete.