Bug module wrapper regex (#806)

* Fix regex in ModuleWrapperCatElem to account for versioned modules * Convert dashes to underscores in version names
hpc · Jan 31, 2025 · 7caf955 · 7caf955
1 parent 9b5ff93
commit 7caf955
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/pavilion/module_wrapper.py b/lib/pavilion/module_wrapper.py
@@ -404,7 +404,7 @@ def _load(self, var_man: VariableSetManager, req_name: str, version: str) \
                 actions.append(ModuleLoad(module_name=mod_name, version=mod_vers))
 
         env_vars = collections.OrderedDict()
-        vers_var_name = '{}_VERSION'.format(self.name.replace('*', 'any'))
+        vers_var_name = '{}_VERSION'.format(self.name.replace('*', 'any').replace('-', '_'))
         if version:
             env_vars[vers_var_name] = version
         else:

diff --git a/lib/pavilion/test_config/file_format.py b/lib/pavilion/test_config/file_format.py
@@ -395,7 +395,7 @@ class TestCatElem(yc.CategoryElem):
 class ModuleWrapperCatElem(yc.CategoryElem):
     """Allow glob wildcards in key names."""
 
-    _NAME_RE = re.compile(r'^[a-zA-Z*?+][a-zA-Z0-9_*+?-]*$')
+    _NAME_RE = re.compile(r'^[a-zA-Z*?+][a-zA-Z0-9_*+?-]*(/[a-zA-Z0-9._-]+)?$')
     type=OrderedDict