From 5739d2832c14a51743a25cb5974719d4bb927128 Mon Sep 17 00:00:00 2001
From: Cody Hiar <codyfh@gmail.com>
Date: Tue, 8 Jan 2019 12:26:12 -0700
Subject: Minor fixes, update readme

---
 Dockerfile  |  4 ++++
 README.md   | 14 +++++++++++++-
 plugins.txt |  1 +
 3 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/Dockerfile b/Dockerfile
index b18e08a..cba072a 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -4,8 +4,12 @@ USER root
 
 RUN apt-get update && apt-get -y install \
     build-essential \
+    libltdl-dev \
+    sudo \
     rsync
 
+RUN echo "jenkins ALL=NOPASSWD: ALL" >> /etc/sudoers
+
 # Install Plugins
 COPY plugins.txt /usr/share/jenkins/ref/plugins.txt
 RUN /usr/local/bin/install-plugins.sh < /usr/share/jenkins/ref/plugins.txt
diff --git a/README.md b/README.md
index 11b6af3..7aad680 100644
--- a/README.md
+++ b/README.md
@@ -1,3 +1,15 @@
 # Docker Jenkins
 
-`jenkins_home` needs 777 permissions
+This is a jenkins repo for building some CI pipelines.
+
+
+## Lessons Learned
+- `/var/jenkins_home` probably needs 777 permissions
+- `docker.sock` probably needs 777 permissions since user/groups get confused
+- SSH creds can be generated by jenkins user. Git commands require those creds
+  added into `credentials` and if you have scripts that use ssh then you will
+  want to add those ssh creds to `publish over ssh` so that pipelines can ssh
+  into other machines
+- Keep the filepath of `jenkins_home` the same on host and in container so if
+  you need volume mounts inside the container, they will be translated
+  correctly onto the host machine
diff --git a/plugins.txt b/plugins.txt
index cc65a7f..37e62e1 100644
--- a/plugins.txt
+++ b/plugins.txt
@@ -76,6 +76,7 @@ pipeline-stage-step:2.3
 pipeline-stage-tags-metadata:1.3.4
 pipeline-stage-view:2.10
 plain-credentials:1.5
+publish-over-ssh:1.20.1
 pubsub-light:1.12
 resource-disposer:0.12
 scm-api:2.3.0
-- 
cgit v1.2.3