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

[Telemetry] Remove initial delay to check and send #26575

Merged
merged 2 commits into from
Dec 4, 2018

Conversation

tsullivan
Copy link
Member

Part of #21241 (comment)

Summary

Before 6.3.0, Telemetry stats had to be explicitly opted-out of to prevent your cluster to send stats. In 6.3.0, the method was changed to opt-in. This means a user doesn't have to read a message about sharing their cluster stats with us and click "Opt out" before anything is sent to Elastic.

Because it was previously the case that the user would have to read and opt out, we had a 2-minute delay in the browser before creating and sending the payload.

This PR takes out the 2 minute delay. When a page loads, code on the page will check immediately if they are opted in, and will send control to the "send if due" logic that has Telemetry sent to Elastic from the browser up to once per day.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

@pickypg
Copy link
Member

pickypg commented Dec 3, 2018

Similar to #25897

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@pickypg pickypg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sold on the immediate-firing.

x-pack/plugins/xpack_main/public/hacks/telemetry.js Outdated Show resolved Hide resolved
@pickypg pickypg mentioned this pull request Dec 3, 2018
1 task
Copy link
Member

@pickypg pickypg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tsullivan
Copy link
Member Author

Stats will get called and transmitted after 1 minute:
image

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@tsullivan tsullivan merged commit ee3fc58 into elastic:master Dec 4, 2018
@tsullivan tsullivan deleted the telemetry/remove-browser-wait branch December 4, 2018 01:27
tsullivan added a commit to tsullivan/kibana that referenced this pull request Dec 4, 2018
* [Telemetry] Remove initial delay to check and send

* do not fire immediately
tsullivan added a commit that referenced this pull request Dec 4, 2018
* [Telemetry] Remove initial delay to check and send

* do not fire immediately
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants