use FQCN when calling a module from action plugin (#967) (#988)

* use FQCN when calling a module from action plugin

https://docs.ansible.com/ansible/latest/porting_guides/porting_guide_2.10.html#action-plugins-which-execute-modules-should-use-fully-qualified-module-names

* doc: add changelog fragment (minor_changes)

* move to shippable/posix/group1 to run tests with ansible 2.9

* move back to shippable/posix/group4

to not share testbed with docker engine on rhel7, that never releases
the xtables lock and leads 'iptables_state' tests to always fail.

(cherry picked from commit ea1fb83b0c)

Co-authored-by: quidame <quidame@poivron.org>

changelogs/fragments/967-use-fqcn-when-calling-a-module-from-action-plugin.yml

@@ -0,0 +1,4 @@
+---
+minor_changes:
+  - iptables_state - use FQCN when calling a module from action plugin
+    (https://github.com/ansible-collections/community.general/pull/967).

plugins/action/system/iptables_state.py

@@ -48,7 +48,7 @@ class ActionModule(ActionBase):
         # At least one iteration is required, even if timeout is 0.
         for i in range(max(1, timeout)):
             async_result = self._execute_module(
-                module_name='async_status',
+                module_name='ansible.builtin.async_status',
                 module_args=module_args,
                 task_vars=task_vars,
                 wrap_async=False)
@@ -177,7 +177,7 @@ class ActionModule(ActionBase):
 
                 async_status_args['mode'] = 'cleanup'
                 garbage = self._execute_module(
-                    module_name='async_status',
+                    module_name='ansible.builtin.async_status',
                     module_args=async_status_args,
                     task_vars=task_vars,
                     wrap_async=False)