superset icon indicating copy to clipboard operation
superset copied to clipboard

feat(docker): add GUNICORN_LOGLEVEL env var

Open drummerwolli opened this issue 2 years ago • 3 comments

SUMMARY

to debug gunicorn, it is useful to specify the log level. This env var gives the possibility to define it in the docker image

TESTING INSTRUCTIONS

in the docker sub directory, run this to build and run the new container:

docker build -t test . && docker run --name test -e GUNICORN_LOGLEVEL=debug -it test

after the build logs, you will see that gunicorn logs now way more verbose than when running with the the -e GUNICORN_LOGLEVEL=debug flag.

ADDITIONAL INFORMATION

  • [ ] Has associated issue:
  • [ ] Required feature flags:
  • [ ] Changes UI
  • [ ] Includes DB Migration (follow approval process in SIP-59)
    • [ ] Migration is atomic, supports rollback & is backwards-compatible
    • [ ] Confirm DB migration upgrade and downgrade tested
    • [ ] Runtime estimates and downtime expectations provided
  • [ ] Introduces new feature or API
  • [ ] Removes existing feature or API

drummerwolli avatar Jun 07 '23 10:06 drummerwolli

bump, anything missing to merge this?

drummerwolli avatar Jan 18 '24 04:01 drummerwolli

This seems harmles... closing/reopening to kick-start CI, then we can (hopefully/finally) merge it!

rusackas avatar Aug 27 '24 02:08 rusackas

Sorry this slipped through the cracks for so long.

rusackas avatar Aug 27 '24 02:08 rusackas

@rusackas thanks for picking it up. anything missing from my side to get it merged?

drummerwolli avatar Aug 30 '24 13:08 drummerwolli

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 66.53%. Comparing base (ede6acd) to head (37001ce). :warning: Report is 3844 commits behind head on master.

:x: Your project status has failed because the head coverage (65.55%) is below the target coverage (100.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #24308   +/-   ##
=======================================
  Coverage   66.53%   66.53%           
=======================================
  Files        1957     1957           
  Lines       75623    75623           
  Branches     8224     8224           
=======================================
  Hits        50312    50312           
  Misses      23202    23202           
  Partials     2109     2109           

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Aug 30 '24 14:08 codecov[bot]