From d81739a93a0453dbf9436b873125844570a97661 Mon Sep 17 00:00:00 2001 From: Bastian Blank Date: Tue, 11 Apr 2017 14:36:28 +0200 Subject: [PATCH] Drop xinetd support in vsftpd --- modules/vsftpd/manifests/site.pp | 56 ------------------------ modules/vsftpd/manifests/site_systemd.pp | 10 ----- 2 files changed, 66 deletions(-) delete mode 100644 modules/vsftpd/manifests/site.pp diff --git a/modules/vsftpd/manifests/site.pp b/modules/vsftpd/manifests/site.pp deleted file mode 100644 index 98cca9776..000000000 --- a/modules/vsftpd/manifests/site.pp +++ /dev/null @@ -1,56 +0,0 @@ -define vsftpd::site ( - $root, - $bind='', - $chown_user='', - $writable=false, - $writable_other=false, - $banner="${name} FTP Server", - $max_clients=100, - $logfile="/var/log/ftp/vsftpd-${name}.debian.org.log", - $ensure=present -){ - - include vsftpd - - case $ensure { - present,absent: {} - default: { fail ( "Invald ensure `$ensure' for $name" ) } - } - - $ftpsite = $name - - $fname = "/etc/vsftpd-${name}.conf" - - file { $fname: - ensure => $ensure, - content => template('vsftpd/vsftpd.conf.erb') - } - - file { "/etc/logrotate.d/vsftpd-${name}": - ensure => absent - } - - munin::check { "vsftpd-${name}": - ensure => $ensure, - script => 'vsftpd' - } - munin::conf { "vsftpd-${name}": - ensure => $ensure, - content => template('vsftpd/munin.erb') - } - - # We don't need a firewall rule because it's added in vsftp.pp - xinetd::service { "vsftpd-${name}": - ensure => $ensure, - bind => $bind, - id => "${name}-ftp", - server => '/usr/sbin/vsftpd', - service => 'ftp', - server_args => $fname, - ferm => false, - instances => $max_clients, - require => File[$fname] - } - - Service['vsftpd']->Service['xinetd'] -} diff --git a/modules/vsftpd/manifests/site_systemd.pp b/modules/vsftpd/manifests/site_systemd.pp index 789c5665e..27a75be2d 100644 --- a/modules/vsftpd/manifests/site_systemd.pp +++ b/modules/vsftpd/manifests/site_systemd.pp @@ -72,20 +72,10 @@ define vsftpd::site_systemd ( Exec['systemctl daemon-reload'], File["/etc/systemd/system/vsftpd-${name}@.service"], File["/etc/systemd/system/vsftpd-${name}.socket"], - Service['xinetd'], ], provider => systemd, } - xinetd::service { [ "vsftpd-${name}", "vsftpd-${name}6", "vsftpd-${name}-v6" ]: - ensure => absent, - id => 'unused', - server => 'unused', - service => 'unused', - ferm => false, - before => Service["vsftpd-${name}.socket"], - } - munin::check { "vsftpd-${name}": ensure => $ensure, script => 'vsftpd' -- 2.20.1