Skip to content

Commit

Permalink
Fix strict-local volume restore stuck in attaching state
Browse files Browse the repository at this point in the history
For a volume being restored, the spec.nodeID is empyty but the status.ownerID is set.
Thus, set r.Spec.HardNodeAffinity to volume.status.ownerID instead.

Longhorn 6239

Signed-off-by: Derek Su <derek.su@suse.com>
  • Loading branch information
derekbit authored and David Ko committed Jul 10, 2023
1 parent de26d0a commit 3ea2dcd
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
11 changes: 7 additions & 4 deletions controller/volume_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -1208,11 +1208,14 @@ func (vc *VolumeController) ReconcileVolumeState(v *longhorn.Volume, es map[stri
continue
}
if v.Spec.DataLocality == longhorn.DataLocalityStrictLocal {
if v.Spec.NodeID == "" {
continue
if v.Status.RestoreRequired {
r.Spec.HardNodeAffinity = v.Status.OwnerID
} else {
if v.Spec.NodeID == "" {
continue
}
r.Spec.HardNodeAffinity = v.Spec.NodeID
}

r.Spec.HardNodeAffinity = v.Spec.NodeID
}

scheduledReplica, multiError, err := vc.scheduler.ScheduleReplica(r, rs, v)
Expand Down
2 changes: 1 addition & 1 deletion types/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ const (
)

// SettingsRelatedToVolume should match the items in datastore.GetLabelsForVolumesFollowsGlobalSettings
// TODO: May need to add the data locality check
// TODO: May need to add the data locality check
var SettingsRelatedToVolume = map[string]string{
string(SettingNameReplicaAutoBalance): LonghornLabelValueIgnored,
string(SettingNameSnapshotDataIntegrity): LonghornLabelValueIgnored,
Expand Down

0 comments on commit 3ea2dcd

Please sign in to comment.