[backup-client][native_rust_library] update backup client to run cleanup on any failure
AcceptedPublic
Actions

Authored by kamil on Thu, Nov 14, 8:17 AM.

Tags

None

Referenced Files

	F3363152: D13930.id45820.diff
	Mon, Nov 25, 12:30 AM

	Unknown Object (File)
	Fri, Nov 22, 11:47 AM

	Unknown Object (File)
	Thu, Nov 21, 2:25 PM

	Unknown Object (File)
	Wed, Nov 20, 8:47 PM

	Unknown Object (File)
	Wed, Nov 20, 1:14 AM

	Unknown Object (File)
	Tue, Nov 19, 9:02 PM

	Unknown Object (File)
	Tue, Nov 19, 3:50 AM

	Unknown Object (File)
	Mon, Nov 18, 2:06 PM

View All 16 Files

Subscribers

ashoat

Details

Reviewers

bartek
tomek

Summary

ENG-9656.

Previously, we ran cleanup only if the final stage failed, potentially leaving some files in storage - this diff fixes this and attempt cleanup on each failure.

Depends on D13929

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Branch

publish-client-backup

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Thu, Nov 14, 8:17 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptThu, Nov 14, 8:17 AM

kamil added a child revision: D13931: [backup-client][native_rust_library] implement creating User Keys backup.Thu, Nov 14, 8:22 AM

Harbormaster completed remote builds in B32661: Diff 45820.Thu, Nov 14, 9:52 AM

kamil published this revision for review.Fri, Nov 15, 2:21 AM

bartek accepted this revision.Tue, Nov 19, 5:41 AM

This revision is now accepted and ready to land.Tue, Nov 19, 5:41 AM

Revision Contents
Changeset List

Path

Size

native/

native_rust_library/

src/

backup.rs

12 lines

Diff 45820

View Options

native/native_rust_library/src/backup.rs

[backup-client][native_rust_library] update backup client to run cleanup on any failureAcceptedPublicActions