From 032b32db58c9470523998be0a24d6bdcf1a0decd Mon Sep 17 00:00:00 2001 From: Szczepan Pecio Date: Mon, 6 Nov 2023 13:22:34 +0100 Subject: [PATCH] Added active waiting for cmake-js due to (Text file busy) race condition (#349) --- QuoteVerification/QuoteVerificationService/Dockerfile | 4 ++-- QuoteVerification/QuoteVerificationService/src/package.json | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/QuoteVerification/QuoteVerificationService/Dockerfile b/QuoteVerification/QuoteVerificationService/Dockerfile index 0566eb0b..4990f65d 100644 --- a/QuoteVerification/QuoteVerificationService/Dockerfile +++ b/QuoteVerification/QuoteVerificationService/Dockerfile @@ -51,8 +51,8 @@ COPY src /qvs/src COPY configuration-default /qvs/configuration-default # build QVS RUN echo 'cmake_QVL_PATH=/qvl/Build/Release/dist' >> /qvs/src/.npmrc # workaround for npm 9+ https://github.com/npm/cli/issues/5852 -RUN cd /qvs/src && npm install -# copy compiled bianries +RUN cd /qvs/src && npm install && npm run wait-for-cmake-js && npm run build-native # workaournd for Text file busy race condition https://github.com/nodejs/docker-node/issues/1918 +# copy compiled binaries RUN mkdir -p /qvs/native/lib/ \ && cp /qvl/Build/Release/dist/lib/*.so /qvs/native/lib/ \ && cp /qvs/src/qvl/cmake-build-release/Release/*.node /qvs/native/ \ diff --git a/QuoteVerification/QuoteVerificationService/src/package.json b/QuoteVerification/QuoteVerificationService/src/package.json index 4b629932..5a51812c 100644 --- a/QuoteVerification/QuoteVerificationService/src/package.json +++ b/QuoteVerification/QuoteVerificationService/src/package.json @@ -1,7 +1,8 @@ { "name": "quoteverificationservice", "scripts": { - "install": "cmake-js build -d qvl/ -O qvl/cmake-build-release", + "wait-for-cmake-js": "timeout 10s bash -c 'until cmake-js --version; do sleep 1; done'", + "build-native": "cmake-js build -d qvl/ -O qvl/cmake-build-release", "swagger": "node swagger.js" }, "version": "1.0.0",