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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: migrate from kuttl to chainsaw #1250

Closed
wants to merge 1 commit into from

Conversation

eddycharly
Copy link

@eddycharly eddycharly commented Feb 17, 2024

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 5 times, most recently from b31bf41 to 397e34a Compare February 17, 2024 21:50
@eddycharly eddycharly marked this pull request as ready for review February 17, 2024 21:59
@eddycharly eddycharly marked this pull request as draft February 17, 2024 23:56
@eddycharly eddycharly force-pushed the chainsaw branch 5 times, most recently from c3d5942 to 04f6fb5 Compare February 18, 2024 20:17
@eddycharly eddycharly marked this pull request as ready for review February 18, 2024 21:23
@iam-veeramalla
Copy link
Collaborator

@eddycharly Really appreciate the contribution. We will go through the PR. Thanks again :)

@eddycharly
Copy link
Author

@iam-veeramalla once everything is green, i will see how to enable concurrent tests. It should be easy, chainsaw has a feature to disable concurrency on a per test basis.

@eddycharly
Copy link
Author

@iam-veeramalla any chance to get this in ?

@eddycharly
Copy link
Author

i bumped chainsaw to the latest version

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants