Skip to content
This repository has been archived by the owner on May 4, 2021. It is now read-only.

Commit

Permalink
Forgive the case when a whiteout file trying to cover nonexistent path (
Browse files Browse the repository at this point in the history
#358)

* Forgive the case when a whiteout file trying to cover nonexistent path

* Update comment
  • Loading branch information
yiranwang52 authored Mar 15, 2021
1 parent 7d88e9f commit 026a501
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion lib/snapshot/mem_layer.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"sort"
"strings"

"github.com/uber/makisu/lib/log"
"github.com/uber/makisu/lib/pathutils"
"github.com/uber/makisu/lib/tario"
)
Expand Down Expand Up @@ -114,7 +115,9 @@ func (f *whiteoutMemFile) updateMemFS(node *memFSNode) error {
if i != len(parts)-1 {
return fmt.Errorf("missing intermediate dir %s in %s", part, f.del)
}
return fmt.Errorf("whiteout nonexistent path %s", f.del)
// This could happen to files that's cleaned up in the background, like
// python package's .dist-info or .pth file.
log.Warnf("Trying to whiteout nonexistent path: %s", f.del)
}
}
return nil
Expand Down

0 comments on commit 026a501

Please sign in to comment.