Preserve behavior where we ignore 'version' in the workflow spec

Author: ecolternv

src/lib/utils/tests/test_workflow.py

@@ -81,6 +81,17 @@ def test_duplicate_workflow_raises(self):
             workflow_utils.parse_workflow_spec(spec)
         self.assertIn('workflow', str(context.exception))
 
+    def test_version_key_allowed(self):
+        spec = """\
+version: 2
+workflow:
+  name: my-wf
+  groups: []
+"""
+        workflow_spec, default_values = workflow_utils.parse_workflow_spec(spec)
+        self.assertIn('name: my-wf', workflow_spec)
+        self.assertIsNone(default_values)
+
     def test_unknown_top_level_key_raises(self):
         spec = """\
 workflow:

src/lib/utils/workflow.py

@@ -45,7 +45,7 @@ def parse_workflow_spec(workflow_spec: str) -> Tuple[str, Dict | None]:
         r'^([a-zA-Z][a-zA-Z0-9_-]*):(.*?)(?=^[a-zA-Z][a-zA-Z0-9_-]*:|\Z)',
         re.DOTALL | re.MULTILINE,
     )
-    allowed_sections = {'workflow', 'default-values'}
+    allowed_sections = {'workflow', 'default-values', 'version'}
     sections: Dict[str, str] = {}
     for match in section_pattern.finditer(workflow_spec):
         key = match.group(1)