Paths

Table of Contentst

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

Authored by kamil on Oct 29 2024, 4:28 PM.

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Oct 29 2024, 4:28 PM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 29 2024, 4:28 PM

Harbormaster failed remote builds in B32408: Diff 45431!Oct 29 2024, 4:55 PM

kamil added inline comments.Oct 30 2024, 10: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

rebase

Harbormaster completed remote builds in B32431: Diff 45458.Oct 30 2024, 11:08 AM

kamil published this revision for review.Oct 30 2024, 4:29 PM

kamil added a child revision: D13839: [native] make it possible to test retrieving latest backup info.Oct 31 2024, 12:24 PM

Neat idea with this rename to backup info

This revision is now accepted and ready to land.Nov 4 2024, 1:01 PM

rebase

Harbormaster completed remote builds in B32537: Diff 45605.Nov 5 2024, 12:51 PM

kamil added a child revision: D13872: [commtest] refactor backup service integration tests.Nov 5 2024, 2:06 PM

Closed by commit rCOMM8f72e02e2b90: [backup-service][backup-client] update `/latest` endpoint to return both…. · Explain WhyNov 7 2024, 12:07 PM

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

Path

Size

native/

native_rust_library/

src/

backup.rs

19 lines

services/

backup/

src/

http/

handlers/

backup.rs

7 lines

mod.rs

4 lines

commtest/

tests/

backup_integration_test.rs

13 lines

backup_performance_test.rs

6 lines

shared/

backup_client/

src/

lib.rs

6 lines

comm-lib/

src/

backup/

mod.rs

5 lines

Diff 45666

View Options

native/native_rust_library/src/backup.rs

View Options

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

View Options

services/backup/src/http/mod.rs

View Options

services/commtest/tests/backup_integration_test.rs

View Options

services/commtest/tests/backup_performance_test.rs

View Options

shared/backup_client/src/lib.rs

View Options

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

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

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 45666

native/native_rust_library/src/backup.rs

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

services/backup/src/http/mod.rs

services/commtest/tests/backup_integration_test.rs

services/commtest/tests/backup_performance_test.rs

shared/backup_client/src/lib.rs

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

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

Revision Contents
Changeset List