Merge ~hloeung/jenkins-agent-charm:master into jenkins-agent-charm:master

[Haw Loeung]

Fixed permissions for directories created

Things like snaps needs to write to the home directory, /var/lib/jenkins/snap. Default host.mkdir permissions is 0o555.

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[Paul Collins (pjdc)]

review: Approve (lgtm)

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 767168b766d68777b81ce83126ce0377d72a1709

diff --git a/reactive/jenkins_slave.py b/reactive/jenkins_slave.py
index 01b99b4..c673499 100644
--- a/reactive/jenkins_slave.py
+++ b/reactive/jenkins_slave.py
@@ -48,8 +48,8 @@ def install():
 
     # And ensure required directories exist and are set up.
     status.maintenance('Installing jenkins-slave (directories)...')
-    host.mkdir('/var/lib/jenkins', owner='jenkins', group='jenkins')
-    host.mkdir('/var/log/jenkins', owner='jenkins', group='jenkins')
+    host.mkdir('/var/lib/jenkins', owner='jenkins', group='jenkins', perms=0o755)
+    host.mkdir('/var/log/jenkins', owner='jenkins', group='jenkins', perms=0o755)
 
     status.maintenance('Installing jenkins-slave (common files)...')
     write_default_conf()