Skip to content

Commit

Permalink
pb-4969: Fixed issue with backuplocation CR variable, which was causing
Browse files Browse the repository at this point in the history
the nfs backup issue.

	- backuplocation CR variable scope was lost inside the if
	  condtion. Because of this the nfs parameters are not passed to kopia job.
	- The kopia writes the snapshot file directly  inside the job
	  pod file system and files are lost once the pod is completed.
	- Since the nfs parameters are not passed, the nfs pvc was not
	  getting mounted inside the nfs job pod.
  • Loading branch information
sivakumar subraani committed Nov 21, 2023
1 parent fdf611b commit b7fd17f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/controllers/dataexport/reconcile.go
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,7 @@ func (c *Controller) sync(ctx context.Context, in *kdmpapi.DataExport) (bool, er
blNamespace = vb.Spec.BackupLocation.Namespace
}

backupLocation, err := readBackupLocation(blName, blNamespace, "")
backupLocation, err = readBackupLocation(blName, blNamespace, "")
if err != nil {
msg := fmt.Sprintf("reading of backuplocation [%v/%v] failed: %v", blNamespace, blName, err)
logrus.Errorf(msg)
Expand Down

0 comments on commit b7fd17f

Please sign in to comment.