Improve winrm import error message (#20267)
The requests python module is needed, however it is not a dependency of the python-winrm package. The python-winrm package does require python-requests_ntlm, which does not seem to pull python-requests. So for the time being (until Red Hat fixes their package) give a more informative error message.
This commit is contained in:
parent
70336c63c9
commit
cfb1f72d1c
1 changed files with 5 additions and 2 deletions
|
@ -41,6 +41,7 @@ from ansible.compat.six.moves.urllib.parse import urlunsplit
|
||||||
from ansible.errors import AnsibleError, AnsibleConnectionFailure
|
from ansible.errors import AnsibleError, AnsibleConnectionFailure
|
||||||
from ansible.errors import AnsibleFileNotFound
|
from ansible.errors import AnsibleFileNotFound
|
||||||
from ansible.module_utils._text import to_bytes, to_native, to_text
|
from ansible.module_utils._text import to_bytes, to_native, to_text
|
||||||
|
from ansible.module_utils.pycompat24 import get_exception
|
||||||
from ansible.plugins.connection import ConnectionBase
|
from ansible.plugins.connection import ConnectionBase
|
||||||
from ansible.plugins.shell.powershell import exec_wrapper, become_wrapper, leaf_exec
|
from ansible.plugins.shell.powershell import exec_wrapper, become_wrapper, leaf_exec
|
||||||
from ansible.utils.hashing import secure_hash
|
from ansible.utils.hashing import secure_hash
|
||||||
|
@ -51,12 +52,14 @@ try:
|
||||||
from winrm import Response
|
from winrm import Response
|
||||||
from winrm.protocol import Protocol
|
from winrm.protocol import Protocol
|
||||||
except ImportError:
|
except ImportError:
|
||||||
raise AnsibleError("winrm is not installed")
|
e = get_exception()
|
||||||
|
raise AnsibleError("winrm or requests is not installed: %s" % str(e))
|
||||||
|
|
||||||
try:
|
try:
|
||||||
import xmltodict
|
import xmltodict
|
||||||
except ImportError:
|
except ImportError:
|
||||||
raise AnsibleError("xmltodict is not installed")
|
e = get_exception()
|
||||||
|
raise AnsibleError("xmltodict is not installed: %s" % str(e))
|
||||||
|
|
||||||
try:
|
try:
|
||||||
from __main__ import display
|
from __main__ import display
|
||||||
|
|
Loading…
Reference in a new issue