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

Don't emit Kubernetes autodiscover events without host #7235

Merged
merged 1 commit into from
Jun 4, 2018

Conversation

exekias
Copy link
Contributor

@exekias exekias commented Jun 1, 2018

Some kubernetes events are generated while the container is being
configured. It doesn't make sense to emit an event yet, as the host will
be update right afterwards.

This PR changes the code to ignore Pod events without an IP address.

Closes #7193

@exekias exekias added enhancement in progress Pull request is currently in progress. libbeat needs tests containers Related to containers use case labels Jun 1, 2018
@ruflin
Copy link
Contributor

ruflin commented Jun 1, 2018

Change LGTM. Don't merge yet as still in progress. Should this have a changelog entry?

@exekias exekias force-pushed the k8s-no-host-events branch from 347910b to 5ee7de7 Compare June 1, 2018 16:56
Some kubernetes events are generated while the container is being
configured. It doesn't make sense to emit an event yet, as the host will
be update right afterwards.

This PR changes the code to ignore Pod events without an IP address.
@exekias exekias force-pushed the k8s-no-host-events branch from 5ee7de7 to 32e9e66 Compare June 1, 2018 17:07
@exekias
Copy link
Contributor Author

exekias commented Jun 1, 2018

Added tests and changelog

@exekias exekias added review and removed in progress Pull request is currently in progress. labels Jun 1, 2018
@ruflin ruflin merged commit 7b5a7e4 into elastic:master Jun 4, 2018
@exekias exekias added needs_backport PR is waiting to be backported to other branches. v6.3.0 labels Jun 8, 2018
@exekias exekias added v6.3.0 and removed needs_backport PR is waiting to be backported to other branches. labels Jun 8, 2018
exekias added a commit to exekias/beats that referenced this pull request Jun 8, 2018
Some kubernetes events are generated while the container is being
configured. It doesn't make sense to emit an event yet, as the host will
be update right afterwards.

This PR changes the code to ignore Pod events without an IP address.

(cherry picked from commit 7b5a7e4)
jsoriano pushed a commit that referenced this pull request Jun 8, 2018
Some kubernetes events are generated while the container is being
configured. It doesn't make sense to emit an event yet, as the host will
be update right afterwards.

This PR changes the code to ignore Pod events without an IP address.

(cherry picked from commit 7b5a7e4)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…elastic#7289)

Some kubernetes events are generated while the container is being
configured. It doesn't make sense to emit an event yet, as the host will
be update right afterwards.

This PR changes the code to ignore Pod events without an IP address.

(cherry picked from commit 65c8c03)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants