Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

Documentation/files/logging: Update Fluentd configmap for 1.8x #138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Documentation/files/logging: Update Fluentd configmap for 1.8x #138

wants to merge 1 commit into from

Conversation

hhoover
Copy link
Contributor

@hhoover hhoover commented Feb 7, 2018

Changes audit log parsing for fluentd as audit logging has changed
in v1.8 of Kubernetes.

@hhoover hhoover requested a review from zbwright February 8, 2018 16:47
<match kubernetes.var.log.containers.fluentd**>
type null
</match>

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prevent fluentd from handling records containing fluentd logs. Allowing this access can lead to an infinite loop, in which an error in sending one message generates another message which also fails to be sent, and so on.

# 2017-02-09T00:15:57.992775796Z AUDIT: id="90c73c7c-97d6-4b65-9461-f94606ff825f" ip="104.132.1.72" method="GET" user="kubecfg" as="<self>" asgroups="<lookup>" namespace="default" uri="/api/v1/namespaces/default/pods"
# 2017-02-09T00:15:57.993528822Z AUDIT: id="90c73c7c-97d6-4b65-9461-f94606ff825f" response="200"
# Audit logging must be enabled, otherwise this is skipped as the file is
# not present. This file will only be present on master nodes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... This file will be present only on master nodes.

Changes audit log parsing for fluentd as audit logging has changed
in v1.8 of Kubernetes.
Copy link
Contributor

@madorn madorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants