Skip to content
This repository has been archived by the owner on Dec 1, 2018. It is now read-only.

[ES] enhance es mapping for event #1379

Closed

Conversation

andyxning
Copy link
Contributor

@andyxning andyxning commented Nov 14, 2016

This PR adds support for adding more info for analyze events in ElasticSearch. This PR adds EventType, EventKind, EventReason and EventMessage as tags and thus allows us to query based on them. Such as, what are the warning events in last two hours. When does the cluster emits FailedScheduling events.

This PR also adds the namespace tag info when the event is of Pod kind.


This change is Reviewable

@k8s-ci-robot
Copy link
Contributor

Can a kubernetes member verify that this patch is reasonable to test? If so, please reply with "@k8s-bot ok to test" on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands will still work. Regular contributors should join the org to skip this step.

If you have questions or suggestions related to this bot's behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

@andyxning
Copy link
Contributor Author

@googlebot i run it correctly in my laptop.

@andyxning
Copy link
Contributor Author

I don't know why the test TestOneExportInTime in k8s.io/heapster/metrics/sinks will fail. I runs ok in my laptop. :(

@andyxning
Copy link
Contributor Author

it turns out that memory is out and something like go build testmain: /home/travis/.gimme/versions/go1.6.linux.amd64/pkg/tool/linux_amd64/link: signal: killed is reported. For more info please see Travis CI build envrionment and Travis CI FAQ about memory limit

@AlmogBaku
Copy link
Contributor

That's an interesting approach, i solved that slight differently in #1313

@piosz piosz changed the title enhance es mapping for event [ES] enhance es mapping for event Nov 23, 2016
@piosz
Copy link
Contributor

piosz commented Nov 23, 2016

Can anyone review it? @AlmogBaku?

@AlmogBaku
Copy link
Contributor

AlmogBaku commented Nov 23, 2016 via email

@piosz
Copy link
Contributor

piosz commented Nov 25, 2016

closing per @AlmogBaku comment

@piosz piosz closed this Nov 25, 2016
@andyxning andyxning deleted the enhance_es_for_eventer branch December 6, 2016 09:01
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.

5 participants