foreman_maintain 1.6.7 → 1.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c513edc079ddea8f18904feeed2d3299c59949183703f7344946c8828abee6b6
-  data.tar.gz: 2fb58c156aaf22a0b1be982768a267926f8f080b063a2169ab6868fa1955b26d
+  metadata.gz: 693bf5fa41d37437188c4990278301d1965819e37fad7b0dc680e1fdff8f8631
+  data.tar.gz: d56694b7ed8093eeaa814fdf950f5c308d76eeffbf97a654ecbb20ef55611759
 SHA512:
-  metadata.gz: bbd8eb809ab9f165b461dd2b29e8af8b5ffdee2d285462d9a9b626e93f922ac20066d166a751524c80edbadb6c465cbb6a5661529591da7188608ff0d48467b3
-  data.tar.gz: 7450630a6ab864eec72be71fd6b948f016d59f8f1e6dcbc11c8110d41bbde62504f16acc40b9bd3ff2e5b902cd316555fb03ebf72c2606b851824fbf8d50fb6f
+  metadata.gz: 40503012245738da00890d0d224141754f00c110f9bbc7c62112b3e11bea7beb066718bfffdc36056ae9afbed29f2ac8ca6d9dd474e4af9a1eee50099a636352
+  data.tar.gz: d71cfc02366e99a507f1453706d0ffcfe01f1dcdb8cf231a03a4b8468716f1c4b3ae25acfa5938f32858bf17fb37d974d511173b047808b84a03e8dd38ce1d1a

data/README.md

@@ -17,7 +17,6 @@ Subcommands:
         --tags tags                 Run only those with all specific set of tags
 
     upgrade                       Upgrade related commands
-      list-versions                 List versions this system is upgradable to
       check --target-version TARGET_VERSION   Run pre-upgrade checks for upgrading to specified version
             --disable-self-upgrade            Disable automatic self upgrade (default: false)
       run --target-version TARGET_VERSION     Run the full upgrade
@@ -56,12 +55,11 @@ Subcommands:
 Foreman-maintain implements upgrade tooling that helps the administrator to go
 through the upgrade process.
 
-Foreman-maintain scans the system to know, what versions are available
-for upgrade on the particular system. To see what versions are available
-for upgrade, run:
+The foreman-maintain tool is intended to self upgrade itself to the next major
+version of the project. This is needed before upgrading, run:
 
 ```
-foreman-maintain upgrade list-versions
+foreman-maintain self-upgrade
 ```
 
 To perform just the pre-upgrade checks for the system, run:

data/definitions/checks/check_ipv6_disable.rb

@@ -0,0 +1,23 @@
+class Checks::CheckIpv6Disable < ForemanMaintain::Check
+  metadata do
+    label :check_ipv6_disable
+    description 'Check if ipv6.disable=1 is set at kernel level'
+  end
+
+  def run
+    cmdline_file = File.read('/proc/cmdline')
+
+    assert(!cmdline_file.include?("ipv6.disable=1"), error_message)
+  end
+
+  def error_message
+    base = "\nThe kernel contains ipv6.disable=1 which is known to break installation and upgrade"\
+           ", remove and reboot before continuining."
+
+    if feature(:instance).downstream
+      base += " See https://access.redhat.com/solutions/5045841 for more details."
+    end
+
+    base
+  end
+end

data/definitions/features/capsule.rb

@@ -7,8 +7,7 @@ class Features::Capsule < ForemanMaintain::Feature
 
     confine do
       !package_manager.installed?(['satellite']) &&
-        package_manager.installed?(['satellite-capsule']) ||
-        package_manager.installed?(['capsule-installer'])
+        package_manager.installed?(['satellite-capsule'])
     end
   end
 

data/definitions/features/dynflow_sidekiq.rb

@@ -8,14 +8,9 @@ class Features::DynflowSidekiq < ForemanMaintain::Feature
   end
 
   def config_files
-    # Workaround until foreman-installer can deploy scaled workers
-    service_symlinks = configured_instances.map do |service|
-      "/etc/systemd/system/multi-user.target.wants/#{service}.service"
-    end
     [
       '/etc/foreman/dynflow',
-      service_symlinks,
-    ].flatten
+    ]
   end
 
   def services

data/definitions/features/foreman_server.rb

@@ -26,7 +26,6 @@ module ForemanMaintain
         [
           '/etc/httpd',
           '/var/www/html/pub/katello-*',
-          '/etc/squid',
           '/etc/foreman',
           '/etc/selinux/targeted/contexts/files/file_contexts.subs',
           '/etc/sysconfig/foreman',

data/definitions/features/foreman_tasks.rb

@@ -125,14 +125,6 @@ class Features::ForemanTasks < ForemanMaintain::Feature
     puts "\nTimeout: #{e.message}. Try again."
   end
 
-  def services
-    feature(:dynflow_sidekiq) ? [] : [system_service(service_name, 30)]
-  end
-
-  def service_name
-    'dynflowd'
-  end
-
   private
 
   def check_task_count(state, spinner)

data/definitions/features/instance.rb

@@ -162,7 +162,6 @@ class Features::Instance < ForemanMaintain::Feature
       'pulp3' => %w[pulpcore pulpcore_database],
       'pulp3_content' => %w[pulpcore pulpcore_database],
       'foreman_tasks' => %w[foreman_tasks],
-      'katello_agent' => %w[katello],
       'katello_events' => %w[katello],
     }
   end

data/definitions/features/katello.rb

@@ -17,14 +17,6 @@ class Features::Katello < ForemanMaintain::Feature
     @current_version ||= package_version('katello')
   end
 
-  def services
-    [
-      system_service('qpidd', 10),
-      system_service('qdrouterd', 10),
-    ]
-  end
-
-  # rubocop:disable  Metrics/MethodLength
   def config_files
     configs = [
       '/etc/pki/katello',
@@ -35,11 +27,6 @@ class Features::Katello < ForemanMaintain::Feature
       '/etc/sysconfig/tomcat*',
       '/etc/tomcat*',
       '/var/lib/candlepin',
-      '/usr/share/foreman/bundler.d/katello.rb',
-      '/etc/qpid',
-      '/etc/qpid-dispatch',
-      '/var/lib/qpidd',
-      '/etc/qpid-dispatch',
     ]
 
     if installer_scenario_answers['certs']
@@ -53,11 +40,9 @@ class Features::Katello < ForemanMaintain::Feature
 
     configs
   end
-  # rubocop:enable  Metrics/MethodLength
 
   def config_files_exclude_for_online
     [
-      '/var/lib/qpidd',
       '/var/lib/candlepin/activemq-artemis',
     ]
   end

data/definitions/procedures/hammer_setup.rb

@@ -2,6 +2,9 @@ class Procedures::HammerSetup < ForemanMaintain::Procedure
   metadata do
     description 'Setup hammer'
     for_feature :hammer
+    preparation_steps do
+      Checks::ServicesUp.new
+    end
   end
 
   def run

data/definitions/procedures/packages/update.rb

@@ -8,6 +8,7 @@ module Procedures::Packages
         :flag => true, :default => false
       param :download_only, 'Download and cache packages only', :flag => true, :default => false
       param :clean_cache, 'If true will cause a DNF cache clean', :flag => true, :default => true
+      param :enabled_repos, 'List of repositories to enable', :array => true
     end
 
     def run
@@ -16,6 +17,7 @@ module Procedures::Packages
       opts = {
         :assumeyes => assumeyes_val,
         :download_only => @download_only,
+        :enabled_repos => @enabled_repos,
       }
       packages_action(:update, @packages, opts)
     rescue ForemanMaintain::Error::ExecutionError => e

data/definitions/procedures/pulpcore/container_handle_image_metadata.rb

@@ -0,0 +1,26 @@
+module Procedures::Pulpcore
+  class ContainerHandleImageMetadata < ForemanMaintain::Procedure
+    include ForemanMaintain::Concerns::SystemService
+    include ForemanMaintain::Concerns::PulpCommon
+
+    metadata do
+      description 'Initialize and expose container image metadata in the pulpcore db'
+      for_feature :pulpcore
+    end
+
+    def run
+      with_spinner('Initialize and expose container image metadata in the pulpcore db') do |spinner|
+        necessary_services = feature(:pulpcore_database).services
+
+        feature(:service).handle_services(spinner, 'start', :only => necessary_services)
+
+        spinner.update('Adding image metadata to pulp. You can continue using the ' \
+               'system normally while the task runs in the background.')
+        execute!(pulpcore_manager('container-handle-image-data'))
+        spinner.update('Adding image metadata to katello. You can continue using the ' \
+               'system normally while the task runs in the background.')
+        execute!('foreman-rake katello:import_container_manifest_labels')
+      end
+    end
+  end
+end

data/definitions/procedures/restore/configs.rb

@@ -14,18 +14,10 @@ module Procedures::Restore
         spinner.update('Restoring configs')
         clean_conflicting_data
         restore_configs(backup)
-        reset_qpid_jrnls
       end
     end
 
     def restore_configs(backup)
-      exclude = ForemanMaintain.available_features.each_with_object([]) do |feat, cfgs|
-        if backup.online_backup?
-          feat.config_files_exclude_for_online.each { |f| cfgs << f.gsub(%r{^/}, '') }
-        end
-        feat.config_files_to_exclude.each { |f| cfgs << f.gsub(%r{^/}, '') }
-      end
-
       tar_options = {
         :overwrite => true,
         :listed_incremental => '/dev/null',
@@ -33,7 +25,6 @@ module Procedures::Restore
         :directory => '/',
         :archive => backup.file_map[:config_files][:path],
         :gzip => true,
-        :exclude => exclude,
       }
 
       feature(:tar).run(tar_options)
@@ -45,15 +36,5 @@ module Procedures::Restore
       # tar is unable to --overwrite dir with symlink
       execute('rm -rf /usr/share/foreman-proxy/.ssh')
     end
