Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/web/manifests/server.pp

Issue 29329028: Issue 3190 - Ensure /var/cache/$cms_repository being present and writable (Closed)
Patch Set: Created Oct. 9, 2015, 7:28 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/web/manifests/server.pp
===================================================================
--- a/modules/web/manifests/server.pp
+++ b/modules/web/manifests/server.pp
@@ -94,7 +94,10 @@
mode => 755,
}
- file {"/var/www/${vhost}":
+ file {[
+ "/var/cache/${repository}",
+ "/var/www/${vhost}",
Fred 2015/10/12 13:07:48 Are the variables $repository and $vhost guarantee
mathias 2015/10/12 13:18:36 Neither by code, nor by documentation. By conventi
+ ]:
ensure => directory,
owner => www,
mode => 755,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld