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

Remove repeated prompts for metrics being stored #23

Closed
gridhead opened this issue Apr 25, 2021 · 2 comments · Fixed by #24
Closed

Remove repeated prompts for metrics being stored #23

gridhead opened this issue Apr 25, 2021 · 2 comments · Fixed by #24
Assignees
@gridhead
Labels
anatomy Changes are more than what meets the eye bug Something isn't working good first issue Good for newcomers

Comments

@gridhead
Copy link
Owner

It makes it difficult for users who are trying to log into the frontend after a long time since the driver service started as the logs would be convoluted and filled with the following.

The users would not be able to scroll back up to find the Sync URI, TermSocket URI and Passcode.

...
 * [Sun Apr 25 03:34:59 2021] Stored system metrics now...
 * [Sun Apr 25 03:35:09 2021] Stored system metrics now...
 * [Sun Apr 25 03:35:19 2021] Stored system metrics now...
 * [Sun Apr 25 03:35:29 2021] Stored system metrics now...
 * [Sun Apr 25 03:35:39 2021] Stored system metrics now...
 * [Sun Apr 25 03:35:50 2021] Stored system metrics now...
 * [Sun Apr 25 03:36:00 2021] Stored system metrics now...
 * [Sun Apr 25 03:36:10 2021] Stored system metrics now...
...
@gridhead gridhead added bug Something isn't working good first issue Good for newcomers anatomy Changes are more than what meets the eye labels Apr 25, 2021
@gridhead
Copy link
Owner Author

Not just the metrics, but the prompts generated from werkzeug can populate long enough to hamper the discovery of Sync URI, TermSocket URI and Passcode in the logs. These logs need to be disabled as well.

Thanks @shivangswain for marking this out.

@gridhead gridhead mentioned this issue May 12, 2021
Merged
@gridhead gridhead linked a pull request May 12, 2021 that will close this issue
Disabled Werkzeug and metric storage logging #24
Merged
@gridhead
Copy link
Owner Author

Closed via #24.

@gridhead gridhead self-assigned this May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gridhead gridhead

Labels
anatomy Changes are more than what meets the eye bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Disabled Werkzeug and metric storage logging gridhead/supervisor-driver-service
1 participant
@gridhead