Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix fluentd configs to collect etcd logs on the master #12045

Merged
merged 2 commits into from Jul 31, 2015

Conversation

a-robinson
Copy link
Contributor

Also remove some cruft from a previous poor merge conflict resolution.

Fixes #11997

@satnam6502 @roberthbailey

@k8s-bot
Copy link

k8s-bot commented Jul 30, 2015

GCE e2e build/test passed for commit 9e123aea890aaa75f7d8b25ab49649ff158d2042.

@k8s-bot
Copy link

k8s-bot commented Jul 30, 2015

GCE e2e build/test failed for commit 686949e6e79585a8d558125673d4defa290d1e01.

@a-robinson
Copy link
Contributor Author

That's awesome, gendocs caught that one of the logging docs had a tag that didn't match the new one. I've pushed a fixed commit.

@a-robinson
Copy link
Contributor Author

I just realized @satnam6502 is out today and @roberthbailey is out this afternoon. @fabioy, want to rubber stamp a quick config fix?

@satnam6502
Copy link
Contributor

I can do it

@satnam6502 satnam6502 self-assigned this Jul 30, 2015
@@ -94,20 +94,13 @@
tag docker
</source>

<match kubernetes.**>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this match rules being removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because it was left in by a bad merge. The real match rule is at the bottom of the file.

@satnam6502 satnam6502 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 30, 2015
@k8s-bot
Copy link

k8s-bot commented Jul 30, 2015

GCE e2e build/test passed for commit 69ab702.

@mikedanese mikedanese closed this Jul 30, 2015
@mikedanese mikedanese reopened this Jul 30, 2015
mikedanese added a commit that referenced this pull request Jul 31, 2015
Fix fluentd configs to collect etcd logs on the master
@mikedanese mikedanese merged commit bfaa527 into kubernetes:master Jul 31, 2015
zmerlynn added a commit that referenced this pull request Jul 31, 2015
…2045-upstream-release-1.0

Automated cherry pick of #12045 upstream release 1.0
@brendandburns brendandburns mentioned this pull request Aug 1, 2015
@a-robinson a-robinson deleted the logs branch August 5, 2015 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants