foreman_maintain 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c76360569117ea968a720fbb493d1348a396b01b
-  data.tar.gz: 746747370c85d10ad6f0dec6dfe7123085485e3a
+  metadata.gz: 8e9f70b0ee10a6f3d86da947c29b44f28425fe3d
+  data.tar.gz: 25c1ffb8b8b12ff26a2ee7cad07f407b7586f276
 SHA512:
-  metadata.gz: 276ab0eadfd9ac48178a4650aab47e54d215d9504f164f5b8a0342dc4271bea01718a27e49e50ac4fe6768c1e50772d296e493223da8b87d2ec2b468dba12ce4
-  data.tar.gz: 94cce0df009844faad60bb89d81a792a563a9850897d949c2722e35227700d21e554b8ffd1f9321eab243c1b45ef3537ee963d7aeab440b0cd3e168e18ee4bb4
+  metadata.gz: ab4457e9ea8319b82ac952b5d389bd2156d384076d2d36db0150b51a04127b62393617327118d715ebb468bcf104ceaefda96cb86d641a0da8724d4ae29cb632
+  data.tar.gz: 4d59d274a7f28c85f4c315766e276890fb9c5285225fd02dcf5aef62e495b08343e8becfcdd7cbb39ddf916b6be38180692dba4ffc83ff9f83fddc9570c12e80

data/README.md

@@ -18,7 +18,7 @@ Subcommands:
 
     upgrade                       Upgrade related commands
       list-versions                 List versions this system is upgradable to
-      check --target-version TARGET_VERSION   Run pre-upgrade checks for upgradeing to specified version
+      check --target-version TARGET_VERSION   Run pre-upgrade checks for upgrading to specified version
       run --target-version TARGET_VERSION     Run the full upgrade
           [--phase=phase TARGET_VERSION]      Run just a specific phase of the upgrade
 ```
@@ -56,7 +56,7 @@ of the system:
     at the current version, while this phase runs.
 
   * **pre-migrations** - these steps perform changes on the system before
-    the actual upgrade stars. An example is disabling access to the system from
+    the actual upgrade starts. An example is disabling access to the system from
     external sources, a.k.a. maintenance mode or disabling sync plans during the run.
 
     After this phase ends, the system is still running the old version, and it's possible

data/definitions/checks/candlepin/validate_db.rb

@@ -0,0 +1,16 @@
+module Checks::Candlepin
+  class ValidateDb < ForemanMaintain::Check
+    metadata do
+      description 'Check to validate candlepin database'
+      tags :pre_upgrade
+
+      confine do
+        feature(:candlepin_database) && feature(:candlepin_database).validate_available_in_cpdb?
+      end
+    end
+
+    def run
+      feature(:candlepin_database).execute_cpdb_validate_cmd
+    end
+  end
+end

data/definitions/checks/disk/performance.rb

@@ -0,0 +1,67 @@
+module Checks
+  module Disk
+    class Performance < ForemanMaintain::Check
+      metadata do
+        label :disk_performance
+        description 'Check for recommended disk speed of pulp, mongodb, pgsql dir.'
+        tags :pre_upgrade
+        preparation_steps { Procedures::Packages::Install.new(:packages => %w[hdparm fio]) }
+
+        confine do
+          feature(:pulp)
+        end
+      end
+
+      EXPECTED_IO = 80
+      DEFAULT_UNIT   = 'MB/sec'.freeze
+      DEFAULT_DIRS   = ['/var/lib/pulp', '/var/lib/mongodb', '/var/lib/pgsql'].freeze
+
+      attr_reader :stats
+
+      def run
+        @stats = ForemanMaintain::Utils::Disk::Stats.new
+        with_spinner(description) do |spinner|
+          io_obj, success = compute_disk_speed(spinner)
+          spinner.update('Finished')
+          puts "\n"
+          puts stats.stdout
+
+          assert(success, io_obj.slow_disk_error_msg)
+        end
+      end
+
+      def check_only_single_device?
+        DEFAULT_DIRS.map do |dir|
+          ForemanMaintain::Utils::Disk::Device.new(dir).name
+        end.uniq.length <= 1
+      end
+
+      def dirs_to_check
+        return DEFAULT_DIRS.first(1) if check_only_single_device?
+        DEFAULT_DIRS
+      end
+
+      private
+
+      def compute_disk_speed(spinner)
+        success = true
+        io_obj = ForemanMaintain::Utils::Disk::NilDevice.new
+
+        dirs_to_check.each do |dir|
+          io_obj = ForemanMaintain::Utils::Disk::Device.new(dir)
+
+          spinner.update("[Speed check In-Progress] device: #{io_obj.name}")
+          stats << io_obj
+
+          next if io_obj.read_speed >= EXPECTED_IO
+
+          success = false
+          logger.info "Slow disk detected #{dir}: #{io_obj.performance}."
+          break
+        end
+
+        [io_obj, success]
+      end
+    end
+  end
+end

data/definitions/features/candlepin_database.rb

@@ -0,0 +1,62 @@
+class Features::CandlepinDatabase < ForemanMaintain::Feature
+  CANDLEPIN_DB_CONFIG = '/etc/candlepin/candlepin.conf'.freeze
+
+  include ForemanMaintain::Concerns::BaseDatabase
+
+  metadata do
+    label :candlepin_database
+
+    confine do
+      file_exists?(CANDLEPIN_DB_CONFIG)
+    end
+  end
+
+  def configuration
+    @configuration || load_configuration
+  end
+
+  def validate_available_in_cpdb?
+    check_option_using_cpdb_help('validate')
+  end
+
+  def check_option_using_cpdb_help(option_name, parent_cmd = '')
+    parent_cmd = '/usr/share/candlepin/cpdb' if parent_cmd.empty?
+    help_cmd = "#{parent_cmd} --help |  grep -c '\\-\\-#{option_name} '"
+    execute?(help_cmd)
+  end
+
+  def execute_cpdb_validate_cmd
+    main_cmd = cpdb_validate_cmd
+    unless main_cmd.empty?
+      main_cmd += format_shell_args(
+        '-u' => configuration['username'], '-p' => configuration[%(password)]
+      )
+      execute!(main_cmd, :hidden_patterns => [configuration['password']])
+    end
+  end
+
+  private
+
+  def load_configuration
+    raw_config = File.read(CANDLEPIN_DB_CONFIG)
+    full_config = Hash[raw_config.scan(/(^[^#\n][^=]*)=(.*)/)]
+    uri = %r{://(([^/:]*):?([^/]*))/(.*)}.match(full_config['org.quartz.dataSource.myDS.URL'])
+    @configuration = {
+      'username' => full_config['org.quartz.dataSource.myDS.user'],
+      'password' => full_config['org.quartz.dataSource.myDS.password'],
+      'database' => uri[4],
+      'host' => uri[2],
+      'port' => uri[3] || '5432',
+      'driver' => full_config['org.quartz.dataSource.myDS.driver'],
+      'url' => full_config['org.quartz.dataSource.myDS.URL']
+    }
+  end
+
+  def cpdb_validate_cmd
+    return '' unless check_option_using_cpdb_help('validate')
+    cmd = '/usr/share/candlepin/cpdb --validate'
+    return cmd unless check_option_using_cpdb_help('verbose', cmd)
+    cmd += ' --verbose'
+    cmd
+  end
+end

data/definitions/features/downstream.rb

@@ -34,16 +34,15 @@ class Features::Downstream < ForemanMaintain::Feature
   def rh_repos(sat_version)
     sat_version = version(sat_version)
     rh_version_major = execute!('facter operatingsystemmajrelease')
+    sat_version_full = "#{sat_version.major}.#{sat_version.minor}"
 
-    sat_repo_id = "rhel-#{rh_version_major}-server-satellite-#{sat_version.major}"\
-                  ".#{sat_version.minor}-rpms"
-    # Override to use Beta repositories for 6.3 until GA
-    if sat_version.to_s == '6.3'
-      sat_repo_id = "rhel-server-#{rh_version_major}-satellite-6-beta-rpms"
-    end
+    sat_repo_id = "rhel-#{rh_version_major}-server-satellite-#{sat_version_full}-rpms"
+    sat_tools_repo_id = "rhel-#{rh_version_major}-server-satellite-tools-#{sat_version_full}-rpms"
 
     ["rhel-#{rh_version_major}-server-rpms",
      "rhel-server-rhscl-#{rh_version_major}-rpms",
+     "rhel-#{rh_version_major}-server-satellite-maintenance-6-rpms",
+     sat_tools_repo_id,
      sat_repo_id]
   end
 

data/definitions/features/foreman_database.rb

@@ -1,21 +1,24 @@
 class Features::ForemanDatabase < ForemanMaintain::Feature
+  FOREMAN_DB_CONFIG = '/etc/foreman/database.yml'.freeze
+
+  include ForemanMaintain::Concerns::BaseDatabase
+
   metadata do
     label :foreman_database
 
     confine do
-      File.exist?('/etc/foreman/database.yml')
+      file_exists?(FOREMAN_DB_CONFIG)
     end
   end
 
-  def query(sql)
-    parse_csv(query_csv(sql))
+  def configuration
+    @configuration || load_configuration
   end
 
-  def query_csv(sql)
-    psql(%{COPY (#{sql}) TO STDOUT WITH CSV HEADER})
-  end
+  private
 
-  def psql(query)
-    execute("su postgres -c 'cd ~; psql -d foreman'", :stdin => query)
+  def load_configuration
+    config = YAML.load(File.read(FOREMAN_DB_CONFIG))
+    @configuration = config['production']
   end
 end

data/definitions/features/foreman_proxy.rb

@@ -20,7 +20,7 @@ class Features::ForemanProxy < ForemanMaintain::Feature
   private
 
   def dhcp_curl_cmd
-    "curl -w '\n%{http_code}' -slient -ks --cert #{cert_path}/client_cert.pem \
+    "curl -w '\n%{http_code}' --silent -ks --cert #{cert_path}/client_cert.pem \
       --key #{cert_path}/client_key.pem \
       --cacert #{cert_path}/proxy_ca.pem https://$(hostname):9090/dhcp"
   end

data/definitions/features/katello_service.rb

@@ -13,7 +13,7 @@ class Features::KatelloService < ForemanMaintain::Feature
       yield if block_given?
     else
       begin
-        filters = "--only #{services.join(',')}"
+        filters = construct_filters(services)
         spinner.update 'Stopping katello running services..'
         execute!("katello-service stop #{filters}")
         yield if block_given?
@@ -28,7 +28,7 @@ class Features::KatelloService < ForemanMaintain::Feature
     if services.empty?
       spinner.update 'No katello service to start'
     else
-      filters = "--only #{services.join(',')}"
+      filters = construct_filters(services)
       spinner.update 'Starting katello services..'
       execute!("katello-service start #{filters}")
     end
@@ -36,7 +36,7 @@ class Features::KatelloService < ForemanMaintain::Feature
 
   def restart(options = {})
     if options[:only] || options[:exclude]
-      filters = construct_filters_for_restart(options)
+      filters = construct_filters(options[:only], options[:exclude])
       execute!("katello-service restart #{filters}")
     else
       execute!('katello-service restart')
@@ -59,6 +59,10 @@ class Features::KatelloService < ForemanMaintain::Feature
     logger.error e.message
   end
 
+  def service_list
+    @service_list ||= katello_service_names.map { |s| s.gsub('.service', '') }
+  end
+
   private
 
   def apply_sleep_before_retry(spinner, result)
@@ -67,13 +71,18 @@ class Features::KatelloService < ForemanMaintain::Feature
     sleep PING_RETRY_INTERVAL
   end
 
-  def construct_filters_for_restart(options)
+  def construct_filters(only_services, exclude_services = [])
     filters = ''
-    if options[:only]
-      filters += "--only #{options[:only].join(',')}" unless options[:only].empty?
+    unless only_services.empty?
+      if feature(:downstream) && feature(:downstream).current_minor_version <= '6.1'
+        exclude_services.concat(service_list - only_services)
+        exclude_services.uniq!
+      else
+        filters += "--only #{only_services.join(',')}"
+      end
     end
-    if options[:exclude]
-      filters += "--exclude #{options[:exclude].join(',')}" unless options[:exclude].empty?
+    unless exclude_services.empty?
+      filters += "--exclude #{exclude_services.join(',')}"
     end
     filters
   end
@@ -98,8 +107,11 @@ class Features::KatelloService < ForemanMaintain::Feature
     find_services_by_state(" -w 'dead'")
   end
 
+  def katello_service_names
+    @katello_service_names ||= execute("katello-service list|awk '{print $1}'").split(/\n/)
+  end
+
   def find_services_by_state(state)
-    katello_service_names = execute("katello-service list|awk '{print $1}'").split(/\n/)
     services_by_state = execute("systemctl --all |grep #{state}|awk '{print $1}'").split(/\n/)
     (katello_service_names & services_by_state).map { |s| s.gsub('.service', '') }
   end

data/definitions/procedures/foreman_tasks/delete.rb

@@ -1,7 +1,12 @@
 module Procedures::ForemanTasks
   class Delete < ForemanMaintain::Procedure
+    ALLOWED_STATES_VALUES = %w[old planning pending].freeze
+
     metadata do
-      param :state, 'In what state should the task be deleted'
+      param :state,
+            'In what state should the task be deleted.'\
+            " Possible values are #{ALLOWED_STATES_VALUES.join(', ')}",
+            :required => true, :allowed_values => ALLOWED_STATES_VALUES
       description 'Delete tasks'
     end
 
@@ -11,7 +16,6 @@ module Procedures::ForemanTasks
 
         if count_tasks_before > 0
           spinner.update "Backup #{@state} tasks"
-
           feature(:foreman_tasks).backup_tasks(@state) do |backup_progress|
             spinner.update backup_progress
           end
@@ -19,11 +23,10 @@ module Procedures::ForemanTasks
           spinner.update "Deleting #{@state} tasks [running]"
           count_tasks_later = feature(:foreman_tasks).delete(@state)
           spinner.update "Deleting #{@state} tasks [DONE]"
+          spinner.update(
+            "Deleted #{@state} stopped and paused tasks: #{count_tasks_before - count_tasks_later}"
+          )
         end
-
-        spinner.update(
-          "Deleted #{@state} stopped and paused tasks: #{count_tasks_before - count_tasks_later}"
-        )
       end
     end
 

data/definitions/procedures/katello_service/restart.rb

@@ -16,8 +16,8 @@ module Procedures::KatelloService
 
     def runtime_message
       msg = 'katello-service restart'
-      msg += "--only #{@only.join(',')}" unless @only.empty?
-      msg += "--exclude #{@exclude.join(',')}" unless @exclude.empty?
+      msg += " --only #{@only.join(',')}" unless @only.empty?
+      msg += " --exclude #{@exclude.join(',')}" unless @exclude.empty?
       msg
     end
   end

data/definitions/scenarios/upgrade_to_satellite_6_2_z.rb

@@ -43,6 +43,7 @@ module Scenarios::Satellite_6_2_z
     end
 
     def compose
+      add_step(Procedures::Repositories::Setup.new(:version => '6.2'))
       add_step(Procedures::Packages::Update.new(:assumeyes => true))
       add_step(Procedures::Installer::Upgrade.new)
     end

data/definitions/scenarios/upgrade_to_satellite_6_3_z.rb

@@ -43,6 +43,7 @@ module Scenarios::Satellite_6_3_z
     end
 
     def compose
+      add_step(Procedures::Repositories::Setup.new(:version => '6.3'))
       add_step(Procedures::Packages::Update.new(:assumeyes => true))
       add_step(Procedures::Installer::Upgrade.new)
     end

data/lib/foreman_maintain.rb

@@ -16,6 +16,7 @@ module ForemanMaintain
   require 'foreman_maintain/concerns/scenario_metadata'
   require 'foreman_maintain/concerns/system_helpers'
   require 'foreman_maintain/concerns/hammer'
+  require 'foreman_maintain/concerns/base_database'
   require 'foreman_maintain/top_level_modules'
   require 'foreman_maintain/yaml_storage'
   require 'foreman_maintain/config'

data/lib/foreman_maintain/cli/base.rb

@@ -98,6 +98,8 @@ module ForemanMaintain
       end
 
       def self.interactive_option
+        delete_duplicate_assumeyes_if_any
+
         option ['-y', '--assumeyes'], :flag,
                'Automatically answer yes for all questions'
 
@@ -110,6 +112,10 @@ module ForemanMaintain
         option ['-f', '--force'], :flag,
                'Force steps that would be skipped as they were already run'
       end
+
+      def self.delete_duplicate_assumeyes_if_any
+        declared_options.delete_if { |opt| opt.handles?('--assumeyes') }
+      end
     end
   end
 end

data/lib/foreman_maintain/concerns/base_database.rb

@@ -0,0 +1,33 @@
+module ForemanMaintain
+  module Concerns
+    module BaseDatabase
+      def configuration
+        raise NotImplementedError
+      end
+
+      def query(sql, config = configuration)
+        parse_csv(query_csv(sql, config))
+      end
+
+      def query_csv(sql, config = configuration)
+        psql(%{COPY (#{sql}) TO STDOUT WITH CSV HEADER}, config)
+      end
+
+      def psql(query, config = configuration)
+        execute("PGPASSWORD='#{config[%(password)]}' #{psql_db_connection_str(config)}",
+                :stdin => query)
+      end
+
+      def ping(config = configuration)
+        psql('SELECT 1 as ping', config)
+      end
+
+      private
+
+      def psql_db_connection_str(config)
+        "psql -d #{config['database']} -h #{config['host'] || 'localhost'} "\
+        " -p #{config['port'] || '5432'} -U #{config['username']}"
+      end
+    end
+  end
+end

data/lib/foreman_maintain/concerns/system_helpers.rb

@@ -138,6 +138,10 @@ module ForemanMaintain
       def version(value)
         Version.new(value)
       end
+
+      def format_shell_args(options = {})
+        options.map { |shell_optn, val| " #{shell_optn} '#{shellescape(val)}'" }.join
+      end
     end
   end
 end

data/lib/foreman_maintain/param.rb

@@ -3,13 +3,14 @@ module ForemanMaintain
     attr_reader :name, :description, :options
 
     def initialize(name, description, options, &block)
-      options.validate_options!(:description, :required, :flag, :array)
+      options.validate_options!(:description, :required, :flag, :array, :allowed_values)
       @name = name
       @description = description || options[:description] || ''
       @options = options
       @required = @options.fetch(:required, false)
       @flag = @options.fetch(:flag, false)
       @block = block
+      @allowed_values = @options.fetch(:allowed_values, [])
       @array = @options.fetch(:array, false)
     end
 
@@ -34,6 +35,7 @@ module ForemanMaintain
       elsif flag?
         value = value ? true : false
       end
+      validate_with_allowed_values(value)
       value
     end
     # rubocop:enable Metrics/PerceivedComplexity,Metrics/CyclomaticComplexity
@@ -45,5 +47,21 @@ module ForemanMaintain
         value.to_s.split(',').map(&:strip)
       end
     end
+
+    def validate_with_allowed_values(value)
+      return if @allowed_values.empty?
+      within_allowed = case value
+                       when Array
+                         (value - @allowed_values).empty?
+                       when Symbol, String
+                         @allowed_values.include?(value.to_s)
+                       else
+                         raise NotImplementedError
+                       end
+      return if within_allowed
+      error_msg = "'#{value}' not allowed for #{name} param."
+      raise ForemanMaintain::Error::UsageError,
+            "#{error_msg} Possible values are #{@allowed_values.join(', ')}"
+    end
   end
 end

data/lib/foreman_maintain/upgrade_runner.rb

@@ -209,7 +209,7 @@ module ForemanMaintain
         response = reporter.ask_decision(<<-MESSAGE.strip_heredoc.strip)
             The pre-upgrade checks indicate that the system is ready for upgrade.
             It's recommended to perform a backup at this stage.
-            Confirm to continue with the the modification part of the upgrade
+            Confirm to continue with the modification part of the upgrade
         MESSAGE
         if [:no, :quit].include?(response)
           ask_to_quit

data/lib/foreman_maintain/utils/command_runner.rb

@@ -93,6 +93,7 @@ module ForemanMaintain
       end
 
       def hide_strings(string)
+        return unless string
         @hidden_patterns.reduce(string) do |result, hidden_pattern|
           result.gsub(hidden_pattern, '[FILTERED]')
         end

data/lib/foreman_maintain/utils/disk.rb

@@ -1,3 +1,4 @@
 require 'foreman_maintain/utils/disk/io'
 require 'foreman_maintain/utils/disk/device'
+require 'foreman_maintain/utils/disk/stats'
 require 'foreman_maintain/utils/disk/nil_device'

data/lib/foreman_maintain/utils/disk/device.rb

@@ -26,6 +26,10 @@ module ForemanMaintain
              Expected disk speed: #{expected_io} #{default_unit}."
         end
 
+        def performance
+          "#{read_speed} #{unit}"
+        end
+
         private
 
         def init_io_device
@@ -46,11 +50,11 @@ module ForemanMaintain
         end
 
         def default_unit
-          Checks::DiskSpeedMinimal::DEFAULT_UNIT
+          Checks::Disk::Performance::DEFAULT_UNIT
         end
 
         def expected_io
-          Checks::DiskSpeedMinimal::EXPECTED_IO
+          Checks::Disk::Performance::EXPECTED_IO
         end
       end
     end

data/lib/foreman_maintain/utils/disk/stats.rb

@@ -0,0 +1,25 @@
+module ForemanMaintain
+  module Utils
+    module Disk
+      class Stats
+        attr_accessor :data
+
+        def initialize
+          @data = {}
+        end
+
+        def <<(io_obj)
+          data[io_obj.dir] = io_obj.performance
+        end
+
+        def stdout
+          if data.keys.length > 1
+            data.map { |dir, perf| "#{dir} : #{perf}" }.join("\n")
+          else
+            "Disk speed : #{data.values.first}"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '0.1.2'.freeze
+  VERSION = '0.1.3'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-08 00:00:00.000000000 Z
+date: 2018-01-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clamp
@@ -111,7 +111,8 @@ files:
 - bin/passenger-recycler
 - config/foreman_maintain.yml.example
 - config/foreman_maintain.yml.packaging
-- definitions/checks/disk_speed_minimal.rb
+- definitions/checks/candlepin/validate_db.rb
+- definitions/checks/disk/performance.rb
 - definitions/checks/foreman_proxy/verify_dhcp_config_syntax.rb
 - definitions/checks/foreman_tasks/invalid/check_old.rb
 - definitions/checks/foreman_tasks/invalid/check_pending_state.rb
@@ -120,6 +121,7 @@ files:
 - definitions/checks/foreman_tasks/not_running.rb
 - definitions/checks/hammer_ping.rb
 - definitions/checks/system_registration.rb
+- definitions/features/candlepin_database.rb
 - definitions/features/downstream.rb
 - definitions/features/foreman_1_11_x.rb
 - definitions/features/foreman_1_7_x.rb
@@ -167,6 +169,7 @@ files:
 - lib/foreman_maintain/cli/health_command.rb
 - lib/foreman_maintain/cli/transform_clamp_options.rb
 - lib/foreman_maintain/cli/upgrade_command.rb
+- lib/foreman_maintain/concerns/base_database.rb
 - lib/foreman_maintain/concerns/finders.rb
 - lib/foreman_maintain/concerns/hammer.rb
 - lib/foreman_maintain/concerns/logger.rb
@@ -200,6 +203,7 @@ files:
 - lib/foreman_maintain/utils/disk/io/block_device.rb
 - lib/foreman_maintain/utils/disk/io/file_system.rb
 - lib/foreman_maintain/utils/disk/nil_device.rb
+- lib/foreman_maintain/utils/disk/stats.rb
 - lib/foreman_maintain/utils/hammer.rb
 - lib/foreman_maintain/version.rb
 - lib/foreman_maintain/yaml_storage.rb

data/definitions/checks/disk_speed_minimal.rb

@@ -1,57 +0,0 @@
-class Checks::DiskSpeedMinimal < ForemanMaintain::Check
-  metadata do
-    label :disk_io
-    description 'Check for recommended disk speed of pulp, mongodb, pgsql dir.'
-    tags :pre_upgrade
-
-    preparation_steps { Procedures::Packages::Install.new(:packages => %w[hdparm fio]) }
-
-    confine do
-      feature(:katello)
-    end
-  end
-
-  EXPECTED_IO = 80
-  DEFAULT_UNIT = 'MB/sec'.freeze
-
-  def run
-    with_spinner(description) do |spinner|
-      io_obj, success = compute_disk_speed(spinner)
-      spinner.update('Finished')
-
-      assert(success, io_obj.slow_disk_error_msg)
-    end
-  end
-
-  def check_only_single_device?
-    feature(:katello).data_dirs.map do |dir|
-      ForemanMaintain::Utils::Disk::Device.new(dir).name
-    end.uniq.length <= 1
-  end
-
-  def dirs_to_check
-    return feature(:katello).data_dirs.first(1) if check_only_single_device?
-    feature(:katello).data_dirs
-  end
-
-  private
-
-  def compute_disk_speed(spinner)
-    success = true
-    io_obj = ForemanMaintain::Utils::Disk::NilDevice.new
-
-    dirs_to_check.each do |dir|
-      io_obj = ForemanMaintain::Utils::Disk::Device.new(dir)
-
-      spinner.update("[Speed check In-Progress] device: #{io_obj.name}")
-
-      next if io_obj.read_speed >= EXPECTED_IO
-
-      success = false
-      logger.info "Slow disk detected #{dir}: #{io_obj.read_speed} #{io_obj.unit}."
-      break
-    end
-
-    [io_obj, success]
-  end
-end