foreman_maintain 1.3.5 → 1.4.1

data/definitions/scenarios/{upgrade_to_satellite_6_14.rb → upgrade_to_satellite_6_15_z.rb}

@@ -1,25 +1,25 @@
-module Scenarios::Satellite_6_14
+module Scenarios::Satellite_6_15_z
   class Abstract < ForemanMaintain::Scenario
     def self.upgrade_metadata(&block)
       metadata do
         tags :upgrade_scenario
         confine do
           feature(:satellite) &&
-            (feature(:satellite).current_minor_version == '6.13' || \
-            ForemanMaintain.upgrade_in_progress == '6.14')
+            (feature(:satellite).current_minor_version == '6.15' || \
+            ForemanMaintain.upgrade_in_progress == '6.15.z')
         end
         instance_eval(&block)
       end
     end
 
     def target_version
-      '6.14'
+      '6.15.z'
     end
   end
 
   class PreUpgradeCheck < Abstract
     upgrade_metadata do
-      description 'Checks before upgrading to Satellite 6.14'
+      description 'Checks before upgrading to Satellite 6.15.z'
       tags :pre_upgrade_checks
       run_strategy :fail_slow
     end
@@ -27,14 +27,13 @@ module Scenarios::Satellite_6_14
     def compose
       add_steps(find_checks(:default))
       add_steps(find_checks(:pre_upgrade))
-      add_step(Checks::Foreman::CheckpointSegments)
-      add_step(Checks::Repositories::Validate.new(:version => '6.14'))
+      add_step(Checks::Repositories::Validate.new(:version => '6.15'))
     end
   end
 
   class PreMigrations < Abstract
     upgrade_metadata do
-      description 'Procedures before migrating to Satellite 6.14'
+      description 'Procedures before migrating to Satellite 6.15.z'
       tags :pre_migrations
     end
 
@@ -45,9 +44,8 @@ module Scenarios::Satellite_6_14
 
   class Migrations < Abstract
     upgrade_metadata do
-      description 'Migration scripts to Satellite 6.14'
+      description 'Migration scripts to Satellite 6.15.z'
       tags :migrations
-      run_strategy :fail_fast
     end
 
     def set_context_mapping
@@ -55,11 +53,11 @@ module Scenarios::Satellite_6_14
     end
 
     def compose
-      add_step(Procedures::Repositories::Setup.new(:version => '6.14'))
+      add_step(Procedures::Repositories::Setup.new(:version => '6.15'))
       modules_to_enable = ["satellite:#{el_short_name}"]
       add_step(Procedures::Packages::EnableModules.new(:module_names => modules_to_enable))
       add_step(Procedures::Packages::Update.new(:assumeyes => true,
-        :yum_options => ['--downloadonly']))
+        :dnf_options => ['--downloadonly']))
       add_step(Procedures::Service::Stop.new)
       add_step(Procedures::Packages::Update.new(:assumeyes => true))
       add_step_with_context(Procedures::Installer::Upgrade)
@@ -69,7 +67,7 @@ module Scenarios::Satellite_6_14
 
   class PostMigrations < Abstract
     upgrade_metadata do
-      description 'Procedures after migrating to Satellite 6.14'
+      description 'Procedures after migrating to Satellite 6.15.z'
       tags :post_migrations
     end
 
@@ -82,7 +80,7 @@ module Scenarios::Satellite_6_14
 
   class PostUpgradeChecks < Abstract
     upgrade_metadata do
-      description 'Checks after upgrading to Satellite 6.14'
+      description 'Checks after upgrading to Satellite 6.15.z'
       tags :post_upgrade_checks
       run_strategy :fail_slow
     end

data/extras/foreman_protector/foreman-protector.conf

@@ -1,3 +1,3 @@
 [main]
 enabled = 0
-whitelist = /etc/yum/pluginconf.d/foreman-protector.whitelist
+whitelist = /etc/dnf/plugins/foreman-protector.whitelist

data/lib/foreman_maintain/concerns/base_database.rb

@@ -75,11 +75,6 @@ module ForemanMaintain
           :hidden_patterns => [config['password']])
       end
 
-      def backup_file_path(config = configuration)
-        dump_file_name = "#{config['database']}_#{Time.now.strftime('%Y-%m-%d_%H-%M-%S')}.dump"
-        "#{backup_dir}/#{dump_file_name}.bz2"
-      end
-
       def dump_db(file, config = configuration)
         execute!(dump_command(config) + " > #{file}", :hidden_patterns => [config['password']])
       end
@@ -115,47 +110,11 @@ module ForemanMaintain
         end
       end
 
-      # TODO: refactor to use dump_db
-      def backup_db_command(file_path, config = configuration)
-        pg_dump_cmd = "pg_dump -Fc #{config['database']}"
-        "runuser - postgres -c '#{pg_dump_cmd}' | bzip2 -9 > #{file_path}"
-      end
-
       # TODO: remove the backup file path tools from here. Lib Utils::Backup?
       def backup_dir
         @backup_dir ||= File.expand_path(ForemanMaintain.config.db_backup_dir)
       end
 
-      def perform_backup(config = configuration)
-        file_path = backup_file_path(config)
-        backup_cmd = backup_db_command(file_path, config)
-        execute!(backup_cmd, :hidden_patterns => [config['password']])
-        puts "\n Note: Database backup file path - #{file_path}"
-        puts "\n In case of any exception, use above dump file to restore DB."
-      end
-
-      def table_exist?(table_name)
-        sql = <<-SQL
-          SELECT EXISTS ( SELECT *
-          FROM information_schema.tables WHERE table_name =  '#{table_name}' )
-        SQL
-        result = query(sql)
-        return false if result.nil? || (result && result.empty?)
-
-        result.first['exists'].eql?('t')
-      end
-
-      def delete_records_by_ids(tbl_name, rec_ids)
-        quotize_rec_ids = rec_ids.map { |el| "'#{el}'" }.join(',')
-        unless quotize_rec_ids.empty?
-          psql(<<-SQL)
-            BEGIN;
-             DELETE FROM #{tbl_name} WHERE id IN (#{quotize_rec_ids});
-            COMMIT;
-          SQL
-        end
-      end
-
       def find_base_directory(directory)
         find_dir_containing_file(directory, 'postgresql.conf')
       end

data/lib/foreman_maintain/concerns/system_helpers.rb

@@ -100,13 +100,13 @@ module ForemanMaintain
       end
 
       def packages_action(action, packages, options = {})
-        options.validate_options!(:assumeyes, :yum_options)
+        options.validate_options!(:assumeyes, :dnf_options)
         case action
         when :install
           package_manager.install(packages, :assumeyes => options[:assumeyes])
         when :update
           package_manager.update(packages, :assumeyes => options[:assumeyes],
-                                           :yum_options => options[:yum_options])
+                                           :dnf_options => options[:dnf_options])
         when :remove
           package_manager.remove(packages, :assumeyes => options[:assumeyes])
         else

data/lib/foreman_maintain/config.rb

@@ -64,7 +64,11 @@ module ForemanMaintain
     end
 
     def config_file_path
-      File.exist?(CONFIG_FILE) ? CONFIG_FILE : File.join(source_path, 'config/foreman_maintain.yml')
+      if defined?(CONFIG_FILE) && File.exist?(CONFIG_FILE)
+        CONFIG_FILE
+      else
+        File.join(source_path, 'config/foreman_maintain.yml')
+      end
     end
 
     def source_path

data/lib/foreman_maintain/package_manager/dnf.rb

@@ -1,14 +1,100 @@
 module ForemanMaintain::PackageManager
-  class Dnf < Yum
-    def clean_cache(assumeyes: false)
-      dnf_action('clean', 'all', :assumeyes => assumeyes)
-      super
+  class Dnf < Base
+    PROTECTOR_CONFIG_FILE = '/etc/dnf/plugins/foreman-protector.conf'.freeze
+    PROTECTOR_WHITELIST_FILE = '/etc/dnf/plugins/foreman-protector.whitelist'.freeze
+
+    def self.parse_envra(envra)
+      # envra format: 0:foreman-1.20.1.10-1.el7sat.noarch
+      parsed = envra.match(/\d*:?(?<name>.*)-[^-]+-[^-]+\.[^.]+/)
+      parsed ? Hash[parsed.names.map(&:to_sym).zip(parsed.captures)].merge(:envra => envra) : nil
+    end
+
+    def lock_versions
+      enable_protector
+    end
+
+    def unlock_versions
+      disable_protector
+    end
+
+    def versions_locked?
+      !!(protector_config =~ /^\s*enabled\s*=\s*1/) &&
+        protector_whitelist_file_nonzero?
+    end
+
+    def protector_whitelist_file_nonzero?
+      File.exist?(PROTECTOR_WHITELIST_FILE) &&
+        !File.zero?(PROTECTOR_WHITELIST_FILE)
     end
 
     def version_locking_supported?
       true
     end
 
