deis / logger

In-memory log buffer used by Deis Workflow.
https://deis.com
MIT License
18 stars 15 forks source link

fix(server): Fix regex to help remove duplicate lines #72

Closed jchauncey closed 8 years ago

jchauncey commented 8 years ago

The controller regex was allowing duplicate log lines.

Testing Steps: 1) Build and deploy new logger 2) Bounce all fluentd pods (this might not be needed but its habit for me) 3) Create an application 4) deis logs and make sure there are not duplicate lines from the controller

It is possible that there are duplicate lines from fluentd (I am investigating that now) but the controller log lines shouldnt be duplicated.

jchauncey commented 8 years ago

closing this PR for now. Im not able to reproduce the problem I was seeing yesterday that caused me to make this change