[services] Backup - Set backup id in create new backup reactor's response
ClosedPublic
Actions

Authored by • karol on Apr 4 2022, 2:24 PM.

Details

Reviewers

tomek
• max
varun
• jim

Commits

rCOMM1a7f4c28508f: [services] Backup - Set backup id in create new backup reactor's response

Summary

Depends on D3596

Setting the backup id for the new backup in the reactor's response is missing, this diff fixes it.

Test Plan

The same as in D3535

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• karol created this revision.Apr 4 2022, 2:24 PM

Herald added subscribers: • abosh, • benschac, atul and 3 others. · View Herald TranscriptApr 4 2022, 2:24 PM

• karol edited the summary of this revision. (Show Details)Apr 4 2022, 2:28 PM

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

• karol added reviewers: tomek, • max, varun, • jim.

• karol added a parent revision: D3596: [services] Blob - Fix test name for storage manager tests.

Services build passed and there are changes only in services.

Harbormaster completed remote builds in B7837: Diff 11005.Apr 4 2022, 7:20 PM

• karol added a child revision: D3626: [services] Backup - client base reactors - terminate on error.Apr 6 2022, 2:48 PM

The diff that introduced this code should have a test plan that would caught this issue

This revision is now accepted and ready to land.Apr 7 2022, 7:08 AM

Closed by commit rCOMM1a7f4c28508f: [services] Backup - Set backup id in create new backup reactor's response. · Explain WhyApr 13 2022, 11:58 AM

This revision was automatically updated to reflect the committed changes.

• karol added a commit: rCOMM1a7f4c28508f: [services] Backup - Set backup id in create new backup reactor's response.

Revision Contents
Changeset List

Path

Size

services/

backup/

docker-server/

contents/

server/

src/

Reactors/

server/

CreateNewBackupReactor.h

1 line

Diff 11402

View Options

services/backup/docker-server/contents/server/src/Reactors/server/CreateNewBackupReactor.h

[services] Backup - Set backup id in create new backup reactor's responseClosedPublicActions