Skip to content
This repository has been archived by the owner on Aug 25, 2023. It is now read-only.

BDRK-3386: Update fluentd-container-insight to support containerd logs #44

Merged
merged 3 commits into from
Aug 4, 2021
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# Order is important; the last matching pattern takes the most
# precedence.
# https://help.github.com/en/articles/about-code-owners
* @basisai/infrastructure
4 changes: 2 additions & 2 deletions charts/eks-container-insights/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v1
apiVersion: v2
appVersion: "1.0"
description: A Helm chart for EKS Container Insights
name: eks-container-insights
version: 0.4.1
version: 0.4.2
tunguyen9889 marked this conversation as resolved.
Show resolved Hide resolved
home: https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/Container-Insights-setup-logs.html
12 changes: 6 additions & 6 deletions charts/eks-container-insights/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ data:
tag *
read_from_head true
<parse>
@type json
time_format %Y-%m-%dT%H:%M:%S.%NZ
@type "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TYPE'] || 'json'}"
time_format "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TIME_FORMAT'] || '%Y-%m-%dT%H:%M:%S.%NZ'}"
</parse>
</source>

Expand All @@ -39,8 +39,8 @@ data:
tag *
read_from_head true
<parse>
@type json
time_format %Y-%m-%dT%H:%M:%S.%NZ
@type "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TYPE'] || 'json'}"
time_format "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TIME_FORMAT'] || '%Y-%m-%dT%H:%M:%S.%NZ'}"
</parse>
</source>

Expand All @@ -53,8 +53,8 @@ data:
tag *
read_from_head true
<parse>
@type json
time_format %Y-%m-%dT%H:%M:%S.%NZ
@type "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TYPE'] || 'json'}"
time_format "#{ENV['FLUENT_CONTAINER_TAIL_PARSER_TIME_FORMAT'] || '%Y-%m-%dT%H:%M:%S.%NZ'}"
</parse>
</source>

Expand Down
3 changes: 3 additions & 0 deletions charts/eks-container-insights/templates/daemonset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,9 @@ spec:
value: regional
# END Archive cluster logs to S3
{{- end }}
{{- with .Values.env }}
{{- toYaml . | nindent 12 }}
{{- end }}
resources:
{{- toYaml .Values.resources | nindent 12 }}
volumeMounts:
Expand Down
9 changes: 8 additions & 1 deletion charts/eks-container-insights/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ serviceAccount:
annotations: {}

image: basisai/eks-container-insights
imageTag: v1.12.4
imageTag: v1.13.0

initImage: busybox
initImageTag: 1.32.0
Expand Down Expand Up @@ -39,6 +39,13 @@ additional_container_logs_filters: []
additional_systemd_logs_filters: []
additional_host_logs_filters: []

env: []
# Use these variables if containerd runtime
# - name: FLUENT_CONTAINER_TAIL_PARSER_TYPE
# value: "cri"
# - name: FLUENT_CONTAINER_TAIL_PARSER_TIME_FORMAT
# value: "%Y-%m-%dT%H:%M:%S.%N%:z"

resources:
limits:
cpu: 200m
Expand Down
2 changes: 1 addition & 1 deletion dockerfiles/eks-container-insights/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
FROM fluent/fluentd-kubernetes-daemonset:v1.12.4-debian-cloudwatch-amd64-1.3
FROM fluent/fluentd-kubernetes-daemonset:v1.13-debian-cloudwatch-amd64-1
tunguyen9889 marked this conversation as resolved.
Show resolved Hide resolved

RUN gem install fluent-plugin-s3 -v 1.6.0 --no-document
6 changes: 3 additions & 3 deletions dockerfiles/fluentd/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
FROM quay.io/fluentd_elasticsearch/fluentd:v3.2.0

RUN fluent-gem install digest-crc --version 0.5.1 \
&& fluent-gem install fluent-plugin-gcs --version 0.4.1 \
&& fluent-gem install fluent-plugin-s3 --version 1.6.0
RUN fluent-gem install digest-crc --version 0.5.1 --no-document \
&& fluent-gem install fluent-plugin-gcs --version 0.4.1 --no-document \
&& fluent-gem install fluent-plugin-s3 --version 1.6.0 --no-document
2 changes: 1 addition & 1 deletion tests/ct.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
validate-maintainers: false
chart-dirs:
- /charts
- /charts
chart-repos:
- jetstack=https://charts.jetstack.io