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

ducktape: Fix constant in connection_rate test #5375

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

VadimPlh
Copy link
Contributor

@VadimPlh VadimPlh commented Jul 7, 2022

Cover letter

Decrease kafka_connection_rate_limit settings to prevent flaky run.
Because client may use several connection for produce/consume and we also can not control time for it.

Fixes #5276

Release notes

  • none

@jcsp
Copy link
Contributor

jcsp commented Jul 8, 2022

Please fill out the commit and/or PR description with the rationale:

  • Why is REFRESH_TOKENS_TIME_SEC better set to 3 than 2?
  • Why is the success condition better set to 1.4 than 1.7?

@mmedenjak mmedenjak added kind/bug Something isn't working area/tests ci-failure labels Jul 8, 2022
@VadimPlh
Copy link
Contributor Author

Please fill out the commit and/or PR description with the rationale:

* Why is REFRESH_TOKENS_TIME_SEC better set to 3 than 2?

* Why is the success condition better set to 1.4 than 1.7?

So I decided to decrease kafka_connection_rate_limit only

@jcsp jcsp merged commit 128386d into redpanda-data:dev Jul 26, 2022
@BenPope
Copy link
Member

BenPope commented Oct 11, 2022

@BenPope
Copy link
Member

BenPope commented Oct 11, 2022

/backport v22.2.x

@vbotbuildovich
Copy link
Collaborator

Failed to run cherry-pick command. I executed the below command:

git cherry-pick -x a0ed303779d73a4866dfe767acaa5c5f0400896c

Workflow run logs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failure of ConnectionRateLimitTest.connection_rate_test
5 participants