conan-io#279 Add logging level for conan-center hook

Signed-off-by: Uilian Ries <[email protected]>

Author: uilianries

README.md

@@ -88,7 +88,16 @@ variable ``CONAN_HOOK_ERROR_LEVEL``:
    - ``CONAN_HOOK_ERROR_LEVEL=40`` it will raise if any error happen.
    - ``CONAN_HOOK_ERROR_LEVEL=30`` it will raise if any error or warning happen.
 
+If you want to decrease logging level, you can adjust it by the environment variable
+``CONAN_HOOK_LOGGING_LEVEL``:
 
+  - ``CONAN_HOOK_LOGGING_LEVEL=40`` it will print out only error messages
+  - ``CONAN_HOOK_LOGGING_LEVEL=WARNING`` it will print out only warning and error messages
+
+It accepts both logging level names and logging level code:
+  - ERROR: 40
+  - WARNING/WARN: 30
+  - INFO: 20
 
 ### [Attribute checker](hooks/attribute_checker.py)
 

hooks/conan-center.py

@@ -2,7 +2,7 @@
 import inspect
 import os
 import re
-from logging import WARNING, ERROR, INFO, DEBUG, NOTSET
+from logging import WARNING, ERROR, INFO, DEBUG, NOTSET, _nameToLevel
 
 import yaml
 from conans import tools, Settings
@@ -67,6 +67,11 @@ def __init__(self, output, kb_id=None):
         if self.kb_id:
             self.kb_url = kb_url(self.kb_id)
         self._error_level = int(os.getenv("CONAN_HOOK_ERROR_LEVEL", str(NOTSET)))
+        self._set_logging_level()
+
+    def _set_logging_level(self):
+        level = os.getenv("CONAN_HOOK_LOGGING_LEVEL", str(NOTSET))
+        self._logging_level = int(level) if level.isdigit() else _nameToLevel[level.upper()]
 
     def _get_message(self, message):
         if self._test_name:
@@ -76,22 +81,26 @@ def _get_message(self, message):
             return message
 
     def success(self, message):
-        self._output.success(self._get_message(message))
+        if self._logging_level < WARNING:
+            self._output.success(self._get_message(message))
 
     def debug(self, message):
         if self._error_level and self._error_level <= DEBUG:
             self._error = True
-        self._output.debug(self._get_message(message))
+        if self._logging_level < INFO:
+            self._output.debug(self._get_message(message))
 
     def info(self, message):
         if self._error_level and self._error_level <= INFO:
             self._error = True
-        self._output.info(self._get_message(message))
+        if self._logging_level < WARNING:
+            self._output.info(self._get_message(message))
 
     def warn(self, message):
         if self._error_level and self._error_level <= WARNING:
             self._error = True
-        self._output.warn(self._get_message(message))
+        if self._logging_level < ERROR:
+            self._output.warn(self._get_message(message))
 
     def error(self, message):
         self._error = True

tests/test_hooks/conan-center/test_conan-center.py

@@ -1035,3 +1035,41 @@ def package_info(self):
         output = self.conan(['create', '.', 'name/version@user/test'])
         self.assertIn("[LIBRARY DOES NOT EXIST (KB-H054)] OK", output)
         self.assertNotIn('does not contain any library', output)
+
+    def test_logging_level(self):
+        conanfile = textwrap.dedent("""\
+        from conans import ConanFile
+        import os
+
+        class FoobarConan(ConanFile):
+            author = "Foobar"
+
+            def package(self):
+                pass
+        """)
+        tools.save('conanfile.py', content=conanfile)
+        output = self.conan(['create', '.', 'name/version@user/test'])
+        self.assertIn("ERROR: [PACKAGE LICENSE (KB-H012)]", output)
+        self.assertIn("WARN: [HEADER_ONLY, NO COPY SOURCE (KB-H005)]", output)
+        self.assertIn("[FPIC MANAGEMENT (KB-H007)] OK", output)
+
+        for level in ["INFO", "20"]:
+            with tools.environment_append({"CONAN_HOOK_LOGGING_LEVEL": level}):
+                output = self.conan(['create', '.', 'name/version@user/test'])
+                self.assertIn("ERROR: [PACKAGE LICENSE (KB-H012)]", output)
+                self.assertIn("WARN: [HEADER_ONLY, NO COPY SOURCE (KB-H005)]", output)
+                self.assertIn("[FPIC MANAGEMENT (KB-H007)] OK", output)
+
+        for level in ["Warning", "30"]:
+            with tools.environment_append({"CONAN_HOOK_LOGGING_LEVEL": level}):
+                output = self.conan(['create', '.', 'name/version@user/test'])
+                self.assertIn("ERROR: [PACKAGE LICENSE (KB-H012)]", output)
+                self.assertIn("WARN: [HEADER_ONLY, NO COPY SOURCE (KB-H005)]", output)
+                self.assertNotIn("[FPIC MANAGEMENT (KB-H007)] OK", output)
+
+        for level in ["error", "40"]:
+            with tools.environment_append({"CONAN_HOOK_LOGGING_LEVEL": level}):
+                output = self.conan(['create', '.', 'name/version@user/test'])
+                self.assertIn("ERROR: [PACKAGE LICENSE (KB-H012)]", output)
+                self.assertNotIn("WARN: [HEADER_ONLY, NO COPY SOURCE (KB-H005)]", output)
+                self.assertNotIn("[FPIC MANAGEMENT (KB-H007)] OK", output)