diff --git a/puppet/Gemfile b/puppet/Gemfile index b0a83c430..9297d4816 100644 --- a/puppet/Gemfile +++ b/puppet/Gemfile @@ -21,5 +21,3 @@ gem 'puppet-lint-version_comparison-check', require: false gem 'puppet-syntax', require: false gem 'rspec-puppet', require: false gem 'rspec-puppet-facts', require: false - -gem 'facterdb', '~> 2.0', require: false diff --git a/puppet/Puppetfile b/puppet/Puppetfile index cd8c27965..7184ee20c 100644 --- a/puppet/Puppetfile +++ b/puppet/Puppetfile @@ -1,7 +1,6 @@ moduledir 'external_modules' mod 'citrin/htpasswd', '1.0.2' -mod 'claranet/timezone', '2.0.1' mod 'puppet/archive', '7.1.0' mod 'puppet/augeasproviders_core', '4.1.0' mod 'puppet/augeasproviders_mounttab', '4.0.0' diff --git a/puppet/data/common.yaml b/puppet/data/common.yaml index 333e78927..a1b4ad191 100644 --- a/puppet/data/common.yaml +++ b/puppet/data/common.yaml @@ -237,3 +237,5 @@ sudo::wheel_config: password redmine::https: true apache::default_vhost: false + +systemd::timezone: UTC diff --git a/puppet/modules/profiles/manifests/base.pp b/puppet/modules/profiles/manifests/base.pp index 541a2161b..dc9920d2f 100644 --- a/puppet/modules/profiles/manifests/base.pp +++ b/puppet/modules/profiles/manifests/base.pp @@ -4,7 +4,7 @@ include motd include puppet include ssh - include timezone + include systemd include unattended if $facts['os']['family'] == 'RedHat' { package { 'ntp': diff --git a/puppet/spec/classes/jenkins_node_spec.rb b/puppet/spec/classes/jenkins_node_spec.rb index 75030d15f..0cace568e 100644 --- a/puppet/spec/classes/jenkins_node_spec.rb +++ b/puppet/spec/classes/jenkins_node_spec.rb @@ -13,14 +13,14 @@ end it { is_expected.to compile.with_all_deps } it { is_expected.to contain_users__account('jenkins').with_sudo(false) } - if facts[:osfamily] == 'Debian' + if facts[:os]['family'] == 'Debian' it { is_expected.to contain_class('sudo') } it { is_expected.to contain_sudo__conf('sudo-puppet-jenkins').with_content('jenkins ALL=NOPASSWD: ALL') } else it { is_expected.not_to contain_class('sudo') } end - if facts[:osfamily] == 'Debian' + if facts[:os]['family'] == 'Debian' it { is_expected.to contain_file('/etc/pbuilder/bullseye64/hooks/A10nozstd').with_ensure('absent') } it { is_expected.to contain_file('/etc/pbuilder/jammy64/hooks/A10nozstd').with_ensure('present') } it { is_expected.to contain_file('/etc/pbuilder/bullseye64/hooks/C10foremanlog').with_ensure('present') } @@ -49,7 +49,7 @@ it { is_expected.to contain_package('jq') } end - if facts[:osfamily] == 'RedHat' + if facts[:os]['family'] == 'RedHat' context "unittest only node" do let(:params) do {uploader: false, packaging: false, unittests: true}