chore(deps): update opentelemetry#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 |
This PR contains the following updates:
v1.24.0→v1.28.01.24.0→1.27.0v1.20.0→v1.28.0Release Notes
open-telemetry/opentelemetry-cpp (open-telemetry/opentelemetry-cpp)
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
WITH_LOG_FILTERING_PREVIEWoption by @dbarker in #4160Bazel on MacOSjob runner to macos-15 by @dbarker in #4176bugprone-throwing-static-initializationwarnings by @RaviTriv in #4206modernize-andcppcoreguidelines-pro-checks to clang-tidy by @dbarker in #4238New Contributors
Full Changelog: open-telemetry/opentelemetry-cpp@v1.27.0...v1.28.0
v1.27.0: releaseCompare Source
v1.27.0 release
Release of:
Security fix
[EXPORTER] OTLP HTTP exporters read unbounded HTTP response
#4078
the exporter could allocate an arbitrary amount of memory when getting
the endpoint HTTP response back.
following the opentelemetry-proto recommendations.
Breaking changes
[SDK] env var durations non conforming to spec
#4020
(
15s,15000ms) or without units (15000).and only in the case no unit is provided,
for example
OTEL_METRIC_EXPORT_INTERVAL=15000,the code interpreted the value,
15000,in seconds instead of milliseconds per the specifications.
without a unit is now interpreted differently, which is a breaking change.
OTEL_EXPORTER_OTLP_TRACES_TIMEOUT=30meant 30 secondsbefore, and now means 30 milliseconds.
To preserve the same behavior, existing values must be adjusted,
to either
30000or30000msor30sin this case.Important changes
[SDK] Move inline implementation from SDK headers to .cc files.
#3887
GetEmptyAttributes()now requires linkingopentelemetry_common.Enable WITH_OTLP_RETRY_PREVIEW by default
#3953
Enable WITH_OTLP_GRPC_SSL_MTLS_PREVIEW by default
#3970
adjust the application code to initialize all members in grpc options.
[EXPORTER] implement non-utf8 string to bytes in OTLP exporters
#3991
and enable ENABLE_OTLP_UTF8_VALIDITY for Bazel
to export non-UTF-8 strings as bytes in OTLP.
Enable ENABLE_OTLP_RETRY_PREVIEW for bazel
#4010
[API] Deprecate opentelemetry::plugin
#4021
[SDK] Fix cardinality-limit overflow attribute name to match the
specification
#4060
otel.metric.overflow(singular) per theMetrics SDK specification.
otel.metrics.overflow(plural), whichdiverged from the spec and from every other language SDK
(Go, Java, JS, .NET).
the old name must be updated to the spec-correct name.
Deprecations
This release contains deprecations, see file DEPRECATED.md
What's Changed
commonandcontextby @dbarker in #3948Logger.Enabled()by @proost in #4011New Contributors
Full Changelog: open-telemetry/opentelemetry-cpp@v1.26.0...v1.27.0
v1.26.0: releaseCompare Source
v1.26.0 release
Release of:
Important changes
[SDK] Swap Tracer/Meter/LoggerConfig disabled for enabled
#3942
the boolean parameter now represents
enabledinstead ofdisabled.to pass
falseto disable.[BUILD] Revisit EventLogger deprecation
#3855
DEPRECATED.mdcontains information about current deprecationplans.
[CONFIGURATION] File configuration - spec stability
#3862
OTEL_EXPERIMENTAL_CONFIG_FILEis now renamed to
OTEL_CONFIG_FILE.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.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.