Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix exception in unixy callback v2_runner_retry #58801

Merged
merged 1 commit into from
Jul 13, 2019
Merged

Conversation

jpic
Copy link
Contributor

@jpic jpic commented Jul 7, 2019

This fixes the warning:

[WARNING]: Failure using method (v2_runner_retry) in callback plugin
(<ansible.plugins.callback.unixymdv.CallbackModule object at 0x7fb4fd4eeef0>): _run_is_verbose() got
an unexpected keyword argument 'verbosity'
SUMMARY

Fixes a little exception, probably on code that needed update

ISSUE TYPE
  • Bugfix Pull Reques
COMPONENT NAME

unixy callback plugin

ADDITIONAL INFORMATION
- hosts: '*'
  tasks:
  - pause:
      prompt:
    until: false

@ansibot
Copy link
Contributor

ansibot commented Jul 7, 2019

cc @akatch
click here for bot help

@ansibot ansibot added affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. community_review In order to be merged, this PR must follow the community review workflow. needs_triage Needs a first human triage before being processed. small_patch support:community This issue/PR relates to code supported by the Ansible community. labels Jul 7, 2019
@@ -236,6 +236,6 @@ def v2_playbook_on_start(self, playbook):

def v2_runner_retry(self, result):
msg = " Retrying... (%d of %d)" % (result._result['attempts'], result._result['retries'])
if self._run_is_verbose(result, verbosity=2):
if self._display.verbosity > 1:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if self._display.verbosity > 1:
if self._run_is_verbose(result):

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree with this suggestion, thank you for submitting the patch @jpic

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done !
Note that line 227 there is self._display.verbosity > 3
https://github.com/ansible/ansible/pull/58801/files#diff-bfc0fb65ff63d5ae485526877e282f97R227
Have a great day

@ansibot ansibot removed the needs_triage Needs a first human triage before being processed. label Jul 8, 2019
This fixes the warning:
[WARNING]: Failure using method (v2_runner_retry) in callback plugin
(<ansible.plugins.callback.unixymdv.CallbackModule object at 0x7fb4fd4eeef0>): _run_is_verbose() got
an unexpected keyword argument 'verbosity'
Copy link
Member

@Akasurde Akasurde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shipit

@akatch
Copy link
Contributor

akatch commented Jul 12, 2019 via email

@ansibot ansibot added shipit This PR is ready to be merged by Core and removed community_review In order to be merged, this PR must follow the community review workflow. labels Jul 12, 2019
@Akasurde Akasurde merged commit 945e35a into ansible:devel Jul 13, 2019
@ansible ansible locked and limited conversation to collaborators Aug 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. shipit This PR is ready to be merged by Core small_patch support:community This issue/PR relates to code supported by the Ansible community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants