chore(deps): update opentelemetry to v1.29.0 - #16199
Conversation
|
/gcbrun |
There was a problem hiding this comment.
Code Review
This pull request updates the opentelemetry-cpp dependency version to 1.27.0 across Bazel configuration files, Dockerfiles, and packaging documentation. The reviewer feedback suggests using a Docker ARG instruction to define the OpenTelemetry version in the Dockerfiles to improve maintainability and avoid duplicating the version string.
| # ```bash | ||
| WORKDIR /var/tmp/build/opentelemetry-cpp | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.24.0.tar.gz | \ | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.27.0.tar.gz | \ |
There was a problem hiding this comment.
To improve maintainability and avoid duplicating the version string across multiple Dockerfiles, use an ARG instruction to define the OpenTelemetry version. This allows the version to be centralized or overridden during builds.
ARG OPENTELEMETRY_VERSION=1.27.0
RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v${OPENTELEMETRY_VERSION}.tar.gz | \
References
- In Dockerfiles, use
ARGto define version strings that are duplicated across multiple files to improve maintainability.
| # ```bash | ||
| WORKDIR /var/tmp/build/opentelemetry-cpp | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.24.0.tar.gz | \ | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.27.0.tar.gz | \ |
There was a problem hiding this comment.
To improve maintainability and avoid duplicating the version string across multiple Dockerfiles, use an ARG instruction to define the OpenTelemetry version. This allows the version to be centralized or overridden during builds.
ARG OPENTELEMETRY_VERSION=1.27.0
RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v${OPENTELEMETRY_VERSION}.tar.gz | \
References
- In Dockerfiles, use
ARGto define version strings that are duplicated across multiple files to improve maintainability.
| # ```bash | ||
| WORKDIR /var/tmp/build/opentelemetry-cpp | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.24.0.tar.gz | \ | ||
| RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v1.27.0.tar.gz | \ |
There was a problem hiding this comment.
To improve maintainability and avoid duplicating the version string across multiple Dockerfiles, use an ARG instruction to define the OpenTelemetry version. This allows the version to be centralized or overridden during builds.
ARG OPENTELEMETRY_VERSION=1.27.0
RUN curl -fsSL https://github.com/open-telemetry/opentelemetry-cpp/archive/v${OPENTELEMETRY_VERSION}.tar.gz | \
References
- In Dockerfiles, use
ARGto define version strings that are duplicated across multiple files to improve maintainability.
31dfdd7 to
7ab86ba
Compare
|
/gcbrun |
7ab86ba to
637f8c0
Compare
|
/gcbrun |
637f8c0 to
b674298
Compare
|
/gcbrun |
b674298 to
5c8d4d1
Compare
|
/gcbrun |
5c8d4d1 to
de11af3
Compare
|
/gcbrun |
de11af3 to
0e04a9d
Compare
|
/gcbrun |
0e04a9d to
f834228
Compare
|
/gcbrun |
f834228 to
8058af8
Compare
|
/gcbrun |
8058af8 to
54ce3f1
Compare
|
/gcbrun |
54ce3f1 to
785be29
Compare
|
/gcbrun |
785be29 to
cc8c22e
Compare
|
/gcbrun |
cc8c22e to
3654e56
Compare
|
/gcbrun |
3654e56 to
0762747
Compare
|
/gcbrun |
0762747 to
18b0f3a
Compare
|
/gcbrun |
18b0f3a to
f58d4a2
Compare
|
/gcbrun |
This PR contains the following updates:
v1.24.0→v1.29.01.24.0→1.29.0v1.20.0→v1.29.0Release Notes
open-telemetry/opentelemetry-cpp (open-telemetry/opentelemetry-cpp)
v1.29.0: releaseCompare Source
v1.29.0 release
Release of:
Breaking changes:
[CONFIGURATION] Align declarative configuration schema defaults with
OpenTelemetry Configuration Schema v1.1.0
#4432
these schema defaults instead of the previous legacy values:
attribute_limits.attribute_value_length_limit,logger_provider.limits.attribute_value_length_limit, andtracer_provider.limits.attribute_value_length_limitdefault to no limit(unlimited attribute value length) instead of 4096 bytes.
batchlog record processorschedule_delaydefaults to 1000 ms insteadof 5000 ms (more frequent batch exports when the field is omitted).
periodicmetric readerintervaldefaults to 60000 ms instead of 5000 ms(less frequent metric exports when the field is omitted).
trace_id_ratio_basedsamplerratiodefaults to 1.0 instead of 0.0(sample all traces instead of dropping all when the field is omitted).
[CONFIGURATION] SDK default component builder libraries and example
#4367
the default SDK component builders are provided optionally through
the
opentelemetry-cpp::configuration_registry_factorylibrary andthe
opentelemetry::sdk::configuration::RegistryFactory::Create()method.[BUILD] Install an explicit list of ext headers instead of the whole
directory
#4327
ext_commoncomponent installedinclude/opentelemetry/extwith a*.hglob. It now installs a named list, and these five headers are nolonger part of the package:
opentelemetry/ext/http/client/curl/http_client_curl.hopentelemetry/ext/http/client/curl/http_operation_curl.hopentelemetry/ext/http/client/detail/default_factory.hopentelemetry/ext/http/server/http_server.hopentelemetry/ext/http/server/socket_tools.hcompile. A curl client is created through
http_client_factory_curl.h,which needs only the abstract
http_client_factory.h. The two serverheaders are an embedded test server; moving them out of
extis trackedin #4332.
[CMAKE] Rename cmake options with prefix
OTELCPP_#4268
OTELCPP_prefix to avoidname collisions with user and third-party project variables. For example,
WITH_OTLP_HTTP,WITH_CONFIGURATION, andWITH_OTLP_RETRY_PREVIEWarenow
OTELCPP_WITH_OTLP_HTTP,OTELCPP_WITH_CONFIGURATION, andOTELCPP_WITH_OTLP_RETRY_PREVIEW.BUILD_TESTINGbecomesOTELCPP_BUILD_TESTING, but is not treated as adeprecated alias:
BUILD_TESTINGsets thedefault value of
OTELCPP_BUILD_TESTING.BUILD_TESTINGno longer controls opentelemetry-cpp tests, which defaultto
OFF.OTELCPP_BUILD_TESTINGexplicitly to enable or disableopentelemetry-cpp tests in either top-level or subproject builds.
OTELCPP_WITH_BENCHMARKnow defaults to the value ofOTELCPP_BUILD_TESTINGinstead of always defaulting toON.deprecation warning is emitted at configure time, and the
OTELCPP_nametakes precedence when both names are set.
[METRICS SDK] Rename Base2 Exponential Histogram Aggregation config field
#4253
max_buckets_was renamed tomax_size_tomatch the configuration schema. Please adjust SDK configuration accordingly.
[CONFIGURATION] Update prometheus config to schema v1.1.0
#4383
translation_strategyhave been replacedUnderscoreEscapingWithSuffixesis replaced byunderscore_escaping_with_suffixesUnderscoreEscapingWithoutSuffixesis replaced byunderscore_escaping_without_suffixes/developmentNoUTF8EscapingWithSuffixesis replaced byno_utf8_escaping_with_suffixes/developmentNoTranslationis replaced byno_translation/development[METRICS SDK] Add an
ExemplarFilterTypeparameter to theMeterContextconstructor when metrics exemplars are enabled. The parameter has a default
value, so existing source calls remain valid, but the constructor signature
and SDK ABI change. Applications using the preview exemplar feature must be
rebuilt.
#4328
[METRICS SDK] Remove the
SystemTimestampparameter from the previewExemplarReservoir::OfferMeasurement()overloads#4267
Implementations and callers must remove the timestamp parameter.
[METRICS SDK] Breaking change to the preview metrics exemplar surface: the
SyncMetricStorage/AsyncMetricStorageconstructors now take anExemplarFilterType, andExemplarData::Createtakes theSpanContextby value.
#4267
Important changes
[CONFIGURATION] Complete resource detection support in SdkBuilder (#4424)
#4424
SetResourceno longer runsOTELResourceDetectorand nolonger injects a default
service.name, since neither is part of theconfiguration model. Use the
servicedetector or set the attributes inthe configuration file instead.
[API] Never set a null global provider or propagator
#4420
trace::Provider::SetTracerProvider(),metrics::Provider::SetMeterProvider(),logs::Provider::SetLoggerProvider(),logs::Provider::SetEventLoggerProvider()andcontext::propagation::GlobalTextMapPropagator::SetGlobalPropagator()now install the corresponding no-op implementation when passed a null
pointer, instead of storing the null. The matching getters therefore never
return
nullptr, as already documented, and code that resets a globalduring cleanup no longer risks a nullptr de-ref on a later use.
Note: this is a correctness fix to an inline API header; the observable
behavior of these methods changes towards the already documented contract
(see
docs/abi-policy.md).[SDK] Enforce Span Status transition rules (#4547)
#4547
SetStatuscall overwrote the recorded status. It nowfollows the specification, which changes behavior for users in three ways:
StatusCode::kUnsetis ignored.kOk, it is final. So a later call can no longerchange an explicit
kOk.kError; it is dropped forkOkandkUnset.ETWSpan::SetStatusin the ETW exporter follows the same rules.Deprecations
This release contains deprecations, see file DEPRECATED.md
What's Changed
SpinLockMutextostd::mutexpart 1 by @dbarker in #4323/proc/self/mountinfoby @dbarker in #4525configuration_coredependency on the SDKmetricslibrary by @dbarker in #4554configuration_coredependency on the SDKlogslibrary by @dbarker in #4553configuration_coredependency on the SDKtracelibrary by @dbarker in #4555New Contributors
Full Changelog: open-telemetry/opentelemetry-cpp@v1.28.0...v1.29.0
v1.28.0: releaseCompare Source
v1.28.0 release
Release of:
Important changes
[API] fix the trace state regex to comply with the w3c trace context level 2 (#4194)
#4194
TraceState::IsValidKey()to comply with the W3C Trace ContextLevel 2, where keys containing
@and keys with more than 241 charactersbefore
@or more than 14 characters after@are now accepted.Only the total 256-character key length limit is enforced.
Note: this is a correctness fix to an inline API header; the observable
behavior of
IsValidKey,IsValidKeyRegEx, andIsValidKeyNonRegExchanges (see
docs/abi-policy.md).[API+SDK] apply filtering EmitLogRecord (#4079)
#4079
[API] (ABI v2)
Logger::EmitLogRecord(...)templates now apply theEnabledfilter chain when aSeverityis in args. v1 behavior isunchanged.
[API/SDK] (ABI v2) Add
Logger::CreateLogRecordvirtual takingconst nostd::variant<trace::SpanContext, context::Context> &for explicit-context record creation.
Logger::EmitLogRecord(args...)also detects a
Context,SpanContextor
TraceId+SpanId[+TraceFlags] in args and routes filtering.[SDK] Add
LogRecordProcessor::HasEnabledFilter()so the SDK Logger caninclude processor-level filtering in its extended-enabled cache. Defaults
to
true. Built-inSimpleLogRecordProcessorandBatchLogRecordProcessoroverride tofalsesince they use the defaultEnabled.
[API/SDK] Replace
Context-only signatures onLogRecordProcessor::Enabled,LogRecordProcessor::EnabledImplementation,Logger::EnabledImplementation(v2), andLogger::CreateLogRecord(v2)with
nostd::variant<trace::SpanContext, context::Context>.Deprecations
This release contains deprecations, see file DEPRECATED.md
What's Changed
Configuration
📅 Schedule: (UTC)
* * * * 1-5)🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.