Page MenuHomePhabricator
Differential D14060

[native] add handler for staff to attempt backup restore
AcceptedPublic
Actions

Authored by kamil on Wed, Nov 27, 9:45 AM.
Tags
None
Referenced Files
F3392997: D14060.diff
Sat, Nov 30, 10:59 AM
Unknown Object (File)
Wed, Nov 27, 5:28 PM
Unknown Object (File)
Wed, Nov 27, 3:47 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
View All 15 Files
Subscribers
ashoat

Details

Reviewers
bartek
tomek
Summary

ENG-9711.

Depends on D14059

Test Plan

Update millisecondsPerDay to be lower value and test if this works.

Diff Detail

Repository
rCOMM Comm
Branch
backup-2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Wed, Nov 27, 9:45 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptWed, Nov 27, 9:45 AM
Harbormaster completed remote builds in B32888: Diff 46103.Wed, Nov 27, 10:10 AM
kamil published this revision for review.Wed, Nov 27, 10:25 AM
kamil added inline comments.
native/backup/backup-handler.js
91–100

can be done using Promise.all

145–160

causing another update after 24 hours is intentional for testing purposes and checking stabillity

kamil added a child revision: D14061: [native] handle invalid CSAT in Backup authenticated API.Thu, Nov 28, 4:53 AM
kamil added inline comments.Thu, Nov 28, 6:19 AM
native/backup/backup-handler.js
148–150
bartek accepted this revision.Fri, Nov 29, 2:59 AM
bartek added inline comments.
native/backup/backup-handler.js
145–160

Looks reasonable

This revision is now accepted and ready to land.Fri, Nov 29, 2:59 AM

Revision Contents
Changeset List

PathSize
native/
backup/
73 lines

Diff 46103

View Options

native/backup/backup-handler.js

Loading...