Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for shareProcessNamespace to the helm chart #8279

Closed
thomasbruggink opened this issue Mar 1, 2022 · 1 comment · Fixed by #8287
Closed

Add support for shareProcessNamespace to the helm chart #8279

thomasbruggink opened this issue Mar 1, 2022 · 1 comment · Fixed by #8287
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.

Comments

@thomasbruggink
Copy link
Contributor

Hi,

I'm trying to setup a side-car to write the NGINX access log to an internal log processor and would like to signal the NGINX process that it needs to re-open the file. With yml this can be done by adding shareProcessNamespace: true which allows a log rotate side-car to signal NGINX to re-open the logs files.
It seems this option is missing in the helm chart deploymeny.

Is it possible to add this as a configuration option?

A similar issue was reported before but it went stale: #7108

@thomasbruggink thomasbruggink added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 1, 2022
@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Mar 1, 2022
@k8s-ci-robot
Copy link
Contributor

@thomasbruggink: This issue is currently awaiting triage.

If Ingress contributors determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@thomasbruggink thomasbruggink changed the title Add support shareProcessNamespace to the helm chart Add support for shareProcessNamespace to the helm chart Mar 1, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs-priority needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one.
Projects
None yet
2 participants