foreman_maintain 1.3.0 → 1.3.1

data/definitions/scenarios/restore.rb

@@ -11,15 +11,15 @@ module ForemanMaintain::Scenarios
     end
 
     # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
-    # rubocop:disable Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
+    # rubocop:disable Metrics/CyclomaticComplexity
     def compose
       backup = ForemanMaintain::Utils::Backup.new(context.get(:backup_dir))
 
       add_steps(find_checks(:root_user))
-      supported_version_check
       add_steps_with_context(Checks::Restore::ValidateBackup,
-                             Checks::Restore::ValidateHostname,
-                             Checks::Restore::ValidateInterfaces)
+        Checks::Restore::ValidateHostname,
+        Checks::Restore::ValidateInterfaces,
+        Checks::Restore::ValidatePostgresqlDumpPermissions)
 
       if context.get(:dry_run)
         self.class.metadata[:run_strategy] = :fail_slow
@@ -27,12 +27,11 @@ module ForemanMaintain::Scenarios
       end
 
       add_steps_with_context(Procedures::Restore::Confirmation,
-                             Procedures::Selinux::SetFileSecurity,
-                             Procedures::Restore::Configs)
+        Procedures::Selinux::SetFileSecurity,
+        Procedures::Restore::Configs)
       add_step_with_context(Procedures::Crond::Stop) if feature(:cron)
       unless backup.incremental?
-        add_steps_with_context(Procedures::Restore::EnsureMongoEngineMatches,
-                               Procedures::Restore::InstallerReset)
+        add_steps_with_context(Procedures::Restore::InstallerReset)
       end
       add_step_with_context(Procedures::Service::Stop)
       add_steps_with_context(Procedures::Restore::ExtractFiles) if backup.tar_backups_exist?
@@ -44,25 +43,24 @@ module ForemanMaintain::Scenarios
       if backup.sql_dump_files_exist? && feature(:instance).postgresql_local?
         add_step(Procedures::Service::Stop.new(:only => ['postgresql']))
       end
-      restore_mongo_dump(backup)
 
       if feature(:instance).postgresql_local? && !backup.online_backup?
         add_step_with_context(Procedures::Restore::ReindexDatabases)
       end
 
-      add_steps_with_context(Procedures::Pulp::Migrate,
-                             Procedures::Pulpcore::Migrate,
-                             Procedures::Restore::CandlepinResetMigrations)
+      add_steps_with_context(
+        Procedures::Pulpcore::Migrate,
+        Procedures::Restore::CandlepinResetMigrations
+      )
 
-      add_steps_with_context(Procedures::Restore::RegenerateQueues) if backup.online_backup?
       add_steps_with_context(Procedures::Service::Start,
-                             Procedures::Service::DaemonReload)
+        Procedures::Service::DaemonReload)
       add_step(Procedures::Installer::Upgrade.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,Metrics/PerceivedComplexity
+    # rubocop:enable Metrics/CyclomaticComplexity
 
     def drop_dbs(backup)
       if backup.file_map[:candlepin_dump][:present] ||
@@ -84,35 +82,21 @@ module ForemanMaintain::Scenarios
       end
     end
 
-    def restore_mongo_dump(backup)
-      if backup.file_map[:mongo_dump][:present]
-        add_steps_with_context(Procedures::Restore::MongoDump)
-      end
-    end
-
-    def supported_version_check
-      if feature(:instance).downstream && feature(:instance).downstream.less_than_version?('6.3')
-        msg = 'ERROR: Restore subcommand is supported by Satellite 6.3+. ' \
-              'Please use katello-restore instead.'
-        abort(msg)
-      end
-    end
-
     def set_context_mapping
       context.map(:backup_dir,
-                  Checks::Restore::ValidateBackup => :backup_dir,
-                  Checks::Restore::ValidateHostname => :backup_dir,
-                  Checks::Restore::ValidateInterfaces => :backup_dir,
-                  Procedures::Restore::Configs => :backup_dir,
-                  Procedures::Restore::DropDatabases => :backup_dir,
-                  Procedures::Restore::CandlepinDump => :backup_dir,
-                  Procedures::Restore::ForemanDump => :backup_dir,
-                  Procedures::Restore::PulpcoreDump => :backup_dir,
-                  Procedures::Restore::ExtractFiles => :backup_dir,
-                  Procedures::Restore::MongoDump => :backup_dir)
+        Checks::Restore::ValidateBackup => :backup_dir,
+        Checks::Restore::ValidateHostname => :backup_dir,
+        Checks::Restore::ValidateInterfaces => :backup_dir,
+        Checks::Restore::ValidatePostgresqlDumpPermissions => :backup_dir,
+        Procedures::Restore::Configs => :backup_dir,
+        Procedures::Restore::DropDatabases => :backup_dir,
+        Procedures::Restore::CandlepinDump => :backup_dir,
+        Procedures::Restore::ForemanDump => :backup_dir,
+        Procedures::Restore::PulpcoreDump => :backup_dir,
+        Procedures::Restore::ExtractFiles => :backup_dir)
 
       context.map(:incremental_backup,
-                  Procedures::Selinux::SetFileSecurity => :incremental_backup)
+        Procedures::Selinux::SetFileSecurity => :incremental_backup)
     end
   end
 

data/definitions/scenarios/self_upgrade.rb

@@ -26,18 +26,14 @@ module ForemanMaintain::Scenarios
     end
 
     def maintenance_repo(version)
-      if el7?
-        "rhel-#{el_major_version}-server-satellite-maintenance-#{version}-rpms"
-      else
-        "satellite-maintenance-#{version}-for-rhel-#{el_major_version}-x86_64-rpms"
-      end
+      "satellite-maintenance-#{version}-for-rhel-#{el_major_version}-x86_64-rpms"
     end
 
     def use_rhsm?
       return false if maintenance_repo_label
 
-      if (repo = ENV['MAINTENANCE_REPO_LABEL'])
-        return false unless repo.empty?
+      if (repo = ENV['MAINTENANCE_REPO_LABEL']) && !repo.empty?
+        return false
       end
 
       true
@@ -64,7 +60,7 @@ module ForemanMaintain::Scenarios
     metadata do
       label :self_upgrade_foreman_maintain
       description "Enables the specified version's maintenance repository and,"\
-  								"\nupdates the satellite-maintain packages"
+                  "\nupdates the satellite-maintain packages"
       manual_detection
     end
 
@@ -76,7 +72,7 @@ module ForemanMaintain::Scenarios
           "--enablerepo=#{id}"
         end
         add_step(Procedures::Packages::Update.new(packages: pkgs_to_update, assumeyes: true,
-                                                  yum_options: yum_options))
+          yum_options: yum_options))
       end
     end
 
@@ -94,14 +90,14 @@ module ForemanMaintain::Scenarios
 
     def rollback_repositories
       installed_release_pkg = package_manager.find_installed_package('foreman-release',
-                                                                     '%{VERSION}')
+        '%{VERSION}')
 
       unless current_version.nil? && installed_release_pkg.nil?
         current_major_version = current_version[0..2]
         installed_foreman_release_major_version = installed_release_pkg[0..2]
         if installed_foreman_release_major_version != current_major_version
           add_step(Procedures::Packages::Uninstall.new(packages: %w[foreman-release katello-repos],
-                                                       assumeyes: true))
+            assumeyes: true))
           add_step(Procedures::Repositories::Setup.new(:version => current_major_version))
         end
       end

data/definitions/scenarios/services.rb

@@ -14,13 +14,13 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Restart => :only)
+        Procedures::Service::Restart => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Restart => :exclude)
+        Procedures::Service::Restart => :exclude)
 
       context.map(:wait_for_server_response,
-                  Procedures::Service::Restart => :wait_for_server_response)
+        Procedures::Service::Restart => :wait_for_server_response)
     end
   end
 
@@ -39,10 +39,10 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Stop => :only)
+        Procedures::Service::Stop => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Stop => :exclude)
+        Procedures::Service::Stop => :exclude)
     end
   end
 
@@ -61,10 +61,10 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Start => :only)
+        Procedures::Service::Start => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Start => :exclude)
+        Procedures::Service::Start => :exclude)
     end
   end
 
@@ -82,10 +82,10 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::List => :only)
+        Procedures::Service::List => :only)
 
       context.map(:exclude,
-                  Procedures::Service::List => :exclude)
+        Procedures::Service::List => :exclude)
     end
   end
 
@@ -104,10 +104,10 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Enable => :only)
+        Procedures::Service::Enable => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Enable => :exclude)
+        Procedures::Service::Enable => :exclude)
     end
   end
 
@@ -126,10 +126,10 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Disable => :only)
+        Procedures::Service::Disable => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Disable => :exclude)
+        Procedures::Service::Disable => :exclude)
     end
   end
 
@@ -147,16 +147,16 @@ module ForemanMaintain::Scenarios
 
     def set_context_mapping
       context.map(:only,
-                  Procedures::Service::Status => :only)
+        Procedures::Service::Status => :only)
 
       context.map(:exclude,
-                  Procedures::Service::Status => :exclude)
+        Procedures::Service::Status => :exclude)
 
       context.map(:brief,
-                  Procedures::Service::Status => :brief)
+        Procedures::Service::Status => :brief)
 
       context.map(:failing,
-                  Procedures::Service::Status => :failing)
+        Procedures::Service::Status => :failing)
     end
   end
 end

data/extras/foreman_protector/foreman-protector.whitelist

@@ -17,12 +17,6 @@ boost-random
 boost-iostreams
 boost-thread
 yum-utils
-# el7 psql client dependencies
-rh-postgresql12-postgresql
-rh-postgresql12-postgresql-libs
-rh-postgresql12-postgresql-syspaths
-rh-postgresql12-runtime
-scl-utils
 # el8 psql client dependencies
 postgresql
 libpq

data/lib/foreman_maintain/cli/advanced/prebuild_bash_completion.rb

@@ -9,7 +9,7 @@ module ForemanMaintain
         answers = feature(:installer).configuration[:answer_file]
         comp_map[:expire] = {
           :file => answers,
-          :sha1sum => execute!("sha1sum #{answers}")
+          :sha1sum => execute!("sha1sum #{answers}"),
         }
         cache_dir = File.dirname(ForemanMaintain.config.completion_cache_file)
         FileUtils.mkdir_p(cache_dir) unless File.directory?(cache_dir)

data/lib/foreman_maintain/cli/advanced_command.rb

@@ -6,8 +6,8 @@ module ForemanMaintain
     class AdvancedCommand < Base
       subcommand 'procedure', 'Run maintain procedures manually', ProcedureCommand
       subcommand 'prebuild-bash-completion',
-                 'Prepare map of options and subcommands for Bash completion',
-                 PrebuildBashCompletionCommand
+        'Prepare map of options and subcommands for Bash completion',
+        PrebuildBashCompletionCommand
     end
   end
 end

data/lib/foreman_maintain/cli/backup_command.rb

@@ -33,20 +33,20 @@ module ForemanMaintain
         def common_backup_options
           # TODO: BACKUP_DIR in f-m config - should be default?
           parameter 'BACKUP_DIR', 'Path to backup dir',
-                    :completion => { :type => :directory },
-                    :attribute_name => :backup_root_dir do |dir|
+            :completion => { :type => :directory },
+            :attribute_name => :backup_root_dir do |dir|
             File.expand_path(dir)
           end
           option ['-s', '--skip-pulp-content'], :flag, 'Do not backup Pulp content'
           option ['-p', '--preserve-directory'], :flag, 'Do not create a time-stamped subdirectory'
           option ['-t', '--split-pulp-tar'], 'SPLIT_SIZE',
-                 'Split pulp data into files of a specified size, i.e. (100M, 50G). ' \
-                 "See '--tape-length' in 'info tar' for all sizes" do |size|
+            'Split pulp data into files of a specified size, i.e. (100M, 50G). ' \
+            "See '--tape-length' in 'info tar' for all sizes" do |size|
             self.class.valid_tape_size(size)
           end
           option ['-i', '--incremental'], 'PREVIOUS_BACKUP_DIR',
