-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
feat(cmp): Print stderr output from command even on success #15921 #15973
Conversation
c7a0ac8
to
d595cf2
Compare
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #15973 +/- ##
==========================================
- Coverage 49.67% 49.66% -0.01%
==========================================
Files 267 267
Lines 46383 46387 +4
==========================================
+ Hits 23039 23040 +1
- Misses 21084 21086 +2
- Partials 2260 2261 +1
☔ View full report in Codecov by Sentry. |
Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
d595cf2
to
9682aea
Compare
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.
Thanks, @peschmae!
I think this is a reasonable/safe cherry pick for 2.9. |
/cherry-pick release-2.9 |
…15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
/cherry-pick release-2.8 |
Cherry-pick failed with |
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> Signed-off-by: Thomas Sands <thomas.sands@ly.st>
…15973) (#16124) * feat(cmp): Print stderr output from command even on success * docs(cmp): Document logging from cmp sidecard for development purposes --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> Signed-off-by: Thomas Sands <thomas.sands@ly.st> Co-authored-by: Mathias Petermann <mathias.petermann@gmail.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> Signed-off-by: jmilic1 <70441727+jmilic1@users.noreply.github.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
…#15921) (argoproj#15973) * feat(cmp): Print stderr output from command even on success Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> * docs(cmp): Document logging from cmp sidecard for development purposes Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com> --------- Signed-off-by: Mathias Petermann <mathias.petermann@gmail.com>
Always log stderr from command execution in cmp sidecar.
If the command executed succesfully, log everything written to stderr to the sidecard in Info level.
This makes developing and debugging plugins easier.
Fixes #15921
Checklist: