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

Add additional volume for Kibana logs when hardened security context is enabled #8380

Merged
merged 3 commits into from
Dec 27, 2024

Conversation

naemono
Copy link
Contributor

@naemono naemono commented Dec 27, 2024

related: #8378

This adds an additional volume to the Kibana pod when the hardened security context is enabled as readOnlyRootFilesystem is set to true.

@naemono naemono added the >bug Something isn't working label Dec 27, 2024
pkg/controller/kibana/pod.go Outdated Show resolved Hide resolved
@pebrc
Copy link
Collaborator

pebrc commented Dec 27, 2024

buildkite test this -f p=gke,E2E_TAGS=kb

Signed-off-by: Michael Montgomery <[email protected]>
pkg/controller/kibana/pod.go Outdated Show resolved Hide resolved
@naemono naemono merged commit 186d4fb into elastic:main Dec 27, 2024
5 checks passed
@naemono naemono deleted the add-logs-emptydir-volume-mount-kibana branch December 27, 2024 19:51
@naemono
Copy link
Contributor Author

naemono commented Jan 15, 2025

💚 All backports created successfully

Status Branch Result
2.16.1-wip

Questions ?

Please refer to the Backport tool documentation

naemono added a commit to naemono/cloud-on-k8s that referenced this pull request Jan 15, 2025
…is enabled (elastic#8380)

* Add additional volume for Kibana logs when hardened security context is enabled.

Co-authored-by: Peter Brachwitz <[email protected]>

---------

Signed-off-by: Michael Montgomery <[email protected]>
Co-authored-by: Peter Brachwitz <[email protected]>
(cherry picked from commit 186d4fb)
naemono added a commit that referenced this pull request Jan 15, 2025
…is enabled (#8380) (#8408)

* Add additional volume for Kibana logs when hardened security context is enabled.

Co-authored-by: Peter Brachwitz <[email protected]>

---------

Signed-off-by: Michael Montgomery <[email protected]>
Co-authored-by: Peter Brachwitz <[email protected]>
(cherry picked from commit 186d4fb)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants