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

Authored by kamil on Nov 14 2024, 4:17 PM.

Details

Reviewers

bartek
tomek

Commits

rCOMMc935e7cacce2: [backup-client][native_rust_library] update backup client to run cleanup on any…

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Nov 14 2024, 4:17 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptNov 14 2024, 4:17 PM

kamil added a child revision: D13931: [backup-client][native_rust_library] implement creating User Keys backup.Nov 14 2024, 4:22 PM

Harbormaster completed remote builds in B32661: Diff 45820.Nov 14 2024, 5:52 PM

kamil published this revision for review.Nov 15 2024, 10:21 AM

bartek accepted this revision.Nov 19 2024, 1:41 PM

This revision is now accepted and ready to land.Nov 19 2024, 1:41 PM

Closed by commit rCOMMc935e7cacce2: [backup-client][native_rust_library] update backup client to run cleanup on any…. · Explain WhyNov 25 2024, 12:07 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMc935e7cacce2: [backup-client][native_rust_library] update backup client to run cleanup on any….

Revision Contents
Changeset List

Path

Size

native/

native_rust_library/

src/

backup.rs

12 lines

Diff 46008

View Options

native/native_rust_library/src/backup.rs

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