-
-    def reset_qpid_jrnls
-      # on restore without pulp data qpid fails to start
-      # https://access.redhat.com/solutions/4645231
-      ['/var/lib/qpidd/.qpidd/', '/var/lib/qpidd/'].each do |qpidd_path|
-        if Dir.exist?("#{qpidd_path}/qls/dat2/")
-          execute("rm -rf #{qpidd_path}/qls/dat2/__db.00*")
-        end
-      end
-    end
   end
 end

data/definitions/procedures/restore/drop_databases.rb

@@ -19,9 +19,7 @@ module Procedures::Restore
         feature(:service).handle_services(spinner, 'start', :only => ['postgresql'])
         drop_foreman(backup, spinner)
         drop_candlepin(backup, spinner)
-        if feature(:pulpcore)
-          drop_pulpcore(backup, spinner)
-        end
+        drop_pulpcore(backup, spinner)
       end
     end
 

data/definitions/procedures/restore/reindex_databases.rb

@@ -1,6 +1,7 @@
 module Procedures::Restore
   class ReindexDatabases < ForemanMaintain::Procedure
     include ForemanMaintain::Concerns::SystemService
+    include ForemanMaintain::Concerns::SystemHelpers
 
     metadata do
       description 'REINDEX databases'
@@ -16,6 +17,11 @@ module Procedures::Restore
 
         spinner.update('Reindexing the databases')
         execute!('runuser - postgres -c "reindexdb -a"')
+        if check_min_version('python3.11-pulp-ansible', '0.20.0')
+          execute!('runuser -c '\
+                   '\'echo "ALTER COLLATION pulp_ansible_semver REFRESH VERSION;"'\
+                   '| psql pulpcore\' postgres')
+        end
       end
     end
   end

data/definitions/procedures/restore/required_packages.rb

@@ -12,7 +12,6 @@ module Procedures::Restore
       backup = ForemanMaintain::Utils::Backup.new(@backup_dir)
       required_packages = []
       required_packages << 'puppetserver' if backup.with_puppetserver?
-      required_packages << 'qpid-cpp-server' if backup.with_qpidd?
       if required_packages.any?
         with_spinner('Installing required packages') do
           ForemanMaintain.package_manager.install(required_packages, assumeyes: true)

data/definitions/scenarios/backup.rb

@@ -19,9 +19,9 @@ module ForemanMaintain::Scenarios
     def compose
       check_valid_strategy
       safety_confirmation
-      accessibility_confirmation
-      prepare_directory
-      add_step_with_context(Procedures::Backup::Metadata, :online_backup => online_backup?)
+      add_step_with_context(Procedures::Backup::AccessibilityConfirmation) if strategy == :offline
+      add_step_with_context(Procedures::Backup::PrepareDirectory)
+      add_step_with_context(Procedures::Backup::Metadata, :online_backup => strategy == :online)
 
       case strategy
       when :online
@@ -29,6 +29,7 @@ module ForemanMaintain::Scenarios
       when :offline
         add_offline_backup_steps
       end
+
       add_step_with_context(Procedures::Backup::CompressData)
     end
 
@@ -64,18 +65,8 @@ module ForemanMaintain::Scenarios
 
     private
 
-    def prepare_directory
-      add_step_with_context(Procedures::Backup::PrepareDirectory)
-    end
-
-    def accessibility_confirmation
-      if strategy == :offline
-        add_step_with_context(Procedures::Backup::AccessibilityConfirmation)
-      end
-    end
-
     def safety_confirmation
-      if online_backup? || include_db_dumps?
+      if strategy == :online || include_db_dumps?
         add_step_with_context(Procedures::Backup::Online::SafetyConfirmation)
       end
     end
@@ -132,10 +123,6 @@ module ForemanMaintain::Scenarios
     def include_db_dumps?
       !!context.get(:include_db_dumps)
     end
-
-    def online_backup?
-      strategy == :online
-    end
   end
 
   class BackupRescueCleanup < ForemanMaintain::Scenario
@@ -149,8 +136,10 @@ module ForemanMaintain::Scenarios
     end
 
     def compose
-      add_step_with_context(Procedures::Service::Start) if strategy != :online
-      add_steps_with_context(find_procedures(:maintenance_mode_off)) if strategy != :online
+      if strategy == :offline
+        add_step_with_context(Procedures::Service::Start)
+        add_steps_with_context(find_procedures(:maintenance_mode_off))
+      end
       add_step_with_context(Procedures::Backup::Clean)
     end
 

data/definitions/scenarios/foreman_upgrade.rb

@@ -134,7 +134,8 @@ module Scenarios::ForemanUpgrade
         Checks::ServerPing,
         Checks::ServicesUp,
         Checks::SystemRegistration,
-        Procedures::Packages::CheckForReboot
+        Procedures::Packages::CheckForReboot,
+        Procedures::Pulpcore::ContainerHandleImageMetadata
       )
     end
   end

data/definitions/scenarios/restore.rb

@@ -11,7 +11,6 @@ module ForemanMaintain::Scenarios
     end
 
     # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
-    # rubocop:disable Metrics/CyclomaticComplexity
     def compose
       backup = ForemanMaintain::Utils::Backup.new(context.get(:backup_dir))
 
@@ -27,7 +26,6 @@ module ForemanMaintain::Scenarios
       end
 
       add_steps_with_context(Procedures::Restore::Confirmation,
-        Procedures::Selinux::SetFileSecurity,
         Procedures::Restore::RequiredPackages,
         Procedures::Restore::Configs)
       add_step_with_context(Procedures::Crond::Stop) if feature(:cron)
@@ -36,13 +34,10 @@ module ForemanMaintain::Scenarios
       end
       add_step_with_context(Procedures::Service::Stop)
       add_steps_with_context(Procedures::Restore::ExtractFiles) if backup.tar_backups_exist?
-      drop_dbs(backup)
-      if backup.sql_dump_files_exist? && feature(:instance).postgresql_local?
-        add_step(Procedures::Service::Start.new(:only => ['postgresql']))
-      end
-      restore_sql_dumps(backup)
-      if backup.sql_dump_files_exist? && feature(:instance).postgresql_local?
-        add_step(Procedures::Service::Stop.new(:only => ['postgresql']))
+
+      if backup.sql_dump_files_exist?
+        add_steps_with_context(Procedures::Restore::DropDatabases)
+        restore_sql_dumps(backup)
       end
 
       if feature(:instance).postgresql_local? &&
@@ -51,38 +46,28 @@ module ForemanMaintain::Scenarios
         add_step_with_context(Procedures::Restore::ReindexDatabases)
       end
 
-      add_steps_with_context(
-        Procedures::Pulpcore::Migrate,
-        Procedures::Restore::CandlepinResetMigrations
-      )
-
-      add_steps_with_context(Procedures::Service::Start,
-        Procedures::Service::DaemonReload)
       add_step(Procedures::Installer::Run.new(:assumeyes => true))
       add_step_with_context(Procedures::Installer::UpgradeRakeTask)
       add_step_with_context(Procedures::Crond::Start) if feature(:cron)
     end
     # rubocop:enable Metrics/MethodLength,Metrics/AbcSize
-    # rubocop:enable Metrics/CyclomaticComplexity
-
-    def drop_dbs(backup)
-      if backup.file_map[:candlepin_dump][:present] ||
-         backup.file_map[:foreman_dump][:present] ||
-         (feature(:pulpcore) && backup.file_map[:pulpcore_dump][:present])
-        add_steps_with_context(Procedures::Restore::DropDatabases)
-      end
-    end
 
     def restore_sql_dumps(backup)
+      if feature(:instance).postgresql_local?
+        add_step(Procedures::Service::Start.new(:only => ['postgresql']))
+      end
       if backup.file_map[:candlepin_dump][:present]
         add_steps_with_context(Procedures::Restore::CandlepinDump)
       end
       if backup.file_map[:foreman_dump][:present]
         add_steps_with_context(Procedures::Restore::ForemanDump)
       end
-      if feature(:pulpcore) && backup.file_map[:pulpcore_dump][:present]
+      if backup.file_map[:pulpcore_dump][:present]
         add_steps_with_context(Procedures::Restore::PulpcoreDump)
       end
+      if feature(:instance).postgresql_local?
+        add_step(Procedures::Service::Stop.new(:only => ['postgresql']))
+      end
     end
 
     def set_context_mapping
@@ -98,9 +83,6 @@ module ForemanMaintain::Scenarios
         Procedures::Restore::ForemanDump => :backup_dir,
         Procedures::Restore::PulpcoreDump => :backup_dir,
         Procedures::Restore::ExtractFiles => :backup_dir)
-
-      context.map(:incremental_backup,
-        Procedures::Selinux::SetFileSecurity => :incremental_backup)
     end
   end
 

data/definitions/scenarios/self_upgrade.rb

@@ -67,11 +67,8 @@ module ForemanMaintain::Scenarios
     def downstream_self_upgrade(pkgs_to_update)
       ForemanMaintain.enable_maintenance_module
 
-      dnf_options = req_repos_to_update_pkgs.map do |id|
-        "--enablerepo=#{id}"
-      end
       add_step(Procedures::Packages::Update.new(packages: pkgs_to_update, assumeyes: true,
-        dnf_options: dnf_options))
+        enabled_repos: req_repos_to_update_pkgs))
     end
 
     def upstream_self_upgrade(pkgs_to_update)

data/definitions/scenarios/update.rb

@@ -28,6 +28,7 @@ module Scenarios::Update
         Checks::SystemRegistration,
         Checks::CheckHotfixInstalled,
         Checks::CheckTmout,
+        Checks::CheckIpv6Disable,
         Checks::CheckUpstreamRepository,
         Checks::Disk::AvailableSpace,
         Checks::Disk::AvailableSpaceCandlepin, # if candlepin

data/definitions/scenarios/upgrade_to_capsule_6_16.rb

@@ -27,6 +27,7 @@ module Scenarios::Capsule_6_16
     def compose
       add_steps(find_checks(:default))
       add_steps(find_checks(:pre_upgrade))
+      add_step(Checks::CheckIpv6Disable)
       add_step(Checks::Disk::AvailableSpacePostgresql13)
       add_step(Checks::Repositories::Validate.new(:version => '6.16'))
     end
@@ -95,6 +96,7 @@ module Scenarios::Capsule_6_16
       add_steps(find_checks(:default))
       add_steps(find_checks(:post_upgrade))
       add_step(Procedures::Packages::CheckForReboot)
+      add_step(Procedures::Pulpcore::ContainerHandleImageMetadata)
     end
   end
 end

data/definitions/scenarios/upgrade_to_satellite_6_16.rb

@@ -27,6 +27,7 @@ module Scenarios::Satellite_6_16
     def compose
       add_steps(find_checks(:default))
       add_steps(find_checks(:pre_upgrade))
+      add_step(Checks::CheckIpv6Disable)
       add_step(Checks::Disk::AvailableSpacePostgresql13)
       add_step(Checks::Repositories::Validate.new(:version => '6.16'))
       add_step(Checks::CheckOrganizationContentAccessMode)
@@ -98,6 +99,7 @@ module Scenarios::Satellite_6_16
       add_steps(find_checks(:default))
       add_steps(find_checks(:post_upgrade))
       add_step(Procedures::Packages::CheckForReboot)
+      add_step(Procedures::Pulpcore::ContainerHandleImageMetadata)
     end
   end
 end

data/lib/foreman_maintain/cli/upgrade_command.rb

@@ -56,16 +56,6 @@ module ForemanMaintain
         !disable_self_upgrade?
       end
 
-      subcommand 'list-versions', 'List versions this system is upgradable to' do
-        disable_self_upgrade_option
-
-        def execute
-          ForemanMaintain.validate_downstream_packages
-          ForemanMaintain.perform_self_upgrade if allow_self_upgrade?
-          print_versions(UpgradeRunner.available_targets)
-        end
-      end
-
       subcommand 'check', 'Run pre-upgrade checks before upgrading to specified version' do
         target_version_option
         interactive_option

data/lib/foreman_maintain/concerns/base_database.rb

@@ -41,12 +41,6 @@ module ForemanMaintain
         raise NotImplementedError
       end
 
-      def config_files
-        [
-          '/etc/systemd/system/postgresql.service',
-        ]
-      end
-
       def local?(config = configuration)
         ['localhost', '127.0.0.1', `hostname`.strip].include? config['host']
       end

data/lib/foreman_maintain/concerns/system_helpers.rb

@@ -100,7 +100,7 @@ module ForemanMaintain
       end
 
       def packages_action(action, packages, options = {})
-        options.validate_options!(:assumeyes, :options, :download_only)
+        options.validate_options!(:assumeyes, :options, :download_only, :enabled_repos)
         case action
         when :install
           package_manager.install(packages, :assumeyes => options[:assumeyes])
@@ -108,7 +108,8 @@ module ForemanMaintain
           package_manager.update(
             packages,
             :assumeyes => options[:assumeyes],
-            :download_only => options[:download_only]
+            :download_only => options[:download_only],
+            :enabled_repos => options[:enabled_repos]
           )
         when :remove
           package_manager.remove(packages, :assumeyes => options[:assumeyes])

data/lib/foreman_maintain/package_manager/apt.rb

@@ -19,10 +19,12 @@ module ForemanMaintain::PackageManager
       apt_action('remove', packages, :assumeyes => assumeyes)
     end
 
-    def update(packages = [], assumeyes: false, download_only: false)
+    # rubocop:disable Lint/UnusedMethodArgument
+    def update(packages = [], assumeyes: false, download_only: false, enabled_repos: [])
       action = packages.any? ? '--only-upgrade install' : 'upgrade'
       apt_action(action, packages, :assumeyes => assumeyes, :download_only => download_only)
     end
+    # rubocop:enable Lint/UnusedMethodArgument
 
     def clean_cache(assumeyes: false)
       apt_action('clean', [], :assumeyes => assumeyes)

data/lib/foreman_maintain/package_manager/dnf.rb

@@ -59,12 +59,13 @@ module ForemanMaintain::PackageManager
       dnf_action('remove', packages, assumeyes: assumeyes)
     end
 
-    def update(packages = [], assumeyes: false, download_only: false)
+    def update(packages = [], assumeyes: false, download_only: false, enabled_repos: [])
       dnf_action(
         'update',
         packages,
         assumeyes: assumeyes,
-        download_only: download_only
+        download_only: download_only,
+        enabled_repos: enabled_repos
       )
     end
 
@@ -132,14 +133,18 @@ module ForemanMaintain::PackageManager
 
     private
 
-    # rubocop:disable Layout/LineLength, Metrics/ParameterLists
-    def dnf_action(action, packages, with_status: false, assumeyes: false, dnf_options: [], valid_exit_statuses: [0], download_only: false)
+    # rubocop:disable Layout/LineLength, Metrics/ParameterLists, Metrics/MethodLength
+    def dnf_action(action, packages, with_status: false, assumeyes: false, dnf_options: [], valid_exit_statuses: [0], download_only: false, enabled_repos: [])
       packages = [packages].flatten(1)
 
       dnf_options << '-y' if assumeyes
       dnf_options << '--downloadonly' if download_only
       dnf_options << '--disableplugin=foreman-protector'
 
+      enabled_repos.map do |id|
+        dnf_options << "--enablerepo=#{id}"
+      end
+
       command = ['dnf', dnf_options.join(' '), action]
 
       command.push(packages.join(' ')) unless packages.empty?
@@ -158,7 +163,7 @@ module ForemanMaintain::PackageManager
         )
       end
     end
-    # rubocop:enable Layout/LineLength, Metrics/ParameterLists
+    # rubocop:enable Layout/LineLength, Metrics/ParameterLists, Metrics/MethodLength
 
     def protector_config
       File.exist?(protector_config_file) ? File.read(protector_config_file) : ''

data/lib/foreman_maintain/utils/backup.rb

@@ -183,24 +183,7 @@ module ForemanMaintain
 
       def validate_hostname?
         # make sure that the system hostname is the same as the backup
-        hostname_from_metadata = metadata.fetch('hostname', nil)
-        if hostname_from_metadata
-          hostname_from_metadata == hostname
-        else
-          config_tarball = file_map[:config_files][:path]
-          tar_cmd = "tar zxf #{config_tarball} etc/httpd/conf/httpd.conf --to-stdout --occurrence=1"
-          status, httpd_config = execute_with_status(tar_cmd)
-
-          # Incremental backups sometimes don't include httpd.conf. Since a "base" backup
-          # is restored before an incremental, we can assume that the hostname is checked
-          # during the base backup restore
-          if status == 0
-            match = httpd_config.match(/\s*ServerName\s+"*([^ "]+)"*\s*$/)
-            match ? match[1] == hostname : false
-          else
-            true
-          end
-        end
+        metadata.fetch('hostname', nil) == hostname
       end
 
       def validate_interfaces
@@ -243,7 +226,7 @@ module ForemanMaintain
       def sql_dump_files_exist?
         file_map[:foreman_dump][:present] ||
           file_map[:candlepin_dump][:present] ||
-          (feature(:pulpcore_database) && file_map[:pulpcore_dump][:present])
+          file_map[:pulpcore_dump][:present]
       end
 
       def incremental?
@@ -262,10 +245,6 @@ module ForemanMaintain
         installed_rpms.any? { |rpm| rpm.start_with?('puppetserver-') }
       end
 
-      def with_qpidd?
-        installed_rpms.any? { |rpm| rpm.start_with?('qpid-cpp-server-') }
-      end
-
       def source_os_version
         metadata.fetch('os_version', 'unknown')
       end

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '1.6.7'.freeze
+  VERSION = '1.6.8'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.6.7
+  version: 1.6.8
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-05-06 00:00:00.000000000 Z
+date: 2024-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clamp
@@ -160,6 +160,7 @@ files:
 - definitions/checks/backup/certs_tar_exist.rb
 - definitions/checks/candlepin/db_up.rb
 - definitions/checks/check_hotfix_installed.rb
+- definitions/checks/check_ipv6_disable.rb
 - definitions/checks/check_tmout.rb
 - definitions/checks/disk/available_space.rb
 - definitions/checks/disk/available_space_candlepin.rb
@@ -211,7 +212,6 @@ files:
 - definitions/features/foreman_proxy.rb
 - definitions/features/foreman_server.rb
 - definitions/features/foreman_tasks.rb
-- definitions/features/gofer.rb
 - definitions/features/hammer.rb
 - definitions/features/installer.rb
 - definitions/features/instance.rb
@@ -274,7 +274,7 @@ files:
 - definitions/procedures/packages/unlock_versions.rb
 - definitions/procedures/packages/update.rb
 - definitions/procedures/packages/update_all_confirmation.rb
-- definitions/procedures/pulpcore/migrate.rb
+- definitions/procedures/pulpcore/container_handle_image_metadata.rb
 - definitions/procedures/pulpcore/trim_rpm_changelogs.rb
 - definitions/procedures/puppet/delete_empty_ca_cert_request_files.rb
 - definitions/procedures/puppet/remove_puppet.rb
@@ -286,7 +286,6 @@ files:
 - definitions/procedures/repositories/enable.rb
 - definitions/procedures/repositories/setup.rb
 - definitions/procedures/restore/candlepin_dump.rb
-- definitions/procedures/restore/candlepin_reset_migrations.rb
 - definitions/procedures/restore/configs.rb
 - definitions/procedures/restore/confirmation.rb
 - definitions/procedures/restore/drop_databases.rb
@@ -297,7 +296,6 @@ files:
 - definitions/procedures/restore/pulpcore_dump.rb
 - definitions/procedures/restore/reindex_databases.rb
 - definitions/procedures/restore/required_packages.rb
-- definitions/procedures/selinux/set_file_security.rb
 - definitions/procedures/service/base.rb
 - definitions/procedures/service/daemon_reload.rb
 - definitions/procedures/service/disable.rb
@@ -319,9 +317,7 @@ files:
 - definitions/scenarios/services.rb
 - definitions/scenarios/update.rb
 - definitions/scenarios/upgrade_to_capsule_6_16.rb
-- definitions/scenarios/upgrade_to_capsule_6_16_z.rb
 - definitions/scenarios/upgrade_to_satellite_6_16.rb
-- definitions/scenarios/upgrade_to_satellite_6_16_z.rb
 - extras/foreman-maintain.sh
 - extras/foreman_protector/dnf/foreman-protector.py
 - extras/foreman_protector/foreman-protector.conf

data/definitions/features/gofer.rb

@@ -1,16 +0,0 @@
-class Features::Gofer < ForemanMaintain::Feature
-  metadata do
-    label :gofer
-
-    confine do
-      find_package('gofer') &&
-        ForemanMaintain::Utils::Service::Systemd.new('goferd', 0).enabled?
-    end
-  end
-
-  def services
-    [
-      system_service('goferd', 30),
-    ]
-  end
-end

data/definitions/procedures/pulpcore/migrate.rb

@@ -1,24 +0,0 @@
-module Procedures::Pulpcore
-  class Migrate < ForemanMaintain::Procedure
-    include ForemanMaintain::Concerns::SystemService
-    include ForemanMaintain::Concerns::PulpCommon
-
-    metadata do
-      description 'Migrate pulpcore db'
-      for_feature :pulpcore
-    end
-
-    def run
-      with_spinner('Migrating pulpcore') do |spinner|
-        necessary_services = feature(:pulpcore_database).services
-        pulp_services = feature(:pulpcore).services
-
-        feature(:service).handle_services(spinner, 'start', :only => necessary_services)
-        feature(:service).handle_services(spinner, 'stop', :only => pulp_services)
-
-        spinner.update('Migrating pulpcore database')
-        execute!(pulpcore_manager('migrate --noinput'))
-      end
-    end
-  end
-end

data/definitions/procedures/restore/candlepin_reset_migrations.rb

@@ -1,14 +0,0 @@
-module Procedures::Restore
-  class CandlepinResetMigrations < ForemanMaintain::Procedure
-    metadata do
-      description 'Ensure Candlepin runs all migrations after restoring the database'
-      confine do
-        feature(:candlepin_database)
-      end
-    end
-
-    def run
-      FileUtils.rm_f('/var/lib/candlepin/.puppet-candlepin-rpm-version')
-    end
-  end
-end

data/definitions/procedures/selinux/set_file_security.rb

@@ -1,25 +0,0 @@
-module Procedures::Selinux
-  class SetFileSecurity < ForemanMaintain::Procedure
-    metadata do
-      description 'Setting file security'
-
-      param :incremental_backup,
-        'Is the backup incremental?',
-        :required => true
-      manual_detection
-      confine do
-        File.directory?('/sys/fs/selinux')
-      end
-    end
-
-    def run
-      with_spinner('Restoring SELinux context') do |spinner|
-        if @incremental_backup
-          spinner.update('Skipping for incremental update')
-        else
-          execute!('restorecon -Rn /')
-        end
-      end
-    end
-  end
-end

data/definitions/scenarios/upgrade_to_capsule_6_16_z.rb

