Skip to content

Commit

Permalink
Merge pull request #13040 from roosterfish/fix_rsync_feature_offer
Browse files Browse the repository at this point in the history
Migration: Accept offered rsync features for BLOCK_AND_RSYNC (v2)
  • Loading branch information
tomponline authored Mar 5, 2024
2 parents 2e9a403 + ef209cb commit 79be0e7
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions lxd/migration/migration_volumes.go
Original file line number Diff line number Diff line change
Expand Up @@ -193,14 +193,8 @@ func MatchTypes(offer *MigrationHeader, fallbackType MigrationFSType, ourTypes [
offeredFeatures = offer.GetZfsFeaturesSlice()
} else if offerFSType == MigrationFSType_BTRFS {
offeredFeatures = offer.GetBtrfsFeaturesSlice()
} else if offerFSType == MigrationFSType_RSYNC {
} else if shared.ValueInSlice(offerFSType, []MigrationFSType{MigrationFSType_RSYNC, MigrationFSType_BLOCK_AND_RSYNC}) {
offeredFeatures = offer.GetRsyncFeaturesSlice()
if !shared.ValueInSlice("bidirectional", offeredFeatures) {
// If no bi-directional support, this means we are getting a response from
// an old LXD server that doesn't support bidirectional negotiation, so
// assume LXD 3.7 level. NOTE: Do NOT extend this list of arguments.
offeredFeatures = []string{"xattrs", "delete", "compress"}
}
}

// Find common features in both our type and offered type.
Expand Down

0 comments on commit 79be0e7

Please sign in to comment.