foreman_maintain 1.6.6 → 1.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 49e4ffcc4a0a5899887cc3bc1dad11b10381deb25add42d732e8285787307959
-  data.tar.gz: e9ae9b2138d6a6ab9cb6c5cd36332467aab83c2f4cd8f8d6b46150067d072053
+  metadata.gz: 693bf5fa41d37437188c4990278301d1965819e37fad7b0dc680e1fdff8f8631
+  data.tar.gz: d56694b7ed8093eeaa814fdf950f5c308d76eeffbf97a654ecbb20ef55611759
 SHA512:
-  metadata.gz: 4b242b0cb68111062213960792f2b712ba5770a5887d088f824a957fa193dc8d3b49425b3e94e0ac9752cf504ee9a7b2ff7d52417a5edb077d6890aaee6c8859
-  data.tar.gz: 8a09714780cf70012aabeab9d919c80b3c62ca2859c5a208b5838bfbb83a658a38c753b9db0508d36d094053570be027428517760212acb30ca431310f172a5f
+  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_install.rb

@@ -6,7 +6,7 @@ class Features::ForemanInstall < ForemanMaintain::Feature
     label :foreman_install
 
     confine do
-      !feature(:instance).downstream && !feature(:katello) && feature(:foreman_server)
+      !feature(:instance).downstream && feature(:foreman_server)
     end
   end
 end

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

@@ -6,9 +6,9 @@ module Procedures::Packages
       param :force, 'Do not skip if package is installed', :flag => true, :default => false
       param :warn_on_errors, 'Do not interrupt scenario on failure',
         :flag => true, :default => false
-      param :dnf_options, 'Extra dnf options if any', :array => true, :default => []
       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,8 +16,8 @@ module Procedures::Packages
       package_manager.clean_cache(:assumeyes => assumeyes_val) if @clean_cache
       opts = {
         :assumeyes => assumeyes_val,
-        :options => @dnf_options,
         :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/repositories/setup.rb

@@ -3,7 +3,7 @@ module Procedures::Repositories
     metadata do
       description 'Setup repositories'
       confine do
-        feature(:instance).downstream || feature(:upstream)
+        feature(:instance).downstream || feature(:instance).upstream_install
       end
       param :version,
         'Version for which repositories needs to be setup',

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

@@ -0,0 +1,142 @@
+module Scenarios::ForemanUpgrade
+  class Abstract < ForemanMaintain::Scenario
+    def self.upgrade_metadata(&block)
+      metadata do
+        tags :upgrade_scenario
+        confine do
+          feature(:foreman_install)
+        end
+        instance_eval(&block)
+      end
+    end
+
+    def target_version
+      'nightly'
+    end
+  end
+
+  class PreUpgradeCheck < Abstract
+    upgrade_metadata do
+      description 'Checks before upgrading'
+      tags :pre_upgrade_checks
+      run_strategy :fail_slow
+    end
+
+    # rubocop:disable Metrics/MethodLength
+    def compose
+      add_steps(
+        Checks::Foreman::FactsNames, # if Foreman database present
+        Checks::ForemanProxy::CheckTftpStorage, # if Satellite with foreman-proxy+tftp
+        Checks::ForemanProxy::VerifyDhcpConfigSyntax, # if foreman-proxy+dhcp-isc
+        Checks::ForemanTasks::NotPaused, # if foreman-tasks present
+        Checks::Puppet::VerifyNoEmptyCacertRequests, # if puppetserver
+        Checks::ServerPing,
+        Checks::ServicesUp,
+        Checks::SystemRegistration,
+        Checks::CheckHotfixInstalled,
+        Checks::CheckTmout,
+        Checks::CheckUpstreamRepository,
+        Checks::Disk::AvailableSpace,
+        Checks::Disk::AvailableSpaceCandlepin, # if candlepin
+        Checks::Foreman::ValidateExternalDbVersion, # if external database
+        Checks::Foreman::CheckCorruptedRoles,
+        Checks::Foreman::CheckDuplicatePermissions,
+        Checks::Foreman::TuningRequirements, # if katello present
+        Checks::ForemanOpenscap::InvalidReportAssociations, # if foreman-openscap
+        Checks::ForemanTasks::Invalid::CheckOld, # if foreman-tasks
+        Checks::ForemanTasks::Invalid::CheckPendingState, # if foreman-tasks
+        Checks::ForemanTasks::Invalid::CheckPlanningState, # if foreman-tasks
+        Checks::ForemanTasks::NotRunning, # if foreman-tasks
+        Checks::NonRhPackages,
+        Checks::PackageManager::Dnf::ValidateDnfConfig,
+        Checks::Repositories::CheckNonRhRepository,
+        Checks::Repositories::Validate
+      )
+    end
+    # rubocop:enable Metrics/MethodLength
+  end
+
+  class PreMigrations < Abstract
+    upgrade_metadata do
+      description 'Procedures before upgrading'
+      tags :pre_migrations
+    end
+
+    def compose
+      add_steps(
+        Procedures::MaintenanceMode::EnableMaintenanceMode,
+        Procedures::Crond::Stop,
+        Procedures::SyncPlans::Disable
+      )
+    end
+  end
+
+  class Migrations < Abstract
+    upgrade_metadata do
+      description 'Upgrade steps'
+      tags :migrations
+      run_strategy :fail_fast
+    end
+
+    def set_context_mapping
+      context.map(:assumeyes, Procedures::Installer::Run => :assumeyes)
+    end
+
+    def compose
+      add_step(Procedures::Repositories::Setup.new(:version => 'nightly'))
+      if el8?
+        modules_to_switch = ['postgresql:13']
+        add_step(Procedures::Packages::SwitchModules.new(:module_names => modules_to_switch))
+      end
+
+      add_step(Procedures::Packages::Update.new(
+        :assumeyes => true,
+        :download_only => true
+      ))
+      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 'Post upgrade procedures'
+      tags :post_migrations
+    end
+
+    def compose
+      add_steps(
+        Procedures::RefreshFeatures,
+        Procedures::Service::Start,
+        Procedures::Crond::Start,
+        Procedures::SyncPlans::Enable,
+        Procedures::MaintenanceMode::DisableMaintenanceMode
+      )
+    end
+  end
+
+  class PostUpgradeChecks < Abstract
+    upgrade_metadata do
+      description 'Checks after upgrading'
+      tags :post_upgrade_checks
+      run_strategy :fail_slow
+    end
+
+    def compose
+      add_steps(
+        Checks::Foreman::FactsNames, # if Foreman database present
+        Checks::ForemanProxy::CheckTftpStorage, # if Satellite with foreman-proxy+tftp
+        Checks::ForemanProxy::VerifyDhcpConfigSyntax, # if foreman-proxy+dhcp-isc
+        Checks::ForemanTasks::NotPaused, # if foreman-tasks present
+        Checks::Puppet::VerifyNoEmptyCacertRequests, # if puppetserver
+        Checks::ServerPing,
+        Checks::ServicesUp,
+        Checks::SystemRegistration,
+        Procedures::Packages::CheckForReboot,
+        Procedures::Pulpcore::ContainerHandleImageMetadata
+      )
+    end
+  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
@@ -59,7 +60,7 @@ module Scenarios::Update
       add_steps(
         Procedures::Packages::Update.new(
           :assumeyes => true,
-          :dnf_options => ['--downloadonly']
+          :download_only => true
         ),
         Procedures::MaintenanceMode::EnableMaintenanceMode,
         Procedures::Crond::Stop,

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