Merge lp:~verterok/ols-jenkaas/bind-mount-and-misc-fixes into lp:~ols-jenkaas-admins/ols-jenkaas/trunk

Proposed by Guillermo Gonzalez
Status: Merged
Merged at revision: 918
Proposed branch: lp:~verterok/ols-jenkaas/bind-mount-and-misc-fixes
Merge into: lp:~ols-jenkaas-admins/ols-jenkaas/trunk
Diff against target: 63 lines (+20/-8)
2 files modified
setup/seed-slave (+13/-0)
setup/seeded-slave (+7/-8)
To merge this branch: bzr merge lp:~verterok/ols-jenkaas/bind-mount-and-misc-fixes
Reviewer Review Type Date Requested Status
Online Services Jenkaas Administrators Pending
Review via email: mp+454438@code.launchpad.net

Commit message

PS5 redeploy fixes and improvements:
 - fix docker-setup call in setup/seeded-slave and use `lxd waitready` instead of 10s sleep
 - renamed G|UID env variables as these are read-only in bash
 - bind mount /var/lib/jenkins in /home/jenkins and update user home dir to make snapd work

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'setup/seed-slave'
2--- setup/seed-slave 2023-10-23 21:03:48 +0000
3+++ setup/seed-slave 2023-10-25 00:29:30 +0000
4@@ -97,6 +97,19 @@
5 # allow jenkins to use lxd
6 sudo adduser jenkins lxd
7
8+# bind mount /var/lib/jenkins to /home/jenkins to make snapd happy
9+if [ ! -f /home/jenkins/.real-dir ]
10+then
11+ mkdir -p -m 755 /home/jenkins
12+ touch /home/jenkins/.real-dir
13+ sudo chown jenkins:jenkins /home/jenkins
14+ sudo mount -o bind /var/lib/jenkins /home/jenkins
15+ echo "/var/lib/jenkins /home/jenkins none defaults,bind 0 0" | sudo tee -a /etc/fstab
16+ # update user home, do it manually as 'usermod -d /home/jenkins jenkins' fails
17+ sudo sed -i 's#/var/lib/jenkins#//home/jenkins#' /etc/passwd
18+ # we need to reboot after this?
19+fi
20+
21 # Force a restart so the lxd/docker group is taken into account
22 # We can't use +0min, it's too fast :-/
23 echo sudo service jenkins-slave restart | at now+1min
24
25=== modified file 'setup/seeded-slave'
26--- setup/seeded-slave 2023-10-24 19:08:43 +0000
27+++ setup/seeded-slave 2023-10-25 00:29:30 +0000
28@@ -54,9 +54,8 @@
29 LXD_INIT_OPTS="--storage-backend zfs --storage-create-loop $POOL_SIZE --storage-pool $POOL"
30 sudo rm -rf /var/snap/lxd/common/lxd
31 sudo systemctl start snap.lxd.daemon.unix.socket snap.lxd.daemon.service
32- echo "sleeping 10 secs"
33- sleep 10
34- echo "waking up"
35+ echo "waiting for lxd to be ready"
36+ lxd waitready
37 fi
38
39 sudo lxd init --auto $LXD_INIT_OPTS
40@@ -71,10 +70,10 @@
41 lxc network attach-profile lxdbr0 default eth0 --force-local || true
42
43 # Ensure lxd mounts can be used
44-UID=$(id -u)
45-GID=$(id -g)
46-sudo usermod --add-subuid $UID-$UID root
47-sudo usermod --add-subgid $GID-$GID root
48+U_ID=$(id -u)
49+G_ID=$(id -g)
50+sudo usermod --add-subuid $U_ID-$U_ID root
51+sudo usermod --add-subgid $G_ID-$G_ID root
52
53 # Ensure containers can be nested
54 for user in root lxd; do
55@@ -110,7 +109,7 @@
56 bzr branch ${JENKAAS_JOBS} ~/jenkaas
57
58 # setup docker, in this case is just in case one of the env vars or registry changed
59-${DIRNAME}/docker-setup
60+~/jenkaas/setup/docker-setup
61
62 if [ "${JENKAAS}" = "production" ] ; then
63 # jenkaas production slaves use a 1400 MTU, tell lxd to do the same for all

Subscribers

People subscribed via source and target branches