Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Standardize log format and level #105

Closed
jmarin opened this issue Jun 3, 2015 · 3 comments
Closed

Standardize log format and level #105

jmarin opened this issue Jun 3, 2015 · 3 comments
Assignees
Milestone

Comments

@jmarin
Copy link
Contributor

jmarin commented Jun 3, 2015

Review logging for each application, applying logging level as appropriate

@jmarin jmarin added this to the M3 milestone Jun 3, 2015
@hkeeler
Copy link
Member

hkeeler commented Jun 4, 2015

We have two issues named "Review logging". Renaming this one to "Standardize log format and level".

@hkeeler hkeeler changed the title Review logging Standardize log format and level Jun 4, 2015
@jmarin
Copy link
Contributor Author

jmarin commented Jun 29, 2015

@hkeeler Can we close this one?

@hkeeler
Copy link
Member

hkeeler commented Jun 29, 2015

Here's where we stand with log message formatting:

  1. The following write to stdout and are parsable to by logstash:
    1. All of the Scala-based grasshopper services's standard logging
    2. grasshopper-parser's Gunicorn access logs
    3. grasshopper-ui's Nginx access logs
  2. The following are in an unknown state:
    1. grasshopper-parser's Python logging. It's setup to log to stdout, but I'm not doing any internal logging, so I haven't set this up. I'm pretty sure it would not be parsable by logstash out-of-the-box, but should be simple to setup.
    2. I have not tested grasshopper-loader and grasshopper-retreiver logging. @wpears was refactoring to a standard NodeJS logging package, but I am not sure of the status of that. @wpears?
  3. The following are known to not currently work:
    1. Scala-based access logs. Issue Add logRequestResponse to Akka HTTP  #113 has been opened for this.

So, if this issue if for just the grasshopper project, yes, we can close this out. If we consider it for the whole project, then it looks like it needs a little work.

@jmarin jmarin closed this as completed Jul 30, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants