Skip to content

BCF-6482: Allow to exclude nodes for FF restore during BMR#84

Open
Andrus Suvalau (svlv) wants to merge 1 commit intodevelopfrom
feature/BCF-6482-allow-to-exclude-nodes-for-ff-restore-during-bmr
Open

BCF-6482: Allow to exclude nodes for FF restore during BMR#84
Andrus Suvalau (svlv) wants to merge 1 commit intodevelopfrom
feature/BCF-6482-allow-to-exclude-nodes-for-ff-restore-during-bmr

Conversation

@svlv
Copy link

Add the COVE_EXCLUDE_NODES_FROM_RESTORE variable to allow to exclude nodes for FF restore during BMR

Copilot AI review requested due to automatic review settings March 16, 2026 14:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new ReaR configuration knob to let users exclude additional filesystem “nodes” from Cove FileSystem restore during bare-metal recovery (BMR), while preserving the existing default exclusions.

Changes:

  • Introduce COVE_EXCLUDE_NODES_FROM_RESTORE as a configurable array in default.conf.
  • Build Cove restore -exclude arguments from COVE_EXCLUDE_NODES_FROM_RESTORE (plus implicit exclusions) in the Cove restore script.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
usr/share/rear/restore/COVE/default/400_restore_with_cove.sh Builds FileSystem restore exclusions from a configurable array and appends them to restore_args.
usr/share/rear/conf/default.conf Adds the new COVE_EXCLUDE_NODES_FROM_RESTORE default (empty) configuration array.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Jira-Ref: BCF-6482: Allow to exclude nodes for FF restore during BMR
@svlv Andrus Suvalau (svlv) force-pushed the feature/BCF-6482-allow-to-exclude-nodes-for-ff-restore-during-bmr branch from 901b900 to ae46dd8 Compare March 17, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants