Paths

Table of Contentst

Differential D14060

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

Authored by kamil on Nov 27 2024, 9:45 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 27, 7:30 AM

	Unknown Object (File)
	Thu, Dec 26, 4:22 PM

	Unknown Object (File)
	Thu, Dec 26, 5:51 AM

	Unknown Object (File)
	Thu, Dec 26, 5:51 AM

	Unknown Object (File)
	Thu, Dec 26, 5:51 AM

	Unknown Object (File)
	Thu, Dec 26, 5:46 AM

	Unknown Object (File)
	Mon, Dec 23, 9:55 AM

	Unknown Object (File)
	Mon, Dec 23, 5:46 AM

View All 35 Files

Subscribers

Details

Reviewers

bartek
tomek

Commits

rCOMM618bdb01cda6: [native] add handler for staff to attempt backup restore

Summary

Depends on D14059

Test Plan

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

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Nov 27 2024, 9:45 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptNov 27 2024, 9:45 AM

Harbormaster completed remote builds in B32888: Diff 46103.Nov 27 2024, 10:10 AM

kamil published this revision for review.Nov 27 2024, 10:25 AM

kamil added inline comments.

native/backup/backup-handler.js
91–100 ↗	(On Diff #46103)	can be done using `Promise.all`
145–160 ↗	(On Diff #46103)	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.Nov 28 2024, 4:53 AM

kamil added inline comments.Nov 28 2024, 6:19 AM

native/backup/backup-handler.js
148–150 ↗	(On Diff #46103)

bartek accepted this revision.Fri, Nov 29, 2:59 AM

bartek added inline comments.

native/backup/backup-handler.js
145–160 ↗	(On Diff #46103)	Looks reasonable

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

address notes

Harbormaster completed remote builds in B32919: Diff 46180.Tue, Dec 3, 5:27 AM

Closed by commit rCOMM618bdb01cda6: [native] add handler for staff to attempt backup restore (authored by kamil). · Explain WhyTue, Dec 3, 6:28 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM618bdb01cda6: [native] add handler for staff to attempt backup restore.

kamil mentioned this in D14077: [native] remove comment from `BackupHandler`.Tue, Dec 3, 6:33 AM

kamil mentioned this in rCOMM066e576587e0: [native] remove comment from `BackupHandler`.Wed, Dec 4, 4:38 AM

Revision Contents
Changeset List

Path

Size

native/

backup/

backup-handler.js

78 lines

Diff 46201

native/backup/backup-handler.js

Loading...