diff options
| author | Sebastiaan van Stijn <github@gone.nl> | 2020-03-06 12:45:15 +0100 |
|---|---|---|
| committer | Sebastiaan van Stijn <github@gone.nl> | 2020-03-06 12:45:15 +0100 |
| commit | 32e554e68195b48d462d5ef96defbb1e64679eae (patch) | |
| tree | dc82a954a9e915d307616234f1134de6b4877225 /layer/migration.go | |
| parent | cc2d166cfd6185059c729173f33db13eeab00695 (diff) | |
| download | docker-32e554e68195b48d462d5ef96defbb1e64679eae.tar.gz | |
Remove "could not get checksum with tar-split" debug message
This code was originally written for v1.10 migration where it signified that layers
were recomputed from pre 1.8 layout and could possibly change the tarball checksums.
It's now being repurposed in the BuildKit adapter but there it doesn't have any
warn condition as all data generated by builder is new anyway.
Currently, debug log entries as the one below may appear in the daemon logs:
[2019-10-02T10:00:06.690674253Z] could not get checksum for "x128nsj79yzfx4j5h6em2w2on" with tar-split: "no tar-split file"
This patch removes the debug log, as it may confuse users ("we couldn't validate
what we downloaded, but we're gonna run it anyway?")
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Diffstat (limited to 'layer/migration.go')
| -rw-r--r-- | layer/migration.go | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/layer/migration.go b/layer/migration.go index 5834659433..80f0ff7ff4 100644 --- a/layer/migration.go +++ b/layer/migration.go @@ -15,7 +15,6 @@ import ( func (ls *layerStore) ChecksumForGraphID(id, parent, oldTarDataPath, newTarDataPath string) (diffID DiffID, size int64, err error) { defer func() { if err != nil { - logrus.Debugf("could not get checksum for %q with tar-split: %q", id, err) diffID, size, err = ls.checksumForGraphIDNoTarsplit(id, parent, newTarDataPath) } }() |