Merge pull request #4817 from rhatdan/codespell

Add codespell to validate spelling mistakes in code.

33 files changed, 45 insertions, 42 deletions

diff --git a/.cirrus.yml b/.cirrus.yml
index d069b5a3e..abf3d0dd8 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -111,7 +111,7 @@ gating_task:
     build_script:
         - '/usr/local/bin/entrypoint.sh podman |& ${TIMESTAMP}'
         - 'cd $GOSRC && ./hack/podman-commands.sh |& ${TIMESTAMP}'
-        # N/B: need 'clean' so some commited files are re-generated.
+        # N/B: need 'clean' so some committed files are re-generated.
         - '/usr/local/bin/entrypoint.sh clean podman-remote |& ${TIMESTAMP}'
         - '/usr/local/bin/entrypoint.sh clean podman BUILDTAGS="exclude_graphdriver_devicemapper selinux seccomp" |& ${TIMESTAMP}'
         - '/usr/local/bin/entrypoint.sh podman-remote-darwin |& ${TIMESTAMP}'
diff --git a/API.md b/API.md
index 283f4fc58..ca71fdf9e 100755
--- a/API.md
+++ b/API.md
@@ -1065,7 +1065,7 @@ varlink call -m unix:/run/podman/io.podman/io.podman.RemoveImage '{"name": "regi
 
 method RemoveImageWithResponse(name: [string](https://godoc.org/builtin#string), force: [bool](https://godoc.org/builtin#bool)) [RemoveImageResponse](#RemoveImageResponse)</div>
 RemoveImageWithResponse takes the name or ID of an image as well as a boolean that determines if containers using that image
-should be deleted.  If the image cannot be found, an [ImageNotFound](#ImageNotFound) error will be returned. The reponse is
+should be deleted.  If the image cannot be found, an [ImageNotFound](#ImageNotFound) error will be returned. The response is
 in the form of a RemoveImageResponse .
 ### <a name="RemovePod"></a>func RemovePod
 <div style="background-color: #E8E8E8; padding: 15px; margin: 10px; border-radius: 10px;">
diff --git a/Makefile b/Makefile
index 3530b73c9..9a4bceb34 100644
--- a/Makefile
+++ b/Makefile
@@ -351,6 +351,9 @@ install-podman-remote-%-docs: podman-remote docs $(MANPAGES)
 man-page-check:
 	hack/man-page-checker
 
+codespell:
+	codespell -S bin,vendor,.git,go.sum,changelog.txt,seccomp.json,.cirrus.yml,"*.xz,*.gz,*.tar,*.tgz,bin2img,*ico,*.png,*.1,*.5,copyimg,*.orig,apidoc.go" -L uint,iff,od,seeked
+
 # When publishing releases include critical build-time details
 .PHONY: release.txt
 release.txt:
diff --git a/cmd/podman/containers_prune.go b/cmd/podman/containers_prune.go
index 78c50268c..6a86c24fd 100644
--- a/cmd/podman/containers_prune.go
+++ b/cmd/podman/containers_prune.go
@@ -49,11 +49,11 @@ func pruneContainersCmd(c *cliconfig.PruneContainersValues) error {
 		reader := bufio.NewReader(os.Stdin)
 		fmt.Printf(`WARNING! This will remove all stopped containers.
 Are you sure you want to continue? [y/N] `)
-		ans, err := reader.ReadString('\n')
+		answer, err := reader.ReadString('\n')
 		if err != nil {
 			return errors.Wrapf(err, "error reading input")
 		}
-		if strings.ToLower(ans)[0] != 'y' {
+		if strings.ToLower(answer)[0] != 'y' {
 			return nil
 		}
 	}
diff --git a/cmd/podman/errors_remote.go b/cmd/podman/errors_remote.go
index 19df2d2d8..378f9398f 100644
--- a/cmd/podman/errors_remote.go
+++ b/cmd/podman/errors_remote.go
@@ -25,7 +25,7 @@ func outputError(err error) {
 		}
 		var ne error
 		switch e := err.(type) {
-		// For some reason golang wont let me list them with commas so listing them all.
+		// For some reason golang won't let me list them with commas so listing them all.
 		case *iopodman.ImageNotFound:
 			ne = errors.New(e.Reason)
 		case *iopodman.ContainerNotFound:
@@ -48,7 +48,7 @@ func outputError(err error) {
 func setExitCode(err error) int {
 	cause := errors.Cause(err)
 	switch e := cause.(type) {
-	// For some reason golang wont let me list them with commas so listing them all.
+	// For some reason golang won't let me list them with commas so listing them all.
 	case *iopodman.ContainerNotFound:
 		return 1
 	case *iopodman.InvalidState:
diff --git a/cmd/podman/images_prune.go b/cmd/podman/images_prune.go
index 2b498f83d..8f187cbd7 100644
--- a/cmd/podman/images_prune.go
+++ b/cmd/podman/images_prune.go
@@ -47,11 +47,11 @@ func pruneImagesCmd(c *cliconfig.PruneImagesValues) error {
 		fmt.Printf(`
 WARNING! This will remove all dangling images.
 Are you sure you want to continue? [y/N] `)
-		ans, err := reader.ReadString('\n')
+		answer, err := reader.ReadString('\n')
 		if err != nil {
 			return errors.Wrapf(err, "error reading input")
 		}
-		if strings.ToLower(ans)[0] != 'y' {
+		if strings.ToLower(answer)[0] != 'y' {
 			return nil
 		}
 	}
diff --git a/cmd/podman/reset.go b/cmd/podman/reset.go
index 9d16dc978..203399047 100644
--- a/cmd/podman/reset.go
+++ b/cmd/podman/reset.go
@@ -52,11 +52,11 @@ WARNING! This will remove:
         - all images
         - all build cache
 Are you sure you want to continue? [y/N] `)
-		ans, err := reader.ReadString('\n')
+		answer, err := reader.ReadString('\n')
 		if err != nil {
 			return errors.Wrapf(err, "error reading input")
 		}
-		if strings.ToLower(ans)[0] != 'y' {
+		if strings.ToLower(answer)[0] != 'y' {
 			return nil
 		}
 	}
diff --git a/cmd/podman/shared/intermediate.go b/cmd/podman/shared/intermediate.go
index bc12bd2a5..e985e4dc0 100644
--- a/cmd/podman/shared/intermediate.go
+++ b/cmd/podman/shared/intermediate.go
@@ -8,7 +8,7 @@ import (
 /*
 attention
 
-in this file you will see alot of struct duplication.  this was done because people wanted a strongly typed
+in this file you will see a lot of struct duplication.  this was done because people wanted a strongly typed
 varlink mechanism.  this resulted in us creating this intermediate layer that allows us to take the input
 from the cli and make an intermediate layer which can be transferred as strongly typed structures over a varlink
 interface.
diff --git a/cmd/podman/shared/intermediate_novarlink.go b/cmd/podman/shared/intermediate_novarlink.go
index 26738ce48..c6f011fe0 100644
--- a/cmd/podman/shared/intermediate_novarlink.go
+++ b/cmd/podman/shared/intermediate_novarlink.go
@@ -6,7 +6,7 @@ package shared
 /*
 attention
 
-in this file you will see alot of struct duplication.  this was done because people wanted a strongly typed
+in this file you will see a lot of struct duplication.  this was done because people wanted a strongly typed
 varlink mechanism.  this resulted in us creating this intermediate layer that allows us to take the input
 from the cli and make an intermediate layer which can be transferred as strongly typed structures over a varlink
 interface.
diff --git a/cmd/podman/system_prune.go b/cmd/podman/system_prune.go
index 74fdcde99..bbca7b881 100644
--- a/cmd/podman/system_prune.go
+++ b/cmd/podman/system_prune.go
@@ -63,11 +63,11 @@ WARNING! This will remove:
         - all dangling images
         - all build cache
 Are you sure you want to continue? [y/N] `, volumeString)
-		ans, err := reader.ReadString('\n')
+		answer, err := reader.ReadString('\n')
 		if err != nil {
 			return errors.Wrapf(err, "error reading input")
 		}
-		if strings.ToLower(ans)[0] != 'y' {
+		if strings.ToLower(answer)[0] != 'y' {
 			return nil
 		}
 	}
diff --git a/cmd/podman/varlink/io.podman.varlink b/cmd/podman/varlink/io.podman.varlink
index ac400a467..b993457ca 100644
--- a/cmd/podman/varlink/io.podman.varlink
+++ b/cmd/podman/varlink/io.podman.varlink
@@ -885,7 +885,7 @@ method UntagImage(name: string, tag: string) -> (image: string)
 method RemoveImage(name: string, force: bool) -> (image: string)
 
 # RemoveImageWithResponse takes the name or ID of an image as well as a boolean that determines if containers using that image
-# should be deleted.  If the image cannot be found, an [ImageNotFound](#ImageNotFound) error will be returned. The reponse is
+# should be deleted.  If the image cannot be found, an [ImageNotFound](#ImageNotFound) error will be returned. The response is
 # in the form of a RemoveImageResponse .
 method RemoveImageWithResponse(name: string, force: bool) -> (response: RemoveImageResponse)
 
diff --git a/cmd/podman/volume_prune.go b/cmd/podman/volume_prune.go
index daea5a4d2..48ed68509 100644
--- a/cmd/podman/volume_prune.go
+++ b/cmd/podman/volume_prune.go
@@ -74,11 +74,11 @@ func volumePruneCmd(c *cliconfig.VolumePruneValues) error {
 		reader := bufio.NewReader(os.Stdin)
 		fmt.Println("WARNING! This will remove all volumes not used by at least one container.")
 		fmt.Print("Are you sure you want to continue? [y/N] ")
-		ans, err := reader.ReadString('\n')
+		answer, err := reader.ReadString('\n')
 		if err != nil {
 			return errors.Wrapf(err, "error reading input")
 		}
-		if strings.ToLower(ans)[0] != 'y' {
+		if strings.ToLower(answer)[0] != 'y' {
 			return nil
 		}
 	}
diff --git a/completions/bash/podman b/completions/bash/podman
index c23d156bc..ca3618b0b 100644
--- a/completions/bash/podman
+++ b/completions/bash/podman
@@ -366,7 +366,7 @@ __podman_to_extglob() {
 # continue processing its completion.
 #
 # TODO if the preceding command has options that accept arguments and an
-# argument is equal ot one of the subcommands, this is falsely detected as
+# argument is equal to one of the subcommands, this is falsely detected as
 # a match.
 __podman_subcommands() {
 	local subcommands="$1"
diff --git a/contrib/cirrus/README.md b/contrib/cirrus/README.md
index de9a33714..49f713a8f 100644
--- a/contrib/cirrus/README.md
+++ b/contrib/cirrus/README.md
@@ -196,7 +196,7 @@ as the standard 'cloud-init' services.
    in the ``test_build_vm_images`` Task (above).
 
 *  Base images do not need to be produced often, but doing so completely
-   manually would be time-consuming and error-prone.  Therefor a special
+   manually would be time-consuming and error-prone.  Therefore a special
    semi-automatic *Makefile* target is provided to assist with producing
    all the base-images: ``libpod_base_images``
 
diff --git a/contrib/spec/podman.spec.in b/contrib/spec/podman.spec.in
index 25c70c392..31e94fa80 100644
--- a/contrib/spec/podman.spec.in
+++ b/contrib/spec/podman.spec.in
@@ -19,7 +19,7 @@
 %endif
 
 # %if ! 0% {?gobuild:1}
-%define gobuild(o:) go build -tags="$BUILDTAGS" -ldflags "${LDFLAGS:-} -B 0x$(head -c20 /dev/urandom|od -An -tx1|tr -d ' \\n')" -a -v -x %{?**};
+%define gobuild(o:) go build -tags="$BUILDTAGS" -ldflags "${LDFLAGS:-} -B 0x$(head -c20 /dev/urandom|of -An -tx1|tr -d ' \\n')" -a -v -x %{?**};
 #% endif
 
 # libpod hack directory
diff --git a/dependencies/analyses/README.md b/dependencies/analyses/README.md
index 67dab6f75..734058045 100644
--- a/dependencies/analyses/README.md
+++ b/dependencies/analyses/README.md
@@ -1,7 +1,7 @@
 # A set of scripts and instructions that help to analyze and debloat go-lang dependencies
 
 Note that all scripts mentioned below follow the [KISS principle](https://en.wikipedia.org/wiki/KISS_principle) on purpose.
-The scripts are meant to be used in combination to aid in understanding the packages' dependencies and how they contribute to the size of the compiled binary.
+The scripts are meant to be used in combination to aid in understanding the package's dependencies and how they contribute to the size of the compiled binary.
 
 ## Size of packages
 
diff --git a/docs/source/markdown/podman-create.1.md b/docs/source/markdown/podman-create.1.md
index fdc2edf39..abde899bf 100644
--- a/docs/source/markdown/podman-create.1.md
+++ b/docs/source/markdown/podman-create.1.md
@@ -799,7 +799,7 @@ You can pass `host` to copy the current configuration from the host.
 
 Sets the username or UID used and optionally the groupname or GID for the specified command.
 
-The followings examples are all valid:
+The following examples are all valid:
 --user [user | user:group | uid | uid:gid | user:gid | uid:group ]
 
 Without this argument the command will be run as root in the container.
diff --git a/docs/source/markdown/podman-run.1.md b/docs/source/markdown/podman-run.1.md
index d9fdd9650..9484ad347 100644
--- a/docs/source/markdown/podman-run.1.md
+++ b/docs/source/markdown/podman-run.1.md
@@ -835,7 +835,7 @@ You can pass `host` to copy the current configuration from the host.
 
 Sets the username or UID used and optionally the groupname or GID for the specified command.
 
-The followings examples are all valid:
+The following examples are all valid:
 --user [user | user:group | uid | uid:gid | user:gid | uid:group ]
 
 Without this argument the command will be run as root in the container.
diff --git a/libpod/image/image.go b/libpod/image/image.go
index 84da21ce5..503282156 100644
--- a/libpod/image/image.go
+++ b/libpod/image/image.go
@@ -1528,7 +1528,7 @@ func GetLayersMapWithImageInfo(imageruntime *Runtime) (map[string]*LayerInfo, er
 		}
 	}
 
-	// scan all layers & add all childs for each layers to layerInfo
+	// scan all layers & add all childid's for each layers to layerInfo
 	for _, layer := range layers {
 		_, ok := layerInfoMap[layer.ID]
 		if ok {
diff --git a/libpod/kube.go b/libpod/kube.go
index 6ae3e3d07..e0f79ac11 100644
--- a/libpod/kube.go
+++ b/libpod/kube.go
@@ -310,13 +310,13 @@ func ocicniPortMappingToContainerPort(portMappings []ocicni.PortMapping) ([]v1.C
 func libpodEnvVarsToKubeEnvVars(envs []string) ([]v1.EnvVar, error) {
 	var envVars []v1.EnvVar
 	for _, e := range envs {
-		splitE := strings.SplitN(e, "=", 2)
-		if len(splitE) != 2 {
+		split := strings.SplitN(e, "=", 2)
+		if len(split) != 2 {
 			return envVars, errors.Errorf("environment variable %s is malformed; should be key=value", e)
 		}
 		ev := v1.EnvVar{
-			Name:  splitE[0],
-			Value: splitE[1],
+			Name:  split[0],
+			Value: split[1],
 		}
 		envVars = append(envVars, ev)
 	}
diff --git a/libpod/networking_linux.go b/libpod/networking_linux.go
index 89dac2b5d..5db749a4d 100644
--- a/libpod/networking_linux.go
+++ b/libpod/networking_linux.go
@@ -255,7 +255,7 @@ func (r *Runtime) setupRootlessNetNS(ctr *Container) (err error) {
 	}
 	defer func() {
 		if err := cmd.Process.Release(); err != nil {
-			logrus.Errorf("unable to release comman process: %q", err)
+			logrus.Errorf("unable to release command process: %q", err)
 		}
 	}()
 
diff --git a/pkg/adapter/containers.go b/pkg/adapter/containers.go
index fdd9f6ab3..4228c3dca 100644
--- a/pkg/adapter/containers.go
+++ b/pkg/adapter/containers.go
@@ -375,7 +375,7 @@ func (r *LocalRuntime) selectDetachKeys(flagValue string) (string, error) {
 
 	config, err := r.GetConfig()
 	if err != nil {
-		return "", errors.Wrapf(err, "unable to retrive runtime config")
+		return "", errors.Wrapf(err, "unable to retrieve runtime config")
 	}
 	if config.DetachKeys != "" {
 		return config.DetachKeys, nil
diff --git a/pkg/adapter/runtime.go b/pkg/adapter/runtime.go
index 5f880e807..121f85755 100644
--- a/pkg/adapter/runtime.go
+++ b/pkg/adapter/runtime.go
@@ -59,7 +59,7 @@ type Volume struct {
 // VolumeFilter is for filtering volumes on the client
 type VolumeFilter func(*Volume) bool
 
-// GetRuntimeNoStore returns a localruntime struct wit an embedded runtime but
+// GetRuntimeNoStore returns a localruntime struct with an embedded runtime but
 // without a configured storage.
 func GetRuntimeNoStore(ctx context.Context, c *cliconfig.PodmanCommand) (*LocalRuntime, error) {
 	runtime, err := libpodruntime.GetRuntimeNoStore(ctx, c)
diff --git a/pkg/hooks/docs/oci-hooks.5.md b/pkg/hooks/docs/oci-hooks.5.md
index b50a6bddc..7d13ffa82 100644
--- a/pkg/hooks/docs/oci-hooks.5.md
+++ b/pkg/hooks/docs/oci-hooks.5.md
@@ -25,7 +25,7 @@ Tools consuming this format may also opt to monitor the hook directories for cha
 
 Hooks are injected in the order obtained by sorting the JSON file names, after converting them to lower case, based on their Unicode code points.
 For example, a matching hook defined in `01-my-hook.json` would be injected before matching hooks defined in `02-another-hook.json` and `01-UPPERCASE.json`.
-It is strongly recommended to make the sort oder unambiguous depending on an ASCII-only prefix (like the `01`/`02` above).
+It is strongly recommended to make the sort order unambiguous depending on an ASCII-only prefix (like the `01`/`02` above).
 
 Each JSON file should contain an object with one of the following schemas.
 
diff --git a/test/e2e/healthcheck_run_test.go b/test/e2e/healthcheck_run_test.go
index 4acea06eb..7633261e3 100644
--- a/test/e2e/healthcheck_run_test.go
+++ b/test/e2e/healthcheck_run_test.go
@@ -42,7 +42,7 @@ var _ = Describe("Podman healthcheck run", func() {
 	})
 
 	It("podman healthcheck on valid container", func() {
-		Skip("Extremely consistent flake - reenable on debugging")
+		Skip("Extremely consistent flake - re-enable on debugging")
 		session := podmanTest.Podman([]string{"run", "-dt", "--name", "hc", healthcheck})
 		session.WaitWithDefaultTimeout()
 		Expect(session.ExitCode()).To(Equal(0))
diff --git a/test/e2e/mount_test.go b/test/e2e/mount_test.go
index dda83ba31..ac52d8c7e 100644
--- a/test/e2e/mount_test.go
+++ b/test/e2e/mount_test.go
@@ -205,7 +205,7 @@ var _ = Describe("Podman mount", func() {
 		Expect(lmount.OutputToString()).To(Equal(""))
 	})
 
-	It("podman list mulitple mounted containers", func() {
+	It("podman list multiple mounted containers", func() {
 		SkipIfRootless()
 
 		setup := podmanTest.Podman([]string{"create", ALPINE, "ls"})
diff --git a/test/e2e/run_signal_test.go b/test/e2e/run_signal_test.go
index 1d57e6211..eee7c14fb 100644
--- a/test/e2e/run_signal_test.go
+++ b/test/e2e/run_signal_test.go
@@ -47,7 +47,7 @@ var _ = Describe("Podman run with --sig-proxy", func() {
 
 	Specify("signals are forwarded to container using sig-proxy", func() {
 		if podmanTest.Host.Arch == "ppc64le" {
-			Skip("Doesnt work on ppc64le")
+			Skip("Doesn't work on ppc64le")
 		}
 		signal := syscall.SIGFPE
 		// Set up a socket for communication
diff --git a/test/e2e/save_test.go b/test/e2e/save_test.go
index 52dab923b..60825f975 100644
--- a/test/e2e/save_test.go
+++ b/test/e2e/save_test.go
@@ -51,7 +51,7 @@ var _ = Describe("Podman save", func() {
 	})
 
 	It("podman save with stdout", func() {
-		Skip("Pipe redirection in ginkgo probably wont work")
+		Skip("Pipe redirection in ginkgo probably won't work")
 		outfile := filepath.Join(podmanTest.TempDir, "alpine.tar")
 
 		save := podmanTest.PodmanNoCache([]string{"save", ALPINE, ">", outfile})
diff --git a/test/endpoint/commit.go b/test/endpoint/commit.go
index 476ac6ca3..ab9af819f 100644
--- a/test/endpoint/commit.go
+++ b/test/endpoint/commit.go
@@ -40,7 +40,7 @@ var _ = Describe("Podman commit", func() {
 		// run the container to be committed
 		_ = endpointTest.startTopContainer("top")
 		result := endpointTest.Varlink("Commit", string(b), false)
-		// This indicates an error occured
+		// This indicates an error occurred
 		Expect(len(result.StdErrToString())).To(BeNumerically(">", 0))
 	})
 
diff --git a/test/system/TODO.md b/test/system/TODO.md
index f6110d2e9..f0d311626 100644
--- a/test/system/TODO.md
+++ b/test/system/TODO.md
@@ -70,7 +70,7 @@ have been omitted as they are verified by repeated implied use.
 - [ ] Container runlabel, exists, checkpoint, exists, restore, stop, prune
 
   - Using pre-existing remote image, start it with 'podman container runlabel --pull'
-  - Run a named container that exits immediatly
+  - Run a named container that exits immediately
   - Confirm 'container exists' zero exit (both containers)
   - Checkpoint the running container
   - Confirm 'container exists' non-zero exit (runlabel container)
diff --git a/test/test_podman_pods.sh b/test/test_podman_pods.sh
index daa8acaee..f2f47f510 100755
--- a/test/test_podman_pods.sh
+++ b/test/test_podman_pods.sh
@@ -39,13 +39,13 @@ fi
 
 
 ########
-# Create a named and unamed pod
+# Create a named and unnamed pod
 ########
 podman pod create --name foobar
 podid=$(podman pod create)
 
 ########
-# Delete a named and unamed pod
+# Delete a named and unnamed pod
 ########
 podman pod rm foobar
 podman pod rm $podid
diff --git a/test/utils/common_function_test.go b/test/utils/common_function_test.go
index 98cb43188..46cce1076 100644
--- a/test/utils/common_function_test.go
+++ b/test/utils/common_function_test.go
@@ -115,7 +115,7 @@ var _ = Describe("Common functions test", func() {
 		bytes, _ := ioutil.ReadAll(read)
 		json.Unmarshal(bytes, compareData)
 
-		Expect(reflect.DeepEqual(testData, compareData)).To(BeTrue(), "Data chaned after we store it to file.")
+		Expect(reflect.DeepEqual(testData, compareData)).To(BeTrue(), "Data changed after we store it to file.")
 	})
 
 	DescribeTable("Test Containerized",
diff --git a/troubleshooting.md b/troubleshooting.md
index d122983d7..0f9440799 100644
--- a/troubleshooting.md
+++ b/troubleshooting.md
@@ -431,7 +431,7 @@ or as root if your user has not enough privileges.
 
 ### 18) `podman run` fails with "bpf create: permission denied error"
 
-The Kernel Lockdown patches deny eBPF programs when Secure Boot is enabled in the BIOS. [Matthew Garrett's post](https://mjg59.dreamwidth.org/50577.html) desribes the relationship between Lockdown and Secure Boot and [Jan-Philip Gehrcke's](https://gehrcke.de/2019/09/running-an-ebpf-program-may-require-lifting-the-kernel-lockdown/) connects this with eBPF. [RH bug 1768125](https://bugzilla.redhat.com/show_bug.cgi?id=1768125) contains some additional details.
+The Kernel Lockdown patches deny eBPF programs when Secure Boot is enabled in the BIOS. [Matthew Garrett's post](https://mjg59.dreamwidth.org/50577.html) describes the relationship between Lockdown and Secure Boot and [Jan-Philip Gehrcke's](https://gehrcke.de/2019/09/running-an-ebpf-program-may-require-lifting-the-kernel-lockdown/) connects this with eBPF. [RH bug 1768125](https://bugzilla.redhat.com/show_bug.cgi?id=1768125) contains some additional details.
 
 #### Symptom