Skip to content

feat(template): Write operator file logs to an emptyDir under /var/log - #656

Merged
lfrancke merged 2 commits into
mainfrom
push-ozpklznmrroq
Oct 7, 2026
Merged

lfrancke merged 2 commits into
mainfrom
push-ozpklznmrroq

Conversation

@lfrancke

@lfrancke lfrancke commented Oct 6, 2026

Copy link
Copy Markdown
Member

This makes it possible for us to switch to a read-only root filesystem. And it causes the logs to survive a container restart.

@lfrancke lfrancke self-assigned this Oct 6, 2026
This makes it possible for us to switch to a read-only root filesystem.
And it causes the logs to survive a container restart.
sbernauer
sbernauer previously approved these changes Oct 6, 2026

@sbernauer sbernauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM % Nicks comments

Co-authored-by: Nick <10092581+NickLarsenNZ@users.noreply.github.com>
@sbernauer

sbernauer commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

I would love to see us doing the same for the product pods :)

@NickLarsenNZ NickLarsenNZ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@lfrancke
lfrancke added this pull request to the merge queue Oct 7, 2026
Merged via the queue into main with commit 4518f63 Oct 7, 2026
2 checks passed
@lfrancke
lfrancke deleted the push-ozpklznmrroq branch October 7, 2026 15:21
@lfrancke

lfrancke commented Oct 9, 2026

Copy link
Copy Markdown
Member Author

reminder for myself: this needs release notes

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

Projects

Development

Successfully merging this pull request may close these issues.

3 participants