Merge lp://staging/~freyes/charm-helpers/lp1417754 into lp://staging/charm-helpers
Proposed by
Felipe Reyes
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 301 | ||||
| Proposed branch: | lp://staging/~freyes/charm-helpers/lp1417754 | ||||
| Merge into: | lp://staging/charm-helpers | ||||
| Diff against target: |
43 lines (+17/-1) 2 files modified
charmhelpers/core/templating.py (+1/-1) tests/core/test_templating.py (+16/-0) |
||||
| To merge this branch: | bzr merge lp://staging/~freyes/charm-helpers/lp1417754 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jorge Niedbalski | Pending | ||
| charmers | Pending | ||
|
Review via email:
|
|||
Description of the change
Dear Charmers,
This patch sets the `perms` argument while calling host.mkdir() inside templating.render() this ensures that the directory where the output on the template will be places has the correct permissions to be able to write.
Thanks,
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
