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.

Sign up for GitHub

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

Jump to bottom

Beats can loose events when using kubernetes autodiscovery #17246

Closed
brunotm opened this issue Mar 25, 2020 · 1 comment · Fixed by #17223
Closed

Beats can loose events when using kubernetes autodiscovery #17246

brunotm opened this issue Mar 25, 2020 · 1 comment · Fixed by #17223
Assignees
@brunotm
Labels
autodiscovery containers Related to containers use case Team:Platforms Label for the Integrations - Platforms team

Comments

@brunotm
Copy link
Contributor

brunotm commented Mar 25, 2020 •
edited by andresrc
Loading

  • Version: 7.6.1 (and previous)
  • Operating System: Linux (any)
  • Steps to Reproduce: Run kubernetes cronjobs or any pod that runs to completion.

OnUpdate() in kubernetes autodiscovery provider uses the ObjectMeta.DeletionTimestamp to determine if a stop event should be emitted. ObjectMeta.DeletionTimestamp is only present for deleted pods and when a pod runs to completion (eg. pods generated by conjobs), OnUpdate() will treat it as an update (stop/start) and emit a pod stop event disrespecting the CleanupTimeout and leading to early termination of running beats.

Fixed on #17223

/cc @blakerouse @exekias
@brunotm brunotm mentioned this issue Mar 25, 2020
Merged
2 tasks
@blakerouse blakerouse added autodiscovery containers Related to containers use case Team:Platforms Label for the Integrations - Platforms team labels Mar 25, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations-platforms (Team:Platforms)

This was referenced Mar 25, 2020
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brunotm brunotm

Labels
autodiscovery containers Related to containers use case Team:Platforms Label for the Integrations - Platforms team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[BUG] Use Pod.Status.Phase for pod updates in kubernetes autodiscovery Unbabel/beats
3 participants
@blakerouse @brunotm @elasticmachine