Skip to content
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

Closed
wants to merge 19 commits into from

Conversation

eddycharly
Copy link
Contributor

@eddycharly eddycharly commented Feb 8, 2024

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:

If you need more infos 👇

@eddycharly eddycharly force-pushed the chainsaw branch 8 times, most recently from eaaa321 to e7b48ab Compare February 8, 2024 09:55
@iblancasa
Copy link
Contributor

cc/ @IshwarKanse

Makefile Outdated Show resolved Hide resolved
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>
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>
@eddycharly eddycharly marked this pull request as ready for review February 13, 2024 10:50
@eddycharly eddycharly requested a review from a team as a code owner February 13, 2024 10:50
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@swiatekm-sumo swiatekm-sumo added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Feb 13, 2024
Copy link
Contributor

@swiatekm-sumo swiatekm-sumo left a 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.

.github/workflows/e2e.yaml Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@eddycharly
Copy link
Contributor Author

Quick blog post covering chainsaw two major new features in v0.1.4 https://kyverno.io/blog/2024/02/15/kyverno-chainsaw-0.1.4-awesome-new-features/ 🤩

@IshwarKanse
Copy link
Contributor

@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.

@eddycharly
Copy link
Contributor Author

@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 !

@eddycharly
Copy link
Contributor Author

Not sure what breaks here, probably something simple but i fail to spot it 🙈

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
@eddycharly
Copy link
Contributor Author

Hope i fixed it now 🤞

Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants