Attempt building opentracing to test dynamic loading

Signed-off-by: Isaac Hier <ihier@uber.com>

Author: Isaac Hier

CMakeLists.txt

@@ -107,18 +107,18 @@ if(JAEGERTRACING_BUILD_CROSSDOCK)
   set(CROSSDOCK_SRC crossdock/Server.cpp)
   add_executable(crossdock ${CROSSDOCK_SRC})
   target_include_directories(crossdock PUBLIC
-    $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/crossdock>)
-  target_link_libraries(crossdock PUBLIC jaegertracing-static)
+    $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/crossdock>)
+  target_link_libraries(crossdock PUBLIC ${JAEGERTRACING_LIB})
 
   set(JAEGER_CROSSDOCK_URL
 "https://raw.githubusercontent.com/jaegertracing/jaeger/master/docker-compose/\
 jaeger-docker-compose.yml")
   file(DOWNLOAD ${JAEGER_CROSSDOCK_URL}
-       "${PROJECT_SOURCE_DIR}/crossdock/jaeger-docker-compose.yml")
+       "${CMAKE_CURRENT_SOURCE_DIR}/crossdock/jaeger-docker-compose.yml")
   find_program(DOCKER_COMPOSE_EXE docker-compose REQUIRED)
   set(DOCKER_COMPOSE_CMD ${DOCKER_COMPOSE_EXE}
-      -f ${PROJECT_SOURCE_DIR}/crossdock/docker-compose.yml
-      -f ${PROJECT_SOURCE_DIR}/crossdock/jaeger-docker-compose.yml)
+      -f ${CMAKE_CURRENT_SOURCE_DIR}/crossdock/docker-compose.yml
+      -f ${CMAKE_CURRENT_SOURCE_DIR}/crossdock/jaeger-docker-compose.yml)
   add_custom_target(crossdock-kill
     COMMAND ${DOCKER_COMPOSE_CMD} kill
     COMMAND ${DOCKER_COMPOSE_CMD} rm --force)
@@ -231,28 +231,35 @@ set(SRC
     src/jaegertracing/utils/UDPClient.cpp
     src/jaegertracing/utils/YAML.cpp)
 
-add_library(jaegertracing SHARED ${SRC})
-target_include_directories(jaegertracing PUBLIC
-  $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/src>
-  $<BUILD_INTERFACE:${PROJECT_BINARY_DIR}/src>)
-target_link_libraries(jaegertracing ${LIBS})
-set_target_properties(jaegertracing PROPERTIES
-  VERSION ${PROJECT_VERSION}
-  SOVERSION ${PROJECT_VERSION_MAJOR})
+if(BUILD_SHARED_LIBS)
+  add_library(jaegertracing SHARED ${SRC})
+  target_include_directories(jaegertracing PUBLIC
+    $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src>
+    $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/src>)
+  target_link_libraries(jaegertracing ${LIBS})
+  set_target_properties(jaegertracing PROPERTIES
+    VERSION ${PROJECT_VERSION}
+    SOVERSION ${PROJECT_VERSION_MAJOR})
+  set(JAEGERTRACING_LIB jaegertracing)
+  list(APPEND JAEGERTRACING_LIBS jaegertracing)
+endif()
 
 add_library(jaegertracing-static STATIC ${SRC})
 target_include_directories(jaegertracing-static PUBLIC
-  $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/src>
-  $<BUILD_INTERFACE:${PROJECT_BINARY_DIR}/src>)
+  $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/src>
+  $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/src>)
 target_link_libraries(jaegertracing-static ${LIBS})
 set_target_properties(jaegertracing-static PROPERTIES
   OUTPUT_NAME jaegertracing)
+if(NOT JAEGERTRACING_LIB)
+  set(JAEGERTRACING_LIB jaegertracing-static)
+endif()
+list(APPEND JAEGERTRACING_LIBS jaegertracing-static)
 
 configure_file(
   src/jaegertracing/Constants.h.in
   src/jaegertracing/Constants.h
   @ONLY)
-set(CMAKE_INCLUDE_CURRENT_DIR ON)
 
 if(BUILD_TESTING)
   add_subdirectory("${CMAKE_CURRENT_SOURCE_DIR}/src/jaegertracing/testutils")
@@ -290,7 +297,7 @@ if(BUILD_TESTING)
       GTEST_HAS_TR1_TUPLE=0
       GTEST_USE_OWN_TR1_TUPLE=0)
   target_link_libraries(
-      UnitTest testutils GTest::main jaegertracing ${LIBS})
+      UnitTest testutils GTest::main ${JAEGERTRACING_LIB} ${LIBS})
   add_test(NAME UnitTest COMMAND UnitTest)
 
   add_executable(DynamicallyLoadTracerTest src/jaegertracing/DynamicallyLoadTracerTest.cpp)
@@ -348,7 +355,7 @@ configure_package_config_file(
 #   * header location after install: <prefix>/include/jaegertracing/Tracer.h
 #   * headers can be included by C++ code `#include <jaegertracing/Tracer.h>`
 install(
-  TARGETS jaegertracing jaegertracing-static
+  TARGETS ${JAEGERTRACING_LIBS}
   EXPORT "${TARGETS_EXPORT_NAME}"
   LIBRARY DESTINATION "lib"
   ARCHIVE DESTINATION "lib"

Dockerfile.plugin

@@ -1,9 +1,9 @@
 FROM debian
 RUN apt-get update
-RUN apt-get install --yes build-essential curl
+RUN apt-get install --yes build-essential curl git
 RUN curl -L -O "https://cmake.org/files/v3.11/cmake-3.11.0-Linux-x86_64.sh" && \
     bash cmake-3.11.0-Linux-x86_64.sh --skip-license
 
-ADD . /src
-WORKDIR /src
+ADD . /src/jaeger-cpp-client
+WORKDIR /src/jaeger-cpp-client
 RUN ./scripts/build-plugin.sh

examples/config.yml

@@ -0,0 +1,19 @@
+service: test-service
+disabled: false
+sampler:
+    type: probabilistic
+    param: 0.001
+reporter:
+    queueSize: 100
+    bufferFlushInterval: 10
+    logSpans: false
+    localAgentHostPort: 127.0.0.1:6831
+headers:
+    jaegerDebugHeader: debug-id
+    jaegerBaggageHeader: baggage
+    TraceContextHeaderName: trace-id
+    traceBaggageHeaderPrefix: testctx-
+baggage_restrictions:
+    denyBaggageOnInitializationFailure: false
+    hostPort: 127.0.0.1:5778
+    refreshInterval: 60

scripts/build-plugin.sh

@@ -4,43 +4,38 @@ set -e
 
 function main() {
     local project_dir
-    if [[ "$(which git)" != "" ]]; then
-        project_dir="$(git rev-parse --show-toplevel)"
-    else
-        project_dir="$PWD"
-    fi
-
-    BUILD_DIR=${BUILD_DIR:-${project_dir}}
-    cd "$BUILD_DIR"
+    project_dir="$(git rev-parse --show-toplevel)"
 
     mkdir -p build
     cd build
     export CFLAGS="$CFLAGS -march=x86-64"
     export CXXFLAGS="$CXXFLAGS -march=x86-64"
+
+    cat <<EOF > export.map
+{
+    global:
+        OpenTracingMakeTracerFactory;
+    local: *;
+};
+EOF
+
     cmake -DCMAKE_BUILD_TYPE=Release \
         -DCMAKE_INSTALL_PREFIX="${BUILD_DIR}/install" \
         -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
+        -DCMAKE_SHARED_LINKER_FLAGS="-Wl,--version-script=export.map" \
         -DBUILD_SHARED_LIBS=OFF \
         -DBUILD_TESTING=OFF \
         -DJAEGERTRACING_COVERAGE=OFF \
         -DJAEGERTRACING_BUILD_CROSSDOCK=OFF \
         -DJAEGERTRACING_WITH_YAML_CPP=ON \
         -DHUNTER_CONFIGURATION_TYPES=Release \
         ..
-    make
+    make -j3
     make install
 
     local hunter_install_root_dir
     hunter_install_root_dir=$(cat "_3rdParty/Hunter/install-root-dir")
 
-    cat <<EOF > export.map
-{
-    global:
-        OpenTracingMakeTracerFactory;
-    local: *;
-};
-EOF
-
     gcc -shared -o libjaegertracing_plugin.so \
         -Wl,--version-script=export.map \
         -Wl,--whole-archive \
@@ -49,7 +44,7 @@ EOF
         ${hunter_install_root_dir}/lib/libopentracing.a \
         ${hunter_install_root_dir}/lib/libthrift.a \
         ${hunter_install_root_dir}/lib/libyaml-cpp.a \
-        -static-libstdc++ -static-libgcc
+        -static-libstdc++ -pthread -static-libgcc
     (ldd libjaegertracing_plugin.so |
         cut -d ' ' -f1 |
         sed -e 's/\s\+//g' \
@@ -61,10 +56,19 @@ EOF
 c
 ld-linux-x86-64
 linux-vdso
+pthread
 EOF
     diff ldd-output.txt ldd-expected.txt  # Non-zero exits due to set -e above.
-
     cp libjaegertracing_plugin.so /
+
+    git clone https://github.com/opentracing/opentracing-cpp.git
+    cd opentracing-cpp
+    mkdir build
+    cd build
+    cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_MOCKTRACER=OFF ..
+    make -j3
+    ./example/dynamic_load/dynamic_load-example /libjaegertracing_plugin.so \
+        ${project_dir}/examples/config.yml
 }
 
 main