add clarity to validate error message (#58)

acornett21 · web-flow · commit 7abfd4e9bf9c · 2021-10-21T11:16:24.000-07:00
Issue #, if available:
N/A

Description of changes:
With the removal of `--aws-account-id` the validate error message that gets propagated is less clear then it used to be. This PR aims to add more clarity to the error message that gets propagated when `ack_user_secrets` does not exist in the namespace that a given controller runs in.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
diff --git a/pkg/config/config.go b/pkg/config/config.go
@@ -153,7 +153,7 @@ func (cfg *Config) SetAWSAccountID() error {
 // Validate ensures the options are valid
 func (cfg *Config) Validate() error {
 	if err := cfg.SetAWSAccountID(); err != nil {
-		return errors.New("unable to determine account ID. Please make sure AWS credentials are setup in controller pod")
+		return fmt.Errorf("unable to determine account ID: %v", err)
 	}
 
 	if cfg.Region == "" {

Original file line number	Diff line number	Diff line change
`@@ -153,7 +153,7 @@ func (cfg *Config) SetAWSAccountID() error {`
`153`	`153`	`// Validate ensures the options are valid`
`154`	`154`	`func (cfg *Config) Validate() error {`
`155`	`155`	`if err := cfg.SetAWSAccountID(); err != nil {`
`156`		`- return errors.New("unable to determine account ID. Please make sure AWS credentials are setup in controller pod")`
	`156`	`+ return fmt.Errorf("unable to determine account ID: %v", err)`
`157`	`157`	`}`
`158`	`158`
`159`	`159`	`if cfg.Region == "" {`