From 1b214aa17c407b08aeaf6d5339123553a771c284 Mon Sep 17 00:00:00 2001 From: ianballou Date: Wed, 8 Nov 2023 10:14:49 -0500 Subject: [PATCH] Fixes #36902 - Support Pulpcore 3.39, drop older versions --- .github/workflows/ci.yml | 2 +- .sync.yml | 2 +- README.md | 14 +------------- manifests/repo.pp | 14 +------------- spec/classes/repo_spec.rb | 15 +-------------- templates/pulpcore-api.service.erb | 2 +- templates/pulpcore-content.service.erb | 3 +-- 7 files changed, 7 insertions(+), 45 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 03a660f..4bb74b2 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -15,6 +15,6 @@ jobs: uses: voxpupuli/gha-puppet/.github/workflows/beaker.yml@v2 with: pidfile_workaround: 'false' - beaker_facter: 'pulpcore_version:Pulp:3.28,3.22,3.21' + beaker_facter: 'pulpcore_version:Pulp:nightly,3.39' rubocop: false cache-version: '1' diff --git a/.sync.yml b/.sync.yml index 38127cd..84ff4b1 100644 --- a/.sync.yml +++ b/.sync.yml @@ -1,3 +1,3 @@ --- .github/workflows/ci.yml: - beaker_facter: 'pulpcore_version:Pulp:3.28,3.22,3.21' + beaker_facter: 'pulpcore_version:Pulp:nightly,3.39' diff --git a/README.md b/README.md index 31bfc02..23ca296 100644 --- a/README.md +++ b/README.md @@ -10,22 +10,10 @@ All supported versions are listed below. For every supported version, acceptance Supported operating systems are listed in `metadata.json` but individual releases can divert from that. For example, if Pulpcore x.y drops EL7, it will still be listed in metadata.json until all versions supported by the module have dropped it. Similarly, if x.z adds support for EL9, it'll be listed in `metadata.json` and all versions that don't support EL9 will have a note. -### Pulpcore 3.28 +### Pulpcore 3.39 Default recommended version. -### Pulpcore 3.22 - -Supported version. - -### Pulpcore 3.21 - -Supported version. The parameter `$hide_guarded_distributions` doesn't work since it's a Pulp 3.22 feature. - -### Pulpcore 3.16 - 3.18 - -Untested, but should work besides the `$hide_guarded_distributions` parameter. Not recommended. - ## Installation layout Pulpcore doesn't mandate a specific layout so this module creates and manages this. There are some constraints, mostly due to SELinux support. diff --git a/manifests/repo.pp b/manifests/repo.pp index a380732..425ea21 100644 --- a/manifests/repo.pp +++ b/manifests/repo.pp @@ -8,7 +8,7 @@ # An optional value for gpgkey to be used for yumrepo, instead of the default. # If an empty string is passed, gpgcheck will be disabled. class pulpcore::repo ( - Variant[Enum['nightly'], Pattern['^\d+\.\d+$']] $version = '3.28', + Variant[Enum['nightly'], Pattern['^\d+\.\d+$']] $version = '3.39', Optional[Stdlib::HTTPUrl] $baseurl = undef, Optional[String[0]] $gpgkey = undef, ) { @@ -23,18 +23,6 @@ notify => Anchor['pulpcore::repo'], } - # Only EL8 has DNF modules - if $dist_tag == 'el8' { - package { 'pulpcore-dnf-module': - ensure => $dist_tag, - name => 'pulpcore', - enable_only => true, - provider => 'dnfmodule', - require => Yumrepo['pulpcore'], - notify => Anchor['pulpcore::repo'], - } - } - # An anchor is used because it can be collected anchor { 'pulpcore::repo': } # lint:ignore:anchor_resource } diff --git a/spec/classes/repo_spec.rb b/spec/classes/repo_spec.rb index 384e2e7..dc19636 100644 --- a/spec/classes/repo_spec.rb +++ b/spec/classes/repo_spec.rb @@ -14,20 +14,7 @@ .with_gpgkey(%r{https://yum.theforeman.org/pulpcore/\d+\.\d+/GPG-RPM-KEY-pulpcore}) .that_notifies('Anchor[pulpcore::repo]') } - - if os_facts[:os]['release']['major'] == '8' - it 'configures the pulpcore module' do - is_expected.to contain_package('pulpcore-dnf-module') - .with_name('pulpcore') - .with_ensure(/^el\d+/) - .with_enable_only(true) - .with_provider('dnfmodule') - .that_requires('Yumrepo[pulpcore]') - .that_notifies('Anchor[pulpcore::repo]') - end - else - it { is_expected.not_to contain_package('pulpcore-dnf-module') } - end + it { is_expected.not_to contain_package('pulpcore-dnf-module') } end describe "with nightly version" do diff --git a/templates/pulpcore-api.service.erb b/templates/pulpcore-api.service.erb index 6dd2b1a..729c3f7 100644 --- a/templates/pulpcore-api.service.erb +++ b/templates/pulpcore-api.service.erb @@ -11,7 +11,7 @@ User=<%= scope['pulpcore::user'] %> Group=<%= scope['pulpcore::group'] %> WorkingDirectory=<%= scope['pulpcore::user_home'] %> RuntimeDirectory=pulpcore-api -ExecStart=/usr/libexec/pulpcore/gunicorn pulpcore.app.wsgi:application \ +ExecStart=/usr/bin/pulpcore-api \ --preload \ --timeout <%= scope['pulpcore::api_service_worker_timeout'] %> \ --workers <%= scope['pulpcore::api_service_worker_count'] %> \ diff --git a/templates/pulpcore-content.service.erb b/templates/pulpcore-content.service.erb index abfd0e6..c23688c 100644 --- a/templates/pulpcore-content.service.erb +++ b/templates/pulpcore-content.service.erb @@ -11,10 +11,9 @@ User=<%= scope['pulpcore::user'] %> Group=<%= scope['pulpcore::group'] %> WorkingDirectory=<%= scope['pulpcore::user_home'] %> RuntimeDirectory=pulpcore-content -ExecStart=/usr/libexec/pulpcore/gunicorn pulpcore.content:server \ +ExecStart=/usr/bin/pulpcore-content \ --preload \ --timeout <%= scope['pulpcore::content_service_worker_timeout'] %> \ - --worker-class 'aiohttp.GunicornWebWorker' \ --workers <%= scope['pulpcore::content_service_worker_count'] %> \ --access-logfile - ExecReload=/bin/kill -s HUP $MAINPID