fix: roles also allow variables, support parsing them in inventory

tjhop · May 17, 2024 · 8cbf358 · 8cbf358
1 parent d6399a5
commit 8cbf358
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/internal/inventory/role.go b/internal/inventory/role.go
@@ -14,9 +14,11 @@ import (
 // Role contains fields that represent a single role in the inventory.
 // - ID: string idenitfying the role (generally the file path to the role)
 // - Modules: a []string of module names that satisfy this role
+// - variables: path to the variables file for this role, if present
 type Role struct {
-	id      string
-	modules []string
+	id        string
+	modules   []string
+	variables string
 }
 
 // String is a stringer to return the role ID
@@ -104,7 +106,8 @@ func (i *Inventory) ParseRoles(ctx context.Context, logger *slog.Logger) error {
 						}
 
 						role.modules = mods
-
+					case "variables":
+						role.variables = filepath.Join(rolePath, "variables")
 					default:
 						iLogger.LogAttrs(
 							ctx,

diff --git a/test/mockup/inventory/roles/common/variables b/test/mockup/inventory/roles/common/variables
@@ -0,0 +1 @@
+mango_role_template_var="Hi, I'm a role variable!"