Apply suggestions from code review

Co-authored-by: Rafael F. <r4f4rfs@gmail.com>
openshift · Jan 31, 2024 · c1a0d27 · c1a0d27
1 parent f01b5a4
commit c1a0d27
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/asset/cluster/tfvars/tfvars.go b/pkg/asset/cluster/tfvars/tfvars.go
@@ -527,7 +527,7 @@ func (t *TerraformVariables) Generate(parents asset.Parents) error {
 
 		tags, err := gcpconfig.GetUserTags(gcpconfig.NewTagManager(client), installConfig.Config)
 		if err != nil {
-			return errors.Wrapf(err, "failed to fetch user-defined tags")
+			return fmt.Errorf("failed to fetch user-defined tags: %w", err)
 		}
 
 		data, err := gcptfvars.TFVars(

diff --git a/pkg/asset/installconfig/gcp/usertags.go b/pkg/asset/installconfig/gcp/usertags.go
@@ -209,7 +209,7 @@ func validateAndStoreUserTags(ctx context.Context, mgr TagManager, project strin
 func getProjectTags(ctx context.Context, mgr TagManager, project string) (effectiveTags, error) {
 	client, err := getTagBindingsClient(ctx, mgr)
 	if err != nil {
-		return nil, errors.Wrapf(err, "failed to create client to fetch effective tags on the %s project", project)
+		return nil, fmt.Errorf("failed to create client to fetch effective tags on the %s project: %w", project, err)
 	}
 	defer client.Close()