Skip to content

Use SafeRequestStore in the DB LB

Yorick Peterserequested to merge
yorick/lb-safe-request-store into master

What does this MR do and why?

In #339515 (closed) it was found that specs can fail when run in a certain order, possibly due to the use of RequestStore (which can leak between tests) instead of SafeRequestStore.

This commit changes the load balancer to use SafeRequestStore, ensuring we don't leak any data between tests.

MR acceptance checklist

Quality

  • Quality checklist confirmed

Performance, reliability, and availability

  • Performance, reliability, and availability checklist confirmed

Documentation

  • Documentation checklist confirmed

Security

  • Security checklist confirmed

Deployment

  • Deployment checklist confirmed
Edited by Yorick Peterse

Merge request reports