Differential D13876

[backup-service] implement endpoint to upload User Keys
ClosedPublic
Actions

Authored by kamil on Nov 5 2024, 9:05 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 20, 9:01 PM

	Unknown Object (File)
	Sat, Dec 14, 7:19 PM

	Unknown Object (File)
	Sat, Dec 14, 4:57 PM

	Unknown Object (File)
	Sun, Dec 8, 3:44 PM

	Unknown Object (File)
	Sun, Dec 8, 4:20 AM

	Unknown Object (File)
	Fri, Nov 29, 3:28 PM

	Unknown Object (File)
	Thu, Nov 28, 12:52 AM

	Unknown Object (File)
	Wed, Nov 27, 2:51 AM

View All 38 Files

Subscribers

Details

Reviewers

varun
bartek

Commits

rCOMM3eeef1e77029: [backup-service] implement endpoint to upload User Keys

Summary

Part of ENG-6145.

Tracked in ENG-9672.

image.png (818×2 px, 211 KB)

Depends on D13875

Test Plan

Tested in D13879

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Nov 5 2024, 9:05 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 5 2024, 9:05 AM

kamil added a child revision: D13877: [backup-service] implement endpoint to upload User Data.Nov 5 2024, 9:07 AM

update

kamil edited the test plan for this revision. (Show Details)Nov 5 2024, 9:16 AM

Harbormaster completed remote builds in B32543: Diff 45611.Nov 5 2024, 9:23 AM

Harbormaster completed remote builds in B32547: Diff 45615.Nov 5 2024, 9:48 AM

kamil published this revision for review.Nov 6 2024, 3:41 AM

kamil edited the summary of this revision. (Show Details)

kamil added inline comments.

services/backup/src/http/handlers/backup.rs
111–122 ↗	(On Diff #45615)	We need an old backup item to add new holder to user data
124–158 ↗	(On Diff #45615)	this could probably implemented better but not sure - curious for reviewers perspective

bartek accepted this revision.Nov 6 2024, 6:12 AM

bartek added inline comments.

services/backup/src/http/handlers/backup.rs
124–158 ↗	(On Diff #45615)	From data safety and consistency perspective, this code makes a lot of sense - we make sure each backup has its own attachment holders. It's slightly worse about performance due to multiple Blob service calls, however this code won't be run frequently. A potential improvement would be to use the batch-holder endpoints, but this is a follow-up for a separate diff, it can be deferred.

This revision is now accepted and ready to land.Nov 6 2024, 6:12 AM

kamil added inline comments.Nov 7 2024, 2:08 AM

services/backup/src/http/handlers/backup.rs
124–158 ↗	(On Diff #45615)	Good call, created ENG-9874

Closed by commit rCOMM3eeef1e77029: [backup-service] implement endpoint to upload User Keys (authored by kamil). · Explain WhyNov 7 2024, 4:07 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM3eeef1e77029: [backup-service] implement endpoint to upload User Keys.

Revision Contents
Changeset List

Path

Size

services/

backup/

src/

http/

handlers/

137 lines

8 lines

Diff 45681

services/backup/src/http/handlers/backup.rs

Loading...

services/backup/src/http/mod.rs

Loading...