ietf-tools / www

A customized CMS for the IETF website
BSD 3-Clause "New" or "Revised" License
25 stars 45 forks source link

docs: Enable logging when running postgres container locally #424

Closed microamp closed 5 months ago

microamp commented 5 months ago

Making it more debug-friendly when testing locally

codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.72%. Comparing base (47d92fb) to head (3f1dd62).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #424 +/- ## ======================================= Coverage 82.72% 82.72% ======================================= Files 124 124 Lines 2599 2599 ======================================= Hits 2150 2150 Misses 449 449 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

rjsparks commented 5 months ago

Approving (since Kesara's offline), but mostly because this app isn't particularly database intensive. If it later turns out that having the db logs is too much and in the way, consider providing a way to easily enable/disable them at runtime instead.

microamp commented 5 months ago

Approving (since Kesara's offline), but mostly because this app isn't particularly database intensive. If it later turns out that having the db logs is too much and in the way, consider providing a way to easily enable/disable them at runtime instead.

This change (documentation) is only for local testing. Having said that, we can make database logging configurable in the Helm chart, so that we can easily enable it in staging.