feat: added sidecar to turn logs into metrics and forward remaining logs

[CptSchnitz]

No description provided.

[shimoncohen]

The nested if/else here is hard to read, suggested two options.

Option 1 — split the whole block by the outer condition. Each rendered result is visible verbatim; no directive value straddles a template boundary.

{{- if .Values.fluentbit.enabled }}
    access_log  /var/log/nginx/access.log  {{ .Values.fluentbit.accessLog.stdoutReadable | ternary "readable" "main" }};
    access_log  syslog:server=127.0.0.1:{{ .Values.fluentbit.accessLog.syslogPort }} main;
{{- else }}
    access_log  /var/log/nginx/access.log  main;
{{- end }}

Cost: the file access_log ... main; literal is duplicated across both branches — a 1-line dupe in exchange for a readable conditional.

Option 2 — name the format, keep one file directive. Zero duplication; the format decision is a named variable up top instead of inline nesting.

{{- $stdoutFormat := "main" }}
{{- if and .Values.fluentbit.enabled .Values.fluentbit.accessLog.stdoutReadable }}
{{- $stdoutFormat = "readable" }}
{{- end }}
    access_log  /var/log/nginx/access.log  {{ $stdoutFormat }};
{{- if .Values.fluentbit.enabled }}
    access_log  syslog:server=127.0.0.1:{{ .Values.fluentbit.accessLog.syslogPort }} main;
{{- end }}

Both read top-to-bottom without straddling template boundaries. Option 1 is a touch clearer at the cost of one duplicated literal; Option 2 avoids the dupe with one extra variable.