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

Add RF change ST tunings to make it more stable #10116

Merged
merged 1 commit into from
May 18, 2024

Conversation

fvaleri
Copy link
Contributor

@fvaleri fvaleri commented May 15, 2024

It looks like the RF change ST is more stable and a bit faster when running locally with this tuning.

Can you please run the Azure pipeline?

Signed-off-by: Federico Valeri <fedevaleri@gmail.com>
@fvaleri fvaleri marked this pull request as draft May 15, 2024 18:51
@fvaleri fvaleri added this to the 0.42.0 milestone May 15, 2024
@see-quick
Copy link
Member

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@fvaleri
Copy link
Contributor Author

fvaleri commented May 16, 2024

The Azure result is consistent with what I see locally:

2024-05-16T09:54:42.2758772Z [INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1454 s -- in io.strimzi.systemtest.operators.topic.TopicReplicasChangeST

Are you guys good with these changes?

@fvaleri fvaleri marked this pull request as ready for review May 16, 2024 14:17
Copy link
Member

@see-quick see-quick left a comment

Choose a reason for hiding this comment

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

Yeah sure. Thanks @fvaleri 👍

@scholzj scholzj merged commit 730d2a7 into strimzi:main May 18, 2024
21 checks passed
@fvaleri fvaleri deleted the rfc-st-tuning branch May 18, 2024 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants