ansible-core 2.19.0b4__py3-none-any.whl → 2.19.0b5__py3-none-any.whl

ansible/module_utils/_internal/_traceback.py

@@ -6,9 +6,10 @@
 from __future__ import annotations
 
 import enum
-import inspect
 import traceback
 
+from . import _stack
+
 
 class TracebackEvent(enum.Enum):
     """The events for which tracebacks can be enabled."""
@@ -16,6 +17,7 @@ class TracebackEvent(enum.Enum):
     ERROR = enum.auto()
     WARNING = enum.auto()
     DEPRECATED = enum.auto()
+    DEPRECATED_VALUE = enum.auto()  # implies DEPRECATED
 
 
 def traceback_for() -> list[str]:
@@ -28,24 +30,25 @@ def is_traceback_enabled(event: TracebackEvent) -> bool:
     return _is_traceback_enabled(event)
 
 
-def maybe_capture_traceback(event: TracebackEvent) -> str | None:
+def maybe_capture_traceback(msg: str, event: TracebackEvent) -> str | None:
     """
     Optionally capture a traceback for the current call stack, formatted as a string, if the specified traceback event is enabled.
-    The current and previous frames are omitted to mask the expected call pattern from error/warning handlers.
+    Frames marked with the `_skip_stackwalk` local are omitted.
     """
+    _skip_stackwalk = True
+
     if not is_traceback_enabled(event):
         return None
 
     tb_lines = []
 
-    if current_frame := inspect.currentframe():
+    if frame_info := _stack.caller_frame():
         # DTFIX-FUTURE: rewrite target-side tracebacks to point at controller-side paths?
-        frames = inspect.getouterframes(current_frame)
-        ignore_frame_count = 2  # ignore this function and its caller
         tb_lines.append('Traceback (most recent call last):\n')
-        tb_lines.extend(traceback.format_stack(frames[ignore_frame_count].frame))
+        tb_lines.extend(traceback.format_stack(frame_info.frame))
+        tb_lines.append(f'Message: {msg}\n')
     else:
-        tb_lines.append('Traceback unavailable.\n')
+        tb_lines.append('(frame not found)\n')  # pragma: nocover
 
     return ''.join(tb_lines)
 

ansible/module_utils/ansible_release.py

@@ -17,6 +17,6 @@
 
 from __future__ import annotations
 
-__version__ = '2.19.0b4'
+__version__ = '2.19.0b5'
 __author__ = 'Ansible, Inc.'
 __codename__ = "What Is and What Should Never Be"

ansible/module_utils/basic.py

@@ -75,7 +75,7 @@ except ImportError:
 # Python2 & 3 way to get NoneType
 NoneType = type(None)
 
-from ._internal import _traceback, _errors, _debugging, _deprecator
+from ._internal import _traceback, _errors, _debugging, _deprecator, _messages
 
 from .common.text.converters import (
     to_native,
@@ -161,10 +161,10 @@ from ansible.module_utils.common.validation import (
     safe_eval,
 )
 from ansible.module_utils.common._utils import get_all_subclasses as _get_all_subclasses
-from ansible.module_utils.common import messages as _messages
 from ansible.module_utils.parsing.convert_bool import BOOLEANS, BOOLEANS_FALSE, BOOLEANS_TRUE, boolean
 from ansible.module_utils.common.warnings import (
     deprecate,
+    error_as_warning,
     get_deprecations,
     get_warnings,
     warn,
@@ -505,9 +505,34 @@ class AnsibleModule(object):
 
         return self._tmpdir
 
-    def warn(self, warning):
-        warn(warning)
-        self.log('[WARNING] %s' % warning)
+    def warn(
+        self,
+        warning: str,
+        *,
+        help_text: str | None = None,
+    ) -> None:
+        _skip_stackwalk = True
+
+        warn(
+            warning=warning,
+            help_text=help_text,
+        )
+
+    def error_as_warning(
+        self,
+        msg: str | None,
+        exception: BaseException,
+        *,
+        help_text: str | None = None,
+    ) -> None:
+        """Display an exception as a warning."""
+        _skip_stackwalk = True
+
+        error_as_warning(
+            msg=msg,
+            exception=exception,
+            help_text=help_text,
+        )
 
     def deprecate(
         self,
@@ -1399,7 +1424,7 @@ class AnsibleModule(object):
         # deprecate(
         #     msg="The `AnsibleModule.jsonify' method is deprecated.",
         #     version="2.27",
-        #     # help_text="",  # DTFIX-RELEASE: fill in this help text
+        #     # help_text="",  # DTFIX-FUTURE: fill in this help text
         # )
 
         try:
@@ -1528,16 +1553,25 @@ class AnsibleModule(object):
         )
 
         if isinstance(exception, BaseException):
-            # Include a `_messages.ErrorDetail` in the result.
-            # The `msg` is included in the list of errors to ensure it is not lost when looking only at `exception` from the result.
-
-            error_summary = _errors.create_error_summary(exception)
-            error_summary = _dataclasses.replace(error_summary, details=(_messages.Detail(msg=msg),) + error_summary.details)
-
-            kwargs.update(exception=error_summary)
+            # Include a `_messages.Event` in the result.
+            # The `msg` is included in the chain to ensure it is not lost when looking only at `exception` from the result.
+
+            kwargs.update(
+                exception=_messages.ErrorSummary(
+                    event=_messages.Event(
+                        msg=msg,
+                        formatted_traceback=_traceback.maybe_capture_traceback(msg, _traceback.TracebackEvent.ERROR),
+                        chain=_messages.EventChain(
+                            msg_reason=_errors.MSG_REASON_DIRECT_CAUSE,
+                            traceback_reason="The above exception was the direct cause of the following error:",
+                            event=_errors.EventFactory.from_exception(exception, _traceback.is_traceback_enabled(_traceback.TracebackEvent.ERROR)),
+                        ),
+                    ),
+                ),
+            )
         elif _traceback.is_traceback_enabled(_traceback.TracebackEvent.ERROR):
             # Include only a formatted traceback string in the result.
-            # The controller will combine this with `msg` to create an `_messages.ErrorDetail`.
+            # The controller will combine this with `msg` to create an `_messages.ErrorSummary`.
 
             formatted_traceback: str | None
 
@@ -1546,7 +1580,7 @@ class AnsibleModule(object):
             elif exception is _UNSET and (current_exception := t.cast(t.Optional[BaseException], sys.exc_info()[1])):
                 formatted_traceback = _traceback.maybe_extract_traceback(current_exception, _traceback.TracebackEvent.ERROR)
             else:
-                formatted_traceback = _traceback.maybe_capture_traceback(_traceback.TracebackEvent.ERROR)
+                formatted_traceback = _traceback.maybe_capture_traceback(msg, _traceback.TracebackEvent.ERROR)
 
             if formatted_traceback:
                 kwargs.update(exception=formatted_traceback)

ansible/module_utils/common/arg_spec.py

@@ -6,6 +6,7 @@ from __future__ import annotations
 
 from copy import deepcopy
 
+from ansible.module_utils.datatag import deprecator_from_collection_name
 from ansible.module_utils.common.parameters import (
     _ADDITIONAL_CHECKS,
     _get_legal_inputs,
@@ -22,7 +23,6 @@ from ansible.module_utils.common.parameters import (
 
 from ansible.module_utils.common.text.converters import to_native
 from ansible.module_utils.common.warnings import deprecate, warn
-from ansible.module_utils.common import messages as _messages
 
 from ansible.module_utils.common.validation import (
     check_mutually_exclusive,
@@ -306,7 +306,7 @@ class ModuleArgumentSpecValidator(ArgumentSpecValidator):
                 msg=d['msg'],
                 version=d.get('version'),
                 date=d.get('date'),
-                deprecator=_messages.PluginInfo._from_collection_name(d.get('collection_name')),
+                deprecator=deprecator_from_collection_name(d.get('collection_name')),
             )
 
         for w in result._warnings:

ansible/module_utils/common/collections.py

@@ -6,6 +6,7 @@
 from __future__ import annotations
 
 
+from ansible.module_utils.common import warnings as _warnings
 from ansible.module_utils.six import binary_type, text_type
 from ansible.module_utils.six.moves.collections_abc import Hashable, Mapping, MutableMapping, Sequence  # pylint: disable=unused-import
 
@@ -102,6 +103,11 @@ def count(seq):
     code is run on Python 2.6.* where collections.Counter is not available. It should be
     deprecated and replaced when support for Python < 2.7 is dropped.
     """
+    _warnings.deprecate(
+        msg="The `ansible.module_utils.common.collections.count` function is deprecated.",
+        version="2.23",
+        help_text="Use `collections.Counter` from the Python standard library instead.",
+    )
     if not is_iterable(seq):
         raise Exception('Argument provided  is not an iterable')
     counters = dict()

ansible/module_utils/common/json.py

@@ -20,7 +20,7 @@ def __getattr__(name: str) -> object:
         #     _warnings.deprecate(
         #         msg="The `AnsibleJSONEncoder` type is deprecated.",
         #         version="2.27",
-        #         help_text="Use a profile-based encoder instead.",  # DTFIX-RELEASE: improve this help text
+        #         help_text="Use a profile-based encoder instead.",  # DTFIX-FUTURE: improve this help text
         #     )
 
         return _get_legacy_encoder()
@@ -31,7 +31,7 @@ def __getattr__(name: str) -> object:
         #     _warnings.deprecate(
         #         msg="The `AnsibleJSONDecoder` type is deprecated.",
         #         version="2.27",
-        #         help_text="Use a profile-based decoder instead.",  # DTFIX-RELEASE: improve this help text
+        #         help_text="Use a profile-based decoder instead.",  # DTFIX-FUTURE: improve this help text
         #     )
 
         return _tagless.Decoder

ansible/module_utils/common/text/converters.py

@@ -245,7 +245,7 @@ def jsonify(data, **kwargs):
     # deprecate(
     #     msg="The `jsonify` function is deprecated.",
     #     version="2.27",
-    #     # help_text="",  # DTFIX-RELEASE: fill in this help text
+    #     # help_text="",  # DTFIX-FUTURE: fill in this help text
     # )
 
     return json.dumps(data, cls=_common_json._get_legacy_encoder(), _decode_bytes=True, **kwargs)
@@ -257,7 +257,7 @@ def container_to_bytes(d, encoding='utf-8', errors='surrogate_or_strict'):
         Specialized for json return because this only handles, lists, tuples,
         and dict container types (the containers that the json module returns)
     """
-    # DTFIX-RELEASE: deprecate
+    # DTFIX-FUTURE: deprecate
 
     if isinstance(d, text_type):
         return to_bytes(d, encoding=encoding, errors=errors)
@@ -277,7 +277,7 @@ def container_to_text(d, encoding='utf-8', errors='surrogate_or_strict'):
     Specialized for json return because this only handles, lists, tuples,
     and dict container types (the containers that the json module returns)
     """
-    # DTFIX-RELEASE: deprecate
+    # DTFIX-FUTURE: deprecate
 
     if isinstance(d, binary_type):
         # Warning, can traceback

ansible/module_utils/common/validation.py

@@ -402,7 +402,7 @@ def check_type_list(value):
     if isinstance(value, list):
         return value
 
-    # DTFIX-RELEASE: deprecate legacy comma split functionality, eventually replace with `_check_type_list_strict`
+    # DTFIX-FUTURE: deprecate legacy comma split functionality, eventually replace with `_check_type_list_strict`
     if isinstance(value, string_types):
         return value.split(",")
     elif isinstance(value, int) or isinstance(value, float):

ansible/module_utils/common/warnings.py

@@ -6,20 +6,77 @@ from __future__ import annotations as _annotations
 
 import typing as _t
 
-from ansible.module_utils._internal import _traceback, _deprecator
-from ansible.module_utils.common import messages as _messages
+from ansible.module_utils._internal import _traceback, _deprecator, _event_utils, _messages, _errors
 from ansible.module_utils import _internal
 
 
-def warn(warning: str) -> None:
+def warn(
+    warning: str,
+    *,
+    help_text: str | None = None,
+    obj: object | None = None,
+) -> None:
     """Record a warning to be returned with the module result."""
-    # DTFIX-RELEASE: shim to controller display warning like `deprecate`
-    _global_warnings[_messages.WarningSummary(
-        details=(
-            _messages.Detail(msg=warning),
+    _skip_stackwalk = True
+
+    if _internal.is_controller:
+        _display = _internal.import_controller_module('ansible.utils.display').Display()
+        _display.warning(
+            msg=warning,
+            help_text=help_text,
+            obj=obj,
+        )
+
+        return
+
+    warning = _messages.WarningSummary(
+        event=_messages.Event(
+            msg=warning,
+            help_text=help_text,
+            formatted_traceback=_traceback.maybe_capture_traceback(warning, _traceback.TracebackEvent.WARNING),
+        ),
+    )
+
+    _global_warnings[warning] = None
+
+
+def error_as_warning(
+    msg: str | None,
+    exception: BaseException,
+    *,
+    help_text: str | None = None,
+    obj: object = None,
+) -> None:
+    """Display an exception as a warning."""
+    _skip_stackwalk = True
+
+    if _internal.is_controller:
+        _display = _internal.import_controller_module('ansible.utils.display').Display()
+        _display.error_as_warning(
+            msg=msg,
+            exception=exception,
+            help_text=help_text,
+            obj=obj,
+        )
+
+        return
+
+    event = _errors.EventFactory.from_exception(exception, _traceback.is_traceback_enabled(_traceback.TracebackEvent.WARNING))
+
+    warning = _messages.WarningSummary(
+        event=_messages.Event(
+            msg=msg,
+            help_text=help_text,
+            formatted_traceback=_traceback.maybe_capture_traceback(msg, _traceback.TracebackEvent.WARNING),
+            chain=_messages.EventChain(
+                msg_reason=_errors.MSG_REASON_DIRECT_CAUSE,
+                traceback_reason=_errors.TRACEBACK_REASON_EXCEPTION_DIRECT_WARNING,
+                event=event,
+            ),
         ),
-        formatted_traceback=_traceback.maybe_capture_traceback(_traceback.TracebackEvent.WARNING),
-    )] = None
+    )
+
+    _global_warnings[warning] = None
 
 
 def deprecate(
@@ -57,30 +114,30 @@ def deprecate(
 
         return
 
-    _global_deprecations[_messages.DeprecationSummary(
-        details=(
-            _messages.Detail(msg=msg, help_text=help_text),
+    warning = _messages.DeprecationSummary(
+        event=_messages.Event(
+            msg=msg,
+            help_text=help_text,
+            formatted_traceback=_traceback.maybe_capture_traceback(msg, _traceback.TracebackEvent.DEPRECATED),
         ),
-        formatted_traceback=_traceback.maybe_capture_traceback(_traceback.TracebackEvent.DEPRECATED),
         version=version,
         date=date,
         deprecator=deprecator,
-    )] = None
+    )
+
+    _global_deprecations[warning] = None
 
 
 def get_warning_messages() -> tuple[str, ...]:
     """Return a tuple of warning messages accumulated over this run."""
-    # DTFIX-RELEASE: add future deprecation comment
-    return tuple(item._format() for item in _global_warnings)
-
-
-_DEPRECATION_MESSAGE_KEYS = frozenset({'msg', 'date', 'version', 'collection_name'})
+    # DTFIX7: add future deprecation comment
+    return tuple(_event_utils.format_event_brief_message(item.event) for item in _global_warnings)
 
 
 def get_deprecation_messages() -> tuple[dict[str, _t.Any], ...]:
     """Return a tuple of deprecation warning messages accumulated over this run."""
-    # DTFIX-RELEASE: add future deprecation comment
-    return tuple({key: value for key, value in item._as_simple_dict().items() if key in _DEPRECATION_MESSAGE_KEYS} for item in _global_deprecations)
+    # DTFIX7: add future deprecation comment
+    return tuple(_event_utils.deprecation_as_dict(item) for item in _global_deprecations)
 
 
 def get_warnings() -> list[_messages.WarningSummary]:
@@ -94,7 +151,7 @@ def get_deprecations() -> list[_messages.DeprecationSummary]:
 
 
 _global_warnings: dict[_messages.WarningSummary, object] = {}
-"""Global, ordered, de-deplicated storage of acculumated warnings for the current module run."""
+"""Global, ordered, de-duplicated storage of accumulated warnings for the current module run."""
 
 _global_deprecations: dict[_messages.DeprecationSummary, object] = {}
-"""Global, ordered, de-deplicated storage of acculumated deprecations for the current module run."""
+"""Global, ordered, de-duplicated storage of accumulated deprecations for the current module run."""

ansible/module_utils/common/yaml.py

@@ -24,7 +24,7 @@ except ImportError:
 else:
     HAS_YAML = True
 
-# DTFIX-RELEASE: refactor this to share the implementation with the controller version
+# DTFIX-FUTURE: refactor this to share the implementation with the controller version
 #              use an abstract base class, with __init_subclass__ for representer registration, and instance methods for overridable representers
 #              then tests can be consolidated intead of having two nearly identical copies
 

ansible/module_utils/datatag.py

@@ -3,13 +3,15 @@ from __future__ import annotations as _annotations
 
 import typing as _t
 
-from ._internal import _datatag, _deprecator
+from ._internal import _datatag, _deprecator, _traceback, _messages
 from ._internal._datatag import _tags
-from .common import messages as _messages
 
 _T = _t.TypeVar('_T')
 
 
+deprecator_from_collection_name = _deprecator.deprecator_from_collection_name
+
+
 def deprecate_value(
     value: _T,
     msg: str,
@@ -36,6 +38,7 @@ def deprecate_value(
         date=date,
         version=version,
         deprecator=_deprecator.get_best_deprecator(deprecator=deprecator, collection_name=collection_name),
+        formatted_traceback=_traceback.maybe_capture_traceback(msg, _traceback.TracebackEvent.DEPRECATED_VALUE),
     )
 
     return deprecated.tag(value)

ansible/module_utils/facts/system/distribution.py

@@ -310,9 +310,22 @@ class DistributionFiles:
                         suse_facts['distribution_release'] = release.group(1)
                         suse_facts['distribution_version'] = collected_facts['distribution_version'] + '.' + release.group(1)
 
-        # See https://www.suse.com/support/kb/doc/?id=000019341 for SLES for SAP
-        if os.path.islink('/etc/products.d/baseproduct') and os.path.realpath('/etc/products.d/baseproduct').endswith('SLES_SAP.prod'):
-            suse_facts['distribution'] = 'SLES_SAP'
+        # Check VARIANT_ID first for SLES4SAP or SL-Micro
+        variant_id_match = re.search(r'^VARIANT_ID="?([^"\n]*)"?', data, re.MULTILINE)
+        if variant_id_match:
+            variant_id = variant_id_match.group(1)
+            if variant_id in ('server-sap', 'sles-sap'):
+                suse_facts['distribution'] = 'SLES_SAP'
+            elif variant_id == 'transactional':
+                suse_facts['distribution'] = 'SL-Micro'
+        else:
+            # Fallback for older SLES 15 using baseproduct symlink
+            if os.path.islink('/etc/products.d/baseproduct'):
+                resolved = os.path.realpath('/etc/products.d/baseproduct')
+                if resolved.endswith('SLES_SAP.prod'):
+                    suse_facts['distribution'] = 'SLES_SAP'
+                elif resolved.endswith('SL-Micro.prod'):
+                    suse_facts['distribution'] = 'SL-Micro'
 
         return True, suse_facts
 

ansible/module_utils/facts/virtual/linux.py

@@ -201,7 +201,7 @@ class LinuxVirtual(Virtual):
                 virtual_facts['virtualization_type'] = 'virtualbox'
                 found_virt = True
 
-        if bios_vendor in ('Amazon EC2', 'DigitalOcean', 'Hetzner'):
+        if bios_vendor in ('Amazon EC2', 'DigitalOcean', 'Hetzner', 'Linode'):
             guest_tech.add('kvm')
             if not found_virt:
                 virtual_facts['virtualization_type'] = 'kvm'

ansible/module_utils/service.py

@@ -36,7 +36,7 @@ import select
 import shlex
 import subprocess
 
-from ansible.module_utils.six import PY2, b
+from ansible.module_utils.six import b
 from ansible.module_utils.common.text.converters import to_bytes, to_text
 
 
@@ -187,12 +187,8 @@ def daemonize(module, cmd):
     if pid == 0:
         os.close(pipe[0])
 
-        # if command is string deal with  py2 vs py3 conversions for shlex
         if not isinstance(cmd, list):
-            if PY2:
-                cmd = shlex.split(to_bytes(cmd, errors=errors))
-            else:
-                cmd = shlex.split(to_text(cmd, errors=errors))
+            cmd = shlex.split(to_text(cmd, errors=errors))
 
         # make sure we always use byte strings
         run_cmd = []
@@ -247,9 +243,6 @@ def daemonize(module, cmd):
                     break
                 return_data += to_bytes(data, errors=errors)
 
-        # Note: no need to specify encoding on py3 as this module sends the
-        # pickle to itself (thus same python interpreter so we aren't mixing
-        # py2 and py3)
         return pickle.loads(to_bytes(return_data, errors=errors))
 
 

ansible/modules/apt_repository.py

@@ -88,8 +88,8 @@ options:
         description:
             - Whether to automatically try to install the Python apt library or not, if it is not already installed.
               Without this library, the module does not work.
-            - Runs C(apt-get install python-apt) for Python 2, and C(apt-get install python3-apt) for Python 3.
-            - Only works with the system Python 2 or Python 3. If you are using a Python on the remote that is not
+            - Runs C(apt-get install python3-apt).
+            - Only works with the system Python. If you are using a Python on the remote that is not
               the system Python, set O(install_python_apt=false) and ensure that the Python apt library
               for your Python version is installed some other way.
         type: bool
@@ -98,8 +98,7 @@ author:
 - Alexander Saltanov (@sashka)
 version_added: "0.7"
 requirements:
-   - python-apt (python 2)
-   - python3-apt (python 3)
+   - python3-apt
    - apt-key or gpg
 """
 
@@ -232,14 +231,15 @@ class SourcesList(object):
         self.files_mapping = {}  # internal DS for tracking symlinks
         # Repositories that we're adding -- used to implement mode param
         self.new_repos = set()
-        self.default_file = self._apt_cfg_file('Dir::Etc::sourcelist')
+        self.default_file = apt_pkg.config.find_file('Dir::Etc::sourcelist')
 
         # read sources.list if it exists
         if os.path.isfile(self.default_file):
             self.load(self.default_file)
 
         # read sources.list.d
-        for file in glob.iglob('%s/*.list' % self._apt_cfg_dir('Dir::Etc::sourceparts')):
+        self.sources_dir = apt_pkg.config.find_dir('Dir::Etc::sourceparts')
+        for file in glob.iglob(f'{self.sources_dir}/*.list'):
             if os.path.islink(file):
                 self.files_mapping[file] = os.readlink(file)
             self.load(file)
@@ -255,7 +255,7 @@ class SourcesList(object):
         if '/' in filename:
             return filename
         else:
-            return os.path.abspath(os.path.join(self._apt_cfg_dir('Dir::Etc::sourceparts'), filename))
+            return os.path.abspath(os.path.join(self.sources_dir, filename))
 
     def _suggest_filename(self, line):
         def _cleanup_filename(s):
@@ -313,28 +313,6 @@ class SourcesList(object):
 
         return valid, enabled, source, comment
 
-    @staticmethod
-    def _apt_cfg_file(filespec):
-        """
-        Wrapper for `apt_pkg` module for running with Python 2.5
-        """
-        try:
-            result = apt_pkg.config.find_file(filespec)
-        except AttributeError:
-            result = apt_pkg.Config.FindFile(filespec)
-        return result
-
-    @staticmethod
-    def _apt_cfg_dir(dirspec):
-        """
-        Wrapper for `apt_pkg` module for running with Python 2.5
-        """
-        try:
-            result = apt_pkg.config.find_dir(dirspec)
-        except AttributeError:
-            result = apt_pkg.Config.FindDir(dirspec)
-        return result
-
     def load(self, file):
         group = []
         f = open(file, 'r')

ansible/modules/async_status.py

@@ -28,6 +28,8 @@ options:
     type: str
     choices: [ cleanup, status ]
     default: status
+notes:
+  - The RV(started) and RV(finished) return values were updated to return V(True) or V(False) instead of V(1) or V(0) in ansible-core 2.19.
 extends_documentation_fragment:
 - action_common_attributes
 - action_common_attributes.flow
@@ -85,15 +87,15 @@ ansible_job_id:
   type: str
   sample: '360874038559.4169'
 finished:
-  description: Whether the asynchronous job has finished (V(1)) or not (V(0))
+  description: Whether the asynchronous job has finished or not
   returned: always
-  type: int
-  sample: 1
+  type: bool
+  sample: true
 started:
-  description: Whether the asynchronous job has started (V(1)) or not (V(0))
+  description: Whether the asynchronous job has started or not
   returned: always
-  type: int
-  sample: 1
+  type: bool
+  sample: true
 stdout:
   description: Any output returned by async_wrapper
   returned: always
@@ -134,7 +136,7 @@ def main():
     log_path = os.path.join(async_dir, jid)
 
     if not os.path.exists(log_path):
-        module.fail_json(msg="could not find job", ansible_job_id=jid, started=1, finished=1)
+        module.fail_json(msg="could not find job", ansible_job_id=jid, started=True, finished=True)
 
     if mode == 'cleanup':
         os.unlink(log_path)
@@ -151,16 +153,16 @@ def main():
     except Exception:
         if not data:
             # file not written yet?  That means it is running
-            module.exit_json(results_file=log_path, ansible_job_id=jid, started=1, finished=0)
+            module.exit_json(results_file=log_path, ansible_job_id=jid, started=True, finished=False)
         else:
             module.fail_json(ansible_job_id=jid, results_file=log_path,
-                             msg="Could not parse job output: %s" % data, started=1, finished=1)
+                             msg="Could not parse job output: %s" % data, started=True, finished=True)
 
     if 'started' not in data:
-        data['finished'] = 1
+        data['finished'] = True
         data['ansible_job_id'] = jid
     elif 'finished' not in data:
-        data['finished'] = 0
+        data['finished'] = False
 
     # just write the module output directly to stdout and exit; bypass other processing done by exit_json since it's already been done
     print(f"\n{json.dumps(data)}")  # pylint: disable=ansible-bad-function