Ensure that warnings triggered by is_validate are triggered by VIP & VR

hjoliver · Jun 12, 2024 · b8ae3b5 · b8ae3b5
1 parent fa22199
commit b8ae3b5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/cylc/flow/scripts/validate_install_play.py b/cylc/flow/scripts/validate_install_play.py
@@ -86,7 +86,7 @@ def get_option_parser() -> COP:
         # no sense in a VIP context.
         if option.kwargs.get('dest') != 'against_source':
             parser.add_option(*option.args, **option.kwargs)
-
+    parser.set_defaults(is_validate=True)
     return parser
 
 
@@ -103,6 +103,9 @@ def main(parser: COP, options: 'Values', workflow_id: Optional[str] = None):
     log_subcommand('validate', source)
     asyncio.run(cylc_validate(parser, options, str(source)))
 
+    # Unset is validate after validation.
+    delattr(options, 'is_validate')
+
     log_subcommand('install', source)
     _, workflow_id = asyncio.run(cylc_install(options, workflow_id))
 

diff --git a/cylc/flow/scripts/validate_reinstall.py b/cylc/flow/scripts/validate_reinstall.py
@@ -97,6 +97,7 @@ def get_option_parser() -> COP:
     )
     for option in VR_OPTIONS:
         parser.add_option(*option.args, **option.kwargs)
+    parser.set_defaults(is_validate=True)
     return parser
 
 
@@ -169,6 +170,9 @@ async def vr_cli(parser: COP, options: 'Values', workflow_id: str):
     log_subcommand('validate --against-source', workflow_id)
     await cylc_validate(parser, options, workflow_id)
 
+    # Unset is validate after validation.
+    delattr(options, 'is_validate')
+
     log_subcommand('reinstall', workflow_id)
     reinstall_ok = await cylc_reinstall(
         options, workflow_id,