Added flask-monitoringdashboard app#438
Conversation
|
What is the performance impact of this extension? |
|
@jace it writes to a second db. I tried load testing locally using |
|
When does it write? In the same request response cycle, or in a background job? Writing to a second db will still freeze the request until the database finishes committing. |
| *.bz2 | ||
| hasjob/assets/node_modules | ||
| admindash-config.cfg | ||
| flask_monitoringdashboard.db |
There was a problem hiding this comment.
We can't use SQLite in production as there are multiple worker processes competing for control over this file.
There was a problem hiding this comment.
By default it uses sqlite and creates this file. I moved it to postgres.
| *.gz | ||
| *.bz2 | ||
| hasjob/assets/node_modules | ||
| admindash-config.cfg |
There was a problem hiding this comment.
Config should be stored in the instance folder.
| return 'anon' | ||
|
|
||
|
|
||
| dashboard.config.get_group_by = get_user_id |
|
Bibhas seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Implemented flask-monitoringdashboard This app gives us some metrics about HTTP requests on the endpoints we choose. Couple of pros are -