X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=modules%2Fvsftpd%2Fmanifests%2Fsite.pp;h=d0129e1dfc42cd15f0c176a05ac5ea5180c091d7;hb=ed3ae2a61ad25951e43b10960c82549624c43e8b;hp=26541cc01aeda0c8ebd1e9b46bffcbfcff3ffa90;hpb=3cab052371737d529a2898605ab36c830f80848a;p=dsa-puppet.git

diff --git a/modules/vsftpd/manifests/site.pp b/modules/vsftpd/manifests/site.pp
index 26541cc0..d0129e1d 100644
--- a/modules/vsftpd/manifests/site.pp
+++ b/modules/vsftpd/manifests/site.pp
@@ -1,8 +1,8 @@
 define vsftpd::site (
-	$name,
 	$source='',
 	$content='',
-	$bind=$::ipaddress,
+	$bind='',
+	$logfile="/var/log/ftp/vsftpd-${name}.debian.org.log",
 	$ensure=present
 ){
 
@@ -22,27 +22,32 @@ define vsftpd::site (
 	if $source {
 		file { $fname:
 			ensure => $ensure,
-			noop   => true,
 			source => $source,
 		}
 	} elsif $content {
 		file { $fname:
 			ensure  => $ensure,
-			noop    => true,
 			content => $content,
 		}
 	} else {
 		fail ( "Need one of source or content for $name" )
 	}
 
+	file { "/etc/logrotate.d/vsftpd-${name}":
+		ensure  => $ensure,
+		content => template('vsftpd/logrotate.erb')
+	}
+
 	# We don't need a firewall rule because it's added in vsftp.pp
 	xinetd::service { "vsftpd-${name}":
 		bind        => $bind,
-		id          => $name,
+		id          => "${name}-ftp",
 		server      => '/usr/sbin/vsftpd',
 		port        => 'ftp',
 		server_args => $fname,
 		ferm        => false,
+		instances   => 200,
+		require     => File[$fname]
 	}
 
 }