Avoid unnecessary use of a persistent connection (#326)

* Avoid unnecessary use of a persistent connection

* Address review comments

* Update add_require_Connection.yaml

---------

Co-authored-by: Ashwini Mhatre <amhatre@amhatre-thinkpadt14sgen2i.pnq.csb>

changelogs/fragments/add_require_Connection.yaml

@@ -0,0 +1,2 @@
+bugfixes:
+  - Avoid unnecessary use of persistent connection in `cli_parse`, `fact_diff`, `update_fact` and `validate` as this action does not require a connection.

plugins/action/cli_parse.py

@@ -37,6 +37,8 @@ ARGSPEC_CONDITIONALS = {
 class ActionModule(ActionBase):
     """action module"""
 
+    _requires_connection = False
+
     PARSER_CLS_NAME = "CliParser"
 
     def __init__(self, *args, **kwargs):

plugins/action/fact_diff.py

@@ -24,6 +24,8 @@ from ansible_collections.ansible.utils.plugins.modules.fact_diff import DOCUMENT
 class ActionModule(ActionBase):
     """action module"""
 
+    _requires_connection = False
+
     def __init__(self, *args, **kwargs):
         super(ActionModule, self).__init__(*args, **kwargs)
         self._supports_async = True

plugins/action/update_fact.py

@@ -26,6 +26,8 @@ from ansible_collections.ansible.utils.plugins.modules.update_fact import DOCUME
 class ActionModule(ActionBase):
     """action module"""
 
+    _requires_connection = False
+
     def __init__(self, *args, **kwargs):
         """Start here"""
         super(ActionModule, self).__init__(*args, **kwargs)

plugins/action/validate.py

@@ -29,6 +29,7 @@ class ActionModule(ActionBase):
     """action module"""
 
     VALIDATE_CLS_NAME = "Validate"
+    _requires_connection = False
 
     def __init__(self, *args, **kwargs):
         super(ActionModule, self).__init__(*args, **kwargs)