Skip to content
This repository has been archived by the owner on Sep 23, 2019. It is now read-only.

Docker logs are too long and not super useful #90

Open
2 of 3 tasks
j-mcavoy opened this issue Sep 11, 2019 · 1 comment
Open
2 of 3 tasks

Docker logs are too long and not super useful #90

j-mcavoy opened this issue Sep 11, 2019 · 1 comment
Assignees
@timothyhollabaugh
Labels
bug Something isn't working good first issue Good for newcomers priority-high Required for a functional product fairly soon scope-small A relativly small change, does not change any architecture
Milestone
API Server

Comments

@j-mcavoy
Copy link
Contributor

j-mcavoy commented Sep 11, 2019 •
edited by timothyhollabaugh
Loading

Logging is the bottleneck of the server currently. We need to remove unnecessary trace statements in the backend api when building for release.

Things we want to log

  • API requests
  • All errors
  • Log the full date in logs, not just time
@j-mcavoy j-mcavoy added bug Something isn't working good first issue Good for newcomers priority-high Required for a functional product fairly soon labels Sep 11, 2019
@j-mcavoy j-mcavoy added this to the API Server milestone Sep 11, 2019
@j-mcavoy j-mcavoy changed the title Reduce slow logs in Rust Reduce slow logs in Docker Sep 11, 2019
@j-mcavoy j-mcavoy changed the title Reduce slow logs in Docker Docker logs are too long and not super useful Sep 11, 2019
@timothyhollabaugh timothyhollabaugh self-assigned this Sep 11, 2019
timothyhollabaugh added a commit that referenced this issue Sep 11, 2019
@timothyhollabaugh
Merge pull request #97 from A-Team-Rowan-University/#90-Docker_logs_a…
79dcfdb 
…re_too_long_and_not_super_useful

#90 docker logs are too long and not super useful
@timothyhollabaugh
Copy link
Member

Also don't log tokio_reactor

@timothyhollabaugh timothyhollabaugh added the scope-small A relativly small change, does not change any architecture label Sep 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@timothyhollabaugh timothyhollabaugh

Labels
bug Something isn't working good first issue Good for newcomers priority-high Required for a functional product fairly soon scope-small A relativly small change, does not change any architecture
Projects
None yet
Milestone
API Server
Development

No branches or pull requests
2 participants
@timothyhollabaugh @j-mcavoy