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

Add regex based cid extractor to add_process_metadata #17360

Merged
merged 4 commits into from
Apr 3, 2020
Merged

Add regex based cid extractor to add_process_metadata #17360

merged 4 commits into from
Apr 3, 2020

Conversation

jtinkus
Copy link
Contributor

@jtinkus jtinkus commented Mar 31, 2020

  • Enhancement

What does this PR do?

Adds regex based cid extractor to add_kubernetes_metadata

Why is it important?

Adds more flexibility and makes possible to bypass bug with kube-proxy cid turning out to be "kube-proxy"

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
    - [ ] I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Relates:
#15947
#17351

Use cases

Screenshots

Logs

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

1 similar comment
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jtinkus
Copy link
Contributor Author

jtinkus commented Mar 31, 2020

@exekias

Here is optional regex based container id extractor functionality added to add_process_metadata.

Regards,

@jtinkus
Copy link
Contributor Author

jtinkus commented Apr 1, 2020

@exekias auto checks green and checklist check-boxes checked now :)

@exekias exekias added enhancement review Team:Platforms Label for the Integrations - Platforms team labels Apr 2, 2020
@elasticmachine
Copy link
Collaborator

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

@exekias
Copy link
Contributor

exekias commented Apr 2, 2020

ok to test

@exekias exekias added the needs_backport PR is waiting to be backported to other branches. label Apr 2, 2020
@exekias
Copy link
Contributor

exekias commented Apr 2, 2020

Thank you so much for contributing! Waiting for a green light from the CI

@exekias exekias merged commit b92c6c3 into elastic:master Apr 3, 2020
exekias pushed a commit to exekias/beats that referenced this pull request Apr 3, 2020
* Add regex based cid extractor to add_process_metadata

Co-authored-by: Jako Tinkus <[email protected]>
(cherry picked from commit b92c6c3)
@exekias exekias added v7.8.0 and removed needs_backport PR is waiting to be backported to other branches. labels Apr 3, 2020
exekias pushed a commit that referenced this pull request May 8, 2020
* Add regex based cid extractor to add_process_metadata

Co-authored-by: Jako Tinkus <[email protected]>
(cherry picked from commit b92c6c3)

Co-authored-by: jtinkus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement review Team:Platforms Label for the Integrations - Platforms team v7.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants