Skip to content

VMware Plugin: Fix NVRAM backup when datacenter is not in root folder#2475

Merged
BareosBot merged 3 commits intobareos:bareos-25from
pstorz:backport/bareos-25/fix-vmware-issue-2410
Dec 12, 2025
Merged

VMware Plugin: Fix NVRAM backup when datacenter is not in root folder#2475
BareosBot merged 3 commits intobareos:bareos-25from
pstorz:backport/bareos-25/fix-vmware-issue-2410

Conversation

@pstorz
Copy link
Member

@pstorz pstorz commented Dec 8, 2025

Backport of PR #2461 to bareos-25

Checklist for the reviewer of the PR (will be processed by the Bareos team)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

General
  • Correct milestone is set
Source code quality (if there were changes to the original PR)
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR

Backport quality

@pstorz pstorz added bug This addresses a bug is a backport to 25 This is a backport from master to bareos-25 labels Dec 8, 2025
@pstorz pstorz requested a review from sduehr December 8, 2025 11:02
@sduehr sduehr added this to the 25.0.2 milestone Dec 11, 2025
Copy link
Member

@sduehr sduehr left a comment

Choose a reason for hiding this comment

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

Thanks, looks good.

sduehr and others added 3 commits December 12, 2025 15:09
In most cases, the DC (datacenter) object is located in the root folder,
but it could also be inside a folder. In that case the plugin failed to
backup the NVRAM files and on restore it didn't find the DC. Both cases
are fixed now.

(cherry picked from commit f9d5dbf)
(cherry picked from commit bf056cd)
@BareosBot BareosBot force-pushed the backport/bareos-25/fix-vmware-issue-2410 branch from af4435d to 6ba6a9a Compare December 12, 2025 15:10
@BareosBot BareosBot merged commit 2a7ebec into bareos:bareos-25 Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug This addresses a bug is a backport to 25 This is a backport from master to bareos-25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants