Skip to content

Merge release/v0.130.x into main#13452

Merged
mx-psi merged 4 commits into
mainfrom
release/v0.130.x
Jul 22, 2025
Merged

Merge release/v0.130.x into main#13452
mx-psi merged 4 commits into
mainfrom
release/v0.130.x

Conversation

@mx-psi
Copy link
Copy Markdown
Member

@mx-psi mx-psi commented Jul 22, 2025

Description

Follow step 3 from the bugfix release procedure:

Once those changes have been merged, create a pull request to the main branch from the release/ branch.

This includes #13449 in main

Link to tracking issue

Updates #13381
Relates to open-telemetry/opentelemetry-go/issues/7039

mx-psi and others added 4 commits July 18, 2025 11:49
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Downgrades opentelemetry-go dependencies effectively reverting #13289
and others

### ❗ Known Issues ❗

- Due to a
[bug](open-telemetry/opentelemetry-go#7039) in
the prometheus exporter, if you are configuring a prometheus exporter,
the collector's internal metrics will be emitted with an unexpected
suffix in its name. For example, the metric
`otelcol_exporter_sent_spans__spans__total` instead of
`otelcol_exporter_sent_spans_total`. The workaround is to manually
configure `without_units: true` in your prometheus exporter config

  ```yaml
  service:
    telemetry:
      metrics:
        readers:
          - pull:
              exporter:
                prometheus:
                  host: 0.0.0.0
                  port: 8888
                  without_units: true
  ```
If you are using the collector's default Prometheus exporter for
exporting internal metrics you are unaffected.

---------

Signed-off-by: alex boten <223565+codeboten@users.noreply.github.com>
Co-authored-by: alex boten <223565+codeboten@users.noreply.github.com>
Co-authored-by: Yang Song <songy23@users.noreply.github.com>
Reverts #13429 so that we can bump
to the to-be-released bugfix version that adds
open-telemetry/opentelemetry-go/pull/7044
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Updates
#13381
Relates to
open-telemetry/opentelemetry-go#7039
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Prepares release v0.130.1/v1.36.1 to release #13429

<!-- Issue number if applicable -->
#### Link to tracking issue

Updates #13381
Relates to open-telemetry/opentelemetry-go/issues/7039

---------

Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
@mx-psi mx-psi marked this pull request as ready for review July 22, 2025 09:11
@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 22, 2025
@mx-psi
Copy link
Copy Markdown
Member Author

mx-psi commented Jul 22, 2025

@open-telemetry/admins I need some help to follow step 5 of the bugfix release guidelines:

Disable the merge queue. An admin of the repo needs to be available for this.

Can somebody help me disable the merge queue temporarily on this repository so that I can merge this? Thank you

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.76%. Comparing base (91ec67d) to head (d588ee2).
Report is 43 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13452      +/-   ##
==========================================
+ Coverage   91.53%   91.76%   +0.22%     
==========================================
  Files         528      533       +5     
  Lines       29469    30040     +571     
==========================================
+ Hits        26974    27565     +591     
+ Misses       1969     1944      -25     
- Partials      526      531       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@svrnm
Copy link
Copy Markdown
Member

svrnm commented Jul 22, 2025

merge queue is disabled

@mx-psi mx-psi merged commit ba583e0 into main Jul 22, 2025
63 of 72 checks passed
@mx-psi mx-psi deleted the release/v0.130.x branch July 22, 2025 10:41
@svrnm
Copy link
Copy Markdown
Member

svrnm commented Jul 22, 2025

merge queue is re-enabled

@mx-psi mx-psi restored the release/v0.130.x branch July 22, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants