Skip to content

Commit 3d05df5

Browse files
committed
Respond to commnets and update examples
1 parent 7ddac0f commit 3d05df5

File tree

6 files changed

+13
-22
lines changed
  • instrumentation-genai
    • opentelemetry-instrumentation-google-genai/examples/manual
    • opentelemetry-instrumentation-openai-v2
    • opentelemetry-instrumentation-vertexai/examples/manual
  • instrumentation/opentelemetry-instrumentation-botocore

6 files changed

+13
-22
lines changed

instrumentation-genai/opentelemetry-instrumentation-google-genai/examples/manual/main.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@
2222

2323
import google.genai
2424

25-
# NOTE: OpenTelemetry Python Logs and Events APIs are in beta
26-
from opentelemetry import _events as otel_events
25+
# NOTE: OpenTelemetry Python Logs API is in beta
2726
from opentelemetry import _logs as otel_logs
2827
from opentelemetry import metrics as otel_metrics
2928
from opentelemetry import trace as otel_trace
@@ -40,7 +39,6 @@
4039
GoogleGenAiSdkInstrumentor,
4140
)
4241
from opentelemetry.instrumentation.requests import RequestsInstrumentor
43-
from opentelemetry.sdk._events import EventLoggerProvider
4442
from opentelemetry.sdk._logs import LoggerProvider
4543
from opentelemetry.sdk._logs.export import BatchLogRecordProcessor
4644
from opentelemetry.sdk.metrics import MeterProvider
@@ -56,12 +54,11 @@ def setup_otel_tracing():
5654
)
5755

5856

59-
def setup_otel_logs_and_events():
57+
def setup_otel_logs():
6058
otel_logs.set_logger_provider(LoggerProvider())
6159
otel_logs.get_logger_provider().add_log_record_processor(
6260
BatchLogRecordProcessor(OTLPLogExporter())
6361
)
64-
otel_events.set_event_logger_provider(EventLoggerProvider())
6562

6663

6764
def setup_otel_metrics():
@@ -77,7 +74,7 @@ def setup_otel_metrics():
7774

7875
def setup_opentelemetry():
7976
setup_otel_tracing()
80-
setup_otel_logs_and_events()
77+
setup_otel_logs()
8178
setup_otel_metrics()
8279

8380

instrumentation-genai/opentelemetry-instrumentation-openai-v2/examples/manual/main.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33

44
from openai import OpenAI
55

6-
# NOTE: OpenTelemetry Python Logs and Events APIs are in beta
7-
from opentelemetry import _events, _logs, metrics, trace
6+
# NOTE: OpenTelemetry Python Logs API is in beta
7+
from opentelemetry import _logs, metrics, trace
88
from opentelemetry.exporter.otlp.proto.grpc._log_exporter import (
99
OTLPLogExporter,
1010
)
@@ -15,7 +15,6 @@
1515
OTLPSpanExporter,
1616
)
1717
from opentelemetry.instrumentation.openai_v2 import OpenAIInstrumentor
18-
from opentelemetry.sdk._events import EventLoggerProvider
1918
from opentelemetry.sdk._logs import LoggerProvider
2019
from opentelemetry.sdk._logs.export import BatchLogRecordProcessor
2120
from opentelemetry.sdk.metrics import MeterProvider
@@ -29,12 +28,11 @@
2928
BatchSpanProcessor(OTLPSpanExporter())
3029
)
3130

32-
# configure logging and events
31+
# configure logging
3332
_logs.set_logger_provider(LoggerProvider())
3433
_logs.get_logger_provider().add_log_record_processor(
3534
BatchLogRecordProcessor(OTLPLogExporter())
3635
)
37-
_events.set_event_logger_provider(EventLoggerProvider())
3836

3937
# configure metrics
4038
metrics.set_meter_provider(

instrumentation-genai/opentelemetry-instrumentation-openai-v2/pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ classifiers = [
2525
"Programming Language :: Python :: 3.13",
2626
]
2727
dependencies = [
28-
"opentelemetry-api >= 1.37.0",
28+
"opentelemetry-api >= 1.37",
2929
"opentelemetry-instrumentation ~= 0.58b0",
3030
"opentelemetry-semantic-conventions ~= 0.58b0"
3131
]

instrumentation-genai/opentelemetry-instrumentation-openai-v2/src/opentelemetry/instrumentation/openai_v2/patch.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,15 @@
1919
from openai import Stream
2020

2121
from opentelemetry._logs import Logger, LogRecord
22+
from opentelemetry.context import get_current
2223
from opentelemetry.semconv._incubating.attributes import (
2324
gen_ai_attributes as GenAIAttributes,
2425
)
2526
from opentelemetry.semconv._incubating.attributes import (
2627
server_attributes as ServerAttributes,
2728
)
2829
from opentelemetry.trace import Span, SpanKind, Tracer
30+
from opentelemetry.trace.propagation import set_span_in_context
2931

3032
from .instruments import Instruments
3133
from .utils import (
@@ -392,17 +394,13 @@ def cleanup(self):
392394
event_attributes = {
393395
GenAIAttributes.GEN_AI_SYSTEM: GenAIAttributes.GenAiSystemValues.OPENAI.value
394396
}
395-
396-
# this span is not current, so we need to manually set the context on event
397-
span_ctx = self.span.get_span_context()
397+
context = set_span_in_context(self.span, get_current())
398398
self.logger.emit(
399399
LogRecord(
400400
event_name="gen_ai.choice",
401401
attributes=event_attributes,
402402
body=body,
403-
trace_id=span_ctx.trace_id,
404-
span_id=span_ctx.span_id,
405-
trace_flags=span_ctx.trace_flags,
403+
context=context,
406404
)
407405
)
408406

instrumentation-genai/opentelemetry-instrumentation-vertexai/examples/manual/main.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,14 @@
33
from vertexai.generative_models import GenerativeModel
44

55
# NOTE: OpenTelemetry Python Logs and Events APIs are in beta
6-
from opentelemetry import _events, _logs, trace
6+
from opentelemetry import _logs, trace
77
from opentelemetry.exporter.otlp.proto.grpc._log_exporter import (
88
OTLPLogExporter,
99
)
1010
from opentelemetry.exporter.otlp.proto.grpc.trace_exporter import (
1111
OTLPSpanExporter,
1212
)
1313
from opentelemetry.instrumentation.vertexai import VertexAIInstrumentor
14-
from opentelemetry.sdk._events import EventLoggerProvider
1514
from opentelemetry.sdk._logs import LoggerProvider
1615
from opentelemetry.sdk._logs.export import BatchLogRecordProcessor
1716
from opentelemetry.sdk.trace import TracerProvider
@@ -28,7 +27,6 @@
2827
_logs.get_logger_provider().add_log_record_processor(
2928
BatchLogRecordProcessor(OTLPLogExporter())
3029
)
31-
_events.set_event_logger_provider(EventLoggerProvider())
3230

3331
# instrument VertexAI
3432
VertexAIInstrumentor().instrument()

instrumentation/opentelemetry-instrumentation-botocore/pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ classifiers = [
2525
"Programming Language :: Python :: 3.13",
2626
]
2727
dependencies = [
28-
"opentelemetry-api ~= 1.37",
28+
"opentelemetry-api >= 1.37",
2929
"opentelemetry-instrumentation == 0.59b0.dev",
3030
"opentelemetry-semantic-conventions == 0.59b0.dev",
3131
"opentelemetry-propagator-aws-xray ~= 1.0",

0 commit comments

Comments
 (0)