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

Web UI: Renamed "Tail -f" to stdout and added "stderr" to get tail -f stderr #1173

Closed
wants to merge 1 commit into from

Conversation

ouroboroscoding
Copy link

I have been using supervisor for years and it has been forever frustrating that there is only the "Tail -f" option when often stderr is the thing I really want to see, in my fork I have replaced "Tail -f" with "stdout" and added an equivalent "stderr" which calls "'logtail/%s/stderr' % processname". I would be forever grateful if you would consider making this change to supervisor.

@mnaberez mnaberez added the web label Nov 18, 2018
@mnaberez mnaberez changed the title Renamed "Tail -f" to stdout and added "stderr" to get tail -f stderr Web UI: Renamed "Tail -f" to stdout and added "stderr" to get tail -f stderr Nov 18, 2018
@mnaberez
Copy link
Member

Related: #254

@jouve
Copy link

jouve commented Jun 26, 2019

@mnaberez what is blocking this PR ?

@djchou
Copy link

djchou commented Apr 27, 2020

Please merge this -- its useful.

@mnaberez mnaberez closed this in a88a193 Apr 27, 2020
@ouroboroscoding
Copy link
Author

Can you at least explain why this was rejected? It's clearly helpful to some people and takes nothing away from the existing software.

@mnaberez
Copy link
Member

Can you at least explain why this was rejected? It's clearly helpful to some people and takes nothing away from the existing software.

Please see the link above to a88a193. I didn't reject it, I just did it slightly differently. This patch adds a new stderr link but it also removes tail -f from the link text. I didn't want to remove tail -f so that it remains clear to users that the function is tail -f. I credited you for the original patch in the changelog. Thank you.

@ouroboroscoding
Copy link
Author

Ah, ok, I missed that. Thanks so much, this is super helpful :)

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

Successfully merging this pull request may close these issues.

4 participants