[services] Backup - Update service implementation for pull backup
AbandonedPublicDraft
Actions

Authored by • karol on Apr 6 2022, 7:35 AM.

Tags

None

Referenced Files

	F3374038: D3628.diff
	Tue, Nov 26, 1:24 PM

	Unknown Object (File)
	Mon, Nov 18, 5:40 PM

	Unknown Object (File)
	Sun, Nov 10, 9:31 AM

	Unknown Object (File)
	Thu, Nov 7, 4:46 PM

	Unknown Object (File)
	Tue, Nov 5, 5:28 PM

	Unknown Object (File)
	Oct 13 2024, 3:42 PM

	Unknown Object (File)
	Oct 13 2024, 3:41 PM

	Unknown Object (File)
	Oct 13 2024, 3:39 PM

Subscribers

Details

Reviewers

tomek
• max

Summary

Depends on D3627

Since I removed a stream for the pull backup request in the previous diff, now we should use the proper reactor.

Test Plan

The same as in D3535

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• karol created this revision.Apr 6 2022, 7:35 AM

Herald added subscribers: • abosh, • benschac, atul and 3 others. · View Herald TranscriptApr 6 2022, 7:35 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 6 2022, 7:41 AM

Harbormaster failed remote builds in B7920: Diff 11102!

• karol edited the summary of this revision. (Show Details)Apr 6 2022, 7:51 AM

• karol edited the test plan for this revision. (Show Details)

• karol added reviewers: tomek, • max.

• karol added a parent revision: D3627: [services] Backup - Update proto file for pull backup.

• karol added a child revision: D3629: [services] Backup - Add blob get client reactor.

• karol abandoned this revision.Apr 6 2022, 8:04 AM

Revision Contents
Changeset List

Path

Size

services/

backup/

docker-server/

contents/

server/

src/

BackupServiceImpl.h

5 lines

BackupServiceImpl.cpp

10 lines

Diff 11102

View Options

services/backup/docker-server/contents/server/src/BackupServiceImpl.h