Skip to content

fix(pyroscope.ebpf): Use meter noop in otel-ebpf-profiler#4920

Merged
dehaansa merged 1 commit intomainfrom
20251124_pyroscope-ebpf-use-meter-noop
Nov 24, 2025
Merged

fix(pyroscope.ebpf): Use meter noop in otel-ebpf-profiler#4920
dehaansa merged 1 commit intomainfrom
20251124_pyroscope-ebpf-use-meter-noop

Conversation

@simonswine
Copy link
Contributor

With a recent change, the default meter provider was no longer
initialized. This change disables the otel-ebpf-profiler internal
metrics to avoid warnings like this to appear:

WARN[0121] Invalid metric id 102, skipping
WARN[0121] Invalid metric id 272, skipping

With a recent change, the default meter provider was no longer
initialized. This change disables the otel-ebpf-profiler internal
metrics to avoid warnings like this to appear:

```
WARN[0121] Invalid metric id 102, skipping
WARN[0121] Invalid metric id 272, skipping
```
@simonswine
Copy link
Contributor Author

@bobrik thanks for noticing and reporting:

#4816 (comment)

@simonswine simonswine marked this pull request as ready for review November 24, 2025 15:37
@simonswine simonswine requested a review from a team as a code owner November 24, 2025 15:37
@simonswine simonswine added the area/pyroscope Issues/PRs primarly affecting `pyroscope.` components label Nov 24, 2025
@dehaansa dehaansa enabled auto-merge (squash) November 24, 2025 15:40
@simonswine
Copy link
Contributor Author

simonswine commented Nov 24, 2025

@jharvey10 it would be great, if you could include this into the next rc for v1.12

@dehaansa dehaansa merged commit 671123c into main Nov 24, 2025
47 checks passed
@dehaansa dehaansa deleted the 20251124_pyroscope-ebpf-use-meter-noop branch November 24, 2025 15:50
jharvey10 pushed a commit that referenced this pull request Nov 24, 2025
With a recent change, the default meter provider was no longer
initialized. This change disables the otel-ebpf-profiler internal
metrics to avoid warnings like this to appear:

```
WARN[0121] Invalid metric id 102, skipping
WARN[0121] Invalid metric id 272, skipping
```
@jharvey10 jharvey10 mentioned this pull request Nov 24, 2025
jharvey10 added a commit that referenced this pull request Nov 24, 2025
* ci: sync publish workflow with main

* Vendor Alertmanager config (#4905)

* Vendor Alertmanager config

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* beyla: add meta_cache_address to beyla.ebpf.attributes.kubernetes (#4871)

* beyla: add meta_cache_address to beyla.ebpf.attributes.kubernetes

* chore: update changelog

* chore: lint

---------

Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>

* Upgrade Beyla component to 2.7.7 (#4891)

* upgrade beyla to 2.7.7

* update to Beyla 2.7.8

* update docs

* fix(pyroscope.ebpf): Use meter noop in otel-ebpf-profiler (#4920)

With a recent change, the default meter provider was no longer
initialized. This change disables the otel-ebpf-profiler internal
metrics to avoid warnings like this to appear:

```
WARN[0121] Invalid metric id 102, skipping
WARN[0121] Invalid metric id 272, skipping
```

* bump rc version in changelog

---------

Co-authored-by: Paulin Todev <paulin.todev@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Stephen Lang <skl@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Nikola Grcevski <6207777+grcevski@users.noreply.github.com>
Co-authored-by: Christian Simon <simon@swine.de>
dehaansa pushed a commit to madhub/alloy that referenced this pull request Dec 10, 2025
With a recent change, the default meter provider was no longer
initialized. This change disables the otel-ebpf-profiler internal
metrics to avoid warnings like this to appear:

```
WARN[0121] Invalid metric id 102, skipping
WARN[0121] Invalid metric id 272, skipping
```
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/pyroscope Issues/PRs primarly affecting `pyroscope.` components frozen-due-to-age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants