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

Implement display_ok_hosts and display_skipped_hosts for unixy #53179

Open
wants to merge 3 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@akatch
Copy link
Contributor

akatch commented Mar 1, 2019

SUMMARY

Implements options display_ok_hosts and display_skipped_hosts like the default callback plugin.
Also implements show_custom_stats and display_failed_stderr.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

unixy callback plugin

ADDITIONAL INFORMATION
# Default output
Executing playbook test.yml

- localhost on hosts: localhost -
Gathering Facts...
  localhost ok
command...
  localhost skipped | msg: skipped, running in check mode
  localhost skipped | msg: skipped, running in check mode
  localhost skipped | msg: skipped, running in check mode
  localhost skipped | msg: skipped, running in check mode
  localhost skipped
command...
  localhost skipped

- Play recap -
  localhost                  : ok=1    changed=0    unreachable=0    failed=0    rescued=0    ignored=0

# display_ok_hosts=false, display_skipped_hosts=false
Executing playbook test.yml

- localhost on hosts: localhost -
Gathering Facts...
command...
command...

- Play recap -
  localhost                  : ok=1    changed=0    unreachable=0    failed=0    rescued=0    ignored=0
ansible-playbook --diff test.yml --check  2.11s user 0.36s system 97% cpu 2.523 total
@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Mar 1, 2019

The test ansible-test sanity --test validate-modules [explain] failed with 1 error:

test/sanity/validate-modules/ignore.txt:164:1: A102 Remove since "lib/ansible/modules/cloud/kubevirt/kubevirt_cdi_upload.py" passes "E203" test

click here for bot help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.