Feature request: omit /healthz from INFO logging

[tammert]

To start off: Atlantis is awesome! Really appreciate the work you've done on it.

Problem
When running on Kubernetes, the livenessProbe/readinessProbe send a lot of requests to the /healthz endpoint, which are logged as follows:

2020/05/26 12:16:20+0000 [INFO] server: GET /healthz – respond HTTP 200
2020/05/26 12:16:36+0000 [INFO] server: GET /healthz – from <IP>
2020/05/26 12:16:36+0000 [INFO] server: GET /healthz – respond HTTP 200
2020/05/26 12:17:20+0000 [INFO] server: GET /healthz – from <IP>
2020/05/26 12:17:20+0000 [INFO] server: GET /healthz – respond HTTP 200
2020/05/26 12:17:36+0000 [INFO] server: GET /healthz – from <IP>
2020/05/26 12:17:36+0000 [INFO] server: GET /healthz – respond HTTP 200

This could be avoided by upping the log level to error, but since many interesting Atlantis/Terraform/lock logging is also info level, those would be lost too.

Proposed solution

• It would be nice if we could use a flag to turn off the logging for certain endpoints, in this case at least for /healthz (and perhaps /events too).
• Alternatively, those messages could be moved to debug, so we can turn them off without removing the more interesting logging.

I'm pretty sure this is not currently possible, but if it is I'd love to hear. Besides that, if you have an opinion on either of the proposed solutions (or maybe something else entirely), I'd love to hear it as I don't mind building it myself and sending a PR for this.

[lkysow]

Hey @tammert yeah I think having a solution for this would be a good idea. Should we try just moving the path logs to debug? I've asked for feedback in the Slack here: https://atlantis-community.slack.com/archives/C5MGGAV0C/p1590511701246700

[tammert]

Yea, that seems like the way to go. Pending the feedback on Slack, the PR is already up as the change was very trivial :)

[tammert]

@lkysow I think you can close this, as the PR has been merged and released in v0.13.0 (thanks for that, no more massive logs for us!).

[lkysow]

Closed by #1056