X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/e8f99cfef7cfbfcf1a1485d69250f24ced3fd609..2b952ed950affd0904da21e2dc7c7bf5cdd7d1ce:/lib/dispatchcloud/node_size.go diff --git a/lib/dispatchcloud/node_size.go b/lib/dispatchcloud/node_size.go index 2fac799f40..6fb46b5f46 100644 --- a/lib/dispatchcloud/node_size.go +++ b/lib/dispatchcloud/node_size.go @@ -6,7 +6,6 @@ package dispatchcloud import ( "errors" - "log" "regexp" "sort" "strconv" @@ -34,12 +33,10 @@ var pdhRegexp = regexp.MustCompile(`^[0-9a-f]{32}\+(\d+)$`) func estimateDockerImageSize(collectionPDH string) int64 { m := pdhRegexp.FindStringSubmatch(collectionPDH) if m == nil { - log.Printf("estimateDockerImageSize: '%v' did not match pdhRegexp, returning 0", collectionPDH) return 0 } n, err := strconv.ParseInt(m[1], 10, 64) if err != nil || n < 122 { - log.Printf("estimateDockerImageSize: short manifest %v or error (%v), returning 0", n, err) return 0 } // To avoid having to fetch the collection, take advantage of @@ -49,7 +46,7 @@ func estimateDockerImageSize(collectionPDH string) int64 { // the size of the manifest. // // Use the following heuristic: - // - Start with the length of the mainfest (n) + // - Start with the length of the manifest (n) // - Subtract 80 characters for the filename and file segment // - Divide by 42 to get the number of block identifiers ('hash\+size\ ' is 32+1+8+1) // - Assume each block is full, multiply by 64 MiB