Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[fluentd-gcp addon] By default ingest audit logs in JSON format#52541

Merged
k8s-github-robot merged 1 commit into
kubernetes:masterfrom
crassirostris:audit-fluentd-format
Sep 19, 2017
Merged

[fluentd-gcp addon] By default ingest audit logs in JSON format#52541
k8s-github-robot merged 1 commit into
kubernetes:masterfrom
crassirostris:audit-fluentd-format

Conversation

@crassirostris
Copy link
Copy Markdown

As #50971 set default audit log format for advanced auditing to JSON and #51943 makes advanced auditing default, fluentd should also ingest audit logs as JSON by default.

[fluentd-gcp addon] By default ingest apiserver audit logs written to file in JSON format.

@crassirostris crassirostris added area/logging area/platform/gce area/provider/gcp Issues or PRs related to gcp provider kind/bug Categorizes issue or PR as related to a bug. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. labels Sep 15, 2017
@crassirostris crassirostris added this to the v1.8 milestone Sep 15, 2017
@crassirostris crassirostris requested a review from piosz September 15, 2017 13:38
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 15, 2017
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 15, 2017
@dims
Copy link
Copy Markdown
Member

dims commented Sep 18, 2017

/retest

@piosz
Copy link
Copy Markdown
Member

piosz commented Sep 18, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 18, 2017
@crassirostris
Copy link
Copy Markdown
Author

/approve no-issue

@k8s-github-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: crassirostris, piosz

Associated issue requirement bypassed by: crassirostris

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot k8s-github-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 18, 2017
@crassirostris
Copy link
Copy Markdown
Author

/retest

1 similar comment
@spiffxp
Copy link
Copy Markdown
Contributor

spiffxp commented Sep 18, 2017

/retest

@fejta-bot
Copy link
Copy Markdown

/retest
This bot automatically retries jobs that failed/flaked on approved PRs (send feedback to @fejta).

Review the full test history for this PR.

@k8s-github-robot
Copy link
Copy Markdown

Automatic merge from submit-queue (batch tested with PRs 48970, 52497, 51367, 52549, 52541). If you want to cherry-pick this change to another branch, please follow the instructions here..

@k8s-github-robot k8s-github-robot merged commit 16670f1 into kubernetes:master Sep 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/logging area/platform/gce area/provider/gcp Issues or PRs related to gcp provider cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/bug Categorizes issue or PR as related to a bug. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants