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

[Security Solution][Detections] Filter out non-security jobs when collecting Jobs telemetry #74454

Closed
wants to merge 1 commit into from

Conversation

rylnd
Copy link
Contributor

@rylnd rylnd commented Aug 5, 2020

Our jobs summary call returns all installed jobs regardless of group; passing groups as jobIds does not perform group filtering.

This adds a helper predicate function on which to filter these results, and updates tests accordingly.

Summarize your PR. If it involves visual changes include a screenshot or gif.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

Our jobs summary call returns all installed jobs regardless of group;
passing groups as jobIds does not perform group filtering.

This adds a helper predicate function on which to filter these results,
and updates tests accordingly.
@rylnd rylnd requested review from a team as code owners August 5, 2020 21:59
@rylnd rylnd closed this Aug 5, 2020
@rylnd rylnd deleted the fix_detections_telemetry branch August 5, 2020 21:59
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
securitySolution 804.7KB +107.0B 804.6KB

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@rylnd
Copy link
Contributor Author

rylnd commented Aug 5, 2020

Closed in favor of #74456

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants