Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[opentelemetry-cpp] Remove feature of Jaeger exporter #36326

Merged
merged 1 commit into from
Jan 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions ports/opentelemetry-cpp/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ vcpkg_check_features(OUT_FEATURE_OPTIONS FEATURE_OPTIONS
zipkin WITH_ZIPKIN
prometheus WITH_PROMETHEUS
elasticsearch WITH_ELASTICSEARCH
jaeger WITH_JAEGER
otlp-http WITH_OTLP_HTTP
otlp-grpc WITH_OTLP_GRPC
)
Expand Down Expand Up @@ -55,7 +54,6 @@ vcpkg_cmake_configure(
${FEATURE_OPTIONS}
MAYBE_UNUSED_VARIABLES
WITH_OTLP_GRPC
WITH_JAEGER
WITH_LOGS_PREVIEW
)

Expand Down
7 changes: 1 addition & 6 deletions ports/opentelemetry-cpp/vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
"$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
"name": "opentelemetry-cpp",
"version-semver": "1.13.0",
"port-version": 1,
"description": [
"OpenTelemetry is a collection of tools, APIs, and SDKs.",
"You use it to instrument, generate, collect, and export telemetry data (metrics, logs, and traces) for analysis in order to understand your software's performance and behavior."
Expand Down Expand Up @@ -29,12 +30,6 @@
"description": "Whether to include the ETW Exporter in the SDK",
"supports": "windows"
},
"jaeger": {
"description": "Whether to include the Jaeger exporter",
"dependencies": [
"thrift"
]
},
"otlp": {
"description": "Whether to include the OpenTelemetry Protocol in the SDK",
"dependencies": [
Expand Down
2 changes: 1 addition & 1 deletion versions/baseline.json
Original file line number Diff line number Diff line change
Expand Up @@ -6406,7 +6406,7 @@
},
"opentelemetry-cpp": {
"baseline": "1.13.0",
"port-version": 0
"port-version": 1
},
"opentelemetry-fluentd": {
"baseline": "2.0.0",
Expand Down
5 changes: 5 additions & 0 deletions versions/o-/opentelemetry-cpp.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
{
"versions": [
{
"git-tree": "6df7abbc5ea5cb1da3d33640ccfa7d0f086319c0",
"version-semver": "1.13.0",
"port-version": 1
},
{
"git-tree": "fe04f9f3867c1225a0636a9f410871b30dce6670",
"version-semver": "1.13.0",
Expand Down