X-Git-Url: https://git.adam-barratt.org.uk/?a=blobdiff_plain;f=modules%2Fdebian_org%2Fmanifests%2Finit.pp;h=cdf1e6273637cf50f19f169b5f1c6a5fd1bbca97;hb=d824f9b2ae6fe566ad3c47c9d4f0c6aaa1057242;hp=02c8c7ace62d56ca89cbf43397f51e58ab4eaf39;hpb=83a1f108c2b7df3e8c3bc5c7918e2a8612e83d0b;p=mirror%2Fdsa-puppet.git

diff --git a/modules/debian_org/manifests/init.pp b/modules/debian_org/manifests/init.pp
index 02c8c7ace..cdf1e6273 100644
--- a/modules/debian_org/manifests/init.pp
+++ b/modules/debian_org/manifests/init.pp
@@ -12,12 +12,8 @@ class debian_org {
 		$servicefiles = 'absent'
 	}
 
-	# the virtual facter needs virt-what on jessie to work
-	if versioncmp($::lsbmajdistrelease, '9') < 0 {
-		package { 'virt-what': ensure => installed }
-	} else {
-		package { 'virt-what': ensure => purged }
-	}
+	# the virtual facter needs virt-what on jessie to work; clean up.
+	package { 'virt-what': ensure => purged }
 
 	$samhain_recipients = hiera('samhain_recipients')
 	$root_mail_alias = hiera('root_mail_alias')
@@ -76,13 +72,6 @@ class debian_org {
 		]:
 	}
 
-	if getfromhash($site::nodeinfo, 'broken-rtc') {
-		package { 'fake-hwclock':
-			ensure => installed,
-			tag    => extra_repo,
-		}
-	}
-
 	package { 'molly-guard':
 		ensure => installed,
 	}
@@ -125,12 +114,10 @@ class debian_org {
 		content => "Etc/UTC\n",
 		notify => Exec['dpkg-reconfigure tzdata -pcritical -fnoninteractive'],
 	}
-	if versioncmp($::lsbmajdistrelease, '9') >= 0 { # jessie has a regular file there, for instance
-		file { '/etc/localtime':
-			ensure => 'link',
-			target => '/usr/share/zoneinfo/Etc/UTC',
-			notify => Exec['dpkg-reconfigure tzdata -pcritical -fnoninteractive'],
-		}
+	file { '/etc/localtime':
+		ensure => 'link',
+		target => '/usr/share/zoneinfo/Etc/UTC',
+		notify => Exec['dpkg-reconfigure tzdata -pcritical -fnoninteractive'],
 	}
 	file { '/etc/puppet/puppet.conf':
 		content => template('debian_org/puppet.conf.erb'),
@@ -171,9 +158,16 @@ class debian_org {
 		notify => Exec['systemctl daemon-reload'],
 	}
 
-	concat { '/etc/cron.d/dsa-puppet-stuff': }
-	concat::fragment { 'dsa-puppet-stuff---header':
-		target => '/etc/cron.d/dsa-puppet-stuff',
+	file { '/etc/cron.d/dsa-puppet-stuff':
+		ensure => 'absent',
+	}
+	file { '/etc/dsa/cron.ignore.dsa-puppet-stuff':
+		ensure => 'absent',
+	}
+
+	concat { '/etc/cron.d/puppet-crontab': }
+	concat::fragment { 'puppet-crontab---header':
+		target => '/etc/cron.d/puppet-crontab',
 		order  => '000',
 		content  => @(EOF)
 			## THIS FILE IS UNDER PUPPET CONTROL. DON'T EDIT IT HERE.
@@ -182,10 +176,10 @@ class debian_org {
 			PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/lib/nagios/plugins
 			| EOF
 	}
-	concat::fragment { 'dsa-puppet-stuff---all':
-		target => '/etc/cron.d/dsa-puppet-stuff',
+	concat::fragment { 'puppet-crontab---all':
+		target => '/etc/cron.d/puppet-crontab',
 		order  => '010',
-		content => template('debian_org/dsa-puppet-stuff.cron.erb'),
+		content => template('debian_org/puppet-crontab.cron.erb'),
 		require => Package['debian.org'],
 	}
 	file { '/etc/ldap/ldap.conf':
@@ -209,8 +203,8 @@ class debian_org {
 		ensure => directory,
 		mode   => '0755',
 	}
-	file { '/etc/dsa/cron.ignore.dsa-puppet-stuff':
-		source  => 'puppet:///modules/debian_org/dsa-puppet-stuff.cron.ignore',
+	file { '/etc/dsa/cron.ignore.puppet-crontab':
+		source  => 'puppet:///modules/debian_org/puppet-crontab.cron.ignore',
 		require => Package['debian.org']
 	}
 	file { '/etc/nsswitch.conf':
@@ -251,10 +245,10 @@ class debian_org {
 		key   => 'vm.vfs_cache_pressure',
 		value => '10',
 	}
-	site::alternative { 'editor':
+	base::alternative { 'editor':
 		linkto => '/usr/bin/vim.basic',
 	}
-	site::alternative { 'view':
+	base::alternative { 'view':
 		linkto => '/usr/bin/vim.basic',
 	}
 	mailalias { 'samhain-reports':
@@ -330,10 +324,13 @@ class debian_org {
 		source => 'puppet:///modules/debian_org/root-dotfiles/vimrc',
 	}
 
-	if versioncmp($::lsbmajdistrelease, '9') >= 0 { # older puppets do facts as strings.
+	if versioncmp($::lsbmajdistrelease, '9') == 0 { # older puppets do facts as strings.
 		if $::processorcount > 1 {
 			package { 'irqbalance': ensure => installed }
 		}
+	} else {
+		# 926967 drops the recommendation on irqbalance in Buster
+		package { 'irqbalance': ensure => purged }
 	}
 
 
@@ -343,6 +340,16 @@ class debian_org {
 		value => '1',
 	}
 
+	# our ipv6 addresses and routes are statically configured.
+	base::sysctl { 'dsa-accept-ra-default':
+		key   => 'net.ipv6.conf.default.accept_ra',
+		value => 0,
+	}
+	base::sysctl { 'dsa-accept-ra-all':
+		key   => 'net.ipv6.conf.all.accept_ra',
+		value => 0,
+	}
+
 	# Disable kpartx udev rules
 	file { '/etc/udev/rules.d/60-kpartx.rules':
 		ensure => $has_lib_udev_rules_d_60_kpartx_rules ? { true  => 'present', default => 'absent' },