FIXMEs

Signed-off-by: Miloslav Trmač <[email protected]>

storage/storage_dest.go

@@ -386,6 +386,7 @@ func (s *storageImageDestination) tryReusingBlobAsPending(blobDigest digest.Dige
 
 	// Check if we've already cached it in a file.
 	if size, ok := s.fileSizes[blobDigest]; ok {
+		// FIXME: What ensures layer identification?
 		return true, private.ReusedBlob{
 			Digest: blobDigest,
 			Size:   size,
@@ -448,7 +449,7 @@ func (s *storageImageDestination) tryReusingBlobAsPending(blobDigest digest.Dige
 
 	if options.TOCDigest != "" && options.LayerIndex != nil {
 		// Check if we have a chunked layer in storage with the same TOC digest.
-		layers, err := s.imageRef.transport.store.LayersByTOCDigest(options.TOCDigest)
+		layers, err := s.imageRef.transport.store.LayersByTOCDigest(options.TOCDigest) // FIXME: We can’t find this layer later!
 
 		if err != nil && !errors.Is(err, storage.ErrLayerUnknown) {
 			return false, private.ReusedBlob{}, fmt.Errorf(`looking for layers with TOC digest %q: %w`, options.TOCDigest, err)
@@ -620,7 +621,7 @@ func (s *storageImageDestination) singleLayerIDComponent(layerIndex int, blobSum
 	defer s.lock.Unlock()
 
 	if d, found := s.indexToTOCDigest[layerIndex]; found {
-		return d.Hex() + "-toc", found
+		return d.Hex() + "-toc", found // FIXME: If this is the first layer, this is not a valid ID.
 	}
 
 	d, found := s.blobDiffIDs[blobSum]