Merge pull request kubernetes#37941 from Crassirostris/fluentd-gcp-config-unification

Automatic merge from submit-queue (batch tested with PRs 37692, 37785, 37647, 37941, 37856)

Use unified gcp fluentd image for gci and cvm

Follow-up of kubernetes#37681

Actually unify the pod specs for CVM and GCI, to simplify the configuration

CC @piosz

Author: Kubernetes Submit Queue

build-tools/lib/release.sh

@@ -317,7 +317,6 @@ function kube::release::package_kube_manifests_tarball() {
   cp "${salt_dir}/cluster-autoscaler/cluster-autoscaler.manifest" "${dst_dir}/"
   cp "${salt_dir}/fluentd-es/fluentd-es.yaml" "${release_stage}/"
   cp "${salt_dir}/fluentd-gcp/fluentd-gcp.yaml" "${release_stage}/"
-  cp "${salt_dir}/fluentd-gcp-gci/fluentd-gcp-gci.yaml" "${release_stage}/"
   cp "${salt_dir}/kube-registry-proxy/kube-registry-proxy.yaml" "${release_stage}/"
   cp "${salt_dir}/kube-proxy/kube-proxy.manifest" "${release_stage}/"
   cp "${salt_dir}/etcd/etcd.manifest" "${dst_dir}"

cluster/gce/gci/configure-helper.sh

@@ -1125,7 +1125,7 @@ function start-fluentd {
   echo "Start fluentd pod"
   if [[ "${ENABLE_NODE_LOGGING:-}" == "true" ]]; then
     if [[ "${LOGGING_DESTINATION:-}" == "gcp" ]]; then
-      cp "${KUBE_HOME}/kube-manifests/kubernetes/fluentd-gcp-gci.yaml" /etc/kubernetes/manifests/
+      cp "${KUBE_HOME}/kube-manifests/kubernetes/fluentd-gcp.yaml" /etc/kubernetes/manifests/
     elif [[ "${LOGGING_DESTINATION:-}" == "elasticsearch" && "${KUBERNETES_MASTER:-}" != "true" ]]; then
       # Running fluentd-es on the master is pointless, as it can't communicate
       # with elasticsearch from there in the default configuration.

cluster/saltbase/salt/fluentd-gcp-gci/fluentd-gcp-gci.yaml

@@ -11,7 +11,7 @@ spec:
   dnsPolicy: Default
   containers:
   - name: fluentd-cloud-logging
-    image: gcr.io/google_containers/fluentd-gcp:1.28
+    image: gcr.io/google_containers/fluentd-gcp:1.29
     resources:
       limits:
         memory: 200Mi
@@ -23,20 +23,14 @@ spec:
     env:
     - name: FLUENTD_ARGS
       value: -q
-    # Jemalloc is a widely used way to decrease memory consumption
-    # in Ruby world. It's a better implementation of malloc(3).
-    - name: "LD_PRELOAD"
-      value: "/opt/td-agent/embedded/lib/libjemalloc.so"
-    # This is quite hacky, but forces Ruby GC to be ivoked more often
-    # resulting in lower memory consumption, which is important for us.
-    - name: "RUBY_GC_HEAP_OLDOBJECT_LIMIT_FACTOR"
-      value: "0.9"
     volumeMounts:
     - name: varlog
       mountPath: /var/log
     - name: varlibdockercontainers
       mountPath: /var/lib/docker/containers
       readOnly: true
+    - name: libsystemddir
+      mountPath: /host/lib
   terminationGracePeriodSeconds: 30
   volumes:
   - name: varlog
@@ -45,4 +39,6 @@ spec:
   - name: varlibdockercontainers
     hostPath:
       path: /var/lib/docker/containers
-
+  - name: libsystemddir
+    hostPath:
+      path: /usr/lib64