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.

Sign up for GitHub

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

Jump to bottom

Improve error handling #2

Closed
tmullender opened this issue Oct 26, 2014 · 1 comment
Closed

Improve error handling #2

tmullender opened this issue Oct 26, 2014 · 1 comment
Assignees
@tmullender
Labels
enhancement
Milestone
1.1.0

Comments

@tmullender
Copy link
Owner

The plugin should

  • log standard error output at warning level
  • have an option to fail the build when the command returns a non zero exit code

so that users can more easily debug failing executions and avoid subsequent phases being executed
@tmullender tmullender added this to the 1.1.0 milestone Oct 26, 2014
tmullender added a commit that referenced this issue Oct 27, 2014
@tmullender
Issue #2 Logging the commands error stream at warning level and optio…
511588a 
…nally throwing a failure exception if the exit status is not zero
tmullender added a commit that referenced this issue Oct 27, 2014
@tmullender
Issue #2 Adding tests for error handling use case
5627a9e
@tmullender tmullender self-assigned this Oct 27, 2014
tmullender added a commit that referenced this issue Oct 30, 2014
@tmullender
Issue #2 Adding option to log command output to a file
f2af8f7
tmullender added a commit that referenced this issue Oct 30, 2014
@tmullender
Issue #2 Adding tests for option to log command output to a file
e0eff52
@tmullender
Copy link
Owner Author

Removed requirement to log output to file in favour of using ansible's tree option

tmullender added a commit that referenced this issue Oct 31, 2014
@tmullender
Issue #2 Removing logDirectory option in favour of ansible's tree option
f4466ce
tmullender added a commit that referenced this issue Dec 2, 2014
@tmullender
Issue #2 Reintroducing logging option as ansible-playbook does not ha…
6935bdd 
…ve tree option
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmullender tmullender

Labels
enhancement
Projects
None yet
Milestone
1.1.0
Development

No branches or pull requests
1 participant
@tmullender