Fixed the netaddr error msg for clarity (#244)

* Fixed the netaddr error msg for clarity

* added changelog

* Used missing_required_lib

---------

Co-authored-by: Kate Case <kcase@redhat.com>
pull/254/head
Vinay M 2023-04-05 18:54:10 +05:30 committed by GitHub
parent 90b62d0b55
commit ed6555526d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 8 deletions

View File

@ -0,0 +1,3 @@
---
trivial:
- fixed netaddr error message to give better clarity that the package is missing

View File

@ -16,6 +16,7 @@ __metaclass__ = type
import types import types
from ansible.errors import AnsibleFilterError from ansible.errors import AnsibleFilterError
from ansible.module_utils.basic import missing_required_lib
from ansible.utils.display import Display from ansible.utils.display import Display
@ -598,10 +599,7 @@ def _need_netaddr(f_name, *args, **kwargs):
""" """
verify python's netaddr for these filters to work verify python's netaddr for these filters to work
""" """
raise AnsibleFilterError( raise AnsibleFilterError(missing_required_lib("netaddr"))
"The %s filter requires python's netaddr be "
"installed on the ansible controller" % f_name,
)
def _address_normalizer(value): def _address_normalizer(value):

View File

@ -87,7 +87,4 @@ def _validate_args(plugin, doc, params):
def _need_netaddr(f_name, *args, **kwargs): def _need_netaddr(f_name, *args, **kwargs):
raise errors.AnsibleFilterError( raise errors.AnsibleFilterError(missing_required_lib("netaddr"))
"The %s filter requires python's netaddr be "
"installed on the ansible controller" % f_name,
)