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

Docker images should redirect error log message to stderr (not stdout) #982

Closed
lcrilly opened this issue Oct 24, 2023 · 0 comments · Fixed by #1128
Closed

Docker images should redirect error log message to stderr (not stdout) #982

lcrilly opened this issue Oct 24, 2023 · 0 comments · Fixed by #1128
Assignees
@thresheek
Labels
z-Difficulty: ⭐️ Difficulty: EASY. Effects an isolated area of the codebase. z-enhancement ⬆️ Product Enhancement z-packages z-roadmap
Milestone
1.32

Comments

@lcrilly
Copy link
Contributor

lcrilly commented Oct 24, 2023

To facilitate effective log streaming from Unit containers it would be better if the Unit log file was symlinked to /dev/stderr instead of /dev/stdout.

This leaves space for stdout to be used for access logging.

Originated in #973
@lcrilly lcrilly added z-enhancement ⬆️ Product Enhancement z-packages z-Difficulty: ⭐️ Difficulty: EASY. Effects an isolated area of the codebase. labels Oct 24, 2023
danielledeleo added a commit to danielledeleo/unit that referenced this issue Dec 5, 2023
@danielledeleo
Go: Use Homebrew include paths
15e1667 
Fixes nginx#982
@danielledeleo danielledeleo mentioned this issue Dec 5, 2023
Merged
@tippexs tippexs added this to the 1.32 milestone Jan 19, 2024
thresheek added a commit to thresheek/unit that referenced this issue Feb 14, 2024
@thresheek
Docker: redirect logs to stderr.
4a1734c 
This allows to use /dev/stdout as application logs if users choose to do
so.

Fixes nginx#982
@thresheek thresheek mentioned this issue Feb 14, 2024
Merged
thresheek added a commit to thresheek/unit that referenced this issue Feb 16, 2024
@thresheek
Docker: redirect logs to stderr
c14b806 
This allows to use /dev/stdout as application logs if users choose to do
so.

Closes: nginx#982
callahad pushed a commit that referenced this issue Feb 20, 2024
@thresheek @callahad
Docker: redirect logs to stderr
183a1e9 
This allows to use /dev/stdout as application logs if users choose to do
so.

Closes: #982
andrey-zelenkov pushed a commit that referenced this issue Feb 27, 2024
@thresheek @andrey-zelenkov
Docker: redirect logs to stderr
cda2c3f 
This allows to use /dev/stdout as application logs if users choose to do
so.

Closes: #982
pkillarjun pushed a commit to pkillarjun/unit that referenced this issue May 29, 2024
@thresheek @callahad
Docker: redirect logs to stderr
ff69549 
This allows to use /dev/stdout as application logs if users choose to do
so.

Closes: nginx#982
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thresheek thresheek

Labels
z-Difficulty: ⭐️ Difficulty: EASY. Effects an isolated area of the codebase. z-enhancement ⬆️ Product Enhancement z-packages z-roadmap
Projects
NGINX Unit Planning
Status: 🚀 Released
Milestone
1.32
Development

Successfully merging a pull request may close this issue.

Docker: redirect logs to stderr thresheek/unit
3 participants
@thresheek @tippexs @lcrilly