Merge lp:~nick-moffitt/charms/precise/virtual-maas/fix-dot-juju-directory-permissions into lp:~virtual-maasers/charms/precise/virtual-maas/trunk

Proposed by Nick Moffitt
Status: Merged
Merged at revision: 50
Proposed branch: lp:~nick-moffitt/charms/precise/virtual-maas/fix-dot-juju-directory-permissions
Merge into: lp:~virtual-maasers/charms/precise/virtual-maas/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
scripts/setup-maas (+1/-1)
To merge this branch: bzr merge lp:~nick-moffitt/charms/precise/virtual-maas/fix-dot-juju-directory-permissions
Reviewer Review Type Date Requested Status
James Page Pending
Review via email: mp+155012@code.launchpad.net

Description of the change

The chown at the end of the ~ubuntu/.juju/environments.yaml creation ignores the ~ubuntu/.juju/ directory itself, which must be writable by the ubuntu user in order for juju to cache charms from the charm store.

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 'scripts/setup-maas'
2--- scripts/setup-maas 2013-02-19 10:04:19 +0000
3+++ scripts/setup-maas 2013-03-22 16:50:26 +0000
4@@ -220,7 +220,7 @@
5 default-series: raring
6 juju-origin: lp:juju
7 EOF
8-chown ubuntu:ubuntu ~ubuntu/.juju/environments.yaml
9+chown -R ubuntu:ubuntu ~ubuntu/.juju
10
11 ### fix enlistment script ipmi (LP: 1064527). do not attempt enlist
12 sed -i.xdist \

Subscribers

People subscribed via source and target branches