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.
Comment thread template/deploy/helm/[[operator]]/templates/deployment.yaml.j2
Comment thread template/deploy/helm/[[operator]]/templates/_telemetry.tpl.j2 Outdated
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

3 participants