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

Use KRaft only on Testing Farm to avoid unexpected ZK failures #9627

Merged
merged 5 commits into from Feb 1, 2024

Conversation

Frawless
Copy link
Member

@Frawless Frawless commented Jan 31, 2024

Type of change

  • Enhancement / new feature

Description

Because we are closer to KRaft to be production ready, we would like to start using Testing Farm for system tests. This PR removes option to run ZK mode on TF as it was failing with unexpected dns errors caused by ZK bug and move all pipelines to use KRaft mode.

The pipelines will still be executable only via comments on the PRs.

Checklist

  • Make sure all tests pass

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless Frawless added this to the 0.40.0 milestone Jan 31, 2024
@Frawless
Copy link
Member Author

/packit test --labels smoke

@kornys
Copy link
Contributor

kornys commented Jan 31, 2024

👍

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

Frawless commented Feb 1, 2024

/packit test --labels smoke

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

Frawless commented Feb 1, 2024

/packit test --labels smoke

@Frawless Frawless marked this pull request as ready for review February 1, 2024 10:25
@Frawless
Copy link
Member Author

Frawless commented Feb 1, 2024

I am not going to run the full regression as part of this. I will try to check results as part of different PR and fix the failures if any occur.

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

Frawless commented Feb 1, 2024

/packit test --labels smoke

Copy link
Member

@im-konge im-konge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

Signed-off-by: Jakub Stejskal <xstejs24@gmail.com>
@Frawless
Copy link
Member Author

Frawless commented Feb 1, 2024

/packit test --labels smoke

@Frawless Frawless added the ready for merge Label for PRs which are ready for merge label Feb 1, 2024
@scholzj scholzj merged commit 6c6c683 into strimzi:main Feb 1, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Label for PRs which are ready for merge System tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants