[identity] Add exponential backoff to refresh_user_prekeys
AcceptedPublic
Actions

Authored by bartek on Wed, Oct 2, 1:10 AM.

Details

Reviewers

varun
will

Summary

Part of ENG-9427.
Added exponential backoff retry counter to RefreshUserPrekeys RPC

Depends on D13568

Test Plan

It's hard to do that on localstack, but I was able to reuse test plan from D13356 to block DB item with transactions and then confirmed that refresh_user_prekeys also retries.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Wed, Oct 2, 1:10 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Oct 2, 1:10 AM

bartek added a child revision: D13570: [identity] Add exponential backoff to upload_one_time_keys.Wed, Oct 2, 1:15 AM

Harbormaster completed remote builds in B32009: Diff 44813.Wed, Oct 2, 1:25 AM

bartek published this revision for review.Wed, Oct 2, 1:26 AM

will accepted this revision.Wed, Oct 2, 12:23 PM

This revision is now accepted and ready to land.Wed, Oct 2, 12:23 PM

varun accepted this revision.Wed, Oct 2, 12:27 PM

varun added inline comments.

services/identity/src/database/device_list.rs
854	i had to look at the code to jog my memory, but confirmed that `sleep_and_retry` respects the retry limit set by `ExponentialBackoffConfig::default()`

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

database/

device_list.rs

44 lines

Diff 44813

View Options

services/identity/src/database/device_list.rs

[identity] Add exponential backoff to refresh_user_prekeysAcceptedPublicActions