chore: add wrapping to errors

addressing pr comments.
replicatedhq · Oct 15, 2024 · 85ac17c · 85ac17c
1 parent 1fa2eba
commit 85ac17c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/pkg/supportbundle/defaultspec/embed.go b/pkg/supportbundle/defaultspec/embed.go
@@ -3,6 +3,7 @@ package defaultspec
 import (
 	_ "embed"
 
+	"github.com/pkg/errors"
 	troubleshootv1beta2 "github.com/replicatedhq/troubleshoot/pkg/apis/troubleshoot/v1beta2"
 	"github.com/replicatedhq/troubleshoot/pkg/supportbundle"
 )
@@ -17,7 +18,7 @@ func Get(isAirgap bool) (troubleshootv1beta2.SupportBundle, error) {
 		var err error
 		spec, err = supportbundle.ParseSupportBundle(raw, !isAirgap)
 		if err != nil {
-			return troubleshootv1beta2.SupportBundle{}, err
+			return troubleshootv1beta2.SupportBundle{}, errors.Wrap(err, "failed to parse support bundle")
 		}
 	}
 	return *spec.DeepCopy(), nil

diff --git a/pkg/supportbundle/spec.go b/pkg/supportbundle/spec.go
@@ -709,7 +709,7 @@ func deduplicatedAfterCollection(supportBundle *troubleshootv1beta2.SupportBundl
 func getDefaultAnalyzers(isKurl, isAirgap bool) ([]*troubleshootv1beta2.Analyze, error) {
 	defaultSpec, err := defaultspec.Get(isAirgap)
 	if err != nil {
-		return nil, err
+		return nil, errors.Wrap(err, "failed to get default spec")
 	}
 
 	defaultAnalyzers := defaultSpec.Spec.Analyzers

diff --git a/pkg/supportbundle/supportbundle.go b/pkg/supportbundle/supportbundle.go
@@ -315,7 +315,7 @@ func CreateSupportBundleAnalysis(appID string, archivePath string, bundle *types
 
 	defaultAnalyzers, err := getDefaultAnalyzers(isKurl, foundApp.IsAirgap)
 	if err != nil {
-		return err
+		return errors.Wrap(err, "failed to get default analyzers")
 	}
 	analyzer.Spec.Analyzers = append(analyzer.Spec.Analyzers, defaultAnalyzers...)
 	analyzer.Spec.Analyzers = append(analyzer.Spec.Analyzers, getDefaultDynamicAnalyzers(foundApp)...)