[currencyservice] Fix OTel C++ build and update OTel version (open-te…

…lemetry#886) * Fix OTel C++ build and update OTel version * changelog
jmichalak9 · May 12, 2023 · 30f204d · 30f204d
1 parent e49c330
commit 30f204d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -27,6 +27,8 @@ release.
   ([#878](https://github.com/open-telemetry/opentelemetry-demo/pull/878))
 * [kafka] remove KRaft mode support workarounds
   ([#880](https://github.com/open-telemetry/opentelemetry-demo/pull/880))
+* [currencyservice] Fix OTel C++ build and update OTel version to 1.9.0
+  ([#886](https://github.com/open-telemetry/opentelemetry-demo/pull/886))
 
 ## 1.4.0
 

diff --git a/src/currencyservice/Dockerfile b/src/currencyservice/Dockerfile
@@ -19,14 +19,16 @@ FROM alpine as builder
 RUN apk update
 RUN apk add git cmake make g++ grpc-dev re2-dev protobuf-dev c-ares-dev
 
-ARG OPENTELEMETRY_CPP_VERSION=1.8.2
+ARG OPENTELEMETRY_CPP_VERSION=1.9.0
 
 RUN git clone https://github.com/open-telemetry/opentelemetry-cpp \
 	&& cd opentelemetry-cpp/ \
     	&& git checkout tags/v${OPENTELEMETRY_CPP_VERSION} -b v${OPENTELEMETRY_CPP_VERSION} \
 	&& mkdir build \
 	&& cd build \
-	&& cmake .. -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF -DWITH_EXAMPLES=OFF -DWITH_OTLP=ON -DWITH_OTLP_GRPC=ON \
+	&& cmake .. -DCMAKE_CXX_STANDARD=17 -DCMAKE_POSITION_INDEPENDENT_CODE=ON \
+				-DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF \
+				-DWITH_EXAMPLES=OFF -DWITH_OTLP=ON -DWITH_OTLP_GRPC=ON \
 	&& make -j$(nproc || sysctl -n hw.ncpu || echo 1) install && cd ../.. && rm -rf opentelemetry-cpp
 
 COPY . /currencyservice