+    def installed?(packages)
+      packages_list = [packages].flatten(1).map { |pkg| "'#{pkg}'" }.join(' ')
+      sys.execute?(%(rpm -q #{packages_list}))
+    end
+
+    def find_installed_package(name, queryformat = '')
+      rpm_cmd = "rpm -q '#{name}'"
+      unless queryformat.empty?
+        rpm_cmd += " --qf '#{queryformat}'"
+      end
+      status, result = sys.execute_with_status(rpm_cmd, interactive: false)
+      if status == 0
+        result
+      end
+    end
+
+    def install(packages, assumeyes: false)
+      dnf_action('install', packages, assumeyes: assumeyes)
+    end
+
+    def reinstall(packages, assumeyes: false)
+      dnf_action('reinstall', packages, assumeyes: assumeyes)
+    end
+
+    def remove(packages, assumeyes: false)
+      dnf_action('remove', packages, assumeyes: assumeyes)
+    end
+
+    def update(packages = [], assumeyes: false, dnf_options: [])
+      dnf_action('update', packages, assumeyes: assumeyes, dnf_options: dnf_options)
+    end
+
+    def check_update(packages: nil, with_status: false)
+      dnf_action(
+        'check-update',
+        packages,
+        assumeyes: true,
+        valid_exit_statuses: [0, 100],
+        with_status: with_status
+      )
+    end
+
+    def update_available?(package)
+      cmd_output = dnf_action('check-update -q', package, with_status: true, assumeyes: false)
+      cmd_output[0] == 100
+    end
+
+    def files_not_owned_by_package(directory)
+      find_cmd = "find #{directory} -exec /bin/sh -c 'rpm -qf {} &> /dev/null || echo {}' \\;"
+      sys.execute(find_cmd).split("\n")
+    end
+
+    def list_installed_packages(queryformat = '%{NAME}-%{VERSION}-%{RELEASE}.%{ARCH}\n')
+      # The queryformat should only include valid tag(s) as per `rpm --querytags` list.
+      # If any special formatting is required with querytag then it should be provided with tag i.e,
+      # "--%{VENDOR}"
+      # The queryformat string must end with '\n'
+      sys.execute!("rpm -qa --qf '#{queryformat}'").split("\n")
+    end
+
+    def clean_cache(assumeyes: false)
+      dnf_action('clean', 'all', :assumeyes => assumeyes)
+    end
+
     def module_enabled?(name)
       _status, result = info(name)
       result.match?(/Stream.+\[e\].+/)
@@ -29,18 +115,59 @@ module ForemanMaintain::PackageManager
 
     private
 
-    def dnf_action(action, packages, with_status: false, assumeyes: false)
+    # rubocop:disable Metrics/LineLength, Metrics/ParameterLists
+    def dnf_action(action, packages, with_status: false, assumeyes: false, dnf_options: [], valid_exit_statuses: [0])
       packages = [packages].flatten(1)
-      yum_options = []
-      yum_options << '-y' if assumeyes
+
+      dnf_options << '-y' if assumeyes
+      dnf_options << '--disableplugin=foreman-protector'
+
+      command = ['dnf', dnf_options.join(' '), action]
+
+      command.push(packages.join(' ')) unless packages.empty?
+      command = command.join(' ')
+
       if with_status
-        sys.execute_with_status("dnf #{yum_options.join(' ')} #{action} #{packages.join(' ')}",
-          :interactive => !assumeyes)
+        sys.execute_with_status(
+          command,
+          :interactive => !assumeyes
+        )
       else
-        sys.execute!("dnf #{yum_options.join(' ')} #{action} #{packages.join(' ')}",
-          :interactive => !assumeyes)
+        sys.execute!(
+          command,
+          :interactive => !assumeyes,
+          :valid_exit_statuses => valid_exit_statuses
+        )
+      end
+    end
+    # rubocop:enable Metrics/LineLength, Metrics/ParameterLists
 
+    def protector_config
+      File.exist?(protector_config_file) ? File.read(protector_config_file) : ''
+    end
+
+    def protector_config_file
+      PROTECTOR_CONFIG_FILE
+    end
+
+    def enable_protector
+      setup_protector(true)
+    end
+
+    def disable_protector
+      setup_protector(false)
+    end
+
+    def setup_protector(enabled)
+      config = protector_config
+      config += "\n" unless config[-1] == "\n"
+      enabled_re = /^\s*enabled\s*=.*$/
+      if enabled_re.match(config)
+        config = config.gsub(enabled_re, "enabled = #{enabled ? '1' : '0'}")
+      else
+        config += "enabled = #{enabled ? '1' : '0'}\n"
       end
+      File.open(protector_config_file, 'w') { |file| file.puts config }
     end
   end
 end

data/lib/foreman_maintain/package_manager.rb

@@ -1,5 +1,4 @@
 require 'foreman_maintain/package_manager/base'
-require 'foreman_maintain/package_manager/yum'
 require 'foreman_maintain/package_manager/dnf'
 require 'foreman_maintain/package_manager/apt'
 

data/lib/foreman_maintain/repository_manager/el.rb

@@ -72,8 +72,8 @@ module ForemanMaintain::RepositoryManager
                         entry.split(':', 2).last.strip
                       end]
 
-      # The EL7 yum repolist output includes extra info in the output,
-      # as example
+      # repolist output includes extra info in the output
+      #
       # rhel-7-server-rpms/7Server/x86_64
       # rhel-server-rhscl-7-rpms/7Server/x86_64
       # This trims anything after first '/' to get correct repo label

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '1.3.5'.freeze
+  VERSION = '1.4.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.3.5
+  version: 1.4.1
 platform: ruby
 authors:
 - Ivan Nečas
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-16 00:00:00.000000000 Z
+date: 2023-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clamp
@@ -158,7 +158,6 @@ files:
 - config/foreman_maintain.yml.packaging
 - config/hammer.yml.example
 - definitions/checks/backup/certs_tar_exist.rb
-- definitions/checks/backup/directory_ready.rb
 - definitions/checks/candlepin/db_up.rb
 - definitions/checks/check_for_newer_packages.rb
 - definitions/checks/check_hotfix_installed.rb
@@ -168,15 +167,12 @@ files:
 - definitions/checks/disk/available_space_postgresql12.rb
 - definitions/checks/disk/performance.rb
 - definitions/checks/env_proxy.rb
-- definitions/checks/foreman/check_checkpoint_segments.rb
 - 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_puppet_capsules.rb
 - definitions/checks/foreman/check_tuning_requirements.rb
 - definitions/checks/foreman/db_up.rb
 - definitions/checks/foreman/facts_names.rb
-- definitions/checks/foreman/puppet_class_duplicates.rb
 - definitions/checks/foreman/validate_external_db_version.rb
 - definitions/checks/foreman_openscap/invalid_report_associations.rb
 - definitions/checks/foreman_proxy/check_tftp_storage.rb
@@ -188,8 +184,7 @@ files:
 - definitions/checks/foreman_tasks/not_running.rb
 - definitions/checks/maintenance_mode/check_consistency.rb
 - definitions/checks/non_rh_packages.rb
-- definitions/checks/original_assets.rb
-- definitions/checks/package_manager/yum/validate_yum_config.rb
+- definitions/checks/package_manager/dnf/validate_dnf_config.rb
 - definitions/checks/pulpcore/db_up.rb
 - definitions/checks/puppet/verify_no_empty_cacert_requests.rb
 - definitions/checks/repositories/check_non_rh_repository.rb
@@ -259,11 +254,9 @@ files:
 - definitions/procedures/crond/start.rb
 - definitions/procedures/crond/stop.rb
 - definitions/procedures/files/remove.rb
-- definitions/procedures/foreman/apipie_cache.rb
 - definitions/procedures/foreman/fix_corrupted_roles.rb
 - definitions/procedures/foreman/remove_duplicate_obsolete_roles.rb
 - definitions/procedures/foreman/remove_duplicate_permissions.rb
-- definitions/procedures/foreman_docker/remove_foreman_docker.rb
 - definitions/procedures/foreman_maintain_features.rb
 - definitions/procedures/foreman_openscap/invalid_report_associations.rb
 - definitions/procedures/foreman_proxy/features.rb
@@ -332,17 +325,16 @@ files:
 - definitions/scenarios/restore.rb
 - definitions/scenarios/self_upgrade.rb
 - definitions/scenarios/services.rb
-- definitions/scenarios/upgrade_to_capsule_6_14.rb
-- definitions/scenarios/upgrade_to_capsule_6_14_z.rb
+- definitions/scenarios/upgrade_to_capsule_6_15.rb
+- definitions/scenarios/upgrade_to_capsule_6_15_z.rb
 - definitions/scenarios/upgrade_to_foreman_nightly.rb
 - definitions/scenarios/upgrade_to_katello_nightly.rb
-- definitions/scenarios/upgrade_to_satellite_6_14.rb
-- definitions/scenarios/upgrade_to_satellite_6_14_z.rb
+- definitions/scenarios/upgrade_to_satellite_6_15.rb
+- definitions/scenarios/upgrade_to_satellite_6_15_z.rb
 - extras/foreman-maintain.sh
 - extras/foreman_protector/dnf/foreman-protector.py
 - extras/foreman_protector/foreman-protector.conf
 - extras/foreman_protector/foreman-protector.whitelist
-- extras/foreman_protector/yum/foreman-protector.py
 - lib/foreman_maintain.rb
 - lib/foreman_maintain/check.rb
 - lib/foreman_maintain/cli.rb
@@ -397,7 +389,6 @@ files:
 - lib/foreman_maintain/package_manager/apt.rb
 - lib/foreman_maintain/package_manager/base.rb
 - lib/foreman_maintain/package_manager/dnf.rb
-- lib/foreman_maintain/package_manager/yum.rb
 - lib/foreman_maintain/param.rb
 - lib/foreman_maintain/procedure.rb
 - lib/foreman_maintain/reporter.rb
@@ -433,7 +424,7 @@ homepage: https://github.com/theforeman/foreman_maintain
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -451,8 +442,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: Foreman maintenance tool belt
 test_files: []

data/definitions/checks/backup/directory_ready.rb

@@ -1,23 +0,0 @@
-module Checks::Backup
-  class DirectoryReady < ForemanMaintain::Check
-    metadata do
-      description 'Check if the directory exists and is writable'
-      tags :backup
-      manual_detection
-      param :backup_dir, 'Directory where to backup to', :required => true
-      param :preserve_dir, 'Directory where to backup to', :flag => true, :default => false
-      param :postgres_access, 'Whether the postgres user needs access', :flag => true,
-        :default => false
-    end
-
-    def run
-      assert(File.directory?(@backup_dir), "Backup directory (#{@backup_dir}) does not exist.")
-      if feature(:instance).postgresql_local? && @postgres_access
-        result = system("runuser - postgres -c 'test -w #{@backup_dir}'")
-        assert(result, "Postgres user needs write access to the backup directory \n" \
-          "Please allow the postgres user write access to #{@backup_dir}" \
-          ' or choose another directory.')
-      end
-    end
-  end
-end

data/definitions/checks/foreman/check_checkpoint_segments.rb

@@ -1,59 +0,0 @@
-module Checks
-  module Foreman
-    class CheckpointSegments < ForemanMaintain::Check
-      metadata do
-        label :check_postgresql_checkpoint_segments
-        description 'Check if checkpoint_segments configuration exists on the system'
-        confine do
-          feature(:foreman) && feature(:installer) &&
-            File.exist?(feature(:installer).custom_hiera_file)
-        end
-      end
-
-      def run
-        failure_message = check_custom_hiera
-        fail! failure_message if failure_message
-      end
-
-      # rubocop:disable Metrics/MethodLength
-      def check_custom_hiera
-        hiera_file = feature(:installer).custom_hiera_file
-        begin
-          config = YAML.load_file(hiera_file)
-          if config.is_a?(Hash) && config.key?('postgresql::server::config_entries')
-            if config['postgresql::server::config_entries'].nil?
-              return <<-MESSAGE.strip_heredoc
-              ERROR: 'postgresql::server::config_entries' cannot be null.
-              Please remove it from following file and re-run the command.
-              - #{hiera_file}
-              MESSAGE
-            elsif config['postgresql::server::config_entries'].key?('checkpoint_segments')
-              message = <<-MESSAGE.strip_heredoc
-              ERROR: Tuning option 'checkpoint_segments' found.
-              This option is no longer valid for PostgreSQL 9.5 or newer.
-              Please remove it from following file and re-run the command.
-              - #{hiera_file}
-              MESSAGE
-              if feature(:katello)
-                message += <<-MESSAGE.strip_heredoc
-                The presence of checkpoint_segments in #{hiera_file} indicates manual tuning.
-                Manual tuning can override values provided by the --tuning parameter.
-                Review #{hiera_file} for values that are already provided by the built in tuning profiles.
-                Built in tuning profiles also provide a supported upgrade path.
-                MESSAGE
-              end
-              return message
-            end
-          elsif config.is_a?(String)
-            fail! "Error: File #{hiera_file} is not a yaml file."
-            exit 1
-          end
-        rescue Psych::SyntaxError
-          fail! "Found syntax error in file: #{hiera_file}"
-          exit 1
-        end
-      end
-      # rubocop:enable Metrics/MethodLength
-    end
-  end
-end

data/definitions/checks/foreman/check_duplicate_roles.rb

@@ -1,42 +0,0 @@
-module Checks
-  module Foreman
-    class CheckDuplicateRoles < ForemanMaintain::Check
-      metadata do
-        label :duplicate_roles
-        for_feature :foreman_database
-        description 'Check for duplicate roles from DB'
-        tags :pre_upgrade
-        confine do
-          check_max_version('foreman', '1.20')
-        end
-      end
-
-      def run
-        duplicate_roles = find_duplicate_roles
-        roles_names = duplicate_roles.map { |r| r['name'] }.uniq
-        assert(
-          duplicate_roles.empty?,
-          "Duplicate entries found for role(s) - #{roles_names.join(', ')} in your DB",
-          :next_steps => [
-            Procedures::Foreman::RemoveDuplicateObsoleteRoles.new,
-            Procedures::KnowledgeBaseArticle.new(
-              :doc => 'fix_db_migrate_failure_on_duplicate_roles'
-            ),
-          ]
-        )
-      end
-
-      def find_duplicate_roles
-        feature(:foreman_database).query(self.class.query_to_get_duplicate_roles)
-      end
-
-      def self.query_to_get_duplicate_roles
-        <<-SQL
-          SELECT r.id, r.name FROM roles r JOIN (
-            SELECT name, COUNT(*) FROM roles GROUP BY name HAVING count(*) > 1
-          ) dr ON r.name = dr.name ORDER BY r.name
-        SQL
-      end
-    end
-  end
-end

data/definitions/checks/foreman/puppet_class_duplicates.rb

@@ -1,48 +0,0 @@
-module Checks
-  module Foreman
-    class PuppetClassDuplicates < ForemanMaintain::Check
-      metadata do
-        label :puppet_class_duplicates
-        for_feature :foreman_database
-        description 'Check for duplicate Puppet class records'
-        tags :pre_upgrade
-        confine do
-          check_max_version('foreman', '1.20')
-        end
-      end
-
-      def run
-        duplicate_names = find_duplicate_names
-        assert(duplicate_names.empty?, duplicate_msg(duplicate_names))
-      end
-
-      private
-
-      def duplicate_msg(duplicate_names)
-        msg = "There are #{duplicate_names.count} Puppet classes with duplicities:\n"
-        classes_list = duplicate_names.reduce('') do |memo, hash|
-          memo.tap { |acc| acc << "#{hash['name']} - #{hash['name_count']}\n" }
-        end
-        help_msg = 'Please head over to Configure -> Classes'
-        help_msg << " and make sure there is only 1 Puppet class record for each name.\n"
-        [msg, classes_list, help_msg].join('')
-      end
-
-      def find_duplicate_names
-        feature(:foreman_database).query(duplicate_names_query)
-      end
-
-      def duplicate_names_query
-        <<-SQL
-          SELECT name, name_count
-          FROM (
-            SELECT name, count(name) AS name_count
-            FROM puppetclasses
-            GROUP BY name
-          ) AS puppetclass_counts
-          WHERE name_count > 1
-        SQL
-      end
-    end
-  end
-end