Skip to content

Commit

Permalink
Merge pull request #48139 from crassirostris/fix-fluentd-config
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue (batch tested with PRs 48139, 48042, 47645, 48054, 48003)

Fix fluentd-gcp configuration to facilitate JSON parsing

There's a bug in #45734, because of which each records gets additional field and google-cloud plugin thinks it's not JSON (https://github.com/GoogleCloudPlatform/fluent-plugin-google-cloud/blob/master/lib/fluent/plugin/out_google_cloud.rb#L569)

Fixes #48108

/cc @piosz @fgrzadkowski
  • Loading branch information
Kubernetes Submit Queue committed Jun 27, 2017
2 parents 9120d58 + b6a0e44 commit f1b58f4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions cluster/addons/fluentd-gcp/fluentd-gcp-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@ data:
# This filter is used to convert process start timestamp to integer
# value for correct ingestion in the prometheus output plugin.
<filter>
<filter process_start>
@type record_transformer
enable_ruby true
auto_typecast true
Expand Down Expand Up @@ -415,7 +415,7 @@ data:
</store>
</match>
metadata:
name: fluentd-gcp-config-v1.0
name: fluentd-gcp-config-v1.1
namespace: kube-system
labels:
addonmanager.kubernetes.io/mode: Reconcile
2 changes: 1 addition & 1 deletion cluster/addons/fluentd-gcp/fluentd-gcp-ds.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ spec:
path: /usr/lib64
- name: config-volume
configMap:
name: fluentd-gcp-config-v1.0
name: fluentd-gcp-config-v1.1
- name: ssl-certs
hostPath:
path: /etc/ssl/certs

0 comments on commit f1b58f4

Please sign in to comment.