diff --git a/syncoid b/syncoid index 5ed00f07..6d37bb9a 100755 --- a/syncoid +++ b/syncoid @@ -404,6 +404,7 @@ sub syncdataset { my $newsyncsnap; my $matchingsnap; + my $usebookmark = 0; # skip snapshot checking/creation in case of resumed receive if (!defined($receivetoken)) { @@ -657,26 +658,29 @@ sub syncdataset { my $targetsize = getzfsvalue($targethost,$targetfs,$targetisroot,'-p used'); my $bookmark = 0; - my $bookmarkcreation = 0; + my $bookmarkguid = 0; + my $bookmarksnap = 0; $matchingsnap = getmatchingsnapshot($sourcefs, $targetfs, \%snaps); - if (! $matchingsnap) { - # no matching snapshots, check for bookmarks as fallback - my %bookmarks = getbookmarks($sourcehost,$sourcefs,$sourceisroot); - # check for matching guid of source bookmark and target snapshot (oldest first) - foreach my $snap ( sort { $snaps{'target'}{$b}{'creation'}<=>$snaps{'target'}{$a}{'creation'} } keys %{ $snaps{'target'} }) { - my $guid = $snaps{'target'}{$snap}{'guid'}; + # find most recent matching bookmark + my %bookmarks = getbookmarks($sourcehost,$sourcefs,$sourceisroot); - if (defined $bookmarks{$guid}) { - # found a match - $bookmark = $bookmarks{$guid}{'name'}; - $bookmarkcreation = $bookmarks{$guid}{'creation'}; - $matchingsnap = $snap; - last; - } + # check for matching guid of source bookmark and target snapshot (oldest first) + foreach my $snap ( sort { $snaps{'target'}{$b}{'creation'}<=>$snaps{'target'}{$a}{'creation'} } keys %{ $snaps{'target'} }) { + my $guid = $snaps{'target'}{$snap}{'guid'}; + + if (defined $bookmarks{$guid}) { + # found a match + $bookmark = $bookmarks{$guid}{'name'}; + $bookmarkguid = $guid; + $bookmarksnap = $snap; + last; } + } + if (! $matchingsnap) { + # no matching snapshots, check for bookmarks as fallback if (! $bookmark) { if ($args{'force-delete'}) { if (!$quiet) { print "Removing $targetfs because no matching snapshots were found\n"; } @@ -722,9 +726,18 @@ sub syncdataset { # return false now in case more child datasets need replication. return 0; + } else { + # use bookmark as fallback + $matchingsnap = $bookmarksnap; + $usebookmark = 1; } + } elsif ($bookmark && $bookmarkguid ne $snaps{'source'}{$matchingsnap}{'guid'} && $bookmarks{$bookmarkguid}{'creation'} > substr($snaps{'source'}{$matchingsnap}{'creation'}, 0, -3)) { + if ($debug) { print "DEBUG: using bookmark $bookmark because it was created after latest matching snapshot $matchingsnap\n"; } + $matchingsnap = $bookmarksnap; + $usebookmark = 1; } + # make sure target is (still) not currently in receive. if (iszfsbusy($targethost,$targetfs,$targetisroot)) { warn "Cannot sync now: $targetfs is already target of a zfs receive process.\n"; @@ -741,7 +754,7 @@ sub syncdataset { my $nextsnapshot = 0; - if ($bookmark) { + if ($usebookmark) { my $bookmarkescaped = escapeshellparam($bookmark); if (!defined $args{'no-stream'}) { @@ -824,7 +837,7 @@ sub syncdataset { # do a normal replication if bookmarks aren't used or if previous # bookmark replication was only done to the next oldest snapshot - if (!$bookmark || $nextsnapshot) { + if (!$usebookmark || $nextsnapshot) { if ($matchingsnap eq $newsyncsnap) { # edge case: bookmark replication used the latest snapshot return 0; @@ -904,7 +917,7 @@ sub syncdataset { # if "--use-hold" parameter is used set hold on newsync snapshot and remove hold on matching snapshot both on source and target # hold name: "syncoid" + identifier + hostname -> in case of replication to multiple targets separate holds can be set for each target by assinging different identifiers to each target. Only if all targets have been replicated all syncoid holds are removed from the matching snapshot and it can be removed - if (defined $args{'use-hold'}) { + if (defined $args{'use-hold'} && !$usebookmark) { my $holdcmd; my $holdreleasecmd; my $hostid = hostname();