opentelemetry-python-contrib

opentelemetry-python-contrib copied to clipboard

aiohttp-server instrumentation doesn't work with several types of importing

1

### Describe your environment Problem not related to the environment ### What happened? Instrumentation doesn't work with several types of importing. Problem with injecting technique through [setattr](https://github.com/open-telemetry/opentelemetry-python-contrib/blob/748c92592d2f476199667629defce4a3bca9ecc9/instrumentation/opentelemetry-instrumentation-aiohttp-server/src/opentelemetry/instrumentation/aiohttp_server/__init__.py#L261). Looks like it...

nesb1

Executing opentelemetry-instrument.exe on an app with a path with space doesn't work

3

### Describe your environment OS: Windows Python version: 3.13.1 Package version: opentelemetry-instrument 0.50b0 ### What happened? Trying to instrument an app with a space in the path doesn't appear to...

eerhardt

> Could we add `gen_ai.response.id` as well? From the [doc](https://cloud.google.com/vertex-ai/docs/reference/rest/v1/GenerateContentResponse) it appears that it's available in the response. _Originally posted by @lzchen in https://github.com/open-telemetry/opentelemetry-python-contrib/pull/3227#discussion_r1943383043_ --- This is missing from the...

aabmass

psycopg: Patch server_cursor_factory

1

# Description When getting a cursor with a name, the Connection class produces a cursor using `server_cursor_factory`. However, this factory isn't instrumented. This PR aims to remedy this issue. ##...

jamestjw

Instrument HttpResponse body stream.

1

**Is your feature request related to a problem?** Yes. We want to instrument botocore "bedrock-runtime" service response. the [result](https://github.com/open-telemetry/opentelemetry-python-contrib/blob/da78275a5560c155492b0b18f48e4db250974226/instrumentation/opentelemetry-instrumentation-botocore/src/opentelemetry/instrumentation/botocore/extensions/types.py#L118) returned contains a "body" which is an [StreamingBody](https://github.com/boto/botocore/blob/5b7129634de27a7bed6fd7bc8aa13a6d7c227cf2/botocore/response.py#L38) object with `self._raw_stream`...

zzhlogin

### Describe your environment OS: macos Python version: 3.11.10 Package version: 0.50b0 ### What happened? I have a django application that uses `DjangoInstrumentor().instrument()` to add instrumentation. If you have a...

kujenga

# Description Leaving this PR as a draft Lambda has confirmed they are ready for this change across all OTel SDKs. Please review in the meantime. This change is apart...

liustve

At this moment, we have an issue with all instrumentation that supports duration metrics based on the stable semantic convention. When using the stable semantic convention opt-in feature, duration histogram...

emdneto

Provide a way for Logs SDK to sample based on span context

4

The OpenAI instrumentation currently only records events if the span is recording: https://github.com/open-telemetry/opentelemetry-python-contrib/blob/2756c1edff42db2984296e0c5dbe668e8de85aa8/instrumentation-genai/opentelemetry-instrumentation-openai-v2/src/opentelemetry/instrumentation/openai_v2/patch.py#L59-L63 This makes sense, since event recording can be quite expensive and we can piggyback on the sampling...

aabmass

Aiohttp-server instrumentation does not follow semantic conventions about http span name

1

### Describe your environment Problem not related to the environment ### What happened? I found that `aiohttp-server` do not follow [convention](https://opentelemetry.io/docs/specs/semconv/http/http-spans/#name) about http span name. There are two things that...

nesb1