Add the logger resource to the log record

Author: xrmx

opentelemetry-sdk/src/opentelemetry/sdk/_logs/_internal/__init__.py

@@ -338,7 +338,9 @@ def dropped_attributes(self) -> int:
         return 0
 
     @classmethod
-    def _from_api_log_record(cls, record: APILogRecord) -> LogRecord:
+    def _from_api_log_record(
+        cls, *, record: APILogRecord, resource: Resource
+    ) -> LogRecord:
         return cls(
             timestamp=record.timestamp,
             observed_timestamp=record.observed_timestamp,
@@ -351,6 +353,7 @@ def _from_api_log_record(cls, record: APILogRecord) -> LogRecord:
             body=record.body,
             attributes=record.attributes,
             event_name=record.event_name,
+            resource=resource,
         )
 
 
@@ -701,8 +704,9 @@ def emit(self, record: APILogRecord):
         and instrumentation info.
         """
         if not isinstance(record, LogRecord):
-            record = LogRecord._from_api_log_record(record)
-            record.resource = self._resource
+            record = LogRecord._from_api_log_record(
+                record=record, resource=self._resource
+            )
         log_data = LogData(record, self._instrumentation_scope)
         self._multi_log_record_processor.on_emit(log_data)
 

opentelemetry-sdk/tests/logs/test_log_record.py

@@ -199,7 +199,10 @@ def test_log_record_from_api_log_record(self):
             event_name="an.event",
         )
 
-        record = LogRecord._from_api_log_record(api_log_record)
+        resource = Resource.create({})
+        record = LogRecord._from_api_log_record(
+            record=api_log_record, resource=resource
+        )
 
         self.assertEqual(record.timestamp, 1)
         self.assertEqual(record.observed_timestamp, 2)
@@ -212,3 +215,4 @@ def test_log_record_from_api_log_record(self):
         self.assertEqual(record.body, "a log line")
         self.assertEqual(record.attributes, {"a": "b"})
         self.assertEqual(record.event_name, "an.event")
+        self.assertEqual(record.resource, resource)