[Backport 7.x] move dotnet tool restore down after setting usr (#5465)

Co-authored-by: Martijn Laarman <Mpdreamz@gmail.com>

Author: github-actions[bot]

.ci/DockerFile

@@ -13,7 +13,6 @@ RUN chown $GROUP_ID:$USER_ID $(pwd)
 COPY ./*.sln ./nuget.config ./*.Build.props ./*.Build.targets ./
 
 COPY ./dotnet-tools.json ./
-RUN dotnet tool restore
 
 # todo standardize on Build.props as Directory.Build.props needs that form
 COPY ./src/*.Build.props  ./src/
@@ -31,19 +30,21 @@ RUN for file in $(find . -name "*.?sproj"); do mkdir -p $(dirname $file)/$(basen
 COPY build/scripts/scripts.fsproj ./build/scripts/
 COPY .ci/Jenkins.csproj ./.ci/
 
-RUN chown -R $GROUP_ID:$USER_ID $(pwd)
+# Install app dependencies
+RUN dotnet restore
+RUN dotnet tool restore
+
 RUN chown -R $GROUP_ID:$USER_ID $(pwd)
 RUN chown -R $GROUP_ID:$USER_ID /tmp/NuGetScratch
-USER user
-
-# Install app dependencies
-RUN dotnet restore 
 
 # copy relevant files (see .dockerignore)
 COPY . .
 
 # making sure enough git info is available inside the container
 RUN git rev-parse HEAD .
 
+# USER user
+
+
 
 

.ci/make.sh

@@ -14,6 +14,7 @@ TASK=$1
 TASK_ARGS=()
 VERSION=$2
 STACK_VERSION=$VERSION
+REPO_BINDING="$repo:/sln"
 set -euo pipefail
 
 output_folder=".ci/output"
@@ -48,6 +49,7 @@ case $CMD in
         TASK=codegen
         # VERSION is BRANCH here for now
         TASK_ARGS=("$VERSION") 
+		REPO_BINDING="${OUTPUT_DIR}:/sln/${output_folder}"
         ;;
     *)
         echo -e "\nUsage:\n\t $CMD is not supported right now\n"
@@ -59,7 +61,7 @@ esac
 docker run \
   --env "DOTNET_VERSION" \
   --name test-runner \
-  --volume "$repo:/sln" \
+  --volume $REPO_BINDING \
   --rm \
   elastic/elasticsearch-net \
   ./build.sh $TASK "${TASK_ARGS[@]}"