Skip to content

Commit

Permalink
Fix fluentd logging configuration
Browse files Browse the repository at this point in the history
Note: this also required updating permissions in the role for our cluster nodes in the IAM console.
  • Loading branch information
Mr0grog committed Feb 16, 2023
1 parent 0d1670c commit bdf0342
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions kubernetes/kube-system/fluentd/fluentd.cloudwatch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ metadata:
namespace: kube-system

---
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
name: fluentd
Expand All @@ -45,7 +45,7 @@ rules:

---
kind: ClusterRoleBinding
apiVersion: rbac.authorization.k8s.io/v1beta1
apiVersion: rbac.authorization.k8s.io/v1
metadata:
name: fluentd
roleRef:
Expand Down Expand Up @@ -100,6 +100,14 @@ spec:
# EDGI: TODO: are these still needed? Not sure where they are used.
- name: FLUENT_UID
value: "0"
# EDGI: custom exclude path and parser to work around matching errors
# (both `_EXCLUDE_PATH` and `_PARSER_TYPE` below)
# https://github.com/fluent/fluentd-kubernetes-daemonset/issues/434
- name: FLUENT_CONTAINER_TAIL_EXCLUDE_PATH
value: /var/log/containers/fluent*
- name: FLUENT_CONTAINER_TAIL_PARSER_TYPE
# value: /^(?<time>.+) (?<stream>stdout|stderr)( (?<logtag>.))? (?<log>.*)$/
value: cri
- name: K8S_NODE_NAME
valueFrom:
fieldRef:
Expand Down

0 comments on commit bdf0342

Please sign in to comment.