[release/7.0-staging] Eliminate cancellation deadlock in RateLimiter implementations (#90285) #90337
Conversation
BrennanConroy
added
Servicing-consider
|
Tagging subscribers to this area: @mangod9 Issue DetailsBackport of #90285 to release/7.0-staging Customer ImpactRate limiting operations could deadlock in specific scenarios. TestingManual testing and automated test with internal hooks added RiskLow. Issue was clear once pointed out.
|
danmoseley
added
Servicing-approved
|
this was approved in email on 8/10 |
|
@carlossanlop does @BrennanConroy merge now? |
|
PR owners are free to merge whenever (in the staging branches), but just so you know, I already flowed everything from staging into internal. Do we absolutely need this in the October Release? If yes, then I'll have to flow this PR alone manually. |
|
I mention the above because Code complete was yesterday for servicing, @danmoseley. |
|
Since this was merged one day after code complete, it was too late and will have to wait until the next month to release it. |
ghost
locked as resolved and limited conversation to collaborators
4 participants
Backport of #90285 to release/7.0-staging
Customer Impact
Rate limiting operations could deadlock in specific scenarios.
Testing
Manual testing and automated test with internal hooks added
Risk
Low. Issue was clear once pointed out.