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

update Grafana dashboard for Cloud, stripping out hard-coded links #557

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

lieut-data
Copy link
Member

Summary

Update the snapshot of the Cloud dashboard we use for monitoring this plugin. Note that this remains architected towards our Mattermost Cloud environment, with a $namespace filter and various datasource variables. Nevertheless, it's a good starting point for building a single-installation version, and a future iteration may find a way to make the $namespace parameter optional

Ticket Link

None.

@lieut-data lieut-data marked this pull request as ready for review March 21, 2024 15:44
@lieut-data lieut-data merged commit 8bd40a3 into main Mar 21, 2024
5 checks passed
@lieut-data lieut-data deleted the updated-grafana-dashboard-for-cloud branch March 21, 2024 15:44
sbishel added a commit that referenced this pull request Apr 8, 2024
* add handling for connecting bot account

* check for permissions and connected user

* add more tests

* Update api_test.go

* update Grafana dashboard for Cloud, stripping out hard-coded links (#557)

* Disable temporoarily auto share DMs (#558)

* [MM-56908] Make promote command accepts @ for both arguments (#550)

* [MM-57072] Use ConstantTimeCompare to prevent timing attack (#549)

* use ConstantTimeCompare to prevent timing attack

* document code

* MM-57367: log when invite sent (#559)

* log when invite sent

* remove listener on uninitialize

* review change

* use IsGroupOrDirect for clarity (#556)

* [MM-57209] Set EmailVerified to true when promoting synthetic users (#548)

* Set EmailVerified to true when promoting synthetic users

* change EmailVerified in the sync job + add and fix tests

* add once

* lint

* add EmailVerified when creating synthetic user from a getter

* remove EmailVerified from the getOrCreateSyntheticUser (not promoted there)

* fix tests

* MM-57120: show connected users stat (#571)

Show the number of connected users on the system console for administrators.

Fixes: https://mattermost.atlassian.net/browse/MM-57350

* [RE:MM-57209]  Set EmailVerified to true when autopromoting through hook (#565)

* MM-57120: delay monitoring job on startup (#570)

Delay the monitoring job by at least 1 minute after startup to avoid a race where the plugin hasn't fully yet deployed in a mostly disabled environment.

```
{
  "timestamp": "2024-03-30 15:06:00.011 -03:00",
  "level": "info",
  "msg": "Delaying the Monitoring System Job until at least 1 minute after startup",
  "caller": "app/plugin_api.go:1004",
  "plugin_id": "com.mattermost.msteams-sync"
}

{
  "timestamp": "2024-03-30 15:07:00.016 -03:00",
  "level": "info",
  "msg": "Running the Monitoring System Job",
  "caller": "app/plugin_api.go:1004",
  "plugin_id": "com.mattermost.msteams-sync"
}
```

Fixes: https://mattermost.atlassian.net/browse/MM-57120

* [MM-56867] Fix metrics by not counting disabled users (#561)

* stop counting disabled msteams user

* fix terminology

* fix test + lint

* MM-56809: support deleting messages from Mattermost (#569)

---------

Co-authored-by: Jesse Hallam <jesse.hallam@gmail.com>
Co-authored-by: Jesús Espino <jespinog@gmail.com>
Co-authored-by: Julien Tant <785518+JulienTant@users.noreply.github.com>
Co-authored-by: Caleb Roseland <caleb@calebroseland.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant