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
Adding a switch to output docker logs #40693
Adding a switch to output docker logs #40693
Conversation
This would be great! |
The test
|
The test
|
ready_for_review |
@mkrizek I have tried using ready_for_review on this PR. Thanks |
@@ -2022,6 +2034,7 @@ def main(): | |||
oom_killer=dict(type='bool'), | |||
oom_score_adj=dict(type='int'), | |||
paused=dict(type='bool', default=False), | |||
output_logs=dict(type='bool', default=False), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move this before paused to make options alphabetically ordered?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kassiansun Updated and ready_for_review again.
ready_for_review |
* Adding a switch to print logs * Update to docs * Adding `version_added` for the new option * Updating version added * Moving `output_logs` before `paued` for alphabetically ordering.
SUMMARY
Adding a switch to output docker logs optionally if selected to do so.
ISSUE TYPE
Currently we are only able to see the logs of a container if the
command
,entrypoint
task fails.However it's worthwhile having the ability to view these logs optionally regardless.