feat: pass through metadata generate requirements errors (#3142)

Author: apeabody

cli/bpmetadata/cmd.go

@@ -268,7 +268,7 @@ func CreateBlueprintMetadata(bpPath string, bpMetadataObj *BlueprintMetadata) (*
 	moduleName := parseBpModuleName(bpPath, repoDetails.Source.BlueprintRootPath)
 	requirements, err := getBlueprintRequirements(rolesCfgPath, svcsCfgPath, versionsCfgPath, mdFlags.perModuleRequirements, moduleName)
 	if err != nil {
-		Log.Info("skipping blueprint requirements since roles and/or services configurations were not found as per https://tinyurl.com/tf-iam and https://tinyurl.com/tf-services")
+		Log.Info("skipping blueprint requirements since roles and/or services configurations were not found as per https://tinyurl.com/tf-iam and https://tinyurl.com/tf-services", "error:", err)
 	} else {
 		bpMetadataObj.Spec.Requirements = requirements
 	}

cli/bpmetadata/tfconfig.go

@@ -23,11 +23,11 @@ import (
 )
 
 const (
-	versionRegEx   = "/v([0-9]+[.0-9]*)$"
-	modulePattern  = `(?:^|/)modules/([^/]+)`
-	perModuleRoles = "per_module_roles"
+	versionRegEx      = "/v([0-9]+[.0-9]*)$"
+	modulePattern     = `(?:^|/)modules/([^/]+)`
+	perModuleRoles    = "per_module_roles"
 	perModuleServices = "per_module_services"
-	rootModuleName = "root"
+	rootModuleName    = "root"
 )
 
 type blueprintVersion struct {
@@ -509,7 +509,7 @@ func parseBlueprintServices(servicesFile *hcl.File, perModuleMode bool, moduleNa
 		diags = gohcl.DecodeExpression(apisAttr.Expr, nil, &s)
 		err = hasHclErrors(diags)
 		if err != nil {
-			return nil, err
+			return nil, fmt.Errorf("error decoding activate_apis: %w", err)
 		}
 
 		// because we're only interested in the top-level modules block