Avoid duplicate confirmations in a safer way

This version instead still requires --yes with --restart to avoid the
confirmation option, but avoids duplicate prompts.

This might be slightly more cumbersome, but ensures consistency: every
situation that could cause a restart is confirmed even if --restart is
given.

client-cli/pvc/cli/cli.py

@@ -169,9 +169,10 @@ def restart_opt(function):
     @wraps(function)
     def confirm_action(*args, **kwargs):
         restart_state = kwargs.get("restart_flag", None)
+        live_state = kwargs.get("live_flag", False)
 
-        if restart_state is None:
+        if restart_state is None and not live_state:
-            # Neither "--restart" or "--no-restart" was passed: prompt for restart or restart if "--unsafe"
+            # Neither "--restart" or "--no-restart" was passed, and "--no-live" was passed: prompt for restart or restart if "--unsafe"
             try:
                 click.confirm(
                     f"Restart VM {kwargs.get('domain')} to apply changes",
@@ -179,6 +180,7 @@ def restart_opt(function):
                     abort=True,
                 )
                 kwargs["restart_flag"] = True
+                kwargs["confirm_flag"] = True
             except Exception:
                 echo(CLI_CONFIG, "Changes will be applied on next VM start/restart.")
                 kwargs["restart_flag"] = False