@@ -1,97 +0,0 @@
-module Scenarios::Capsule_6_16_z
-  class Abstract < ForemanMaintain::Scenario
-    def self.upgrade_metadata(&block)
-      metadata do
-        tags :upgrade_scenario
-        confine do
-          feature(:capsule) &&
-            (feature(:capsule).current_minor_version == '6.16' || \
-              ForemanMaintain.upgrade_in_progress == '6.16.z')
-        end
-        instance_eval(&block)
-      end
-    end
-
-    def target_version
-      '6.16.z'
-    end
-  end
-
-  class PreUpgradeCheck < Abstract
-    upgrade_metadata do
-      description 'Checks before upgrading to Capsule 6.16.z'
-      tags :pre_upgrade_checks
-      run_strategy :fail_slow
-    end
-
-    def compose
-      add_steps(find_checks(:default))
-      add_steps(find_checks(:pre_upgrade))
-      add_step(Checks::Repositories::Validate.new(:version => '6.16'))
-    end
-  end
-
-  class PreMigrations < Abstract
-    upgrade_metadata do
-      description 'Procedures before migrating to Capsule 6.16.z'
-      tags :pre_migrations
-    end
-
-    def compose
-      add_step(Procedures::Repositories::Setup.new(:version => '6.16'))
-      if el8?
-        modules_to_enable = ["satellite-capsule:#{el_short_name}"]
-        add_step(Procedures::Packages::EnableModules.new(:module_names => modules_to_enable))
-      end
-      add_step(Procedures::Packages::Update.new(
-        :assumeyes => true,
-        :download_only => true
-      ))
-      add_steps(find_procedures(:pre_migrations))
-    end
-  end
-
-  class Migrations < Abstract
-    upgrade_metadata do
-      description 'Migration scripts to Capsule 6.16.z'
-      tags :migrations
-    end
-
-    def set_context_mapping
-      context.map(:assumeyes, Procedures::Installer::Run => :assumeyes)
-    end
-
-    def compose
-      add_step(Procedures::Service::Stop.new)
-      add_step(Procedures::Packages::Update.new(:assumeyes => true, :clean_cache => false))
-      add_step_with_context(Procedures::Installer::Run)
-    end
-  end
-
-  class PostMigrations < Abstract
-    upgrade_metadata do
-      description 'Procedures after migrating to Capsule 6.16.z'
-      tags :post_migrations
-    end
-
-    def compose
-      add_step(Procedures::RefreshFeatures)
-      add_step(Procedures::Service::Start.new)
-      add_steps(find_procedures(:post_migrations))
-    end
-  end
-
-  class PostUpgradeChecks < Abstract
-    upgrade_metadata do
-      description 'Checks after upgrading to Capsule 6.16.z'
-      tags :post_upgrade_checks
-      run_strategy :fail_slow
-    end
-
-    def compose
-      add_steps(find_checks(:default))
-      add_steps(find_checks(:post_upgrade))
-      add_step(Procedures::Packages::CheckForReboot)
-    end
-  end
-end

data/definitions/scenarios/upgrade_to_satellite_6_16_z.rb

@@ -1,98 +0,0 @@
-module Scenarios::Satellite_6_16_z
-  class Abstract < ForemanMaintain::Scenario
-    def self.upgrade_metadata(&block)
-      metadata do
-        tags :upgrade_scenario
-        confine do
-          feature(:satellite) &&
-            (feature(:satellite).current_minor_version == '6.16' || \
-            ForemanMaintain.upgrade_in_progress == '6.16.z')
-        end
-        instance_eval(&block)
-      end
-    end
-
-    def target_version
-      '6.16.z'
-    end
-  end
-
-  class PreUpgradeCheck < Abstract
-    upgrade_metadata do
-      description 'Checks before upgrading to Satellite 6.16.z'
-      tags :pre_upgrade_checks
-      run_strategy :fail_slow
-    end
-
-    def compose
-      add_steps(find_checks(:default))
-      add_steps(find_checks(:pre_upgrade))
-      add_step(Checks::Repositories::Validate.new(:version => '6.16'))
-    end
-  end
-
-  class PreMigrations < Abstract
-    upgrade_metadata do
-      description 'Procedures before migrating to Satellite 6.16.z'
-      tags :pre_migrations
-    end
-
-    def compose
-      add_step(Procedures::Repositories::Setup.new(:version => '6.16'))
-      if el8?
-        modules_to_enable = ["satellite:#{el_short_name}"]
-        add_step(Procedures::Packages::EnableModules.new(:module_names => modules_to_enable))
-      end
-      add_step(Procedures::Packages::Update.new(
-        :assumeyes => true,
-        :download_only => true
-      ))
-      add_steps(find_procedures(:pre_migrations))
-    end
-  end
-
-  class Migrations < Abstract
-    upgrade_metadata do
-      description 'Migration scripts to Satellite 6.16.z'
-      tags :migrations
-    end
-
-    def set_context_mapping
-      context.map(:assumeyes, Procedures::Installer::Run => :assumeyes)
-    end
-
-    def compose
-      add_step(Procedures::Service::Stop.new)
-      add_step(Procedures::Packages::Update.new(:assumeyes => true, :clean_cache => false))
-      add_step_with_context(Procedures::Installer::Run)
-      add_step(Procedures::Installer::UpgradeRakeTask)
-    end
-  end
-
-  class PostMigrations < Abstract
-    upgrade_metadata do
-      description 'Procedures after migrating to Satellite 6.16.z'
-      tags :post_migrations
-    end
-
-    def compose
-      add_step(Procedures::RefreshFeatures)
-      add_step(Procedures::Service::Start.new)
-      add_steps(find_procedures(:post_migrations))
-    end
-  end
-
-  class PostUpgradeChecks < Abstract
-    upgrade_metadata do
-      description 'Checks after upgrading to Satellite 6.16.z'
-      tags :post_upgrade_checks
-      run_strategy :fail_slow
-    end
-
-    def compose
-      add_steps(find_checks(:default))
-      add_steps(find_checks(:post_upgrade))
-      add_step(Procedures::Packages::CheckForReboot)
-    end
-  end
-end