foreman_maintain 1.3.0 → 1.3.2

data/lib/foreman_maintain/utils/bash.rb

@@ -68,37 +68,36 @@ module ForemanMaintain
 
         def traverse_tree(dict, path)
           return [dict, []] if path.nil? || path.empty?
-          result = if dict.key?(path.first)
-                     if path.first.start_with?('-')
-                       parse_option(dict, path)
-                     else
-                       parse_subcommand(dict, path)
-                     end
-                   elsif dict[:params]
-                     # traverse params one by one
-                     parse_params(dict, path)
-                   else
-                     # not found
-                     [{}, path]
-                   end
-          result
+          if dict.key?(path.first)
+            if path.first.start_with?('-')
+              parse_option(dict, path)
+            else
+              parse_subcommand(dict, path)
+            end
+          elsif dict[:params]
+            # traverse params one by one
+            parse_params(dict, path)
+          else
+            # not found
+            [{}, path]
+          end
         end
 
         def parse_params(dict, path)
-          traverse_tree({ :params => dict[:params][1..-1] }, path[1..-1])
+          traverse_tree({ :params => dict[:params][1..] }, path[1..])
         end
 
         def parse_subcommand(dict, path)
-          traverse_tree(dict[path.first], path[1..-1])
+          traverse_tree(dict[path.first], path[1..])
         end
 
         def parse_option(dict, path)
           if dict[path.first][:type] == :flag # flag
-            traverse_tree(dict, path[1..-1])
+            traverse_tree(dict, path[1..])
           elsif path.length >= 2 # option with value
-            traverse_tree(dict, path[2..-1])
+            traverse_tree(dict, path[2..])
           else # option with value missing
-            [dict[path.first], path[1..-1]]
+            [dict[path.first], path[1..]]
           end
         end
 

data/lib/foreman_maintain/utils/command_runner.rb

@@ -21,15 +21,13 @@ module ForemanMaintain
       end
 
       def run
-        if logger
-          logger.debug(hide_strings("Running command #{@command} with stdin #{@stdin.inspect}"))
-        end
+        logger&.debug(hide_strings("Running command #{@command} with stdin #{@stdin.inspect}"))
         if @interactive
           run_interactively
         else
           run_non_interactively
         end
-        logger.debug("output of the command:\n #{hide_strings(output)}") if logger
+        logger&.debug("output of the command:\n #{hide_strings(output)}")
       end
 
       def interactive?
@@ -52,14 +50,13 @@ module ForemanMaintain
 
       def execution_error
         raise Error::ExecutionError.new(hide_strings(@command),
-                                        exit_status,
-                                        hide_strings(@stdin),
-                                        @interactive ? nil : hide_strings(@output))
+          exit_status,
+          hide_strings(@stdin),
+          @interactive ? nil : hide_strings(@output))
       end
 
       private
 
-      # rubocop:disable Metrics/MethodLength
       def run_interactively
         # use tmp files to capture output and exit status of the command when
         # running interactively
@@ -82,7 +79,6 @@ module ForemanMaintain
         log_file.close
         exit_file.close
       end
-      # rubocop:enable Metrics/MethodLength
 
       def run_non_interactively
         IO.popen(full_command, 'r+') do |f|

data/lib/foreman_maintain/utils/hash_tools.rb

@@ -1,9 +1,9 @@
 module ForemanMaintain::Utils
   module HashTools
-    def self.deep_merge!(h, other_h)
-      other_h = symbolize_hash(other_h)
+    def self.deep_merge!(hash, other_hash)
+      other_hash = symbolize_hash(other_hash)
 
-      h.merge!(other_h) do |_key, old_val, new_val|
+      hash.merge!(other_hash) do |_key, old_val, new_val|
         if old_val.is_a?(Hash) && new_val.is_a?(Hash)
           deep_merge!(old_val, new_val)
         elsif old_val.is_a?(Array) && new_val.is_a?(Array)
@@ -14,8 +14,8 @@ module ForemanMaintain::Utils
       end
     end
 
-    def self.symbolize_hash(h)
-      h.inject({}) { |sym_hash, (k, v)| sym_hash.update(k.to_sym => v) }
+    def self.symbolize_hash(hash)
+      hash.inject({}) { |sym_hash, (key, value)| sym_hash.update(key.to_sym => value) }
     end
   end
 end

data/lib/foreman_maintain/utils/response.rb

@@ -2,6 +2,7 @@ module ForemanMaintain
   module Utils
     class Response
       attr_reader :data, :message
+
       def initialize(success, message, data: {})
         @success = success
         @message = message

data/lib/foreman_maintain/utils/service/remote_db.rb

@@ -2,6 +2,7 @@ module ForemanMaintain::Utils
   module Service
     class RemoteDB < Abstract
       attr_reader :component, :db_feature
+
       def initialize(name, priority, options = {})
         super
         @db_feature = options.fetch(:db_feature)

data/lib/foreman_maintain/utils/service/systemd.rb

@@ -2,6 +2,7 @@ module ForemanMaintain::Utils
   module Service
     class Systemd < Abstract
       attr_reader :instance_parent_unit
+
       def initialize(name, priority, options = {})
         super
         @sys = SystemHelpers.new
@@ -32,6 +33,10 @@ module ForemanMaintain::Utils
         execute('stop')
       end
 
+      def restart
+        execute('restart')
+      end
+
       def enable
         execute('enable')
       end
@@ -46,8 +51,7 @@ module ForemanMaintain::Utils
 
       def exist?
         if @sys.systemd_installed?
-          systemd = service_enabled_status
-          systemd == 'enabled' || systemd == 'disabled'
+          ['enabled', 'disabled'].include?(service_enabled_status)
         else
           File.exist?("/etc/init.d/#{@name}")
         end
@@ -73,14 +77,12 @@ module ForemanMaintain::Utils
         # Enable and disable does not work well with globs since they treat them literally.
         # We are skipping the pulpcore-workers@* for these actions until they are configured in
         # a more managable way with systemd
-        # rubocop:disable Layout/IndentAssignment
         msg =
-"
+          "
 \nWARNING: Skipping #{action} for #{name} as there are a variable amount of services to manage
 and this command will not respond to glob operators. These services have been configured by
 the installer and it is recommended to keep them enabled to prevent misconfiguration.\n
 "
-        # rubocop:enable Layout/IndentAssignment
         puts msg
       end
     end

data/lib/foreman_maintain/utils/service.rb

@@ -5,7 +5,7 @@ require 'foreman_maintain/utils/service/remote_db'
 module ForemanMaintain::Utils
   def self.system_service(name, priority, options = {})
     db_feature = options.fetch(:db_feature, nil)
-    if db_feature && db_feature.respond_to?(:local?) && !db_feature.local?
+    if db_feature.respond_to?(:local?) && !db_feature.local?
       Service::RemoteDB.new(name, priority, options)
     else
       Service::Systemd.new(name, priority, options)

data/lib/foreman_maintain/utils.rb

@@ -4,6 +4,5 @@ require 'foreman_maintain/utils/disk'
 require 'foreman_maintain/utils/bash'
 require 'foreman_maintain/utils/hash_tools'
 require 'foreman_maintain/utils/curl_response'
-require 'foreman_maintain/utils/mongo_core'
 require 'foreman_maintain/utils/service'
 require 'foreman_maintain/utils/system_helpers'

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '1.3.0'.freeze
+  VERSION = '1.3.2'.freeze
 end

data/lib/foreman_maintain/yaml_storage.rb

@@ -2,6 +2,7 @@ module ForemanMaintain
   class YamlStorage
     extend Forwardable
     attr_reader :sub_key, :data
+
     def_delegators :data, :[], :[]=, :delete
 
     def initialize(sub_key, data = {})

data/lib/foreman_maintain.rb

@@ -60,7 +60,7 @@ module ForemanMaintain
       'warn' => ::Logger::WARN,
       'error' => ::Logger::ERROR,
       'fatal' => ::Logger::FATAL,
-      'unknown' => ::Logger::UNKNOWN
+      'unknown' => ::Logger::UNKNOWN,
     }.freeze
 
     def setup(options = {})
@@ -98,7 +98,7 @@ module ForemanMaintain
       $LOAD_PATH.concat(config.definitions_dirs)
       config.definitions_dirs.each do |definitions_dir|
         file_paths = File.expand_path(File.join(definitions_dir, '**', '*.rb'))
-        Dir.glob(file_paths).each { |f| require f }
+        Dir.glob(file_paths).sort.each { |f| require f }
       end
     end
 
@@ -143,7 +143,7 @@ module ForemanMaintain
       # convert size in KB to Bytes
       log_fsize = config.log_file_size.to_i * 1024
       @logger = Logger.new(config.log_filename, 10, log_fsize).tap do |logger|
-        logger.level = LOGGER_LEVEL_MAPPING[config.log_level] || Logger::DEBUG
+        logger.level = LOGGER_LEVEL_MAPPING[config.log_level] || Logger::INFO
         logger.datetime_format = '%Y-%m-%d %H:%M:%S%z '
       end
       pickup_log_messages
@@ -180,10 +180,11 @@ module ForemanMaintain
       puts "Checking for new version of #{package_name}..."
 
       enable_maintenance_module
+      package_manager = ForemanMaintain.package_manager
 
-      if ForemanMaintain.package_manager.update_available?(main_package_name)
+      if package_manager.update_available?(main_package_name)
         puts "\nUpdating #{package_name} package."
-        ForemanMaintain.package_manager.update(main_package_name, :assumeyes => true)
+        package_manager.update(main_package_name, :assumeyes => true)
         puts "\nThe #{package_name} package successfully updated."\
              "\nRe-run #{command} with required options!"
         exit 75
@@ -192,7 +193,7 @@ module ForemanMaintain
     end
 
     def enable_maintenance_module
-      return unless el? && !el7?
+      return unless el?
 
       maintenance_module = 'satellite-maintenance:el8'
       package_manager = ForemanMaintain.package_manager

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.2
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-28 00:00:00.000000000 Z
+date: 2023-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clamp
@@ -67,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: mocha
+  name: minitest-reporters
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +81,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest-stub-const
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,21 +95,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.50.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 0.50.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest-stub-const
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: theforeman-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -159,7 +173,6 @@ files:
 - definitions/checks/foreman/check_corrupted_roles.rb
 - definitions/checks/foreman/check_duplicate_permission.rb
 - definitions/checks/foreman/check_duplicate_roles.rb
-- definitions/checks/foreman/check_https_proxies.rb
 - definitions/checks/foreman/check_puppet_capsules.rb
 - definitions/checks/foreman/check_tuning_requirements.rb
 - definitions/checks/foreman/db_up.rb
@@ -175,23 +188,18 @@ files:
 - definitions/checks/foreman_tasks/not_paused.rb
 - definitions/checks/foreman_tasks/not_running.rb
 - definitions/checks/maintenance_mode/check_consistency.rb
-- definitions/checks/mongo/db_up.rb
-- definitions/checks/mongo/tools_installed.rb
 - definitions/checks/non_rh_packages.rb
 - definitions/checks/original_assets.rb
 - definitions/checks/package_manager/yum/validate_yum_config.rb
 - definitions/checks/pulpcore/db_up.rb
-- definitions/checks/pulpcore/group_ownership_check.rb
-- definitions/checks/puppet/provide_upgrade_guide.rb
 - definitions/checks/puppet/verify_no_empty_cacert_requests.rb
-- definitions/checks/puppet/warn_about_puppet_removal.rb
-- definitions/checks/remote_execution/verify_settings_file_already_exists.rb
 - definitions/checks/repositories/check_non_rh_repository.rb
 - definitions/checks/repositories/check_upstream_repository.rb
 - definitions/checks/repositories/validate.rb
 - definitions/checks/restore/validate_backup.rb
 - definitions/checks/restore/validate_hostname.rb
 - definitions/checks/restore/validate_interfaces.rb
+- definitions/checks/restore/validate_postgresql_dump_permissions.rb
 - definitions/checks/root_user.rb
 - definitions/checks/server_ping.rb
 - definitions/checks/services_up.rb
@@ -216,10 +224,8 @@ files:
 - definitions/features/iptables.rb
 - definitions/features/katello.rb
 - definitions/features/katello_install.rb
-- definitions/features/mongo.rb
 - definitions/features/mosquitto.rb
 - definitions/features/nftables.rb
-- definitions/features/pulp2.rb
 - definitions/features/pulpcore.rb
 - definitions/features/pulpcore_database.rb
 - definitions/features/puppet_server.rb
@@ -236,11 +242,9 @@ files:
 - definitions/procedures/backup/metadata.rb
 - definitions/procedures/backup/offline/candlepin_db.rb
 - definitions/procedures/backup/offline/foreman_db.rb
-- definitions/procedures/backup/offline/mongo.rb
 - definitions/procedures/backup/offline/pulpcore_db.rb
 - definitions/procedures/backup/online/candlepin_db.rb
 - definitions/procedures/backup/online/foreman_db.rb
-- definitions/procedures/backup/online/mongo.rb
 - definitions/procedures/backup/online/pulpcore_db.rb
 - definitions/procedures/backup/online/safety_confirmation.rb
 - definitions/procedures/backup/prepare_directory.rb
@@ -250,17 +254,9 @@ files:
 - definitions/procedures/backup/snapshot/mount_base.rb
 - definitions/procedures/backup/snapshot/mount_candlepin_db.rb
 - definitions/procedures/backup/snapshot/mount_foreman_db.rb
-- definitions/procedures/backup/snapshot/mount_mongo.rb
 - definitions/procedures/backup/snapshot/mount_pulp.rb
 - definitions/procedures/backup/snapshot/mount_pulpcore_db.rb
 - definitions/procedures/backup/snapshot/prepare_mount.rb
-- definitions/procedures/candlepin/delete_orphaned_records_from_env_content.rb
-- definitions/procedures/content/fix_pulpcore_artifact_permissions.rb
-- definitions/procedures/content/migration_reset.rb
-- definitions/procedures/content/migration_stats.rb
-- definitions/procedures/content/prepare.rb
-- definitions/procedures/content/prepare_abort.rb
-- definitions/procedures/content/switchover.rb
 - definitions/procedures/crond/start.rb
 - definitions/procedures/crond/stop.rb
 - definitions/procedures/files/remove.rb
@@ -295,11 +291,6 @@ files:
 - definitions/procedures/packages/unlock_versions.rb
 - definitions/procedures/packages/update.rb
 - definitions/procedures/packages/update_all_confirmation.rb
-- definitions/procedures/prep_6_10_upgrade.rb
-- definitions/procedures/pulp/cleanup_old_metadata_files.rb
-- definitions/procedures/pulp/migrate.rb
-- definitions/procedures/pulp/print_remove_instructions.rb
-- definitions/procedures/pulp/remove.rb
 - definitions/procedures/pulpcore/migrate.rb
 - definitions/procedures/pulpcore/trim_rpm_changelogs.rb
 - definitions/procedures/puppet/delete_empty_ca_cert_request_files.rb
@@ -316,14 +307,11 @@ files:
 - definitions/procedures/restore/configs.rb
 - definitions/procedures/restore/confirmation.rb
 - definitions/procedures/restore/drop_databases.rb
-- definitions/procedures/restore/ensure_mongo_engine_matches.rb
 - definitions/procedures/restore/extract_files.rb
 - definitions/procedures/restore/foreman_dump.rb
 - definitions/procedures/restore/installer_reset.rb
-- definitions/procedures/restore/mongo_dump.rb
 - definitions/procedures/restore/postgres_owner.rb
 - definitions/procedures/restore/pulpcore_dump.rb
-- definitions/procedures/restore/regenerate_queues.rb
 - definitions/procedures/restore/reindex_databases.rb
 - definitions/procedures/selinux/set_file_security.rb
 - definitions/procedures/service/base.rb
@@ -338,56 +326,18 @@ files:
 - definitions/procedures/sync_plans/disable.rb
 - definitions/procedures/sync_plans/enable.rb
 - definitions/scenarios/backup.rb
