Skip to content

Commit 44eba4b

Browse files
ixxeL2097Frederic Spiers
andauthored
feat(nginx.conf): add option to set log level for nginx sidecar containers and proxy pods (#52)
Co-authored-by: Frederic Spiers <[email protected]>
1 parent 5c187fc commit 44eba4b

File tree

4 files changed

+6
-2
lines changed

4 files changed

+6
-2
lines changed

helm/ggbridge/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@ A Helm chart for installing ggbridge
7676
| proxy.affinity | object | `{}` | Affinity for pod assignment |
7777
| proxy.annotations | object | `{}` | Set proxy annotations |
7878
| proxy.labels | object | `{}` | Set proxy labels |
79+
| proxy.logLevel | string | `"notice"` | Set nginx sidecar container and proxy pod log level (default: notice) |
7980
| proxy.networkPolicy.allowExternal | bool | `true` | When true, server will accept connections from any source |
8081
| proxy.networkPolicy.enabled | bool | `true` | Specifies whether a NetworkPolicy should be created |
8182
| proxy.networkPolicy.extraEgress | list | `[]` | Add extra egress rules to the NetworkPolicy |

helm/ggbridge/files/nginx.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ load_module "/usr/lib/nginx/modules/ngx_stream_module.so";
1919

2020
worker_processes 1;
2121

22-
error_log stderr notice;
22+
error_log stderr {{ .Values.proxy.logLevel }};
2323
pid /var/run/nginx.pid;
2424

2525
events {

helm/ggbridge/files/proxy/nginx.conf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,13 @@
55

66
{{- $fullname := ternary (include "ggbridge.server.fullname" $context) (include "ggbridge.client.fullname" $context) (eq $context.Values.mode "server") -}}
77
{{- $ports := $context.Values.proxy.service.ports -}}
8+
{{- $logLevel := $context.Values.proxy.logLevel -}}
89

910
load_module "/usr/lib/nginx/modules/ngx_stream_module.so";
1011

1112
worker_processes 1;
1213

13-
error_log stderr notice;
14+
error_log stderr {{ $logLevel }};
1415
pid /var/run/nginx.pid;
1516

1617
events {

helm/ggbridge/values.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -379,6 +379,8 @@ server:
379379
annotations: {}
380380

381381
proxy:
382+
# -- Set nginx sidecar container and proxy pod log level (default: notice)
383+
logLevel: notice
382384
# -- Number of pods for each deployment
383385
replicaCount: 1
384386
updateStrategy:

0 commit comments

Comments
 (0)