otlp-json 0.9.2__py3-none-any.whl → 0.9.4__py3-none-any.whl

otlp_json/__init__.py

@@ -7,6 +7,8 @@ from typing import Any, TYPE_CHECKING
 if TYPE_CHECKING:
     from typing_extensions import TypeAlias
 
+    from opentelemetry.trace import Link
+    from opentelemetry._logs import LogRecord
     from opentelemetry.sdk.trace import ReadableSpan, Event
     from opentelemetry.sdk.resources import Resource
     from opentelemetry.sdk.util.instrumentation import InstrumentationScope
@@ -66,26 +68,27 @@ def encode_spans(spans: Sequence[ReadableSpan]) -> bytes:
 
 
 def _resource(resource: Resource):
-    rv = {"attributes": []}
-    for k, v in resource.attributes.items():
+    # TODO: add schema_url once that lands in opentelemetry-sdk
+    return _attributes(resource)
+
+
+def _attributes(
+    thing: Resource | InstrumentationScope | ReadableSpan | Event | Link | LogRecord,
+) -> dict[str, Any]:
+    rv = {"attributes": [], "dropped_attributes_count": 0}
+
+    assert thing.attributes is not None
+    for k, v in thing.attributes.items():
         try:
             rv["attributes"].append({"key": k, "value": _value(v)})
         except ValueError:
             pass
 
-    # NOTE: blocks that contain droppedAttributesCount:
-    # - Event
-    # - Link
-    # - InstrumentationScope
-    # - LogRecord (out of scope for this library)
-    # - Resource
-    rv["dropped_attributes_count"] = len(resource.attributes) - len(rv["attributes"])  # type: ignore
+    rv["dropped_attributes_count"] = len(thing.attributes) - len(rv["attributes"])  # type: ignore
 
-    if not rv["attributes"]:
-        del rv["attributes"]
-
-    if not rv["dropped_attributes_count"]:
-        del rv["dropped_attributes_count"]
+    for k in ("attributes", "dropped_attributes_count"):
+        if not rv[k]:
+            del rv[k]
 
     return rv
 
@@ -99,8 +102,9 @@ def _homogeneous_array(value: list[_LEAF_VALUE]) -> list[_LEAF_VALUE]:
 
 def _value(value: _VALUE) -> dict[str, Any]:
     # Attribute value can be a primitive type, excluging None...
-    # TODO: protobuf allows bytes, but I think OTLP doesn't.
-    # TODO: protobuf allows k:v pairs, but I think OTLP doesn't.
+    # protobuf allows bytes, but I think OTLP spec does not?
+    # protobuf allows k:v pairs, but I think OTLP doesn't.
+    # TODO: read up the spec and validate the allowed type range.
     for klass, (key, post) in _VALUE_TYPES.items():
         if isinstance(value, klass):
             return {key: post(value)}
@@ -109,7 +113,10 @@ def _value(value: _VALUE) -> dict[str, Any]:
 
 
 def _scope(scope: InstrumentationScope):
-    rv = {"name": scope.name}
+    rv = {
+        "name": scope.name,
+        **_attributes(scope),
+    }
     if scope.version:
         rv["version"] = scope.version
     return rv
@@ -123,29 +130,16 @@ def _span(span: ReadableSpan):
         "traceId": _trace_id(span.context.trace_id),
         "spanId": _span_id(span.context.span_id),
         "flags": 0x100 | ([0, 0x200][bool(span.parent and span.parent.is_remote)]),
-        "startTimeUnixNano": str(span.start_time),  # TODO: is it ever optional?
-        "endTimeUnixNano": str(span.end_time),  # -"-
+        "startTimeUnixNano": str(span.start_time),
+        "endTimeUnixNano": str(span.end_time),  # can this be unset?
         "status": _status(span.status),
-        "attributes": [],
+        **_attributes(span),
     }
 
     if span.parent:
         rv["parentSpanId"] = _span_id(span.parent.span_id)
 
-    for k, v in span.attributes.items():  # type: ignore
-        try:
-            rv["attributes"].append({"key": k, "value": _value(v)})
-        except ValueError:
-            pass
-
-    rv["dropped_attributes_count"] = len(span.attributes) - len(rv["attributes"])  # type: ignore
-
-    if not rv["attributes"]:
-        del rv["attributes"]
-
-    if not rv["dropped_attributes_count"]:
-        del rv["dropped_attributes_count"]
-
+    # TODO: is this field really nullable?
     if span.events:
         rv["events"] = [_event(e) for e in span.events]
 
@@ -165,7 +159,7 @@ def _span_id(span_id: int) -> str:
 
 
 def _status(status: Status) -> dict[str, Any]:
-    # FIXME
+    # FIXME: need an example of bad status
     return {}
 
 
@@ -173,21 +167,7 @@ def _event(event: Event) -> dict[str, Any]:
     rv = {
         "name": event.name,
         "timeUnixNano": str(event.timestamp),
-        "attributes": [],
+        **_attributes(event),
     }
-
-    for k, v in event.attributes.items():  # type: ignore
-        try:
-            rv["attributes"].append({"key": k, "value": _value(v)})
-        except ValueError:
-            pass
-
-    rv["dropped_attributes_count"] = len(event.attributes) - len(rv["attributes"])  # type: ignore
-
-    if not rv["attributes"]:
-        del rv["attributes"]
-
-    if not rv["dropped_attributes_count"]:
-        del rv["dropped_attributes_count"]
-
+    # TODO: any optional attributes?
     return rv

{otlp_json-0.9.2.dist-info → otlp_json-0.9.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: otlp-json
-Version: 0.9.2
+Version: 0.9.4
 Summary: 🐍Lightweight OTEL span to JSON converter, no dependencies, pure Python🐍
 Project-URL: Repository, https://github.com/dimaqq/otlp-json
 Project-URL: Issues, https://github.com/dimaqq/otlp-json/issues

otlp_json-0.9.4.dist-info/RECORD

@@ -0,0 +1,5 @@
+otlp_json/__init__.py,sha256=Ur95iX9xoeylZxq_ZKr5jqUNPvlp7ELcgxwU71SC42A,5364
+otlp_json/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+otlp_json-0.9.4.dist-info/METADATA,sha256=swioA9pQ_LXYVL-J0WgTcKjkbdaC7xh70m2rQ4FSBSs,2810
+otlp_json-0.9.4.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+otlp_json-0.9.4.dist-info/RECORD,,

otlp_json-0.9.2.dist-info/RECORD

@@ -1,4 +0,0 @@
-otlp_json/__init__.py,sha256=KFI4PCFiyTOw3wTUClS1hIeUyAMCCZp1F9F6K6Qi_IQ,5862
-otlp_json-0.9.2.dist-info/METADATA,sha256=1CqS5onFoYgi8n4ft4DuQnsHwV_kp4aHC5zoQVxLIZA,2810
-otlp_json-0.9.2.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-otlp_json-0.9.2.dist-info/RECORD,,