Skip to content

Commit 257622c

Browse files
authored
Merge pull request #4389 from liulanzheng/master
fix usage loss for snapshots with labels in MergeSnapshotter
2 parents 36c5550 + 6fe2ee8 commit 257622c

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

snapshot/merge.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -185,19 +185,25 @@ func mergeUsageOf(info snapshots.Info) (usage snapshots.Usage, ok bool, rerr err
185185
if info.Labels == nil {
186186
return snapshots.Usage{}, false, nil
187187
}
188+
hasMergeUsageLabel := false
188189
if str, ok := info.Labels[mergeUsageSizeLabel]; ok {
189190
i, err := strconv.Atoi(str)
190191
if err != nil {
191192
return snapshots.Usage{}, false, err
192193
}
193194
usage.Size = int64(i)
195+
hasMergeUsageLabel = true
194196
}
195197
if str, ok := info.Labels[mergeUsageInodesLabel]; ok {
196198
i, err := strconv.Atoi(str)
197199
if err != nil {
198200
return snapshots.Usage{}, false, err
199201
}
200202
usage.Inodes = int64(i)
203+
hasMergeUsageLabel = true
204+
}
205+
if !hasMergeUsageLabel {
206+
return snapshots.Usage{}, false, nil
201207
}
202208
return usage, true, nil
203209
}

0 commit comments

Comments
 (0)