Skip to content

opentelemetry-cpp: add version 1.26.0#30041

Merged
jcar87 merged 2 commits into
conan-io:masterfrom
Hailios:opentelemetry-cpp-1.26.0
May 1, 2026
Merged

opentelemetry-cpp: add version 1.26.0#30041
jcar87 merged 2 commits into
conan-io:masterfrom
Hailios:opentelemetry-cpp-1.26.0

Conversation

@Hailios
Copy link
Copy Markdown
Contributor

@Hailios Hailios commented Apr 22, 2026

Summary

Changes to recipe: opentelemetry-cpp/[1.24.0]

Motivation

update opentelemetry-cpp to 1.26.0
open-telemetry/opentelemetry-cpp@v1.24.0...v1.26.0

Details

recipes/opentelemetry-cpp/all/conandata.yml (the new version)
recipes/opentelemetry-cpp/config.yml (the new version)


  • Read the contributing guidelines
  • Checked that this PR is not a duplicate: list of PRs by recipe
  • If this is a bug fix, please link related issue or provide bug details
  • Tested locally with at least one configuration using a recent version of Conan

Add a 👍 reaction to pull requests you find important to help the team prioritize, thanks!

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 22, 2026

CLA assistant check
All committers have signed the CLA.

@marcalff
Copy link
Copy Markdown

remove zipkin from the conanfile.py due to my interpretation of the breaking changes of version 1.25.0 [CONFIGURATION] File configuration - remove zipkin
https://github.com/open-telemetry/opentelemetry-cpp/releases/tag/v1.25.0

@Hailios

Thanks for reaching out.

There is some confusion here.

I am a maintainer of opentelemetry-cpp.

For clarity:

Zipkin is still supported in opentelemetry-cpp.

What is not supported, is to use zipkin from a yaml configuration file, which is a new feature known as "declarative configuration", see repository https://github.com/open-telemetry/opentelemetry-configuration.

So, it makes sense to keep zipkin here, for the time being.

Now, in the long term, be advised that zipkin is deprecated from the specifications, and will be removed later, but this will happen much later, not now.

See https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk_exporters/zipkin.md

@Hailios
Copy link
Copy Markdown
Contributor Author

Hailios commented Apr 22, 2026

I reverted the changes to conanfile.py such that zipkin is left as it was. thanks!

@Hailios
Copy link
Copy Markdown
Contributor Author

Hailios commented Apr 24, 2026

@AbrilRBS

I have signed the CLA multiple times by clicking on the failing CI check, using different computers with different browsers. check still fails. What should I do to get past the license/cla check?

Copy link
Copy Markdown
Contributor

@jcar87 jcar87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@jcar87 jcar87 merged commit 4d72e38 into conan-io:master May 1, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants