Merge branch 'fluentbit-decouple' into 'master'

Decouple Fluentbit from rest of logging stack

Closes #922

See merge request platform-one/big-bang/bigbang!1186

chart/templates/logging/elasticsearch-kibana/namespace.yaml

-{{- if and (not .Values.loki.enabled) (not .Values.promtail.enabled) (or .Values.logging.enabled .Values.clusterAuditor.enabled) }}
+{{- if and (not .Values.loki.enabled) (not .Values.promtail.enabled) (or .Values.logging.enabled .Values.fluentbit.enabled .Values.clusterAuditor.enabled) }}
 {{- /* Default to istio being turned on, but disable if user sets istio to disable in the custom passthrough values.
 We have to do it this way because ownership of "istio.enabled" is owned by the chart but also BigBang.  Sourcing values from the passthrough values also means
   we get to simplify the api space of BigBang just a little bit more.

chart/templates/logging/fluentbit/fluentbit-helmrelease.yaml

@@ -37,11 +37,14 @@ spec:
       kind: Secret
       valuesKey: "overlays"
 
-  {{/* ECK and Logging _always_ depend on .Values.logging being enabled, so can assume they exist here */}}
+  {{/* To support a "standalone" fluentbit for shipping to central location we conditionally depend on logging enabled */}}
+  {{- if or .Values.logging.enabled .Values.gatekeeper.enabled .Values.istio.enabled .Values.monitoring.enabled }}
   dependsOn:
+    {{- if .Values.logging.enabled }}
     - name: ek
       namespace: {{ .Release.Namespace }}
-    {{- if  .Values.gatekeeper.enabled }}
+    {{- end }}
+    {{- if .Values.gatekeeper.enabled }}
     - name: gatekeeper
       namespace: {{ .Release.Namespace }}
     {{- end }}
@@ -54,3 +57,4 @@ spec:
       namespace: {{ .Release.Namespace }}
     {{- end }}
   {{- end }}
+{{- end }}