Skip to content

Commit 74a3b77

Browse files
author
Mik Vyatskov
committed
Use unified gcp fluentd image for gci and cvm
1 parent aef15aa commit 74a3b77

File tree

4 files changed

+7
-74
lines changed

4 files changed

+7
-74
lines changed

build-tools/lib/release.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,6 @@ function kube::release::package_kube_manifests_tarball() {
317317
cp "${salt_dir}/cluster-autoscaler/cluster-autoscaler.manifest" "${dst_dir}/"
318318
cp "${salt_dir}/fluentd-es/fluentd-es.yaml" "${release_stage}/"
319319
cp "${salt_dir}/fluentd-gcp/fluentd-gcp.yaml" "${release_stage}/"
320-
cp "${salt_dir}/fluentd-gcp-gci/fluentd-gcp-gci.yaml" "${release_stage}/"
321320
cp "${salt_dir}/kube-registry-proxy/kube-registry-proxy.yaml" "${release_stage}/"
322321
cp "${salt_dir}/kube-proxy/kube-proxy.manifest" "${release_stage}/"
323322
cp "${salt_dir}/etcd/etcd.manifest" "${dst_dir}"

cluster/gce/gci/configure-helper.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1109,7 +1109,7 @@ function start-fluentd {
11091109
echo "Start fluentd pod"
11101110
if [[ "${ENABLE_NODE_LOGGING:-}" == "true" ]]; then
11111111
if [[ "${LOGGING_DESTINATION:-}" == "gcp" ]]; then
1112-
cp "${KUBE_HOME}/kube-manifests/kubernetes/fluentd-gcp-gci.yaml" /etc/kubernetes/manifests/
1112+
cp "${KUBE_HOME}/kube-manifests/kubernetes/fluentd-gcp.yaml" /etc/kubernetes/manifests/
11131113
elif [[ "${LOGGING_DESTINATION:-}" == "elasticsearch" && "${KUBERNETES_MASTER:-}" != "true" ]]; then
11141114
# Running fluentd-es on the master is pointless, as it can't communicate
11151115
# with elasticsearch from there in the default configuration.

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

Lines changed: 0 additions & 62 deletions
This file was deleted.

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

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ spec:
1111
dnsPolicy: Default
1212
containers:
1313
- name: fluentd-cloud-logging
14-
image: gcr.io/google_containers/fluentd-gcp:1.28
14+
image: gcr.io/google_containers/fluentd-gcp:1.29
1515
resources:
1616
limits:
1717
memory: 200Mi
@@ -23,20 +23,14 @@ spec:
2323
env:
2424
- name: FLUENTD_ARGS
2525
value: -q
26-
# Jemalloc is a widely used way to decrease memory consumption
27-
# in Ruby world. It's a better implementation of malloc(3).
28-
- name: "LD_PRELOAD"
29-
value: "/opt/td-agent/embedded/lib/libjemalloc.so"
30-
# This is quite hacky, but forces Ruby GC to be ivoked more often
31-
# resulting in lower memory consumption, which is important for us.
32-
- name: "RUBY_GC_HEAP_OLDOBJECT_LIMIT_FACTOR"
33-
value: "0.9"
3426
volumeMounts:
3527
- name: varlog
3628
mountPath: /var/log
3729
- name: varlibdockercontainers
3830
mountPath: /var/lib/docker/containers
3931
readOnly: true
32+
- name: libsystemddir
33+
mountPath: /host/lib
4034
terminationGracePeriodSeconds: 30
4135
volumes:
4236
- name: varlog
@@ -45,4 +39,6 @@ spec:
4539
- name: varlibdockercontainers
4640
hostPath:
4741
path: /var/lib/docker/containers
48-
42+
- name: libsystemddir
43+
hostPath:
44+
path: /usr/lib64

0 commit comments

Comments
 (0)