feat(v2 upgrade/controller): upgrading node shouldn't be responsible …

…for backup resources Longhorn 9104 Signed-off-by: Derek Su <[email protected]>
derekbit · Dec 15, 2024 · b23af6c · b23af6c
1 parent aefe41d
commit b23af6c
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/controller/backup_controller.go b/controller/backup_controller.go
@@ -596,6 +596,14 @@ func (bc *BackupController) isResponsibleFor(b *longhorn.Backup, defaultEngineIm
 		err = errors.Wrap(err, "error while checking isResponsibleFor")
 	}()
 
+	upgradeRequested, err := bc.ds.IsNodeDataEngineUpgradeRequested(bc.controllerID)
+	if err != nil {
+		return false, err
+	}
+	if upgradeRequested {
+		return false, nil
+	}
+
 	isResponsible := isControllerResponsibleFor(bc.controllerID, bc.ds, b.Name, "", b.Status.OwnerID)
 
 	currentOwnerEngineAvailable, err := bc.ds.CheckEngineImageReadiness(defaultEngineImage, b.Status.OwnerID)