Merge pull request #32 from mpepping/root-bash-shell

Use /bin/bash for the root user (#31)
mpepping · Nov 20, 2023 · 90f33dd · 90f33dd
2 parents b7c576a + f7fcbcd
commit 90f33dd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -16,6 +16,7 @@ RUN apk add --no-cache \
       openssh-client \
       openssl \
       procps \
+      shadow \
       skopeo \
       socat \
       sudo \
@@ -25,7 +26,8 @@ RUN apk add --no-cache \
 
 ADD include/ /
 
-RUN addgroup -g 1000 podshell && \
+RUN usermod -s /bin/bash root && \
+    addgroup -g 1000 podshell && \
     adduser -D -u 1000 -G podshell -s /bin/bash -g "Podshell User" podshell && \
     su - podshell -c "/usr/local/bin/_add_binenv"
 

diff --git a/Makefile b/Makefile
@@ -1,5 +1,5 @@
 # vim:ft=make:
-APP_NAME="shell"
+APP_NAME=ghcr.io/mpepping/podshell
 
 # HELP
 # This will output the help for each task
@@ -20,7 +20,7 @@ clean: ## Remove the image
 	docker rmi $(APP_NAME):latest
 
 start: ## Start the container
-	docker run -it --rm $(APP_NAME):latest --name podshell
+	docker run -it --rm --name podshell $(APP_NAME):latest
 
 stop: ## Stop the container
 	docker rm -f podshell