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 all 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.5.0
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.3-debian-cloudwatch-amd64-1.2

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