3b5dff2783
The CLI.ManifestStore method is a shallow wrapper around manifeststore.NewStore and has no dependency on the CLI itself. However, due to its signature resulted in various dependencies becoming a dependency of the "command" package. Consequence of this was that cli-plugins, which need the cli/command package, would also get those dependencies. - This patch inlines the code to produce the store, skipping the wrapper. - Define a local interface for some tests where a dummy store was used. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
76 lines
2.0 KiB
Go
76 lines
2.0 KiB
Go
package manifest
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
|
|
"github.com/docker/cli/cli"
|
|
"github.com/docker/cli/cli/command"
|
|
"github.com/docker/cli/cli/manifest/store"
|
|
"github.com/pkg/errors"
|
|
"github.com/spf13/cobra"
|
|
)
|
|
|
|
type createOpts struct {
|
|
amend bool
|
|
insecure bool
|
|
}
|
|
|
|
func newCreateListCommand(dockerCli command.Cli) *cobra.Command {
|
|
opts := createOpts{}
|
|
|
|
cmd := &cobra.Command{
|
|
Use: "create MANIFEST_LIST MANIFEST [MANIFEST...]",
|
|
Short: "Create a local manifest list for annotating and pushing to a registry",
|
|
Args: cli.RequiresMinArgs(2),
|
|
RunE: func(cmd *cobra.Command, args []string) error {
|
|
return createManifestList(cmd.Context(), dockerCli, args, opts)
|
|
},
|
|
}
|
|
|
|
flags := cmd.Flags()
|
|
flags.BoolVar(&opts.insecure, "insecure", false, "Allow communication with an insecure registry")
|
|
flags.BoolVarP(&opts.amend, "amend", "a", false, "Amend an existing manifest list")
|
|
return cmd
|
|
}
|
|
|
|
func createManifestList(ctx context.Context, dockerCLI command.Cli, args []string, opts createOpts) error {
|
|
newRef := args[0]
|
|
targetRef, err := normalizeReference(newRef)
|
|
if err != nil {
|
|
return errors.Wrapf(err, "error parsing name for manifest list %s", newRef)
|
|
}
|
|
|
|
manifestStore := newManifestStore(dockerCLI)
|
|
_, err = manifestStore.GetList(targetRef)
|
|
switch {
|
|
case store.IsNotFound(err):
|
|
// New manifest list
|
|
case err != nil:
|
|
return err
|
|
case !opts.amend:
|
|
return errors.Errorf("refusing to amend an existing manifest list with no --amend flag")
|
|
}
|
|
|
|
// Now create the local manifest list transaction by looking up the manifest schemas
|
|
// for the constituent images:
|
|
manifests := args[1:]
|
|
for _, manifestRef := range manifests {
|
|
namedRef, err := normalizeReference(manifestRef)
|
|
if err != nil {
|
|
// TODO: wrap error?
|
|
return err
|
|
}
|
|
|
|
manifest, err := getManifest(ctx, dockerCLI, targetRef, namedRef, opts.insecure)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if err := manifestStore.Save(targetRef, namedRef, manifest); err != nil {
|
|
return err
|
|
}
|
|
}
|
|
_, _ = fmt.Fprintln(dockerCLI.Out(), "Created manifest list", targetRef.String())
|
|
return nil
|
|
}
|