-
Notifications
You must be signed in to change notification settings - Fork 117
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
test: add chainsaw tests #438
Conversation
5a26284
to
1d490ad
Compare
Somehow related blog post https://kyverno.io/blog/2023/12/12/kyverno-chainsaw-the-ultimate-end-to-end-testing-tool/ |
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.
This looks great. I would be okay with going ahead and removing the existing e2e tests just so we don't flood Circle with tests on the next PR.
name: "Chainsaw End-To-End Kubernetes 1.24" | ||
kind_node_image: "kindest/node:v1.24.7@sha256:577c630ce8e509131eab1aea12c022190978dd2f745aac5eb1fe65c0807eb315" | ||
<<: *chainsaw_configuration | ||
- rok8s/kubernetes_e2e_tests: |
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 think we should drop 1.23 and 1.24 and move to 1.25-1.27 instead.
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 agree, maybe in a follow up PR ? This is not related to this change.
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.
That's fine with me
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@sudermanjr done |
LGTM. Thanks! |
This PR adds chainsaw e2e tests.
Checklist