mirror of
https://github.com/containers/podman.git
synced 2025-10-30 01:17:00 +08:00
There exists a unit test to ensure that shell completion functions are defined. However there was no check about the quality of the provided shell completions. Lets change that. The idea is to create a general test that makes sure we are suggesting containers,pods,images... for the correct commands. This works by reading the command use line and checking for each arg if we provide the correct suggestions for this arg. It includes the following tests: - flag suggestions if [options] is set - container, pod, image, network, volume, registry completion - path completion for the appropriate arg KEYWORDS (`PATH`,`CONTEXT`,etc.) - no completion if there are no args - completion for more than one arg if it ends with `...]` The test does not cover completion values for flags and not every arg KEYWORD is supported. This is still a huge improvement and covers most use cases. This test spotted several inconsistencies between the completion and the command use line. All of them have been adjusted to make the test pass. The biggest advantage is that the completions always match the latest command changes. So if someone changes the arguments for a command this ensures that the completions must be adjusted. Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
144 lines
4.6 KiB
Go
144 lines
4.6 KiB
Go
package images
|
|
|
|
import (
|
|
"context"
|
|
"os"
|
|
"strings"
|
|
|
|
"github.com/containers/common/pkg/completion"
|
|
"github.com/containers/podman/v2/cmd/podman/common"
|
|
"github.com/containers/podman/v2/cmd/podman/parse"
|
|
"github.com/containers/podman/v2/cmd/podman/registry"
|
|
"github.com/containers/podman/v2/libpod/define"
|
|
"github.com/containers/podman/v2/pkg/domain/entities"
|
|
"github.com/containers/podman/v2/pkg/util"
|
|
"github.com/pkg/errors"
|
|
"github.com/spf13/cobra"
|
|
"golang.org/x/crypto/ssh/terminal"
|
|
)
|
|
|
|
var (
|
|
validFormats = []string{define.OCIManifestDir, define.OCIArchive, define.V2s2ManifestDir, define.V2s2Archive}
|
|
containerConfig = registry.PodmanConfig()
|
|
)
|
|
|
|
var (
|
|
saveDescription = `Save an image to docker-archive or oci-archive on the local machine. Default is docker-archive.`
|
|
|
|
saveCommand = &cobra.Command{
|
|
Use: "save [options] IMAGE [IMAGE...]",
|
|
Short: "Save image(s) to an archive",
|
|
Long: saveDescription,
|
|
RunE: save,
|
|
Args: func(cmd *cobra.Command, args []string) error {
|
|
if len(args) == 0 {
|
|
return errors.Errorf("need at least 1 argument")
|
|
}
|
|
format, err := cmd.Flags().GetString("format")
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if !util.StringInSlice(format, validFormats) {
|
|
return errors.Errorf("format value must be one of %s", strings.Join(validFormats, " "))
|
|
}
|
|
return nil
|
|
},
|
|
ValidArgsFunction: common.AutocompleteImages,
|
|
Example: `podman save --quiet -o myimage.tar imageID
|
|
podman save --format docker-dir -o ubuntu-dir ubuntu
|
|
podman save > alpine-all.tar alpine:latest`,
|
|
}
|
|
imageSaveCommand = &cobra.Command{
|
|
Args: saveCommand.Args,
|
|
Use: saveCommand.Use,
|
|
Short: saveCommand.Short,
|
|
Long: saveCommand.Long,
|
|
RunE: saveCommand.RunE,
|
|
ValidArgsFunction: saveCommand.ValidArgsFunction,
|
|
Example: `podman image save --quiet -o myimage.tar imageID
|
|
podman image save --format docker-dir -o ubuntu-dir ubuntu
|
|
podman image save > alpine-all.tar alpine:latest`,
|
|
}
|
|
)
|
|
|
|
var (
|
|
saveOpts entities.ImageSaveOptions
|
|
)
|
|
|
|
func init() {
|
|
registry.Commands = append(registry.Commands, registry.CliCommand{
|
|
Mode: []entities.EngineMode{entities.ABIMode, entities.TunnelMode},
|
|
Command: saveCommand,
|
|
})
|
|
saveFlags(saveCommand)
|
|
|
|
registry.Commands = append(registry.Commands, registry.CliCommand{
|
|
Mode: []entities.EngineMode{entities.ABIMode, entities.TunnelMode},
|
|
Command: imageSaveCommand,
|
|
Parent: imageCmd,
|
|
})
|
|
saveFlags(imageSaveCommand)
|
|
}
|
|
|
|
func saveFlags(cmd *cobra.Command) {
|
|
flags := cmd.Flags()
|
|
|
|
flags.BoolVar(&saveOpts.Compress, "compress", false, "Compress tarball image layers when saving to a directory using the 'dir' transport. (default is same compression type as source)")
|
|
|
|
formatFlagName := "format"
|
|
flags.StringVar(&saveOpts.Format, formatFlagName, define.V2s2Archive, "Save image to oci-archive, oci-dir (directory with oci manifest type), docker-archive, docker-dir (directory with v2s2 manifest type)")
|
|
_ = cmd.RegisterFlagCompletionFunc(formatFlagName, common.AutocompleteImageSaveFormat)
|
|
|
|
outputFlagName := "output"
|
|
flags.StringVarP(&saveOpts.Output, outputFlagName, "o", "", "Write to a specified file (default: stdout, which must be redirected)")
|
|
_ = cmd.RegisterFlagCompletionFunc(outputFlagName, completion.AutocompleteDefault)
|
|
|
|
flags.BoolVarP(&saveOpts.Quiet, "quiet", "q", false, "Suppress the output")
|
|
flags.BoolVarP(&saveOpts.MultiImageArchive, "multi-image-archive", "m", containerConfig.Engine.MultiImageArchive, "Interpret additional arguments as images not tags and create a multi-image-archive (only for docker-archive)")
|
|
}
|
|
|
|
func save(cmd *cobra.Command, args []string) (finalErr error) {
|
|
var (
|
|
tags []string
|
|
succeeded = false
|
|
)
|
|
if cmd.Flag("compress").Changed && (saveOpts.Format != define.OCIManifestDir && saveOpts.Format != define.V2s2ManifestDir && saveOpts.Format == "") {
|
|
return errors.Errorf("--compress can only be set when --format is either 'oci-dir' or 'docker-dir'")
|
|
}
|
|
if len(saveOpts.Output) == 0 {
|
|
saveOpts.Quiet = true
|
|
fi := os.Stdout
|
|
if terminal.IsTerminal(int(fi.Fd())) {
|
|
return errors.Errorf("refusing to save to terminal. Use -o flag or redirect")
|
|
}
|
|
pipePath, cleanup, err := setupPipe()
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if cleanup != nil {
|
|
defer func() {
|
|
errc := cleanup()
|
|
if succeeded {
|
|
writeErr := <-errc
|
|
if writeErr != nil && finalErr == nil {
|
|
finalErr = writeErr
|
|
}
|
|
}
|
|
}()
|
|
}
|
|
saveOpts.Output = pipePath
|
|
}
|
|
if err := parse.ValidateFileName(saveOpts.Output); err != nil {
|
|
return err
|
|
}
|
|
if len(args) > 1 {
|
|
tags = args[1:]
|
|
}
|
|
|
|
err := registry.ImageEngine().Save(context.Background(), args[0], tags, saveOpts)
|
|
if err == nil {
|
|
succeeded = true
|
|
}
|
|
return err
|
|
}
|