Skip to content

STS 537: wipe pvcs of hbase when requested - #51

Merged
fvlankvelt merged 6 commits into
mainfrom
sts-537-wipe-pvcs
Oct 16, 2026
Merged

STS 537: wipe pvcs of hbase when requested#51
fvlankvelt merged 6 commits into
mainfrom
sts-537-wipe-pvcs

Conversation

@fvlankvelt

@fvlankvelt fvlankvelt commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

LouisLotter
LouisLotter reviewed Oct 16, 2026
View reviewed changes
Comment thread internal/orchestration/restore/hbase.go
Comment thread internal/orchestration/restore/hbase.go
@fvlankvelt
fvlankvelt enabled auto-merge October 16, 2026 12:26
LouisLotter
LouisLotter reviewed Oct 16, 2026
View reviewed changes

@LouisLotter LouisLotter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is now green, and the one-release-per-namespace clarification is accepted. This CLI PR can be released independently of verify-backup #21 once the error paths below are fixed. The remaining concern is within #51: a failed wipe still allows the restore to proceed.

Comment thread cmd/stackgraph/restore.go Outdated
Comment thread internal/orchestration/restore/hbase.go Outdated
@fvlankvelt
fvlankvelt merged commit e43d6ac into main Oct 16, 2026
6 checks passed
@fvlankvelt
fvlankvelt deleted the sts-537-wipe-pvcs branch October 16, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LouisLotter LouisLotter LouisLotter left review comments
@craffit craffit craffit approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fvlankvelt @craffit @LouisLotter