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.

Sign up for GitHub

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

Jump to bottom

dashboards-how-to #6935

Merged
merged 3 commits into from
Feb 29, 2024
Merged

dashboards-how-to #6935

merged 3 commits into from
Feb 29, 2024

Conversation

isacikgoz
Copy link
Member

Summary

Add a section to creating dashboards and how to use the dump file generated by the metrics plugin.

@github-actions GitHub Actions
Copy link

Newest code from isacikgoz has been published to preview environment for Git SHA c413108

agnivade
agnivade approved these changes Feb 20, 2024
View reviewed changes
Copy link
Member

@agnivade agnivade left a comment

Choose a reason for hiding this comment

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

Looks good after these changes are done

source/scale/performance-monitoring.rst Outdated

Using the Mattermost Metrics plugin, you can download and share the collected data with Mattermost to understand application performance, troubleshoot system stability and performance, as well as inform route cause analysis.

Furthermore, to use the dump file genereated by the plugin, you can simply clone the `Dockprom <https://github.com/stefanprodan/dockpromo>`__ Repository. Then you should change the prometheus data volume to point the dump that youjust downloaded. Note that the downloaded file is compressed, to be able to use it, you should decmpress it first.
Copy link
Member

Choose a reason for hiding this comment

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

  • typo at generated.
  • Also space between "youjust".
  • Typo at "decompress" as well

@github-actions GitHub Actions
Copy link

Newest code from isacikgoz has been published to preview environment for Git SHA 82c608f

cwarnermm
cwarnermm reviewed Feb 20, 2024
View reviewed changes
Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thanks for applying these updates to the same docs branch! I've provided editorial/style guide feedback inline

source/scale/performance-monitoring.rst Outdated Show resolved Hide resolved
source/scale/performance-monitoring.rst Outdated Show resolved Hide resolved
source/scale/performance-monitoring.rst Outdated Show resolved Hide resolved
source/scale/performance-monitoring.rst Outdated Show resolved Hide resolved
@isacikgoz @cwarnermm
Apply suggestions from code review
27eb665 
Co-authored-by: Carrie Warner (Mattermost) <74422101+cwarnermm@users.noreply.github.com>
@isacikgoz
Copy link
Member Author

Thanks @cwarnermm committed the suggestions :)

@isacikgoz isacikgoz requested a review from cwarnermm February 21, 2024 09:05
@github-actions GitHub Actions
Copy link

Newest code from isacikgoz has been published to preview environment for Git SHA 27eb665

