-
Notifications
You must be signed in to change notification settings - Fork 25
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
Functional testing #42
Conversation
…template properly. Modified checkin to be usable as the only required task.
…ty; not required. Use generate-config instead of reload.sh preStart. Get rid of example backend. Reduced ACME related tasks to one.
…_ENV=staging. Break container pilot health check logic out into health-check script.
I've got an initial test rig done here. I want to get this running on Jenkins next and extend the test to have backends fail before merging this issue.
|
I've extended the tests but they're not stable right now. I'm getting erratic failures which all look to be timeout vs provisioning time and CNS latency issues.
|
Ok, I think this is ready:
|
Looks great to me. First time taking a good look at https://github.com/autopilotpattern/testing. Good stuff! Those tests are nice and easy to understand. Also, having the example app tucked away in the examples folder is so much cleaner IMHO. Less confusing trying to understand the repo now. |
This PR will carry #37 and move the example backend into a functional testing rig.
cc @misterbisson @jasonpincin