Skip to content

Commit 6fe2ee8

Browse files
committed
fix usage loss for snapshots with labels in MergeSnapshotter
Signed-off-by: liulanzheng <[email protected]>
1 parent 3d50b97 commit 6fe2ee8

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)