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

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

Tags

None

Referenced Files

	F5892660: D3628.id.diff
	Thu, Apr 17, 8:21 AM

	Unknown Object (File)
	Thu, Apr 17, 7:51 AM

	Unknown Object (File)
	Wed, Apr 16, 3:48 PM

	Unknown Object (File)
	Tue, Apr 8, 6:30 AM

	Unknown Object (File)
	Sun, Apr 6, 10:00 AM

	Unknown Object (File)
	Sun, Apr 6, 10:00 AM

	Unknown Object (File)
	Sun, Apr 6, 9:59 AM

	Unknown Object (File)
	Thu, Apr 3, 5:45 AM

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