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

[7.x] [Telemetry] Diagnostic Alert Telemetry (#84422) #85631

Merged
merged 1 commit into from
Dec 10, 2020
Merged

[7.x] [Telemetry] Diagnostic Alert Telemetry (#84422) #85631

merged 1 commit into from
Dec 10, 2020

Conversation

pjhampton
Copy link
Contributor

Backports the following commits to 7.x:

* Port @tsg's work on task manager.

Remove 2nd var to track telemetry opt in.

Add ES client to start querying index.

Use query to get docs from a dummy index.

Change how index is queried.

Get diagnostic alerts to send to staging cluster.

Record last timestamp.

PoC on telemetry opt in via 2 processes.

Revert to original solution

* Update on agreed method. Fixes race condition.

* Expand wildcards.

* stage.

* Add rule.ruleset collection.

* Update telemetry sender with correct query for loading diag alerts.

* Add similar task tests to endpont artifact work.

* Fix broken import statement.

* Create sender mocks.

* Update test to check for func call.

* Update unused reference.

* record last run.

* Update index.

* fix import

* Fix test.

* test fix.

* Pass unit to time diff calc.

* Tests should pass now hopefully.

* Add additional process fields to allowlist.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@pjhampton pjhampton changed the title [7.x] [7.11][Telemetry] Diagnostic Alert Telemetry (#84422) [7.x] [Telemetry] Diagnostic Alert Telemetry (#84422) Dec 10, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Distributable file count

id before after diff
default 47260 48022 +762

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

@pjhampton pjhampton merged commit 03523d7 into elastic:7.x Dec 10, 2020
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