Merge lp:~lamont/maas/bug-1568847 into lp:~maas-maintainers/maas/packaging

Proposed by LaMont Jones
Status: Merged
Approved by: LaMont Jones
Approved revision: no longer in the source branch.
Merged at revision: 481
Proposed branch: lp:~lamont/maas/bug-1568847
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 28 lines (+6/-2)
2 files modified
debian/changelog (+2/-0)
debian/extras/maas-proxy-common.sh (+4/-2)
To merge this branch: bzr merge lp:~lamont/maas/bug-1568847
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+291515@code.launchpad.net

Commit message

Only try to create the squid cache if we have a config file.

Description of the change

Only try to create the squid cache if we have a config file.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm! Make sure that the cache directory is created since the systemd job shouldn'y really run in /var/lib/maas/maas-proxy.conf doesn't exist.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2016-04-07 22:24:07 +0000
+++ debian/changelog 2016-04-11 14:30:49 +0000
@@ -1,6 +1,8 @@
1maas (2.0.0~beta2+bzr4889-0ubuntu1) UNRELEASED; urgency=medium1maas (2.0.0~beta2+bzr4889-0ubuntu1) UNRELEASED; urgency=medium
22
3 * UNRELEASED3 * UNRELEASED
4 * Only create the squid cache if we have a configuration file.
5 LP: #1568847
46
5 -- Andres Rodriguez <andreserl@ubuntu.com> Thu, 07 Apr 2016 18:22:00 -04007 -- Andres Rodriguez <andreserl@ubuntu.com> Thu, 07 Apr 2016 18:22:00 -0400
68
79
=== modified file 'debian/extras/maas-proxy-common.sh'
--- debian/extras/maas-proxy-common.sh 2016-03-29 21:46:13 +0000
+++ debian/extras/maas-proxy-common.sh 2016-04-11 14:30:49 +0000
@@ -15,7 +15,9 @@
15 if [ -d /var/log/maas/proxy ]; then15 if [ -d /var/log/maas/proxy ]; then
16 chown -R proxy:proxy /var/log/maas/proxy16 chown -R proxy:proxy /var/log/maas/proxy
17 fi17 fi
18 if [ ! -d /var/cache/maas-proxy/00 ]; then18 if [ -f /var/lib/maas/maas-proxy.conf ]; then
19 $SQUID -z -N -f /var/lib/maas/maas-proxy.conf19 if [ ! -d /var/cache/maas-proxy/00 ]; then
20 $SQUID -z -N -f /var/lib/maas/maas-proxy.conf
21 fi
20 fi22 fi
21}23}

Subscribers

People subscribed via source and target branches