sovereign 0.29.0a3__tar.gz → 0.29.1__tar.gz

{sovereign-0.29.0a3 → sovereign-0.29.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: sovereign
-Version: 0.29.0a3
+Version: 0.29.1
 Summary: Envoy Proxy control-plane written in Python
 Home-page: https://pypi.org/project/sovereign/
 License: Apache-2.0
@@ -18,6 +18,7 @@ Classifier: Natural Language :: English
 Classifier: Operating System :: POSIX :: Linux
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
@@ -37,7 +38,7 @@ Requires-Dist: cachelib (>=0.10.2,<0.11.0)
 Requires-Dist: cachetools (>=5.3.2,<6.0.0)
 Requires-Dist: cashews[redis] (>=6.3.0,<7.0.0) ; extra == "caching"
 Requires-Dist: croniter (>=1.4.1,<2.0.0)
-Requires-Dist: cryptography (>=42.0.5,<43.0.0)
+Requires-Dist: cryptography (>=43.0.1,<44.0.0)
 Requires-Dist: datadog (>=0.47.0,<0.48.0) ; extra == "statsd"
 Requires-Dist: fastapi (>=0.110.0,<0.111.0)
 Requires-Dist: glom (>=23.3.0,<24.0.0)
@@ -48,7 +49,7 @@ Requires-Dist: pydantic (>=2.7.2,<3.0.0)
 Requires-Dist: pydantic-settings (>=2.3.1,<3.0.0)
 Requires-Dist: redis (<=5.0.0)
 Requires-Dist: requests (>=2.31.0,<3.0.0)
-Requires-Dist: sentry-sdk (>=1.23.1,<2.0.0) ; extra == "sentry"
+Requires-Dist: sentry-sdk (>=2.14.0,<3.0.0) ; extra == "sentry"
 Requires-Dist: structlog (>=23.1.0,<24.0.0)
 Requires-Dist: ujson (>=5.8.0,<6.0.0) ; extra == "ujson"
 Requires-Dist: uvicorn (>=0.23.2,<0.24.0)

{sovereign-0.29.0a3 → sovereign-0.29.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "sovereign"
-version = "0.29.0a3"
+version = "0.29.1"
 description = "Envoy Proxy control-plane written in Python"
 license = "Apache-2.0"
 packages = [
@@ -73,10 +73,10 @@ Jinja2 = "^3.1.2"
 structlog = "^23.1.0"
 cachelib = "^0.10.2"
 glom = "^23.3.0"
-cryptography = "^42.0.5"
+cryptography = "^43.0.1"
 fastapi = "^0.110.0"
 uvloop = "^0.19.0"
-sentry-sdk = "^1.23.1"
+sentry-sdk = "^2.14.0"
 boto3 = {version = "^1.28.62", optional = true}
 datadog = {version = "^0.47.0", optional = true}
 ujson = {version = "^5.8.0", optional = true}

{sovereign-0.29.0a3 → sovereign-0.29.1}/src/sovereign/discovery.py

@@ -23,7 +23,6 @@ except ImportError:
 from sovereign import XDS_TEMPLATES, config, logs, template_context
 from sovereign.utils.version_info import compute_hash
 from sovereign.schemas import XdsTemplate, DiscoveryRequest, ProcessedTemplate
-from sovereign.tracing import Tracer
 
 
 try:
@@ -106,8 +105,7 @@ def response(request: DiscoveryRequest, xds_type: str) -> ProcessedTemplate:
         resource_names=request.resources,
         **template_context.get_context(request, template),
     )
-    with Tracer("template rendering"):
-        content = template(**context)
+    content = template(**context)
 
     # Deserialize YAML output from Jinja2
     if not template.is_python_source:
@@ -118,19 +116,13 @@ def response(request: DiscoveryRequest, xds_type: str) -> ProcessedTemplate:
         content = deserialize_config(content)
 
     # Early return if the template is identical
-    with Tracer("nested test"):
-        with Tracer("hashing"):
-            config_version = compute_hash(content)
-        if (
-            config_version == request.version_info
-            and not config.discovery_cache.enabled
-        ):
-            return ProcessedTemplate(version_info=config_version, resources=[])
-
-        if not isinstance(content, dict):
-            raise RuntimeError(f"Attempting to filter unstructured data: {content}")
-        with Tracer("filtering"):
-            resources = filter_resources(content["resources"], request.resources)
+    config_version = compute_hash(content)
+    if config_version == request.version_info and not config.discovery_cache.enabled:
+        return ProcessedTemplate(version_info=config_version, resources=[])
+
+    if not isinstance(content, dict):
+        raise RuntimeError(f"Attempting to filter unstructured data: {content}")
+    resources = filter_resources(content["resources"], request.resources)
     return ProcessedTemplate(resources=resources, version_info=config_version)
 
 

{sovereign-0.29.0a3 → sovereign-0.29.1}/src/sovereign/dynamic_config/__init__.py

@@ -1,3 +1,4 @@
+import inspect
 from typing import Any, Dict, Optional
 
 from pydantic import BaseModel
@@ -13,20 +14,20 @@ class Loadable(BaseModel):
     serialization: Optional[str] = None
 
     def load(self, default: Any = None) -> Any:
-        if self.protocol not in custom_loaders:
+        if self.protocol not in LOADERS:
             raise KeyError(
-                f"Could not find CustomLoader {self.protocol}. Available: {custom_loaders}"
+                f"Could not find CustomLoader {self.protocol}. Available: {LOADERS}"
             )
-        loader_ = custom_loaders[self.protocol]
+        loader_ = LOADERS[self.protocol]
 
         ser = self.serialization
         if ser is None:
             ser = loader_.default_deser
-        if ser not in deserializers:
+        if ser not in DESERIALIZERS:
             raise KeyError(
-                f"Could not find Deserializer {ser}. Available: {deserializers}"
+                f"Could not find Deserializer {ser}. Available: {DESERIALIZERS}"
             )
-        deserializer = deserializers[ser]
+        deserializer = DESERIALIZERS[ser]
 
         try:
             data = loader_.load(self.path)
@@ -63,23 +64,25 @@ class Loadable(BaseModel):
         )
 
 
-custom_loaders: Dict[str, CustomLoader] = {}
-loader_entry_point = EntryPointLoader("loaders")
-for entry_point in loader_entry_point.groups["loaders"]:
+LOADERS: Dict[str, CustomLoader] = {}
+for entry_point in EntryPointLoader("loaders").groups["loaders"]:
     custom_loader = entry_point.load()
-    try:
-        func = custom_loader()
-    except AttributeError:
-        raise AttributeError("CustomLoader does not implement .load()")
-    custom_loaders[entry_point.name] = func
+    func = custom_loader()
+    method = getattr(func, "load")
+    if not inspect.ismethod(method):
+        raise AttributeError(
+            f"CustomLoader {entry_point.name} does not implement .load()"
+        )
+    LOADERS[entry_point.name] = func
 
 
-deserializers: Dict[str, ConfigDeserializer] = {}
-deser_entry_point = EntryPointLoader("deserializers")
-for entry_point in deser_entry_point.groups["deserializers"]:
+DESERIALIZERS: Dict[str, ConfigDeserializer] = {}
+for entry_point in EntryPointLoader("deserializers").groups["deserializers"]:
     deserializer = entry_point.load()
-    try:
-        func = deserializer()
-    except AttributeError:
-        raise AttributeError("Deserializer does not implement .deserialize()")
-    deserializers[entry_point.name] = func
+    func = deserializer()
+    method = getattr(func, "deserialize")
+    if not inspect.ismethod(method):
+        raise AttributeError(
+            f"Deserializer {entry_point.name} does not implement .deserialize()"
+        )
+    DESERIALIZERS[entry_point.name] = func

{sovereign-0.29.0a3 → sovereign-0.29.1}/src/sovereign/schemas.py

@@ -112,6 +112,7 @@ class DiscoveryCacheConfig(BaseModel):
     suppress: bool = False  # False = Don't suppress connection errors. True = suppress connection errors
     socket_keepalive: bool = True  # Try to keep connections to redis around.
     ttl: int = 60
+    extra_keys: Dict[str, Any] = {}
 
     @model_validator(mode="after")
     def set_default_protocol(self) -> Self:

{sovereign-0.29.0a3 → sovereign-0.29.1}/src/sovereign/views/discovery.py

@@ -117,21 +117,24 @@ async def perform_discovery(
         authenticate(req)
     if discovery_cache.enabled:
         logs.access_logger.queue_log_fields(CACHE_XDS_HIT=False)
-        cache_key = compute_hash(
-            [
-                api_version,
-                resource_type,
-                req.envoy_version,
-                req.resource_names,
-                req.desired_controlplane,
-                req.hide_private_keys,
-                req.type_url,
-                req.node.cluster,
-                req.node.locality,
-                req.node.metadata.get("auth", None),
-                req.node.metadata.get("num_cpus", None),
-            ]
-        )
+        metadata_keys = discovery_cache.extra_keys.get("metadata", [])
+        extra_metadata = [req.node.metadata.get(key, None) for key in metadata_keys]
+        hash_keys = [
+            api_version,
+            resource_type,
+            req.envoy_version,
+            req.resource_names,
+            req.desired_controlplane,
+            req.hide_private_keys,
+            req.type_url,
+            req.node.cluster,
+            req.node.locality,
+            req.node.metadata.get("auth", None),
+            req.node.metadata.get("num_cpus", None),
+        ] + extra_metadata
+
+        cache_key = compute_hash(hash_keys)
+
         if template := await cache.get(key=cache_key, default=None):
             logs.access_logger.queue_log_fields(CACHE_XDS_HIT=True)
             return template  # type: ignore[no-any-return]