Skip to content

release-21.2: kv: deflake TestTenantRateLimiter#70737

Merged
RaduBerinde merged 1 commit intorelease-21.2from
blathers/backport-release-21.2-70723
Sep 28, 2021
Merged

release-21.2: kv: deflake TestTenantRateLimiter#70737
RaduBerinde merged 1 commit intorelease-21.2from
blathers/backport-release-21.2-70723

Conversation

@blathers-crl
Copy link
Copy Markdown

@blathers-crl blathers-crl bot commented Sep 25, 2021

Backport 1/1 commits from #70723 on behalf of @RaduBerinde.

/cc @cockroachdb/release

A recent change (#70328) increased the rate limit and that exposed a
fragility in this test. This change restores the test to use the
original (small) rate limit.

Fixes #70456.

Release note: None

Release justification: test-only fix.

Release justification:

@RaduBerinde
kv: deflake TestTenantRateLimiter
65a29ec 
A recent change (#70328) increased the rate limit and that exposed a
fragility in this test. This change restores the test to use the
original (small) rate limit.

Fixes #70456.

Release note: None

Release justification: test-only fix.
@blathers-crl blathers-crl bot requested a review from a team as a code owner September 25, 2021 05:48
@blathers-crl blathers-crl bot force-pushed the blathers/backport-release-21.2-70723 branch from a4053e9 to 65a29ec Compare September 25, 2021 05:48
@blathers-crl blathers-crl bot requested a review from ajwerner September 25, 2021 05:48
@blathers-crl
Copy link
Copy Markdown
Author

blathers-crl bot commented Sep 25, 2021

Thanks for opening a backport.

Please check the backport criteria before merging:

  • Patches should only be created for serious issues.
  • Patches should not break backwards-compatibility.
  • Patches should change as little code as possible.
  • Patches should not change on-disk formats or node communication protocols.
  • Patches should not add new functionality.
  • Patches must not add, edit, or otherwise modify cluster versions; or add version gates.
If some of the basic criteria cannot be satisfied, ensure that the exceptional criteria are satisfied within.
  • There is a high priority need for the functionality that cannot wait until the next release and is difficult to address in another way.
  • The new functionality is additive-only and only runs for clusters which have specifically “opted in” to it (e.g. by a cluster setting).
  • New code is protected by a conditional check that is trivial to verify and ensures that it only runs for opt-in clusters.
  • The PM and TL on the team that owns the changed code have signed off that the change obeys the above rules.

Add a brief release justification to the body of your PR to justify this backport.

Some other things to consider:

  • What did we do to ensure that a user that doesn’t know & care about this backport, has no idea that it happened?
  • Will this work in a cluster of mixed patch versions? Did we test that?
  • If a user upgrades a patch version, uses this feature, and then downgrades, what happens?

@cockroach-teamcity
Copy link
Copy Markdown
Member

cockroach-teamcity commented Sep 25, 2021

This change is Reviewable

ajwerner
ajwerner approved these changes Sep 27, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@ajwerner ajwerner left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @RaduBerinde)

@RaduBerinde RaduBerinde mentioned this pull request Sep 28, 2021
Closed
15 tasks
@RaduBerinde RaduBerinde merged commit 2fef2fb into release-21.2 Sep 28, 2021
@RaduBerinde RaduBerinde deleted the blathers/backport-release-21.2-70723 branch September 28, 2021 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ajwerner ajwerner ajwerner approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@RaduBerinde RaduBerinde

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cockroach-teamcity @ajwerner @RaduBerinde