فهرست منبع

Remove unused distribution/metadata/V1IDService

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Sebastiaan van Stijn 3 سال پیش
والد
کامیت
14fdd97b69
2فایلهای تغییر یافته به همراه0 افزوده شده و 138 حذف شده
  1. 0 51
      distribution/metadata/v1_id_service.go
  2. 0 87
      distribution/metadata/v1_id_service_test.go

+ 0 - 51
distribution/metadata/v1_id_service.go

@@ -1,51 +0,0 @@
-package metadata // import "github.com/docker/docker/distribution/metadata"
-
-import (
-	v1 "github.com/docker/docker/image/v1"
-	"github.com/docker/docker/layer"
-	"github.com/pkg/errors"
-)
-
-// V1IDService maps v1 IDs to layers on disk.
-type V1IDService struct {
-	store Store
-}
-
-// NewV1IDService creates a new V1 ID mapping service.
-func NewV1IDService(store Store) *V1IDService {
-	return &V1IDService{
-		store: store,
-	}
-}
-
-// namespace returns the namespace used by this service.
-func (idserv *V1IDService) namespace() string {
-	return "v1id"
-}
-
-// Get finds a layer by its V1 ID.
-func (idserv *V1IDService) Get(v1ID, registry string) (layer.DiffID, error) {
-	if idserv.store == nil {
-		return "", errors.New("no v1IDService storage")
-	}
-	if err := v1.ValidateID(v1ID); err != nil {
-		return layer.DiffID(""), err
-	}
-
-	idBytes, err := idserv.store.Get(idserv.namespace(), registry+","+v1ID)
-	if err != nil {
-		return layer.DiffID(""), err
-	}
-	return layer.DiffID(idBytes), nil
-}
-
-// Set associates an image with a V1 ID.
-func (idserv *V1IDService) Set(v1ID, registry string, id layer.DiffID) error {
-	if idserv.store == nil {
-		return nil
-	}
-	if err := v1.ValidateID(v1ID); err != nil {
-		return err
-	}
-	return idserv.store.Set(idserv.namespace(), registry+","+v1ID, []byte(id))
-}

+ 0 - 87
distribution/metadata/v1_id_service_test.go

@@ -1,87 +0,0 @@
-package metadata // import "github.com/docker/docker/distribution/metadata"
-
-import (
-	"os"
-	"testing"
-
-	"github.com/docker/docker/layer"
-	"gotest.tools/v3/assert"
-)
-
-func TestV1IDService(t *testing.T) {
-	tmpDir, err := os.MkdirTemp("", "v1-id-service-test")
-	if err != nil {
-		t.Fatalf("could not create temp dir: %v", err)
-	}
-	defer os.RemoveAll(tmpDir)
-
-	metadataStore, err := NewFSMetadataStore(tmpDir)
-	if err != nil {
-		t.Fatalf("could not create metadata store: %v", err)
-	}
-	v1IDService := NewV1IDService(metadataStore)
-
-	ns := v1IDService.namespace()
-
-	assert.Equal(t, "v1id", ns)
-
-	testVectors := []struct {
-		registry string
-		v1ID     string
-		layerID  layer.DiffID
-	}{
-		{
-			registry: "registry1",
-			v1ID:     "f0cd5ca10b07f35512fc2f1cbf9a6cefbdb5cba70ac6b0c9e5988f4497f71937",
-			layerID:  layer.DiffID("sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4"),
-		},
-		{
-			registry: "registry2",
-			v1ID:     "9e3447ca24cb96d86ebd5960cb34d1299b07e0a0e03801d90b9969a2c187dd6e",
-			layerID:  layer.DiffID("sha256:86e0e091d0da6bde2456dbb48306f3956bbeb2eae1b5b9a43045843f69fe4aaa"),
-		},
-		{
-			registry: "registry1",
-			v1ID:     "9e3447ca24cb96d86ebd5960cb34d1299b07e0a0e03801d90b9969a2c187dd6e",
-			layerID:  layer.DiffID("sha256:03f4658f8b782e12230c1783426bd3bacce651ce582a4ffb6fbbfa2079428ecb"),
-		},
-	}
-
-	// Set some associations
-	for _, vec := range testVectors {
-		err := v1IDService.Set(vec.v1ID, vec.registry, vec.layerID)
-		if err != nil {
-			t.Fatalf("error calling Set: %v", err)
-		}
-	}
-
-	// Check the correct values are read back
-	for _, vec := range testVectors {
-		layerID, err := v1IDService.Get(vec.v1ID, vec.registry)
-		if err != nil {
-			t.Fatalf("error calling Get: %v", err)
-		}
-		if layerID != vec.layerID {
-			t.Fatal("Get returned incorrect layer ID")
-		}
-	}
-
-	// Test Get on a nonexistent entry
-	_, err = v1IDService.Get("82379823067823853223359023576437723560923756b03560378f4497753917", "registry1")
-	if err == nil {
-		t.Fatal("expected error looking up nonexistent entry")
-	}
-
-	// Overwrite one of the entries and read it back
-	err = v1IDService.Set(testVectors[0].v1ID, testVectors[0].registry, testVectors[1].layerID)
-	if err != nil {
-		t.Fatalf("error calling Set: %v", err)
-	}
-	layerID, err := v1IDService.Get(testVectors[0].v1ID, testVectors[0].registry)
-	if err != nil {
-		t.Fatalf("error calling Get: %v", err)
-	}
-	if layerID != testVectors[1].layerID {
-		t.Fatal("Get returned incorrect layer ID")
-	}
-}