@cwarnermm cwarnermm added the 1: PM Review Requires review by a product manager label Feb 21, 2024
@isacikgoz isacikgoz merged commit 49affeb into performance-metrics-plugin Feb 29, 2024
5 checks passed
@isacikgoz isacikgoz deleted the add-import-steps branch February 29, 2024 21:13
cwarnermm added a commit that referenced this pull request Apr 24, 2024
@cwarnermm @amyblais
@esarafianou @emdecr @isacikgoz
Added Metrics plugin (#6916)
4dddf1a 
* Update conf.py

* Clarified that system roles also control access to related API endpoints (#6901)

* Clarified that system roles include related API endpoints

* Update source/onboard/system-admin-roles.rst

Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>

---------

Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>

* Added new error codes page (#6908)

* Remove legacy MySQL references (#6896)

* Removed legacy references to MySQL v5.x

* Added missing redirect

* Removed "Sync" to align with product naming (#6898)

* Clarified Ent/Pro differentiation (#6899)

* Clarified that bots don't count as active users (#6902)

* Added interactive demo link (#6905)

* Added link to Kubernetes YAML docs via mattermost repo (#6906)

* Clarified supported AllowCorsFrom values (#6897)

* Updated legacy E10/E20 label text (#6900)

* Updated legacy E10/E20 label text

* Add example for E10/E20 label setup

* Test markup for labels

* Another markup test for labels

* And another markup test for labels

* Use raw html markup for E10/E20 labels

* Updated styling for legacy E10/E20 labels

* Change class name

---------

Co-authored-by: emdecr <delacruz.emily@gmail.com>

* Added transcription end user docs (#6910)

* Added Metrics plugin

* Update source/scale/performance-monitoring.rst

* Removed HA modes details

* dashboards-how-to (#6935)

* dashboards-how-to

Co-authored-by: Carrie Warner (Mattermost) <74422101+cwarnermm@users.noreply.github.com>

* Split perf monitoring into 3 parts: 3rd party integrations, plugin, & metrics

* Incorporated reviewer feedback

* Update source/guides/scale-mattermost.rst

Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>

* Update source/guides/scale-mattermost.rst

Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>

* Incorporated reviewer feedback, added redirects, & updated links

* Fixed broken link

---------

Co-authored-by: Amy Blais <29708087+amyblais@users.noreply.github.com>
Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>
Co-authored-by: emdecr <delacruz.emily@gmail.com>
Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>
cwarnermm added a commit that referenced this pull request May 9, 2024
@cwarnermm @mattermost-build
@amyblais @esarafianou @emdecr @isacikgoz @mickmister
Adding Mattermost-supported plugins to product docs (#7030)
cf302c5 
* Adding Mattermost-supported plugins to product docs

* Ongoing early iteration

* Ongoing early iteration & rewrites

* Final scaffolding/structure drafts

* Added missing link

* split out features from integrations

* Added Metrics plugin (#6916)

* Update conf.py

* Clarified that system roles also control access to related API endpoints (#6901)

* Clarified that system roles include related API endpoints

* Update source/onboard/system-admin-roles.rst

Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>

---------

Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>

* Added new error codes page (#6908)

* Remove legacy MySQL references (#6896)

* Removed legacy references to MySQL v5.x

* Added missing redirect

* Removed "Sync" to align with product naming (#6898)

* Clarified Ent/Pro differentiation (#6899)

* Clarified that bots don't count as active users (#6902)

* Added interactive demo link (#6905)

* Added link to Kubernetes YAML docs via mattermost repo (#6906)

* Clarified supported AllowCorsFrom values (#6897)

* Updated legacy E10/E20 label text (#6900)

* Updated legacy E10/E20 label text

* Add example for E10/E20 label setup

* Test markup for labels

* Another markup test for labels

* And another markup test for labels

* Use raw html markup for E10/E20 labels

* Updated styling for legacy E10/E20 labels

* Change class name

---------

Co-authored-by: emdecr <delacruz.emily@gmail.com>

* Added transcription end user docs (#6910)

* Added Metrics plugin

* Update source/scale/performance-monitoring.rst

* Removed HA modes details

* dashboards-how-to (#6935)

* dashboards-how-to

Co-authored-by: Carrie Warner (Mattermost) <74422101+cwarnermm@users.noreply.github.com>

* Split perf monitoring into 3 parts: 3rd party integrations, plugin, & metrics

* Incorporated reviewer feedback

* Update source/guides/scale-mattermost.rst

Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>

* Update source/guides/scale-mattermost.rst

Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>

* Incorporated reviewer feedback, added redirects, & updated links

* Fixed broken link

---------

Co-authored-by: Amy Blais <29708087+amyblais@users.noreply.github.com>
Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>
Co-authored-by: emdecr <delacruz.emily@gmail.com>
Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>

* Filename updates

* Incorporated AI Copilot, added screenshots & plugin updates

* Updated AI Copilot integration details

* Added AI Actions icon

* Incorporated remaining updates; ready for reviews

* Link fix

* Archived Google Calendar plugin docs (moved to GH repo)

* Updated docs page redirects to point to GH repo

* Migrating GH repo README content to product docs

* Update source/comply/export-mattermost-channel-data.rst

Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>

* Update source/integrate/jira-interoperability.rst

Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>

* Final PR updates consolidating pre-build MM integrations

---------

Co-authored-by: Mattermost Build <build@mattermost.com>
Co-authored-by: Amy Blais <29708087+amyblais@users.noreply.github.com>
Co-authored-by: Eva Sarafianou <eva.sarafianou@gmail.com>
Co-authored-by: emdecr <delacruz.emily@gmail.com>
Co-authored-by: Ibrahim Serdar Acikgoz <serdaracikgoz86@gmail.com>
Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agnivade agnivade agnivade approved these changes

@cwarnermm cwarnermm cwarnermm approved these changes

@nab-77 nab-77 Awaiting requested review from nab-77

Assignees
No one assigned
Labels
1: PM Review Requires review by a product manager
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@isacikgoz @agnivade @cwarnermm