(chore): Resolve lint errors

Signed-off-by: schristoff <28318173+schristoff@users.noreply.github.com>

mage/docs/docs.go

@@ -37,11 +37,13 @@ func Docs() {
 	if baseURL != "" {
 		cmd.Args("-b", baseURL)
 	}
-	cmd.RunV()
+	err := cmd.RunV()
+	mgx.Must(err)
 }
 
 func removePreviewContainer() {
-	docker.RemoveContainer(PreviewContainer)
+	err := docker.RemoveContainer(PreviewContainer)
+	mgx.Must(err)
 }
 
 // Preview the website locally using a Docker container.
@@ -59,12 +61,13 @@ func DocsPreview() {
 	}
 	setDockerUser := fmt.Sprintf("--user=%s:%s", currentUser.Uid, currentUser.Gid)
 	pwd, _ := os.Getwd()
-	must.Run("docker", "run", "-d", "-v", pwd+":/src",
+	err = must.Run("docker", "run", "-d", "-v", pwd+":/src",
 		"-v", operatorDocs+":/src/docs/content/operator",
 		setDockerUser,
 		"-p", "1313:1313", "--name", PreviewContainer, "-w", "/src/docs",
 		"klakegg/hugo:0.78.1-ext-alpine", "server", "-D", "-F", "--noHTTPCache",
 		"--watch", "--bind=0.0.0.0")
+	mgx.Must(err)
 
 	for {
 		output, _ := must.OutputS("docker", "logs", "porter-docs")
@@ -74,7 +77,8 @@ func DocsPreview() {
 		time.Sleep(time.Second)
 	}
 
-	must.Run("open", "http://localhost:1313/docs/")
+	err = must.Run("open", "http://localhost:1313/docs/")
+	mgx.Must(err)
 }
 
 // Build a branch preview of the website.

pkg/build/dockerfile-generator_test.go

@@ -63,8 +63,8 @@ func TestPorter_buildCustomDockerfile(t *testing.T) {
 COPY mybin /cnab/app/
 
 `
-		c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
-
+		err = c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+		require.NoError(t, err)
 		mp := mixin.NewTestMixinProvider()
 		g := NewDockerfileGenerator(c.Config, m, tmpl, mp)
 		gotlines, err := g.buildDockerfile(context.Background())
@@ -94,8 +94,8 @@ COPY mybin /cnab/app/
 COPY mybin /cnab/app/
 
 `
-		c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
-
+		err = c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+		require.NoError(t, err)
 		mp := mixin.NewTestMixinProvider()
 		g := NewDockerfileGenerator(c.Config, m, tmpl, mp)
 		gotlines, err := g.buildDockerfile(context.Background())
@@ -123,7 +123,8 @@ COPY mybin /cnab/app/
 COPY mybin /cnab/app/
 
 `
-		c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+		err = c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+		require.NoError(t, err)
 
 		mp := mixin.NewTestMixinProvider()
 		g := NewDockerfileGenerator(c.Config, m, tmpl, mp)
@@ -225,7 +226,8 @@ func TestPorter_appendBuildInstructionsIfMixinTokenIsNotPresent(t *testing.T) {
 ARG BUNDLE_DIR
 COPY mybin /cnab/app/
 `
-	c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+	err = c.TestContext.AddTestFileContents([]byte(customFrom), "Dockerfile.template")
+	require.NoError(t, err)
 
 	mp := mixin.NewTestMixinProvider()
 	g := NewDockerfileGenerator(c.Config, m, tmpl, mp)

pkg/cache/cache_test.go

@@ -266,12 +266,14 @@ func TestCache_StoreBundle_Overwrite(t *testing.T) {
 	cb := CachedBundle{BundleReference: cnab.BundleReference{Reference: kahn1dot01}}
 	cb.SetCacheDir(cacheDir)
 	cfg.FileSystem.Create(cb.BuildManifestPath())
-	cfg.FileSystem.Create(cb.BuildRelocationFilePath())
+	_, err := cfg.FileSystem.Create(cb.BuildRelocationFilePath())
+	require.NoError(t, err)
 	junkPath := filepath.Join(cb.cacheDir, "junk.txt")
-	cfg.FileSystem.Create(junkPath)
+	_, err = cfg.FileSystem.Create(junkPath)
+	require.NoError(t, err)
 
 	// Refresh the cache
-	cb, err := c.StoreBundle(cb.BundleReference)
+	cb, err = c.StoreBundle(cb.BundleReference)
 	require.NoError(t, err, "StoreBundle failed")
 
 	exists, _ := cfg.FileSystem.Exists(cb.BuildBundlePath())

pkg/cnab/config-adapter/stamp.go

@@ -135,7 +135,8 @@ func (c *ManifestConverter) GenerateStamp(ctx context.Context) (Stamp, error) {
 	if err != nil {
 		// The digest is only used to decide if we need to rebuild, it is not an error condition to not
 		// have a digest.
-		log.Error(fmt.Errorf("WARNING: Could not digest the porter manifest file: %w", err))
+		log.Warn(fmt.Sprint("WARNING: Could not digest the porter manifest file: %w", err))
+
 		stamp.ManifestDigest = "unknown"
 	} else {
 		stamp.ManifestDigest = digest

pkg/config/config.go

@@ -242,7 +242,7 @@ func (c *Config) GetPorterPath(ctx context.Context) (string, error) {
 	// We try to resolve back to the original location
 	hardPath, err := evalSymlinks(porterPath)
 	if err != nil { // if we have trouble resolving symlinks, skip trying to help people who used symlinks
-		log.Error(fmt.Errorf("WARNING could not resolve %s for symbolic links: %w", porterPath, err))
+		return "", log.Error(fmt.Errorf("WARNING could not resolve %s for symbolic links: %w", porterPath, err))
 	} else if hardPath != porterPath {
 		log.Debugf("Resolved porter binary from %s to %s", porterPath, hardPath)
 		porterPath = hardPath

pkg/config/helpers.go

@@ -2,6 +2,7 @@ package config
 
 import (
 	"context"
+	"log"
 	"os"
 	"path/filepath"
 	"strconv"
@@ -42,14 +43,27 @@ func (c *TestConfig) SetupUnitTest() {
 	c.SetHomeDir(home)
 
 	// Fake out the porter home directory
-	c.FileSystem.Create(filepath.Join(home, "porter"))
-	c.FileSystem.Create(filepath.Join(home, "runtimes", "porter-runtime"))
+	var err error
+	if _, err = c.FileSystem.Create(filepath.Join(home, "porter")); err != nil {
+		log.Fatal(err)
+	}
+	if _, err = c.FileSystem.Create(filepath.Join(home, "runtimes", "porter-runtime")); err != nil {
+		log.Fatal(err)
+	}
 
 	mixinsDir := filepath.Join(home, "mixins")
-	c.FileSystem.Create(filepath.Join(mixinsDir, "exec/exec"))
-	c.FileSystem.Create(filepath.Join(mixinsDir, "exec/runtimes/exec-runtime"))
-	c.FileSystem.Create(filepath.Join(mixinsDir, "testmixin/testmixin"))
-	c.FileSystem.Create(filepath.Join(mixinsDir, "testmixin/runtimes/testmixin-runtime"))
+	if _, err = c.FileSystem.Create(filepath.Join(mixinsDir, "exec/exec")); err != nil {
+		log.Fatal(err)
+	}
+	if _, err = c.FileSystem.Create(filepath.Join(mixinsDir, "exec/runtimes/exec-runtime")); err != nil {
+		log.Fatal(err)
+	}
+	if _, err = c.FileSystem.Create(filepath.Join(mixinsDir, "testmixin/testmixin")); err != nil {
+		log.Fatal(err)
+	}
+	if _, err = c.FileSystem.Create(filepath.Join(mixinsDir, "testmixin/runtimes/testmixin-runtime")); err != nil {
+		log.Fatal(err)
+	}
 }
 
 // SetupIntegrationTest initializes the filesystem with the supporting files in

pkg/config/loader.go

@@ -3,9 +3,9 @@ package config
 import (
 	"bytes"
 	"context"
+	"errors"
 	"fmt"
 	"sort"
-	"strings"
 
 	"get.porter.sh/porter/pkg/tracing"
 	"github.com/osteele/liquid"
@@ -30,19 +30,31 @@ func LoadFromEnvironment() DataStoreLoaderFunc {
 }
 
 func BindViperToEnvironmentVariables(v *viper.Viper) {
-	v.SetEnvPrefix("PORTER")
-	v.SetEnvKeyReplacer(strings.NewReplacer("-", "_", ".", "_"))
-	v.AutomaticEnv()
 
 	// Bind open telemetry environment variables
 	// See https://github.com/open-telemetry/opentelemetry-go/tree/main/exporters/otlp/otlptrace
-	v.BindEnv("telemetry.endpoint", "OTEL_EXPORTER_OTLP_ENDPOINT", "OTEL_EXPORTER_OTLP_TRACES_ENDPOINT")
-	v.BindEnv("telemetry.protocol", "OTEL_EXPORTER_OTLP_PROTOCOL", "OTEL_EXPORTER_OTLP_TRACES_PROTOCOL")
-	v.BindEnv("telemetry.insecure", "OTEL_EXPORTER_OTLP_INSECURE", "OTEL_EXPORTER_OTLP_TRACES_INSECURE")
-	v.BindEnv("telemetry.certificate", "OTEL_EXPORTER_OTLP_CERTIFICATE", "OTEL_EXPORTER_OTLP_TRACES_CERTIFICATE")
-	v.BindEnv("telemetry.headers", "OTEL_EXPORTER_OTLP_HEADERS", "OTEL_EXPORTER_OTLP_TRACES_HEADERS")
-	v.BindEnv("telemetry.compression", "OTEL_EXPORTER_OTLP_COMPRESSION", "OTEL_EXPORTER_OTLP_TRACES_COMPRESSION")
-	v.BindEnv("telemetry.timeout", "OTEL_EXPORTER_OTLP_TIMEOUT", "OTEL_EXPORTER_OTLP_TRACES_TIMEOUT")
+	var err error
+	if err = v.BindEnv("telemetry.endpoint", "OTEL_EXPORTER_OTLP_ENDPOINT", "OTEL_EXPORTER_OTLP_TRACES_ENDPOINT"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.protocol", "OTEL_EXPORTER_OTLP_PROTOCOL", "OTEL_EXPORTER_OTLP_TRACES_PROTOCOL"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.insecure", "OTEL_EXPORTER_OTLP_INSECURE", "OTEL_EXPORTER_OTLP_TRACES_INSECURE"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.certificate", "OTEL_EXPORTER_OTLP_CERTIFICATE", "OTEL_EXPORTER_OTLP_TRACES_CERTIFICATE"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.headers", "OTEL_EXPORTER_OTLP_HEADERS", "OTEL_EXPORTER_OTLP_TRACES_HEADERS"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.compression", "OTEL_EXPORTER_OTLP_COMPRESSION", "OTEL_EXPORTER_OTLP_TRACES_COMPRESSION"); err != nil {
+		errors.Unwrap(err)
+	}
+	if err = v.BindEnv("telemetry.timeout", "OTEL_EXPORTER_OTLP_TIMEOUT", "OTEL_EXPORTER_OTLP_TRACES_TIMEOUT"); err != nil {
+		errors.Unwrap(err)
+	}
 }
 
 // LoadFromFilesystem loads data with the following precedence:

pkg/editor/editor.go

@@ -61,7 +61,13 @@ func (e *Editor) Run(ctx context.Context) ([]byte, error) {
 	if err != nil {
 		return nil, err
 	}
-	defer e.FileSystem.Remove(tempFile.Name())
+
+	defer func() error {
+		if err = e.FileSystem.Remove(tempFile.Name()); err != nil {
+			return err
+		}
+		return nil
+	}()
 
 	_, err = tempFile.Write(e.contents)
 	if err != nil {

pkg/exec/builder/errors.go

@@ -74,7 +74,9 @@ func (h IgnoreErrorHandler) HandleError(ctx context.Context, err ExitError, stdo
 	for _, allowMatch := range h.Output.Regex {
 		expression, regexErr := regexp.Compile(allowMatch)
 		if regexErr != nil {
-			span.Error(fmt.Errorf("Could not ignore failed command because the Regex specified by the mixin step definition (%q) is invalid:%s", allowMatch, regexErr.Error()))
+			if err := span.Error(fmt.Errorf("Could not ignore failed command because the Regex specified by the mixin step definition (%q) is invalid:%s", allowMatch, regexErr.Error())); err != nil {
+				return err
+			}
 			return err
 		}
 

pkg/exec/version_test.go

@@ -19,7 +19,8 @@ func TestPrintVersion(t *testing.T) {
 	opts := version.Options{}
 	err := opts.Validate()
 	require.NoError(t, err)
-	m.PrintVersion(opts)
+	err = m.PrintVersion(opts)
+	require.NoError(t, err)
 
 	gotOutput := m.TestConfig.TestContext.GetOutput()
 	wantOutput := "exec v1.2.3 (abc123) by Porter Authors"
@@ -38,7 +39,8 @@ func TestPrintJsonVersion(t *testing.T) {
 	opts.RawFormat = string(printer.FormatJson)
 	err := opts.Validate()
 	require.NoError(t, err)
-	m.PrintVersion(opts)
+	err = m.PrintVersion(opts)
+	require.NoError(t, err)
 
 	gotOutput := m.TestConfig.TestContext.GetOutput()
 	wantOutput := `{

pkg/pkgmgmt/client/install.go

@@ -200,14 +200,18 @@ func (fs *FileSystem) downloadFile(ctx context.Context, url url.URL, destPath st
 		return log.Error(fmt.Errorf("unable to check if directory exists %s: %w", parentDir, err))
 	}
 
-	cleanup := func() {}
+	cleanup := func() error { return nil }
 	if !parentDirExists {
 		err = fs.FileSystem.MkdirAll(parentDir, pkg.FileModeDirectory)
 		if err != nil {
 			return log.Error(fmt.Errorf("unable to create parent directory %s: %w", parentDir, err))
 		}
-		cleanup = func() {
-			fs.FileSystem.RemoveAll(parentDir) // If we can't download the file, don't leave traces of it
+		cleanup = func() error {
+			// If we can't download the file, don't leave traces of it
+			if err = fs.FileSystem.RemoveAll(parentDir); err != nil {
+				return err
+			}
+			return nil
 		}
 	}
 

pkg/pkgmgmt/client/runner.go

@@ -84,7 +84,9 @@ func (r *Runner) Run(ctx context.Context, commandOpts pkgmgmt.CommandOptions) er
 		}
 		go func() {
 			defer stdin.Close()
-			io.WriteString(stdin, commandOpts.Input)
+			if _, err := io.WriteString(stdin, commandOpts.Input); err != nil {
+				span.Error(err)
+			}
 		}()
 	}