-- definitions/scenarios/content.rb
 - definitions/scenarios/maintenance_mode.rb
 - definitions/scenarios/packages.rb
-- definitions/scenarios/prep_6_10_upgrade.rb
 - definitions/scenarios/puppet.rb
 - definitions/scenarios/restore.rb
 - definitions/scenarios/self_upgrade.rb
 - definitions/scenarios/services.rb
-- definitions/scenarios/upgrade_to_capsule_6_10.rb
-- definitions/scenarios/upgrade_to_capsule_6_10_z.rb
-- definitions/scenarios/upgrade_to_capsule_6_11.rb
-- definitions/scenarios/upgrade_to_capsule_6_11_z.rb
-- definitions/scenarios/upgrade_to_capsule_6_12.rb
-- definitions/scenarios/upgrade_to_capsule_6_12_z.rb
-- definitions/scenarios/upgrade_to_capsule_6_13.rb
-- definitions/scenarios/upgrade_to_capsule_6_13_z.rb
 - definitions/scenarios/upgrade_to_capsule_6_14.rb
 - definitions/scenarios/upgrade_to_capsule_6_14_z.rb
-- definitions/scenarios/upgrade_to_capsule_6_8.rb
-- definitions/scenarios/upgrade_to_capsule_6_8_z.rb
-- definitions/scenarios/upgrade_to_capsule_6_9.rb
-- definitions/scenarios/upgrade_to_capsule_6_9_z.rb
 - definitions/scenarios/upgrade_to_foreman_nightly.rb
 - definitions/scenarios/upgrade_to_katello_nightly.rb
-- definitions/scenarios/upgrade_to_satellite_6_10.rb
-- definitions/scenarios/upgrade_to_satellite_6_10_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_11.rb
-- definitions/scenarios/upgrade_to_satellite_6_11_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_12.rb
-- definitions/scenarios/upgrade_to_satellite_6_12_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_13.rb
-- definitions/scenarios/upgrade_to_satellite_6_13_z.rb
 - definitions/scenarios/upgrade_to_satellite_6_14.rb
 - definitions/scenarios/upgrade_to_satellite_6_14_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_2.rb
-- definitions/scenarios/upgrade_to_satellite_6_2_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_3.rb
-- definitions/scenarios/upgrade_to_satellite_6_3_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_4.rb
-- definitions/scenarios/upgrade_to_satellite_6_4_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_5.rb
-- definitions/scenarios/upgrade_to_satellite_6_5_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_6.rb
-- definitions/scenarios/upgrade_to_satellite_6_6_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_7.rb
-- definitions/scenarios/upgrade_to_satellite_6_7_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_8.rb
-- definitions/scenarios/upgrade_to_satellite_6_8_z.rb
-- definitions/scenarios/upgrade_to_satellite_6_9.rb
-- definitions/scenarios/upgrade_to_satellite_6_9_z.rb
 - extras/foreman-maintain.sh
 - extras/foreman_protector/dnf/foreman-protector.py
 - extras/foreman_protector/foreman-protector.conf
@@ -405,7 +355,6 @@ files:
 - lib/foreman_maintain/cli/advanced_command.rb
 - lib/foreman_maintain/cli/backup_command.rb
 - lib/foreman_maintain/cli/base.rb
-- lib/foreman_maintain/cli/content_command.rb
 - lib/foreman_maintain/cli/health_command.rb
 - lib/foreman_maintain/cli/maintenance_mode_command.rb
 - lib/foreman_maintain/cli/packages_command.rb
@@ -472,7 +421,6 @@ files:
 - lib/foreman_maintain/utils/disk/nil_device.rb
 - lib/foreman_maintain/utils/disk/stats.rb
 - lib/foreman_maintain/utils/hash_tools.rb
-- lib/foreman_maintain/utils/mongo_core.rb
 - lib/foreman_maintain/utils/response.rb
 - lib/foreman_maintain/utils/service.rb
 - lib/foreman_maintain/utils/service/abstract.rb
@@ -493,7 +441,10 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
+  - - "<"
+    - !ruby/object:Gem::Version
+      version: '4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/definitions/checks/foreman/check_https_proxies.rb

@@ -1,34 +0,0 @@
-module Checks
-  class CheckHttpsProxies < ForemanMaintain::Check
-    metadata do
-      label :https_proxies
-      for_feature :foreman_database
-      description 'Check for HTTPS proxies from the database'
-      manual_detection
-    end
-
-    def run
-      https_proxies = find_https_proxies
-      unless https_proxies.empty?
-        https_proxy_names = https_proxies.map { |proxy| proxy['name'] }
-        question = "Syncing repositories through an 'HTTP Proxy' that uses the HTTPS\n"\
-                    "protocol is not supported directly with Satellite 6.10.\n"\
-                    "The following proxies use HTTPS: #{https_proxy_names.join(', ')}.\n"\
-                    "For a suggested solution see https://access.redhat.com/solutions/6414991\n"\
-                    'Continue upgrade?'
-        answer = ask_decision(question, actions_msg: 'y(yes), q(quit)')
-        abort! if answer != :yes
-      end
-    end
-
-    def find_https_proxies
-      feature(:foreman_database).query(self.class.query_to_get_https_proxies)
-    end
-
-    def self.query_to_get_https_proxies
-      <<-SQL
-        SELECT \"http_proxies\".* FROM \"http_proxies\" WHERE (http_proxies.url ilike 'https://%')
-      SQL
-    end
-  end
-end

data/definitions/checks/mongo/db_up.rb

@@ -1,33 +0,0 @@
-module Checks
-  module Mongo
-    class DBUp < ForemanMaintain::Check
-      metadata do
-        description 'Make sure Mongo DB is up'
-        label :mongo_db_up
-        for_feature :mongo
-      end
-
-      def run
-        status = false
-        if feature(:mongo).mongo_cmd_available?
-          with_spinner('Checking connection to the Mongo DB') do
-            status = feature(:mongo).ping
-          end
-          assert(status, 'Mongo DB is not responding. ' \
-            'It needs to be up and running to perform the following steps.',
-                 :next_steps => start_mongodb)
-        else
-          feature(:mongo).raise_mongo_client_missing_error
-        end
-      end
-
-      def start_mongodb
-        if feature(:mongo).local?
-          [Procedures::Service::Start.new(:only => feature(:mongo).services)]
-        else
-          [] # there is nothing we can do for remote db
-        end
-      end
-    end
-  end
-end

data/definitions/checks/mongo/tools_installed.rb

@@ -1,31 +0,0 @@
-module Checks::Mongo
-  class ToolsInstalled < ForemanMaintain::Check
-    metadata do
-      manual_detection
-      description 'Checks whether the tools for Mongo DB are installed'
-      for_feature :mongo
-    end
-
-    def run
-      if feature(:mongo).server_version =~ /^3\.4/
-        tools_pkg = 'rh-mongodb34-mongo-tools'
-        result = find_package(tools_pkg)
-      else
-        result = execute?('which mongodump')
-        tools_pkg = 'mongodb'
-      end
-      handle_result(result, tools_pkg)
-    end
-
-    private
-
-    def handle_result(result, tools_pkg)
-      assert(result,
-             "#{tools_pkg} was not found among installed package.\nThis package is needed to " \
-               'do various operations such as backup, restore and import with Mongo DB.',
-             :next_steps => [
-               Procedures::Packages::Install.new(:packages => [tools_pkg])
-             ])
-    end
-  end
-end

data/definitions/checks/pulpcore/group_ownership_check.rb

@@ -1,18 +0,0 @@
-module Checks
-  module Pulpcore
-    class GroupOwnershipCheck < ForemanMaintain::Check
-      metadata do
-        description 'Check the group owner of /var/lib/pulp/content directory'
-        label :group_ownership_check_of_pulp_content
-        manual_detection
-      end
-
-      def run
-        group_id = File.stat('/var/lib/pulp/content/').gid
-        if Etc.getgrgid(group_id).name != 'pulp'
-          fail! "Please run 'foreman-maintain prep-6.10-upgrade' prior to upgrading."
-        end
-      end
-    end
-  end
-end