-
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
chore: migrate from kuttl to chainsaw #2607
Conversation
eaaa321
to
e7b48ab
Compare
cc/ @IshwarKanse |
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
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.
Some comments on managing chainsaw in a local environment.
Quick blog post covering chainsaw two major new features in |
@eddycharly The work done on Chainsaw is really impressive. I'm looking forward to use it for other operator testing as well and for our internal QE testing. |
@IshwarKanse glad to know you like it ! |
Not sure what breaks here, probably something simple but i fail to spot it 🙈 |
Hope i fixed it now 🤞 |
This PR migrates e2e tests from kuttl to chainsaw.
Chainsaw was created because kuttl stopped to be actively maintained (it's actually a single maintainer now).
We are also solving most of the kuttl limitations that exist today and wrote a command to automatically migrate from kuttl to chainsaw. It supercharges e2e tests with better logs, config maps assertions, assertions trees and many more things 💪
This is a relatively young project but it has already been adopted by a couple of e2e tests heavy projects (kyverno, keptn, fairwinds rbac-manager, grafana-operator, redis-operator and others).
To give more context, you can see similar PRs at:
Sadly, kuttl haven't done a release since jan 2023.
Here you can find a good video showcasing chainsaw: https://www.youtube.com/watch?v=Ejof-wtAdQM&ab_channel=CamilaMacedo
To make things more concrete, there are a couple of issues in this repo, a few files are not valid and kuttl doesn't complain because unmarshalling is not strict so the non-valid fields are simply ignored. This is a problem because the behaviour of the tests is not the expected one, for example:
opentelemetry-operator/tests/e2e-opampbridge/opampbridge/00-assert.yaml
Lines 67 to 70 in c534eaf
opentelemetry-operator/tests/e2e-prometheuscr/create-sm-prometheus-exporters/07-delete.yaml
Lines 6 to 8 in 94c8420
If you need more infos 👇