Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Enable performance telemetry tracking for production deployments #9758

Merged
merged 4 commits into from
Feb 10, 2022

Conversation

jwilander
Copy link
Member

Summary

Per discussion here https://community.mattermost.com/core/pl/seageg6dhidkmmx9rz9urgk9mw.

Release Note

Enable performance telemetry tracking for production deployments not running in developer mode. Disabled when telemetry is toggled off.

@jwilander jwilander added 2: Dev Review Requires review by a core commiter QA Deferred Agreement with QA that these changes will be tested post-merge labels Feb 4, 2022
@jwilander
Copy link
Member Author

@hmhealey PTAL

Copy link
Member

@hmhealey hmhealey left a comment

Choose a reason for hiding this comment

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

One small suggestion, but otherwise looks good

actions/telemetry_actions.jsx Outdated Show resolved Hide resolved
@jwilander jwilander merged commit 167fae1 into master Feb 10, 2022
@jwilander jwilander deleted the perf-telemetry-for-prod branch February 10, 2022 13:54
@amyblais amyblais added Changelog/Done Required changelog entry has been written Docs/Needed Requires documentation labels Feb 10, 2022
@amyblais amyblais added this to the v6.5.0 milestone Feb 10, 2022
@cwarnermm
Copy link
Member

@jwilander - Are telemetry documentation updates needed as part of this PR?

@jwilander
Copy link
Member Author

@cwarnermm nope, this falls into the same categories called out in the documentation so all good 👍

@cwarnermm cwarnermm added Docs/Not Needed Does not require documentation and removed Docs/Needed Requires documentation labels Feb 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2: Dev Review Requires review by a core commiter Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation QA Deferred Agreement with QA that these changes will be tested post-merge release-note
Projects
None yet
6 participants