-
Notifications
You must be signed in to change notification settings - Fork 40
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
Automated e2e testing in a4x2 #6749
Comments
Related: https://github.com/oxidecomputer/omicron/tree/main/live-tests I think @faithanalog has been looking at some steps toward running these in CI. |
indeed I have, though I wasn't aware of the existing efforts in 4585. that'll be helpful |
I've been working on what is essentially a rust version of a4x2-prepare.sh , with the intent to then create what would essentially be a rust version of a4x2-deploy.sh I think doing this (but saving myself some effort by referencing the existing scripts) is probably the thing to keep doing, particularly because I can make them easy to run on a local dev workstation as well, so someone trying changes can get the same results without having to round trip through buildomat while iterating. That someone being me for now; I don't feel like roundtripping through buildomat while getting stuff working :P |
https://github.com/oxidecomputer/omicron/tree/artemis/a4x2-package This is the branch I've been working on the rust port in. Not ready for review yet, I'm just tying it to this issue. I have it building an a4x2 bundle, and deploying it with working control plane, working locally on my machine, so that's pretty nice. Next steps are
We'll be able to improve the xtasks while running locally, even without CI running the tests. Should help out people who are interested in trying things on a4x2 but haven't found the time to sit down and learn it in depth. |
Omicron #4585 introduces very basic automated testing for the
a4x2
topology in CI. However, this needs to expand beyond simple "does it work" tests.Update: automated testing need to be removed due to overloading CI resources. We'll need to solve that problem to make forward progress here
The following is a list of things we should cover.
Networking
Tests from other categories to be added here.
The text was updated successfully, but these errors were encountered: