foreman_maintain 1.12.3 → 1.13.0

data/definitions/reports/lab_features.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Reports
+  class LabFeatures < ForemanMaintain::Report
+    metadata do
+      description 'Checks if lab features are enabled'
+    end
+
+    def run
+      data_field('lab_features_enabled') do
+        lab_features_setting = sql_setting('lab_features')
+        return false if lab_features_setting.nil?
+
+        # Parse the YAML setting value and convert to boolean
+        YAML.safe_load(lab_features_setting) == true
+      end
+    end
+  end
+end

data/definitions/reports/personal_access_token.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Reports
+  class PersonalAccessToken < ForemanMaintain::Report
+    metadata do
+      description 'Report about Personal Access Token usage'
+    end
+
+    def run
+      # Total count of non-revoked personal access tokens
+      data_field('pat_counts') do
+        sql_count('personal_access_tokens WHERE revoked = false')
+      end
+
+      # Count of tokens that were used recently (updated in last 2 months)
+      data_field('pat_recently_used_count') do
+        sql_count("personal_access_tokens WHERE updated_at >= NOW() - INTERVAL '2 months'")
+      end
+
+      # Count of revoked personal access tokens
+      data_field('revoked_pats_count') do
+        sql_count('personal_access_tokens WHERE revoked = true')
+      end
+    end
+  end
+end

data/definitions/reports/selinux.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Reports
+  class Selinux < ForemanMaintain::Report
+    metadata do
+      description 'Report about SELinux enforcement status'
+    end
+
+    def run
+      data_field('selinux_enforced') { selinux_enforced? }
+    end
+
+    private
+
+    def selinux_enforced?
+      # Check if getenforce command exists and SELinux is installed
+      return false unless command_present?('getenforce')
+
+      # Execute getenforce command and check if SELinux is enforcing
+      status = execute('getenforce').strip.downcase
+      status == 'enforcing'
+    rescue StandardError
+      # If any error occurs, assume SELinux is not enforced
+      false
+    end
+  end
+end

data/definitions/reports/webhooks.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module Reports
+  class Webhooks < ForemanMaintain::Report
+    metadata do
+      description 'Report about webhook usage'
+    end
+
+    def run
+      data_field('webhooks_enabled_count') { sql_count('webhooks WHERE enabled = true') }
+      data_field('webhooks_subscribed_events') { webhooks_subscribed_events }
+      data_field('shell_hooks_count') { sql_count('webhooks WHERE proxy_authorization = true') }
+    end
+
+    private
+
+    def webhooks_subscribed_events
+      # Extract all events from webhooks table and join into comma-separated string
+      events_data = query('SELECT events FROM webhooks')
+
+      all_events = []
+      events_data.each do |row|
+        events_value = row['events']
+        next if events_value.nil? || events_value.empty?
+
+        # Parse the array-like string - handle both JSON array format and simple arrays
+        # Remove outer brackets/braces, quotes, and split by comma
+        cleaned_value = events_value.gsub(/^[\[{]|[\]}]$/, '').gsub(/["']/, '')
+        parsed_events = cleaned_value.split(',').map(&:strip)
+        all_events.concat(parsed_events)
+      end
+
+      # Remove duplicates and join with commas
+      all_events.uniq.reject(&:empty?).join(',')
+    end
+  end
+end

data/definitions/scenarios/backup.rb

@@ -18,8 +18,9 @@ module ForemanMaintain::Scenarios
 
     def compose
       check_valid_strategy
-      add_step_with_context(Checks::Backup::IncrementalParentType,
-        :online_backup => strategy == :online)
+      add_step(Checks::Foreman::DBIndex)
+      add_step(Checks::Candlepin::DBIndex)
+      add_step(Checks::Pulpcore::DBIndex)
       add_step(Checks::ForemanTasks::NotRunning.new(:wait_for_tasks => wait_for_tasks?))
       add_step(Checks::Pulpcore::NoRunningTasks.new(:wait_for_tasks => wait_for_tasks?))
       add_step_with_context(Procedures::Backup::AccessibilityConfirmation) if strategy == :offline
@@ -55,7 +56,6 @@ module ForemanMaintain::Scenarios
       context.map(:preserve_dir,
         Procedures::Backup::PrepareDirectory => :preserve_dir)
       context.map(:incremental_dir,
-        Checks::Backup::IncrementalParentType => :incremental_dir,
         Procedures::Backup::PrepareDirectory => :incremental_dir,
         Procedures::Backup::Metadata => :incremental_dir)
       context.map(:proxy_features,

data/definitions/scenarios/foreman_upgrade.rb

@@ -72,6 +72,7 @@ module Scenarios::Foreman
       add_steps(
         Procedures::MaintenanceMode::EnableMaintenanceMode,
         Procedures::Crond::Stop,
+        Procedures::Timer::Stop,
         Procedures::SyncPlans::Disable
       )
     end
@@ -113,6 +114,7 @@ module Scenarios::Foreman
         Procedures::RefreshFeatures,
         Procedures::Service::Start,
         Procedures::Crond::Start,
+        Procedures::Timer::Start,
         Procedures::SyncPlans::Enable,
         Procedures::MaintenanceMode::DisableMaintenanceMode
       )

data/definitions/scenarios/maintenance_mode.rb

@@ -10,6 +10,7 @@ module ForemanMaintain::Scenarios
     def compose
       add_step(Procedures::MaintenanceMode::EnableMaintenanceMode)
       add_step(Procedures::Crond::Stop)
+      add_step(Procedures::Timer::Stop)
       add_step(Procedures::SyncPlans::Disable)
     end
   end
@@ -25,6 +26,7 @@ module ForemanMaintain::Scenarios
     def compose
       add_step(Procedures::SyncPlans::Enable)
       add_step(Procedures::Crond::Start)
+      add_step(Procedures::Timer::Start)
       add_step(Procedures::MaintenanceMode::DisableMaintenanceMode)
     end
   end

data/definitions/scenarios/restore.rb

@@ -29,26 +29,23 @@ module ForemanMaintain::Scenarios
         Procedures::Restore::RequiredPackages,
         Procedures::Restore::Configs)
       add_step_with_context(Procedures::Crond::Stop)
+      add_step_with_context(Procedures::Timer::Stop)
       unless backup.incremental?
         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?
 
       if backup.sql_needs_dump_restore?
         add_steps_with_context(Procedures::Restore::DropDatabases)
         restore_sql_dumps(backup)
       end
 
-      if feature(:instance).postgresql_local? &&
-         !backup.online_backup? &&
-         backup.different_source_os?
-        add_step_with_context(Procedures::Restore::ReindexDatabases)
-      end
+      add_steps_with_context(Procedures::Restore::ExtractFiles) if backup.tar_backups_exist?
 
       add_step(Procedures::Installer::Run.new(:assumeyes => true))
       add_step_with_context(Procedures::Installer::UpgradeRakeTask)
       add_step_with_context(Procedures::Crond::Start)
+      add_step_with_context(Procedures::Timer::Start)
     end
     # rubocop:enable Metrics/MethodLength,Metrics/AbcSize
 
@@ -119,6 +116,7 @@ module ForemanMaintain::Scenarios
 
     def compose
       add_step_with_context(Procedures::Crond::Stop)
+      add_step_with_context(Procedures::Timer::Stop)
     end
   end
 end

data/definitions/scenarios/satellite_upgrade.rb

@@ -74,6 +74,7 @@ module Scenarios::Satellite
       add_steps(
         Procedures::MaintenanceMode::EnableMaintenanceMode,
         Procedures::Crond::Stop,
+        Procedures::Timer::Stop,
         Procedures::SyncPlans::Disable,
       )
     end
@@ -97,6 +98,7 @@ module Scenarios::Satellite
           :assumeyes => true,
           :download_only => true
         ),
+        Procedures::Iop::Update.new(:version => target_version),
         Procedures::Service::Stop,
         Procedures::Packages::Update.new(:assumeyes => true, :clean_cache => false),
       )
@@ -116,8 +118,10 @@ module Scenarios::Satellite
         Procedures::RefreshFeatures,
         Procedures::Service::Start,
         Procedures::Crond::Start,
+        Procedures::Timer::Start,
         Procedures::SyncPlans::Enable,
         Procedures::MaintenanceMode::DisableMaintenanceMode,
+        Procedures::Iop::ImagePrune,
       )
     end
   end

data/definitions/scenarios/update.rb

@@ -69,8 +69,10 @@ module Scenarios::Update
           :assumeyes => true,
           :download_only => true
         ),
+        Procedures::Iop::Update.new(:version => feature(:instance).current_major_version),
         Procedures::MaintenanceMode::EnableMaintenanceMode,
         Procedures::Crond::Stop,
+        Procedures::Timer::Stop,
         Procedures::SyncPlans::Disable
       )
     end
@@ -104,8 +106,10 @@ module Scenarios::Update
         Procedures::RefreshFeatures,
         Procedures::Service::Start,
         Procedures::Crond::Start,
+        Procedures::Timer::Start,
         Procedures::SyncPlans::Enable,
-        Procedures::MaintenanceMode::DisableMaintenanceMode
+        Procedures::MaintenanceMode::DisableMaintenanceMode,
+        Procedures::Iop::ImagePrune,
       )
     end
   end

data/lib/foreman_maintain/cli/base.rb

@@ -137,7 +137,7 @@ module ForemanMaintain
         option '--label', '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?
+          raise ArgumentError, 'no value provided' if label.nil? || label.empty?
           underscorize(label).to_sym
         end
       end
@@ -147,7 +147,7 @@ module ForemanMaintain
           '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?
+          raise ArgumentError, 'no value provided' if tags.nil? || tags.empty?
           tags.map { |tag| underscorize(tag).to_sym }
         end
       end
@@ -166,7 +166,7 @@ module ForemanMaintain
         if opts.include?('whitelist')
           option(['-w', '--whitelist'], 'whitelist',
             'Comma-separated list of labels of steps to be skipped') do |whitelist|
-            raise ArgumentError, 'value not specified' if whitelist.nil? || whitelist.empty?
+            raise ArgumentError, 'no value provided' if whitelist.nil? || whitelist.empty?
             whitelist.split(',').map(&:strip)
           end
         end

data/lib/foreman_maintain/concerns/base_database.rb

@@ -140,6 +140,40 @@ module ForemanMaintain
                 ' Make sure system has psql utility installed.'
       end
 
+      def amcheck
+        # executing the check requires superuser privileges
+        return unless local?
+
+        return unless amcheck_installed?
+
+        psqlcmd = "runuser - postgres -c 'psql --set=ON_ERROR_STOP=on #{configuration['database']}'"
+
+        amcheck_query = <<~SQL
+          SELECT bt_index_check(index => c.oid, heapallindexed => i.indisunique),
+               c.relname,
+               c.relpages
+          FROM pg_index i
+          JOIN pg_opclass op ON i.indclass[0] = op.oid
+          JOIN pg_am am ON op.opcmethod = am.oid
+          JOIN pg_class c ON i.indexrelid = c.oid
+          JOIN pg_namespace n ON c.relnamespace = n.oid
+          WHERE am.amname = 'btree' AND n.nspname = 'public'
+          -- Don't check temp tables, which may be from another session:
+          AND c.relpersistence != 't'
+          -- Function may throw an error when this is omitted:
+          AND c.relkind = 'i' AND i.indisready AND i.indisvalid
+          ORDER BY c.relpages DESC;
+        SQL
+
+        execute_with_status(psqlcmd, :stdin => amcheck_query)
+      end
+
+      def amcheck_installed?
+        sql = "select 'amcheck_installed' from pg_extension where extname='amcheck'"
+        result = query(sql)
+        !result.nil? && !result.empty?
+      end
+
       private
 
       def base_env

data/lib/foreman_maintain/concerns/systemd.rb

@@ -0,0 +1,50 @@
+module ForemanMaintain
+  module Concerns
+    module Systemd
+      def action_noun(action)
+        action_word_modified(action) + 'ing'
+      end
+
+      def action_past_tense(action)
+        action_word_modified(action) + 'ed'
+      end
+
+      private
+
+      def format_status(output, exit_code, options)
+        status = ''
+        if !options[:failing] || exit_code > 0
+          if options[:brief]
+            status += format_brief_status(exit_code)
+          elsif !(output.nil? || output.empty?)
+            status += "\n" + output
+          end
+        end
+        status
+      end
+
+      def format_brief_status(exit_code)
+        result = (exit_code == 0) ? reporter.status_label(:success) : reporter.status_label(:fail)
+        padding = reporter.max_length - reporter.last_line.to_s.length - 30
+        "#{' ' * padding} #{result}"
+      end
+
+      def allowed_action?(action)
+        %w[start stop restart status enable disable].include?(action)
+      end
+
+      def action_word_modified(action)
+        case action
+        when 'status'
+          'display'
+        when 'enable', 'disable'
+          action.chomp('e')
+        when 'stop'
+          action + 'p'
+        else
+          action
+        end
+      end
+    end
+  end
+end

data/lib/foreman_maintain/feature.rb

@@ -25,6 +25,15 @@ module ForemanMaintain
       []
     end
 
+    # Override method with list of applicable timers for feature.
+    # Services have a number for priority in order to ensure
+    # they are started and stopped in the correct order.
+    # example:
+    # [ system_service('foo', 10), system_service('bar', 20) ]
+    def timers
+      []
+    end
+
     # Override to generate additional feature instances that can't be
     # autodetected directly
     def additional_features

data/lib/foreman_maintain/utils/backup.rb

@@ -7,9 +7,8 @@ module ForemanMaintain
     class Backup
       include Concerns::SystemHelpers
 
-      attr_accessor :standard_files, :katello_online_files, :katello_offline_files,
-        :foreman_online_files, :foreman_offline_files, :fpc_offline_files,
-        :fpc_online_files
+      attr_accessor :standard_files, :katello_online_files,
+        :foreman_online_files, :fpc_online_files
 
       ONLINE_BACKUP = 'online'.freeze
       OFFLINE_BACKUP = 'offline'.freeze
@@ -19,16 +18,12 @@ module ForemanMaintain
         @backup_dir = backup_dir
         @standard_files = ['config_files.tar.gz']
         @foreman_online_files = ['foreman.dump']
-        @foreman_offline_files = ['pgsql_data.tar.gz']
         @katello_online_files = @foreman_online_files + ['candlepin.dump', 'pulpcore.dump']
-        @katello_offline_files = ['pgsql_data.tar.gz']
         @fpc_online_files = ['pulpcore.dump']
-        @fpc_offline_files = ['pgsql_data.tar.gz']
       end
 
       def file_map
         @file_map ||= {
-          :pgsql_data => map_file(@backup_dir, 'pgsql_data.tar.gz'),
           :pulp_data => map_file(@backup_dir, 'pulp_data.tar'),
           :foreman_dump => map_file(@backup_dir, 'foreman.dump'),
           :iop_advisor_dump => map_file(@backup_dir, 'iop_advisor.dump'),
@@ -72,15 +67,15 @@ module ForemanMaintain
       end
 
       def valid_fpc_backup?
-        fpc_online_backup? || fpc_standard_backup?
+        fpc_online_backup?
       end
 
       def valid_katello_backup?
-        katello_online_backup? || katello_standard_backup?
+        katello_online_backup?
       end
 
       def valid_foreman_backup?
-        foreman_standard_backup? || foreman_online_backup?
+        foreman_online_backup?
       end
 
       def check_file_existence(existence_map)
@@ -99,41 +94,22 @@ module ForemanMaintain
         true
       end
 
-      def katello_standard_backup?
-        present = [:pgsql_data]
-        absent = [:candlepin_dump, :foreman_dump, :pulpcore_dump]
-        check_file_existence(:present => present,
-          :absent => absent)
-      end
-
       def katello_online_backup?
         present = [:candlepin_dump, :foreman_dump, :pulpcore_dump]
-        absent = [:pgsql_data]
-        check_file_existence(:present => present,
-          :absent => absent)
-      end
-
-      def fpc_standard_backup?
-        present = [:pgsql_data]
-        absent = [:candlepin_dump, :foreman_dump, :pulpcore_dump]
+        absent = []
         check_file_existence(:present => present,
           :absent => absent)
       end
 
       def fpc_online_backup?
         present = [:pulpcore_dump]
-        absent = [:pgsql_data, :candlepin_dump, :foreman_dump]
+        absent = [:candlepin_dump, :foreman_dump]
         check_file_existence(:present => present, :absent => absent)
       end
 
-      def foreman_standard_backup?
-        check_file_existence(:present => [:pgsql_data],
-          :absent => [:candlepin_dump, :foreman_dump, :pulpcore_dump])
-      end
-
       def foreman_online_backup?
         check_file_existence(:present => [:foreman_dump],
-          :absent => [:candlepin_dump, :pgsql_data, :pulpcore_dump])
+          :absent => [:candlepin_dump, :pulpcore_dump])
       end
 
       def validate_hostname?
@@ -174,12 +150,7 @@ module ForemanMaintain
       end
 
       def tar_backups_exist?
-        file_map[:pulp_data][:present] ||
-          file_map[:pgsql_data][:present]
-      end
-
-      def sql_tar_files_exist?
-        file_map[:pgsql_data][:present]
+        file_map[:pulp_data][:present]
       end
 
       def sql_dump_files_exist?
@@ -189,7 +160,7 @@ module ForemanMaintain
       end
 
       def sql_needs_dump_restore?
-        !sql_tar_files_exist? && sql_dump_files_exist?
+        sql_dump_files_exist?
       end
 
       def incremental?

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '1.12.3'.freeze
+  VERSION = '1.13.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.12.3
+  version: 1.13.0
 platform: ruby
 authors:
 - Ivan Nečas
@@ -156,7 +156,7 @@ files:
 - config/foreman_maintain.yml.packaging
 - config/hammer.yml.example
 - definitions/checks/backup/certs_tar_exist.rb
-- definitions/checks/backup/incremental_parent_type.rb
+- definitions/checks/candlepin/db_index.rb
 - definitions/checks/candlepin/db_up.rb
 - definitions/checks/check_hotfix_installed.rb
 - definitions/checks/check_ipv6_disable.rb
@@ -174,6 +174,7 @@ files:
 - definitions/checks/foreman/check_external_db_evr_permissions.rb
 - definitions/checks/foreman/check_puppet_capsules.rb
 - definitions/checks/foreman/check_tuning_requirements.rb
+- definitions/checks/foreman/db_index.rb
 - definitions/checks/foreman/db_up.rb
 - definitions/checks/foreman/facts_names.rb
 - definitions/checks/foreman/validate_external_db_version.rb
@@ -193,6 +194,7 @@ files:
 - definitions/checks/maintenance_mode/check_consistency.rb
 - definitions/checks/non_rh_packages.rb
 - definitions/checks/package_manager/dnf/validate_dnf_config.rb
+- definitions/checks/pulpcore/db_index.rb
 - definitions/checks/pulpcore/db_up.rb
 - definitions/checks/pulpcore/no_running_tasks.rb
 - definitions/checks/puppet/verify_no_empty_cacert_requests.rb
@@ -245,6 +247,7 @@ files:
 - definitions/features/service.rb
 - definitions/features/sync_plans.rb
 - definitions/features/tar.rb
+- definitions/features/timer.rb
 - definitions/procedures/backup/accessibility_confirmation.rb
 - definitions/procedures/backup/clean.rb
 - definitions/procedures/backup/compress_data.rb
@@ -275,6 +278,8 @@ files:
 - definitions/procedures/hammer_setup.rb
 - definitions/procedures/installer/run.rb
 - definitions/procedures/installer/upgrade_rake_task.rb
+- definitions/procedures/iop/image_prune.rb
+- definitions/procedures/iop/update.rb
 - definitions/procedures/knowledge_base_article.rb
 - definitions/procedures/maintenance_mode/disable_maintenance_mode.rb
 - definitions/procedures/maintenance_mode/enable_maintenance_mode.rb
@@ -316,7 +321,6 @@ files:
 - definitions/procedures/restore/iop_vmaas_dump.rb
 - definitions/procedures/restore/iop_vulnerability_dump.rb
 - definitions/procedures/restore/pulpcore_dump.rb
-- definitions/procedures/restore/reindex_databases.rb
 - definitions/procedures/restore/required_packages.rb
 - definitions/procedures/service/base.rb
 - definitions/procedures/service/daemon_reload.rb
@@ -329,26 +333,34 @@ files:
 - definitions/procedures/service/stop.rb
 - definitions/procedures/sync_plans/disable.rb
 - definitions/procedures/sync_plans/enable.rb
+- definitions/procedures/timer/start.rb
+- definitions/procedures/timer/stop.rb
 - definitions/reports/advisor_on_prem_remediations.rb
 - definitions/reports/alternate_content_sources.rb
+- definitions/reports/bookmarks.rb
 - definitions/reports/compliance.rb
 - definitions/reports/content.rb
+- definitions/reports/disconnected_environment.rb
 - definitions/reports/external_auth_source.rb
 - definitions/reports/grouping.rb
 - definitions/reports/image_mode_hosts.rb
 - definitions/reports/instance.rb
 - definitions/reports/inventory.rb
 - definitions/reports/kerberos.rb
+- definitions/reports/lab_features.rb
 - definitions/reports/ldap_auth_source.rb
 - definitions/reports/networking.rb
 - definitions/reports/oidc_usage.rb
+- definitions/reports/personal_access_token.rb
 - definitions/reports/platform.rb
 - definitions/reports/provisioning.rb
 - definitions/reports/rbac.rb
 - definitions/reports/recurring_logics.rb
+- definitions/reports/selinux.rb
 - definitions/reports/template
 - definitions/reports/virt_who.rb
 - definitions/reports/vmware.rb
+- definitions/reports/webhooks.rb
 - definitions/scenarios/backup.rb
 - definitions/scenarios/foreman_upgrade.rb
 - definitions/scenarios/maintenance_mode.rb
@@ -405,6 +417,7 @@ files:
 - lib/foreman_maintain/concerns/scenario_metadata.rb
 - lib/foreman_maintain/concerns/system_helpers.rb
 - lib/foreman_maintain/concerns/system_service.rb
+- lib/foreman_maintain/concerns/systemd.rb
 - lib/foreman_maintain/concerns/upstream.rb
 - lib/foreman_maintain/concerns/versions.rb
 - lib/foreman_maintain/config.rb

data/definitions/checks/backup/incremental_parent_type.rb

@@ -1,33 +0,0 @@
-module Checks::Backup
-  class IncrementalParentType < ForemanMaintain::Check
-    metadata do
-      description 'Check if the incremental backup has the right type'
-      tags :backup
-      param :incremental_dir, 'Path to existing backup directory'
-      param :online_backup, 'Select for online backup', :flag => true, :default => false
-      manual_detection
-    end
-
-    def run
-      return unless @incremental_dir
-
-      backup = ForemanMaintain::Utils::Backup.new(@incremental_dir)
-
-      existing_type = backup.backup_type
-      new_type = if @online_backup
-                   ForemanMaintain::Utils::Backup::ONLINE_BACKUP
-                 else
-                   ForemanMaintain::Utils::Backup::OFFLINE_BACKUP
-                 end
-      msg = "The existing backup is an #{existing_type} backup, "\
-        "but an #{new_type} backup was requested."
-      assert(existing_type == new_type, msg)
-
-      unless @online_backup
-        msg = "The existing backup has PostgreSQL as a tarball, "\
-          "but the new one will have a dump."
-        assert(!backup.sql_tar_files_exist?, msg)
-      end
-    end
-  end
-end

data/definitions/procedures/restore/reindex_databases.rb

@@ -1,28 +0,0 @@
-module Procedures::Restore
-  class ReindexDatabases < ForemanMaintain::Procedure
-    include ForemanMaintain::Concerns::SystemService
-    include ForemanMaintain::Concerns::SystemHelpers
-
-    metadata do
-      description 'REINDEX databases'
-
-      confine do
-        feature(:instance).postgresql_local?
-      end
-    end
-
-    def run
-      with_spinner('Reindexing the databases') do |spinner|
-        feature(:service).handle_services(spinner, 'start', :only => ['postgresql'])
-
-        spinner.update('Reindexing the databases')
-        execute!('runuser - postgres -c "reindexdb -a"')
-        if check_min_version('python3.11-pulp-ansible', '0.20.0')
-          execute!('runuser -c '\
-                   '\'echo "ALTER COLLATION pulp_ansible_semver REFRESH VERSION;"'\
-                   '| psql pulpcore\' postgres')
-        end
-      end
-    end
-  end
-end