Skip to content

[minor] Option to exclude Applications PVC backup and restore and other fixes#2396

Merged
whitfiea merged 7 commits into
masterfrom
brpvc
Jun 18, 2026
Merged

[minor] Option to exclude Applications PVC backup and restore and other fixes#2396
whitfiea merged 7 commits into
masterfrom
brpvc

Conversation

@sanjayprab

@sanjayprab sanjayprab commented Jun 17, 2026

Copy link
Copy Markdown
Contributor
  • Option to include/exclude Apps' PVC backup and restore in B&R pipelines and interactive/noninteractive modes
  • Fixes to the custom storage class fields passed to Facilities app task

Note:
If user excludes PVC backup and during restore they forgetfully choose to include PVC, the code will look for the pvc backup data in the archives and skips pvc backup if its absent without breaking the flow.

needs:

Tests:

  1. Backup Flow showing PVC backup option
Screenshot 2026-06-17 at 11 02 21
  1. Backup pipeline showing Facilities PVC backup is skipped
Screenshot 2026-06-17 at 11 04 39
  1. Restore Flow showing PVC restore option
Screenshot 2026-06-17 at 18 06 33
  1. Restore Facilities app task showing the right values passed
Screenshot 2026-06-17 at 18 05 54

@whitfiea whitfiea left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need to update the backup and restore docs with these new options?

@sanjayprab

Copy link
Copy Markdown
Contributor Author

We need to update the backup and restore docs with these new options?

Thanks. Pushed the docs changes.

@sanjayprab sanjayprab requested a review from whitfiea June 18, 2026 10:22
@whitfiea whitfiea enabled auto-merge (squash) June 18, 2026 10:25
@whitfiea whitfiea merged commit 89b4dc5 into master Jun 18, 2026
11 checks passed
@whitfiea whitfiea deleted the brpvc branch June 18, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants