Bugfix: set resources only for fluentd-gcp container.

There is more than one container in fluentd-gcp deployment. Previous implementation was setting resources for multiple containers, not just the fluent-gcp one.

Bugfix: set resources only for fluentd-gcp container.
b7935b96 · Daniel Kłobuszewski · f6cb2fce · b7935b96
Commit b7935b96 authored Aug 09, 2017 by Daniel Kłobuszewski
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

configure-helper.sh cluster/gce/gci/configure-helper.sh +1 -1

No files found.
--- a/cluster/gce/gci/configure-helper.sh
+++ b/cluster/gce/gci/configure-helper.sh
@@ -1606,7 +1606,7 @@ function wait-for-apiserver-and-update-fluentd {
  kubectl set resources --dry-run --local -f ${fluentd_gcp_yaml} \
    --limits=memory=${FLUENTD_GCP_MEMORY_LIMIT} \
    --requests=cpu=${FLUENTD_GCP_CPU_REQUEST},memory=${FLUENTD_GCP_MEMORY_REQUEST} \
-    -o yaml > ${fluentd_gcp_yaml}.tmp
+    --containers=fluentd-gcp -o yaml > ${fluentd_gcp_yaml}.tmp
  mv ${fluentd_gcp_yaml}.tmp ${fluentd_gcp_yaml}
 }