Skip to content

[PM-40516] Fix Extended Cache Service Registration - #8007

Merged
sven-bitwarden merged 1 commit into
rcfrom
cherry-pick-pm-40516
Jul 17, 2026
Merged

[PM-40516] Fix Extended Cache Service Registration#8007
sven-bitwarden merged 1 commit into
rcfrom
cherry-pick-pm-40516

Conversation

@sven-bitwarden

@sven-bitwarden sven-bitwarden commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

🍒⛏️ PM-40516

…8004)

The shared-Redis path registered a single unnamed IFusionCacheBackplane
singleton and bound every named cache to it via WithRegisteredBackplane().
A RedisBackplane instance is bound to one pub/sub channel at Subscribe time,
so sharing one instance across caches collapses them onto a single channel.
Cross-instance invalidations were misrouted, leaving stale L1 entries on
other nodes (e.g. an OrganizationAbilities RemoveAsync publishing on the
OrganizationReports channel).

Register the backplane as a keyed singleton per cache name and resolve it
with WithRegisteredKeyedBackplaneByCacheName(), matching the keyed-Redis
path. The multiplexer and distributed cache remain shared.

(cherry picked from commit a93cf75)
@github-actions

Copy link
Copy Markdown
Contributor

Claude Code is reviewing this pull request...

If this comment does not update with results, check the Actions log.

@sven-bitwarden sven-bitwarden added the t:bugfix Change Type - Bugfix label Jul 17, 2026
@sven-bitwarden
sven-bitwarden merged commit 5d4461a into rc Jul 17, 2026
42 of 47 checks passed
@sven-bitwarden
sven-bitwarden deleted the cherry-pick-pm-40516 branch July 17, 2026 16:53
@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.57%. Comparing base (a48972b) to head (915b345).
⚠️ Report is 1 commits behind head on rc.

Additional details and impacted files
@@           Coverage Diff           @@
##               rc    #8007   +/-   ##
=======================================
  Coverage   61.56%   61.57%           
=======================================
  Files        2256     2256           
  Lines       99391    99393    +2     
  Branches     8984     8984           
=======================================
+ Hits        61194    61197    +3     
+ Misses      36045    36044    -1     
  Partials     2152     2152           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:bugfix Change Type - Bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants