Skip to content

Commit ed9f38d

Browse files
authored
ci: Fix Docker permissions for CI (#25)
* Ensure correct permissions assigned to junit-output dir * Update Docker permissions for CI codegen * Drop run-as user arg from local Docker run * Put permissions back for integration test runner * Simplify npm install * Run as root within container
1 parent fde2e92 commit ed9f38d

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

.buildkite/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,6 @@ RUN apt-get clean -y && \
1111
WORKDIR /usr/src/app
1212

1313
COPY package*.json .
14-
RUN npm install --production=false
14+
RUN npm install
1515

1616
COPY . .

.ci/make.sh

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,6 @@ if [[ -z "${BUILDKITE+x}" ]] || [[ -z "${CI+x}" ]]; then
147147
docker run \
148148
--volume "$repo:/usr/src/app" \
149149
--volume "$(realpath $repo/../elastic-client-generator-js):/usr/src/elastic-client-generator-js" \
150-
-u "$(id -u):$(id -g)" \
151150
--env "WORKFLOW=$WORKFLOW" \
152151
--name make-elasticsearch-js \
153152
--rm \
@@ -159,7 +158,6 @@ else
159158
docker run \
160159
--volume "$repo:/usr/src/app" \
161160
--volume /usr/src/app/node_modules \
162-
-u "$(id -u):$(id -g)" \
163161
--env "WORKFLOW=$WORKFLOW" \
164162
--name make-elasticsearch-js \
165163
--rm \

.dockerignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,4 @@ npm-debug.log
33
.git
44
.nyc_output
55
lib
6+
junit-output

0 commit comments

Comments
 (0)