Fix open-telemetry#1938

CMakeLists.txt

@@ -175,8 +175,6 @@ option(WITH_ELASTICSEARCH
 
 option(WITH_ZPAGES "Whether to include the Zpages Server in the SDK" OFF)
 
-option(WITH_JAEGER "DEPRECATED - Whether to include the Jaeger exporter" OFF)
-
 option(WITH_NO_GETENV "Whether the platform supports environment variables" OFF)
 
 option(BUILD_TESTING "Whether to enable tests" ON)
@@ -251,25 +249,6 @@ function(install_windows_deps)
       PARENT_SCOPE)
 endfunction()
 
-if(WITH_JAEGER)
-  if(WITH_NO_DEPRECATED_CODE)
-    message(FATAL_ERROR "Jaeger is DEPRECATED.")
-  else()
-    message(WARNING "Jaeger is DEPRECATED.")
-  endif()
-
-  find_package(Thrift QUIET)
-  if(Thrift_FOUND)
-    find_package(Boost REQUIRED)
-    include_directories(${Boost_INCLUDE_DIR})
-  else()
-    # Install Thrift and propagate via vcpkg toolchain file
-    if(WIN32 AND (NOT DEFINED CMAKE_TOOLCHAIN_FILE))
-      install_windows_deps()
-    endif()
-  endif()
-endif()
-
 if(MSVC)
   # Options for Visual C++ compiler: /Zc:__cplusplus - report an updated value
   # for recent C++ language standards. Without this option MSVC returns the
@@ -368,7 +347,6 @@ endif()
 
 if(WITH_OTLP_HTTP
    OR WITH_ELASTICSEARCH
-   OR WITH_JAEGER
    OR WITH_ZIPKIN
    OR BUILD_W3CTRACECONTEXT_TEST
    OR WITH_EXAMPLES_HTTP)

DEPRECATED.md

@@ -34,9 +34,7 @@ N/A
 
 ## [Third party dependencies]
 
-### Jaeger
-
-See [Jaeger exporter](#jaeger-exporter)
+N/A
 
 ## [Build tools]
 
@@ -56,59 +54,7 @@ N/A
 
 ## [opentelemetry-cpp Exporter]
 
-### Jaeger exporter
-
-#### Announcement (Jaeger)
-
-* Version: 1.8.2
-* Date: 2023-01-31
-* PR: [DEPRECATION] Deprecate the Jaeger exporter
-  [#1923](https://github.com/open-telemetry/opentelemetry-cpp/pull/1923)
-
-#### Motivation (Jaeger)
-
-The Jaeger client libraries are deprecated, as announced
-[here](https://www.jaegertracing.io/docs/1.41/client-libraries/).
-
-The initial Jaeger announcement in release 1.35 reads as:
-
-"
-We plan to continue accepting pull requests and making new releases of
-Jaeger clients through the end of 2021. In January 2022 we will enter a code
-freeze period for 6 months, during which time we will no longer accept pull
-requests with new features, with the exception of security-related fixes.
-After that we will archive the client library repositories and will no
-longer accept new changes.
-"
-
-At time of writing, Jan 2023, the client libraries have been archived 6
-months ago already.
-
-#### Scope (Jaeger)
-
-The following are deprecated and planned for removal:
-
-* the API header `opentelemetry/trace/propagation/jaeger.h`, including:
-  * the C++ class `JaegerPropagator`
-* all the code located under `exporters/jaeger/`, including:
-  * the jaeger exporter C++ class (`JaegerExporter`)
-  * the related factory (`JaegerExporterFactory`)
-  * the related options (`JaegerExporterOptions`)
-* the jaeger exporter library(`opentelemetry_exporter_jaeger_trace`)
-* the jaeger build options in CMake (`WITH_JAEGER`)
-* the dependency on thrift
-
-#### Mitigation (Jaeger)
-
-Jaeger supports natively the OTLP protocol, starting with jaeger 1.35.
-
-An application instrumented with opentelemetry needs to change how the SDK
-and exporter are configured to replace the Jaeger exporter with the OTLP
-exporter (both OTLP HTTP and OTLP GRPC are supported).
-
-#### Planned removal (Jaeger)
-
-* Date: After April 1st, 2023
+N/A
 
 ## [Documentation]
 

api/test/trace/propagation/BUILD

@@ -36,20 +36,3 @@ cc_test(
         "@com_google_googletest//:gtest_main",
     ],
 )
-
-cc_test(
-    name = "jaeger_propagation_test",
-    srcs = [
-        "jaeger_propagation_test.cc",
-        "util.h",
-    ],
-    tags = [
-        "api",
-        "test",
-        "trace",
-    ],
-    deps = [
-        "//api",
-        "@com_google_googletest//:gtest_main",
-    ],
-)

api/test/trace/propagation/CMakeLists.txt

@@ -10,16 +10,3 @@ foreach(testname http_text_format_test b3_propagation_test)
     TEST_PREFIX trace.
     TEST_LIST ${testname})
 endforeach()
-
-if(NOT WITH_NO_DEPRECATED_CODE)
-  foreach(testname jaeger_propagation_test)
-
-    add_executable(${testname} "${testname}.cc")
-    target_link_libraries(${testname} ${GTEST_BOTH_LIBRARIES}
-                          ${CMAKE_THREAD_LIBS_INIT} opentelemetry_api)
-    gtest_add_tests(
-      TARGET ${testname}
-      TEST_PREFIX trace.
-      TEST_LIST ${testname})
-  endforeach()
-endif()