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

[WIP] Diagnostic telemetry sender #84268

Closed
wants to merge 2 commits into from

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Nov 24, 2020

Summary

This is for the implementation of the Kibana task that reads the data from the diagnostics index and ships it to our telemetry service.

The rough plan is to:

  • create a task manager task that is executed every ~5 minutes

  • on each execution:

    • check if telemetry is enabled. If it's disabled, do nothing
    • query the index we decide to for the time since last execution to present, sorting by the event.ingested field. Limit the result set to 100 events.
  • call the queueTelemetryEvents function from the EventsTelemetry component added in [Security] Alert Telemetry for the Security app #77200

  • record the last execution time

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@tsg tsg added Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! release_note:skip Skip the PR/issue when compiling release notes labels Nov 24, 2020
@pjhampton
Copy link
Contributor

Tracking this work here: #84422

@pjhampton pjhampton closed this Nov 26, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Distributable file count

id before after diff
default 43051 43052 +1

History

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants