Move logging.basicConfig() from api to cli

Logging should be set in the cli rather than the api to avoid overwritting logging settings set by the user. Related to GitHub issue #58
operator-framework · Mar 12, 2019 · d04f08e · d04f08e
1 parent 4628bfa
commit d04f08e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/operatorcourier/api.py b/operatorcourier/api.py
@@ -16,7 +16,6 @@
 from operatorcourier.format import format_bundle
 from operatorcourier.nest import nest_bundles
 
-logging.basicConfig()
 logger = logging.getLogger(__name__)
 
 

diff --git a/operatorcourier/cli.py b/operatorcourier/cli.py
@@ -1,12 +1,14 @@
 import argparse
 import pkg_resources
 import sys
+import logging
 from operatorcourier import api
 
 
 def main():
     """Generate the CLI bits
     """
+    logging.basicConfig()
     try:
         parser = _CliParser()
         parser.parse()