open-telemetry / open-telemetry/opentelemetry-cpp-contrib
opentelemetry-cpp v1.16.0 deprecates API methods currently in use
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 153
- Forks
- 184
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 26
Description
Release v1.16.0 of opentelemetry-cpp includes PR https://github.com/open-telemetry/opentelemetry-cpp/pull/2664, which removes ABI 2 access for several methods.
Two of them are called or mocked in
- exporters/fluentd/test/trace/fluentd_recordable_test.cc
- instrumentation/otel-webserver-module/test/unit/mocks/mock_OpenTelemetry.h
A description of the two batches of relevant changes are at https://github.com/open-telemetry/opentelemetry-cpp/blob/v1.16.0/CHANGELOG.md#1160-2024-06-21
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the v1.16.0 changelog and inspect the affected calls and mocks in exporters/fluentd/test/trace/fluentd_recordable_test.cc and instrumentation/otel-webserver-module/test/unit/mocks/mock_OpenTelemetry.h. Update the two uses affected by the ABI 2 access removal, then run the relevant Fluentd recordable and otel-webserver-module unit tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- observability-sre
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100