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

Wave 0.3.0 tries to write logs to /tmp #52

Closed
voidengineer opened this issue May 13, 2019 · 4 comments · Fixed by #150
Closed

Wave 0.3.0 tries to write logs to /tmp #52

voidengineer opened this issue May 13, 2019 · 4 comments · Fixed by #150
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@voidengineer
Copy link

I recently tried to update to Wave 0.3.0.
Since I run all pods with a restricted PodSecurityPolicy with a read-only root filesystem, Wave fails with the message:

cannot create log: open /tmp/wave.wave-1.nobody.log.INFO.*****: read-only file system

With Wave 0.2.0 this did not happen.

@JoelSpeed JoelSpeed added the bug Something isn't working label May 13, 2019
@JoelSpeed
Copy link
Collaborator

@voidengineer Could you please share your configuration for Wave?

No changes have been made to the logging in Wave between v0.2.0 and v0.3.0 that I'm aware of. The only thing that may have introduced this change is an update to the controller-runtime dependency or changes to the Dockerfile, though I think the latter is less likely

@JoelSpeed JoelSpeed added Hacktoberfest help wanted Extra attention is needed labels Oct 1, 2019
@github-actions
Copy link

This issue has been inactive for 60 days. If the issue is still relevant please comment to re-activate the issue. If no action is taken within 7 days, the issue will be marked closed.

@joschi36
Copy link

I noticed another problem. With long-running pods, the node filesystem gets full. For instance, a wave pod running for 34 days produced 18 GB of Logs. The amount of logs was because of an issue on our side, but I think it should never log to the container FS.

@toelke toelke removed the Stale label Feb 12, 2024
@toelke toelke reopened this Feb 12, 2024
@jabdoa2
Copy link
Contributor

jabdoa2 commented Apr 27, 2024

I can actually confirm that this is still the case. It seems to contain redundant logs. I will have a look at the logger setup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants