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

Fixes #TBFL - Allow viewing of the entire output at once #92

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

adamruzicka
Copy link
Contributor

Prompted by community.theforeman.org/t/output-of-the-job-execution-is-inconsistent

For before see screenshots in #91

Under the hood, all the outputs are logged to all the hosts, but they are treated as debug if they don't really belong to the host in question. Then in the webui, these can be filtered out.

I'm opening this as a draft for now until we figure out if this is something we want as this is a best effort solution at best.

Pros:

  • provides additional context
  • provides additional visibility into how things are actually executed and which things run together

Cons:

  • the output gets stored in multiple copies on both the proxy and in foreman
  • the wider output is shown by default, this could be addressed with some ansible-specific changes in foreman_ansible
  • the use of debug paints the lines red, which is fixable, but not here

With debug:
image

With debug filtered out:
image

@adamruzicka
Copy link
Contributor Author

This also seems to make the output more easily digestible when different execution strategies (think serial: 1 or strategy: free) are used in the playbook

With serial: 1

image

vs
image

Random wait playbook with strategy: free

image

vs
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant