[currencyservice] Create multiple build jobs to optimize build time (o…

…pen-telemetry#569)
jmichalak9 · Nov 11, 2022 · 653ded8 · 653ded8
1 parent af74a0f
commit 653ded8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -134,3 +134,5 @@ significant modifications will be credited to OpenTelemetry Authors.
 ([#512](https://github.com/open-telemetry/opentelemetry-demo/pull/512))
 * Replaced PHP-CLI to PHP-Apache for a more realistic service
 ([#563](https://github.com/open-telemetry/opentelemetry-demo/pull/563))
+* Optimize currencyservice build time with parallel build jobs
+([#569](https://github.com/open-telemetry/opentelemetry-demo/pull/569))
diff --git a/src/currencyservice/Dockerfile b/src/currencyservice/Dockerfile
@@ -35,12 +35,12 @@ RUN git clone --shallow-submodules --depth 1 --recurse-submodules -b v${GRPC_VER
     -DgRPC_BUILD_GRPC_PYTHON_PLUGIN=OFF \
     -DgRPC_BUILD_GRPC_RUBY_PLUGIN=OFF \
     ../.. \
-  && make -j2 \
+  && make -j$(nproc || sysctl -n hw.ncpu || echo 1) \
   && make install \
   && cd ../../.. \
   && rm -rf grpc
 
-# install opentelemetry
+# Install OpenTelemetry
 RUN git clone https://github.com/open-telemetry/opentelemetry-cpp \
     && cd opentelemetry-cpp/ \
     && git checkout tags/v${OPENTELEMETRY_VERSION} -b v${OPENTELEMETRY_VERSION} \