-                 'Backup changes since previous backup',
-                 :completion => { :type => :directory } do |dir|
+            'Backup changes since previous backup',
+            :completion => { :type => :directory } do |dir|
             unless File.directory?(dir)
               raise ArgumentError, "Previous backup directory does not exist: #{dir}"
             end
@@ -55,8 +55,8 @@ module ForemanMaintain
           end
           proxy_name = ForemanMaintain.detector.feature(:capsule) ? 'Capsule' : 'Foreman Proxy'
           option '--features', 'FEATURES',
-                 "#{proxy_name} features to include in the backup. " \
-                     'Valid features are tftp, dns, dhcp, openscap, and all.', :multivalued => true
+            "#{proxy_name} features to include in the backup. " \
+                'Valid features are tftp, dns, dhcp, openscap, and all.', :multivalued => true
         end
         # rubocop:enable  Metrics/MethodLength
 
@@ -76,7 +76,7 @@ module ForemanMaintain
         Scenarios::BackupRescueCleanup.new({
           :backup_dir => backup_dir,
           :strategy => strategy,
-          :preserve_dir => preserve_directory?
+          :preserve_dir => preserve_directory?,
         }.merge(options))
       end
 
@@ -88,7 +88,7 @@ module ForemanMaintain
           :proxy_features => features,
           :tar_volume_size => split_pulp_tar,
           :skip_pulp_content => skip_pulp_content?,
-          :incremental_dir => incremental
+          :incremental_dir => incremental,
         }.merge(options))
       end
 
@@ -123,7 +123,7 @@ module ForemanMaintain
 
       def execute
         perform_backup(:offline,
-                       :include_db_dumps => include_db_dumps?)
+          :include_db_dumps => include_db_dumps?)
       end
     end
 
@@ -133,21 +133,21 @@ module ForemanMaintain
       common_backup_options
       option '--include-db-dumps', :flag, 'Also dump full database schema before snapshot backup'
       option ['-d', '--snapshot-mount-dir'], 'SNAPSHOT_MOUNT_DIR',
-             "Override default directory ('/var/snap/') where the snapshots will be mounted",
-             :default => '/var/snap/' do |dir|
+        "Override default directory ('/var/snap/') where the snapshots will be mounted",
+        :default => '/var/snap/' do |dir|
         unless File.directory?(dir)
           raise ArgumentError, "Snapshot mount directory does not exist: #{dir}"
         end
         dir
       end
       option ['-b', '--snapshot-block-size'], 'SNAPSHOT_BLOCK_SIZE',
-             'Override default block size (2G)', :default => '2G'
+        'Override default block size (2G)', :default => '2G'
 
       def execute
         perform_backup(:snapshot,
-                       :snapshot_mount_dir => snapshot_mount_dir,
-                       :snapshot_block_size => snapshot_block_size,
-                       :include_db_dumps => include_db_dumps?)
+          :snapshot_mount_dir => snapshot_mount_dir,
+          :snapshot_block_size => snapshot_block_size,
+          :include_db_dumps => include_db_dumps?)
       end
     end
 

data/lib/foreman_maintain/cli/base.rb

@@ -57,10 +57,10 @@ module ForemanMaintain
       def run_scenario(scenarios, rescue_scenario = nil)
         @runner ||=
           ForemanMaintain::Runner.new(reporter, scenarios,
-                                      :assumeyes => option_wrapper('assumeyes?'),
-                                      :whitelist => option_wrapper('whitelist') || [],
-                                      :force => option_wrapper('force?'),
-                                      :rescue_scenario => rescue_scenario)
+            :assumeyes => option_wrapper('assumeyes?'),
+            :whitelist => option_wrapper('whitelist') || [],
+            :force => option_wrapper('force?'),
+            :rescue_scenario => rescue_scenario)
         runner.run
       end
 
@@ -135,8 +135,8 @@ module ForemanMaintain
 
       def self.label_option
         option '--label', 'label',
-               'Run only a specific check with a label. ' \
-                 '(Use "list" command to see available labels)' do |label|
+          'Run only a specific check with a label. ' \
+            '(Use "list" command to see available labels)' do |label|
           raise ArgumentError, 'value not specified' if label.nil? || label.empty?
           underscorize(label).to_sym
         end
@@ -144,9 +144,9 @@ module ForemanMaintain
 
       def self.tags_option
         option('--tags', 'tags',
-               'Run only those with all specific set of tags. ' \
-                 '(Use list-tags command to see available tags)',
-               :multivalued => true) do |tags|
+          'Run only those with all specific set of tags. ' \
+            '(Use list-tags command to see available tags)',
+          :multivalued => true) do |tags|
           raise ArgumentError, 'value not specified' if tags.nil? || tags.empty?
           tags.map { |tag| underscorize(tag).to_sym }
         end
@@ -158,14 +158,14 @@ module ForemanMaintain
 
         if opts.include?('assumeyes')
           option ['-y', '--assumeyes'], :flag,
-                 'Automatically answer yes for all questions' do |assume|
+            'Automatically answer yes for all questions' do |assume|
             ForemanMaintain.reporter.assumeyes = assume
           end
         end
 
         if opts.include?('whitelist')
           option(['-w', '--whitelist'], 'whitelist',
-                 'Comma-separated list of labels of steps to be skipped') do |whitelist|
+            'Comma-separated list of labels of steps to be skipped') do |whitelist|
             raise ArgumentError, 'value not specified' if whitelist.nil? || whitelist.empty?
             whitelist.split(',').map(&:strip)
           end
@@ -173,12 +173,12 @@ module ForemanMaintain
 
         if opts.include?('force')
           option ['-f', '--force'], :flag,
-                 'Force steps that would be skipped as they were already run'
+            'Force steps that would be skipped as they were already run'
         end
 
         if opts.include?('plaintext')
           option(['--plaintext'], :flag,
-                 'Print the output in plaintext and disable the spinner') do |plaintext|
+            'Print the output in plaintext and disable the spinner') do |plaintext|
             ForemanMaintain.reporter.plaintext = plaintext
           end
         end

data/lib/foreman_maintain/cli/maintenance_mode_command.rb

@@ -42,7 +42,7 @@ module ForemanMaintain
         end
 
         def fetch_procedure(scenario, procedure_class_name)
-          scenario.steps.find { |procedure| procedure.class.eql?(procedure_class_name) }
+          scenario.steps.find { |procedure| procedure.instance_of?(procedure_class_name) }
         end
       end
     end

data/lib/foreman_maintain/cli/plugin_command.rb

@@ -3,7 +3,7 @@ module ForemanMaintain
     class PluginCommand < Base
       subcommand 'purge-puppet', 'Remove the Puppet feature' do
         option ['-f', '--remove-all-data'], :flag, 'Purge all the Puppet data',
-               :attribute_name => :remove_data
+          :attribute_name => :remove_data
 
         def execute
           run_scenarios_and_exit(Scenarios::Puppet::RemovePuppet.new(:remove_data => remove_data?))

data/lib/foreman_maintain/cli/restore_command.rb

@@ -3,13 +3,13 @@ module ForemanMaintain
     class RestoreCommand < Base
       interactive_option
       parameter 'BACKUP_DIR', 'Path to backup directory to restore',
-                :attribute_name => :backup_dir, :completion => { :type => :directory }
+        :attribute_name => :backup_dir, :completion => { :type => :directory }
 
       option ['-i', '--incremental'], :flag, 'Restore an incremental backup',
-             :attribute_name => :incremental, :completion => { :type => :directory }
+        :attribute_name => :incremental, :completion => { :type => :directory }
       option ['-n', '--dry-run'], :flag,
-             'Check if backup could be restored, without performing the restore',
-             :attribute_name => :dry_run
+        'Check if backup could be restored, without performing the restore',
+        :attribute_name => :dry_run
 
       def execute
         scenario = Scenarios::Restore.new(

data/lib/foreman_maintain/cli/self_upgrade_command.rb

@@ -2,8 +2,8 @@ module ForemanMaintain
   module Cli
     class SelfUpgradeCommand < Base
       option ['--maintenance-repo-label'], 'REPOSITORY_LABEL',\
-             'Repository label from which packages should be updated.'\
-             'This can be used when standard CDN repositories are unavailable.'
+        'Repository label from which packages should be updated.'\
+        'This can be used when standard CDN repositories are unavailable.'
       def execute
         run_scenario(upgrade_scenario)
       end

data/lib/foreman_maintain/cli/service_command.rb

@@ -35,8 +35,8 @@ module ForemanMaintain
         service_options
         if feature(:katello)
           option ['-p', '--wait-for-server-response', '--wait-for-hammer-ping'], :flag,
-                 'Wait for hammer ping to return successfully before terminating',
-                 :attribute_name => :wait_for_server_response
+            'Wait for hammer ping to return successfully before terminating',
+            :attribute_name => :wait_for_server_response
         end
 
         def execute

data/lib/foreman_maintain/cli/transform_clamp_options.rb

@@ -26,7 +26,7 @@ module ForemanMaintain
         private
 
         def option_sym(option)
-          option = underscorize(option.switches.first[2..-1].to_s)
+          option = underscorize(option.switches.first[2..].to_s)
           option.to_sym unless option.empty?
         end
 

data/lib/foreman_maintain/cli/upgrade_command.rb

@@ -3,20 +3,20 @@ module ForemanMaintain
     class UpgradeCommand < Base
       def self.target_version_option
         option '--target-version', 'TARGET_VERSION', 'Target version of the upgrade',
-               :required => false
+          :required => false
       end
 
       def self.disable_self_upgrade_option
         option '--disable-self-upgrade', :flag, 'Disable automatic self upgrade',
-               :default => false
+          :default => false
       end
 
       def current_target_version
         current_target_version = ForemanMaintain::UpgradeRunner.current_target_version
         if current_target_version && target_version && target_version != current_target_version
           raise Error::UsageError,
-                "Can't set target version #{target_version}, "\
-                "#{current_target_version} already in progress"
+            "Can't set target version #{target_version}, "\
+            "#{current_target_version} already in progress"
         end
         @target_version = current_target_version if current_target_version
         return true if current_target_version
@@ -43,9 +43,9 @@ module ForemanMaintain
         return @upgrade_runner if defined? @upgrade_runner
         validate_target_version!
         @upgrade_runner = ForemanMaintain::UpgradeRunner.new(target_version, reporter,
-                                                             :assumeyes => assumeyes?,
-                                                             :whitelist => whitelist || [],
-                                                             :force => force?).tap(&:load)
+          :assumeyes => assumeyes?,
+          :whitelist => whitelist || [],
+          :force => force?).tap(&:load)
       end
 
       def print_versions(target_versions)

data/lib/foreman_maintain/cli.rb

@@ -10,7 +10,6 @@ require 'foreman_maintain/cli/service_command'
 require 'foreman_maintain/cli/restore_command'
 require 'foreman_maintain/cli/maintenance_mode_command'
 require 'foreman_maintain/cli/packages_command'
-require 'foreman_maintain/cli/content_command'
 require 'foreman_maintain/cli/plugin_command'
 require 'foreman_maintain/cli/self_upgrade_command'
 
@@ -26,19 +25,10 @@ module ForemanMaintain
       subcommand 'restore', 'Restore a backup', RestoreCommand
       subcommand 'packages', 'Lock/Unlock package protection, install, update', PackagesCommand
       subcommand 'advanced', 'Advanced tools for server maintenance', AdvancedCommand
-      subcommand 'content', 'Content related commands', ContentCommand
       subcommand 'plugin', 'Manage optional plugins on your server', PluginCommand
       subcommand 'self-upgrade', 'Perform major version self upgrade', SelfUpgradeCommand
       subcommand 'maintenance-mode', 'Control maintenance-mode for application',
-                 MaintenanceModeCommand
-      if ForemanMaintain.detector.feature(:satellite) &&
-         ForemanMaintain.detector.feature(:satellite).current_minor_version == '6.9'
-        subcommand 'prep-6.10-upgrade', 'Preparations for the Satellite 6.10 upgrade' do
-          def execute
-            run_scenarios_and_exit(Scenarios::Prep610Upgrade.new)
-          end
-        end
-      end
+        MaintenanceModeCommand
 
       def run(*arguments)
         logger.info("Running foreman-maintain command with arguments #{arguments.inspect}")