Fix running gulp in docker image

This simplifies Dockerfile and fixes bugs where it would produce errors
on build.

Fixes #315

Author: bryk

.dockerignore

@@ -1 +1 @@
-.git
+.git

Dockerfile

@@ -37,24 +37,7 @@ RUN curl -sSL https://get.docker.com/builds/Linux/x86_64/docker-1.9.1 > /usr/bin
 # Current directory is always /dashboard
 WORKDIR /dashboard
 
-# Make an normal user, required for npm and bower, they are user programs and should not be called as root
-RUN useradd normaluser -m -s /bin/bash && \
-    chown -R normaluser /dashboard
-# Our normal user should have access to /dashboard
-
-# Switch user to normaluser
-USER normaluser
-
-# Copy over package.json bower.json and postinstall.sh.
-# Why? Because of docker caching, npm install will only run again if one of these have changed
-COPY package.json bower.json ./
-COPY build/postinstall.sh build/
-
-# Install all npm deps, bower deps and godep. This will take a while.
-RUN npm install
-
-# Copy over the rest of the source
 COPY ./ ./
 
-# Switch back to the root user when where going to communicate with docker
-USER root
+# Install all npm deps, bower deps and godep. This will take a while.
+RUN npm install --unsafe-perm

build/postinstall.sh

@@ -16,7 +16,7 @@
 # This is a script that runs on npm install postinstall phase.
 # It contains all prerequisites required to use the build system.
 
-./node_modules/.bin/bower install
+./node_modules/.bin/bower install --allow-root
 
 # Godep is required by the project. Install it in the tools directory.
 GOPATH=`pwd`/.tools/go go get github.com/tools/godep

build/run-gulp-in-docker.sh

@@ -14,7 +14,7 @@
 # limitations under the License.
 
 # This is a script that runs gulp in a docker container,
-# for machines that doesn't have nodejs, go and java installed.
+# for machines that don't have nodejs, go and java installed.
 
 DOCKER_RUN_OPTS=${DOCKER_RUN_OPTS:-}
 DASHBOARD_IMAGE_NAME="kubernetes-dashboard-build-image"
@@ -31,4 +31,4 @@ docker run \
 	-v /var/run/docker.sock:/var/run/docker.sock \
 	${DOCKER_RUN_OPTS} \
 	${DASHBOARD_IMAGE_NAME} \
-	${DEFAULT_COMMAND} $@
+	${DEFAULT_COMMAND} $@

package.json

@@ -70,6 +70,6 @@
     "node": ">=4.2.2"
   },
   "scripts": {
-    "postinstall": "./build/postinstall.sh"
+    "postinstall": "build/postinstall.sh"
   }
 }