Page MenuHomePhabricator
Differential D13809

[backup-service][backup-client] update `/latest` endpoint to return both `backupID` and `userID`
ClosedPublic
Actions

Authored by kamil on Tue, Oct 29, 9:28 AM.
Tags
None
Referenced Files
F3328734: D13809.id45666.diff
Wed, Nov 20, 3:00 PM
Unknown Object (File)
Sat, Nov 16, 1:41 AM
Unknown Object (File)
Thu, Nov 14, 8:24 PM
Unknown Object (File)
Thu, Nov 14, 2:06 PM
Unknown Object (File)
Wed, Nov 13, 3:54 AM
Unknown Object (File)
Tue, Nov 12, 4:02 PM
Unknown Object (File)
Mon, Nov 11, 11:35 PM
Unknown Object (File)
Mon, Nov 11, 12:38 PM
View All 25 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMM8f72e02e2b90: [backup-service][backup-client] update `/latest` endpoint to return both…
Summary

ENG-6145.

We need this to return userID too.
Renaming path and following types to make it more readable.

Depends on D13808

Test Plan

run commtest

Diff Detail

Repository
rCOMM Comm
Branch
backup-work-3
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Tue, Oct 29, 9:28 AM
kamil held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptTue, Oct 29, 9:28 AM
Harbormaster failed remote builds in B32408: Diff 45431!Tue, Oct 29, 9:55 AM
kamil added inline comments.Wed, Oct 30, 3:35 AM
native/native_rust_library/src/backup.rs
329 ↗(On Diff #45431)

this is going to be updated later as part of Native changes for minimal version of backup

kamil updated this revision to Diff 45458.Wed, Oct 30, 3:50 AM

rebase

Harbormaster completed remote builds in B32431: Diff 45458.Wed, Oct 30, 4:08 AM
kamil published this revision for review.Wed, Oct 30, 9:29 AM
kamil added a child revision: D13839: [native] make it possible to test retrieving latest backup info.Thu, Oct 31, 5:24 AM
bartek accepted this revision.Mon, Nov 4, 5:01 AM

Neat idea with this rename to backup info

This revision is now accepted and ready to land.Mon, Nov 4, 5:01 AM
kamil updated this revision to Diff 45605.Tue, Nov 5, 4:21 AM

rebase

Harbormaster completed remote builds in B32537: Diff 45605.Tue, Nov 5, 4:51 AM
kamil added a child revision: D13872: [commtest] refactor backup service integration tests.Tue, Nov 5, 6:06 AM
Closed by commit rCOMM8f72e02e2b90: [backup-service][backup-client] update `/latest` endpoint to return both… (authored by kamil). · Explain WhyThu, Nov 7, 4:07 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM8f72e02e2b90: [backup-service][backup-client] update `/latest` endpoint to return both….

Revision Contents
Changeset List

PathSize
native/
native_rust_library/
src/
19 lines
services/
backup/
src/
http/
handlers/
7 lines
4 lines
commtest/
tests/
13 lines
6 lines
shared/
backup_client/
src/
6 lines
comm-lib/
src/
backup/
5 lines

Diff 45458

View Options

native/native_rust_library/src/backup.rs

Loading...
View Options

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

Loading...
View Options

services/backup/src/http/mod.rs

Loading...
View Options

services/commtest/tests/backup_integration_test.rs

Loading...
View Options

services/commtest/tests/backup_performance_test.rs

Loading...
View Options

shared/backup_client/src/lib.rs

Loading...
View Options

shared/comm-lib/src/backup/mod.rs

Loading...