libmozdata 0.2.8__tar.gz → 0.2.10__tar.gz

{libmozdata-0.2.8/libmozdata.egg-info → libmozdata-0.2.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 1.1
 Name: libmozdata
-Version: 0.2.8
+Version: 0.2.10
 Summary: Library to access and aggregate several Mozilla data sources.
 Home-page: https://github.com/mozilla/libmozdata
 Author: Mozilla Release Management

libmozdata-0.2.10/VERSION

@@ -0,0 +1 @@
+0.2.10

{libmozdata-0.2.8 → libmozdata-0.2.10}/libmozdata/config.py

@@ -11,12 +11,19 @@ except ImportError:
 
 
 class Config(object):
-    def get(section, option, default=None):
-        raise NotImplementedError
+    def __init__(self):
+        self.local_config = {}
+
+    def set_default(self, section, option, value):
+        self.local_config[(section, option)] = value
+
+    def get(self, section, option, default=None):
+        return self.local_config.get((section, option), default)
 
 
 class ConfigIni(Config):
     def __init__(self, path=None):
+        super().__init__()
         self.config = ConfigParser()
         if path is not None:
             self.config.read(path)
@@ -32,7 +39,9 @@ class ConfigIni(Config):
 
     def get(self, section, option, default=None, type=str):
         if not self.config.has_option(section, option):
-            return default
+            if default is not None:
+                return default
+            return super().get(section, option)
 
         res = self.config.get(section, option)
         if type == list or type == set:
@@ -45,12 +54,17 @@ class ConfigIni(Config):
 
 
 class ConfigEnv(Config):
-    def get(self, section, option, default=None):
+    def get(self, section, option, default=None, type=str):
         env = os.environ.get("LIBMOZDATA_CFG_" + section.upper() + "_" + option.upper())
         if not env:
-            return default
+            if default is not None:
+                return default
+            return super().get(section, option)
 
-        return env
+        if type == list or type == set:
+            return type([s.strip(" /t") for s in env.split(",")])
+        else:
+            return type(env)
 
 
 __config = ConfigIni()
@@ -69,3 +83,8 @@ def get(section, option, default=None, type=str, required=False):
     if required:
         assert value is not None, f"Option {option} in section {section} is not set"
     return value
+
+
+def set_default_value(section, option, value):
+    global __config
+    __config.set_default(section, option, value)

{libmozdata-0.2.8 → libmozdata-0.2.10}/libmozdata/lando.py

@@ -14,7 +14,7 @@ class LandoWarnings(object):
     """
 
     def __init__(self, api_url, api_key):
-        self.api_url = f"{api_url}/diff_warnings"
+        self.api_url = f"{api_url}/diff_warnings/"
         self.api_key = api_key
         self.USER_AGENT = config.get("User-Agent", "name", required=True)
 
@@ -26,7 +26,7 @@ class LandoWarnings(object):
             warning_id = warning["id"]
 
             response = requests.delete(
-                f"{self.api_url}/{warning_id}",
+                f"{self.api_url}{warning_id}",
                 headers={
                     "X-Phabricator-API-Key": self.api_key,
                     "User-Agent": self.USER_AGENT,
@@ -34,7 +34,9 @@ class LandoWarnings(object):
             )
 
             if response.status_code != 200:
-                raise Exception(f"Failed to delete warning with ID {warning_id}!")
+                raise Exception(
+                    f"Failed to delete warning with ID {warning_id} with error {response.status_code}:\n{response.text}"
+                )
 
     def add_warning(self, warning, revision_id, diff_id):
         """
@@ -55,7 +57,7 @@ class LandoWarnings(object):
         )
         if response.status_code != 201:
             raise Exception(
-                f"Failed to add warnings for revision_id {revision_id} and diff_id {diff_id}!"
+                f"Failed to add warnings for revision_id {revision_id} and diff_id {diff_id} with error {response.status_code}:\n{response.text}"
             )
 
     def get_warnings(self, revision_id, diff_id):
@@ -76,7 +78,7 @@ class LandoWarnings(object):
         )
         if response.status_code != 200:
             raise Exception(
-                f"Failed to get warnings for revision_id {revision_id} and diff_id {diff_id}!"
+                f"Failed to get warnings for revision_id {revision_id} and diff_id {diff_id} with error {response.status_code}:\n{response.text}"
             )
 
         return response.json()

{libmozdata-0.2.8 → libmozdata-0.2.10/libmozdata.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 1.1
 Name: libmozdata
-Version: 0.2.8
+Version: 0.2.10
 Summary: Library to access and aggregate several Mozilla data sources.
 Home-page: https://github.com/mozilla/libmozdata
 Author: Mozilla Release Management

libmozdata-0.2.8/VERSION

@@ -1 +0,0 @@
-0.2.8