Skip to content

Commit a998423

Browse files
committed
Fix debug log bug
Issue violations as discrete array in returned error Add check for verbosity or debug to SDK warnings handler Signed-off-by: Webster Mudge <wmudge@cloudera.com>
1 parent d673de2 commit a998423

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

plugins/module_utils/cdp_common.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ def process_debug(cls, f):
4141
def _impl(self, *args, **kwargs):
4242
result = f(self, *args, **kwargs)
4343
if self.debug:
44-
self.log_out = self.cdpy.get_log()
44+
self.log_out = self.cdpy.sdk.get_log()
4545
self.log_lines.append(self.log_out.splitlines())
4646
return result
4747

@@ -73,11 +73,12 @@ def _get_param(self, param, default=None):
7373

7474
def _cdp_module_throw_error(self, error: 'CdpError'):
7575
"""Error handler for CDPy SDK"""
76-
self.module.fail_json(msg=str(error.message), error=str(error.__dict__))
76+
self.module.fail_json(msg=str(error.message), error=str(error.__dict__), violations=error.violations)
7777

7878
def _cdp_module_throw_warning(self, warning: 'CdpWarning'):
7979
"""Warning handler for CDPy SDK"""
80-
self.module.warn(warning.message)
80+
if self.module._debug or self.module._verbosity >= 2:
81+
self.module.warn(warning.message)
8182

8283
@staticmethod
8384
def argument_spec(**spec):

0 commit comments

Comments
 (0)