Giuseppe Lavagetto has submitted this change and it was merged.
Change subject: rsync: qualify vars
......................................................................
rsync: qualify vars
Change-Id: I9f1cfcb09d7edb654182e447d767d588358ed616
---
M modules/rsync/templates/header.erb
M modules/rsync/templates/module.erb
M modules/rsync/templates/rsync.default.erb
3 files changed, 20 insertions(+), 20 deletions(-)
Approvals:
Giuseppe Lavagetto: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/rsync/templates/header.erb
b/modules/rsync/templates/header.erb
index 358ae5b..1ce0ba8 100644
--- a/modules/rsync/templates/header.erb
+++ b/modules/rsync/templates/header.erb
@@ -3,10 +3,10 @@
uid = nobody
gid = nogroup
-use chroot = <%= use_chroot %>
-<% if log_file != 'UNSET' %>log file = <%= log_file %><% end %>
+use chroot = <%= @use_chroot %>
+<% if @log_file != 'UNSET' %>log file = <%= @log_file %><% end %>
log format = %t %a %m %f %b
syslog facility = local3
timeout = 300
-address = <%= address %>
-<% if motd_file != 'UNSET' %>motd file = <%= motd_file %><% end %>
+address = <%= @address %>
+<% if @motd_file != 'UNSET' %>motd file = <%= @motd_file %><% end %>
diff --git a/modules/rsync/templates/module.erb
b/modules/rsync/templates/module.erb
index 33aacbf..f37a474 100644
--- a/modules/rsync/templates/module.erb
+++ b/modules/rsync/templates/module.erb
@@ -1,19 +1,19 @@
# This file is being maintained by Puppet.
# DO NOT EDIT
-[ <%= name %> ]
-path = <%= path %>
-read only = <%= read_only %>
-write only = <%= write_only %>
-list = <%= list %>
-uid = <%= uid %>
-gid = <%= gid %>
-max connections = <%= max_connections %>
-<% if Integer(max_connections) > 0 %>lock file = <%= lock_file %><% end %>
-<% if comment != :undef %>comment = <%= comment %><% end %>
-<% if secrets_file != :undef %>secrets file = <%= secrets_file %><% end %>
-<% if auth_users != :undef %>auth users = <%= auth_users.to_a.join(', ')%><%
end %>
-<% if hosts_allow != :undef %>hosts allow = <%= hosts_allow.to_a.join(' ')%><%
end %>
-<% if hosts_deny != :undef %>hosts deny = <%= hosts_deny.to_a.join(' ')%><%
end %>
+[ <%= @name %> ]
+path = <%= @path %>
+read only = <%= @read_only %>
+write only = <%= @write_only %>
+list = <%= @list %>
+uid = <%= @uid %>
+gid = <%= @gid %>
+max connections = <%= @max_connections %>
+<% if Integer(@max_connections) > 0 %>lock file = <%= @lock_file %><% end
%>
+<% if @comment %>comment = <%= @comment %><% end %>
+<% if @secrets_file %>secrets file = <%= @secrets_file %><% end %>
+<% if @auth_users %>auth users = <%= @auth_users.to_a.join(', ')%><% end %>
+<% if @hosts_allow %>hosts allow = <%= @hosts_allow.to_a.join(' ')%><% end %>
+<% if @hosts_deny %>hosts deny = <%= @hosts_deny.to_a.join(' ')%><% end %>
diff --git a/modules/rsync/templates/rsync.default.erb
b/modules/rsync/templates/rsync.default.erb
index eb138b2..e120f48 100644
--- a/modules/rsync/templates/rsync.default.erb
+++ b/modules/rsync/templates/rsync.default.erb
@@ -10,7 +10,7 @@
# Use "inetd" if you want to start the rsyncd from inetd,
# all this does is prevent the init.d script from printing a message
# about not starting rsyncd (you still need to modify inetd's config
yourself).
-RSYNC_ENABLE=<%= use_xinetd == true ? 'false' : 'true' %>
+RSYNC_ENABLE=<%= @use_xinetd == true ? 'false' : 'true' %>
# which file should be used as the configuration file for rsync.
# This file is used instead of the default /etc/rsyncd.conf
@@ -18,7 +18,7 @@
# using a remote shell. When using a different file for
# rsync you might want to symlink /etc/rsyncd.conf to
# that file.
-RSYNC_CONFIG_FILE=<%= rsync_conf %>
+RSYNC_CONFIG_FILE=<%= @rsync_conf %>
# what extra options to give rsync --daemon?
# that excludes the --daemon; that's always done in the init.d script
--
To view, visit https://gerrit.wikimedia.org/r/159462
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I9f1cfcb09d7edb654182e447d767d588358ed616
Gerrit-PatchSet: 5
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Matanya <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: ArielGlenn <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Matanya <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
