-
Notifications
You must be signed in to change notification settings - Fork 438
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
Adding helm tests for production recommendations #8768
Conversation
c6339cc
to
5942d35
Compare
// The aim of the test is to ensure that the readiness probe is configured on the gateway proxy | ||
// and the gateway-proxy deployment is ready before helm marks the upgrade as successful. | ||
// Ref: https://github.com/solo-io/gloo/issues/8288 | ||
Context("Custom readiness probe", func() { |
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.
Replacing the custom readiness probe test with the Production recommendations tests as the latter tests the former along with other prod settings
Visit the preview URL for this PR (updated for commit b36cdca): https://gloo-edge--pr8768-helm-prod-rec-xgga4pu8.web.app (expires Fri, 27 Oct 2023 14:32:15 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
ec6b37c
to
3722529
Compare
3722529
to
2663a57
Compare
Issues linked to changelog: |
4f97d67
to
2b7cf4c
Compare
2b7cf4c
to
221ca03
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 like this and I think it's a nice step towards codifying our production recommendations! For each of these tests, we assert that the components are health (which is good) but not much beyond that. I had kind of envisioned we could also update the other suites (https://github.com/solo-io/gloo/blob/main/test/kube2e/gloo/artifacts/helm.yaml for example) so that when we ran the tests we configured as many production recommendations as well. That way we assert both that everything is healthy AND that the behavior matches what we would expect. What do you think about that?
Thanks @sam-heilbron ! I would like to use these same prod recommendations for all the tests but setting |
Will create a new PR along with the custom readiness probe fix |
Description
Adds helm kube e2e tests to mimic the recommendations for production deployments mentioned here
Context
This is to ensure that we test gloo as users would deploy it in production and catch potential issues early
Interesting decisions
N/A
Testing steps
Run the helm kube2e tests
Notes for reviewers
The issue mentioned here needs to be fixed
Checklist:
BOT NOTES:
resolves #8804