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

Send traefik logs to stdout instead stderr #2054

Merged
merged 1 commit into from
Sep 6, 2017

Conversation

m3co-code
Copy link
Contributor

The default setting in logrus is to send the logs to stderr. The documentation states already that the output should be sent to stdout.

@m3co-code
Copy link
Contributor Author

@ldez should we also pick this for 1.4 in case design review is successful?

@ldez
Copy link
Contributor

ldez commented Sep 5, 2017

@marco-jantke yes and you can do this now 😃

@ldez ldez added this to the 1.4 milestone Sep 5, 2017
@m3co-code m3co-code changed the base branch from master to v1.4 September 5, 2017 22:49
@m3co-code
Copy link
Contributor Author

Ok, done.

Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ldez ldez changed the title send traefik logs to stdout Send traefik logs to stdout Sep 6, 2017
The default setting in logrus is to send the logs to stderr. The
documentaion states already that the output should be sent to stdout.
@traefiker traefiker merged commit 5157a6a into traefik:v1.4 Sep 6, 2017
@m3co-code m3co-code deleted the traefik-logs-to-stdout branch September 6, 2017 10:28
@ldez ldez added the kind/enhancement a new or improved feature. label Sep 8, 2017
@ldez ldez changed the title Send traefik logs to stdout Send traefik logs to stdout instead stderr Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logs kind/enhancement a new or improved feature. size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants