foreman_maintain 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6fabde56ed1a12feb739b6776178693bc779dae8ae3d71ba11b5612c09cd3837
-  data.tar.gz: 24c1ea2e3a56c4a72e444808853d61e0abf56134e071b4d89c4ee501bb6fbedf
+  metadata.gz: 467793fe18d5403460cbb877a9b42e8b6c9f82aaf3e876988c444baf0d472e17
+  data.tar.gz: f963b22148f4e08f0daee4a2af1fd479bc655aff1ec6b14c41fe249b51d4e188
 SHA512:
-  metadata.gz: 13fa993d11f4b56cc24d538fd7b9d1b86e39baaad5971266ab8de8a07151d875c6066481b91edadfc5beabd041f4e09b07b298b42ebdc45a1c7c5852d7ac0268
-  data.tar.gz: 00b68e16dca36bf7c71634a98aa7e4d78846aa5ca5b4cfcbc183dd3b492522c53c02793a215da18048f480a002487e95fe6d9965e5b6c1d382a889fb4dd46598
+  metadata.gz: cfce108a55c8f37f82da2b07081a3362f47267a95926e105c74dfc6602eb48ee424603026c9aec32d26e263a1e84deab68a7008953bcfece418029533d14376c
+  data.tar.gz: 8e8f0b94739d7e03f1ca3e8f197013780c030ab46a9d5c6f81c4ee7d2169d37c5d9a21fdd779544c947abb87b8346f67b2b9de7ebe2707c94b6afc4ec9e7298b

data/definitions/features/candlepin_database.rb

@@ -22,26 +22,12 @@ class Features::CandlepinDatabase < ForemanMaintain::Feature
     @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
-    return [true, nil] if main_cmd.empty?
-    main_cmd += format_shell_args(
-      '-u' => configuration['username'], '-p' => configuration[%(password)]
-    )
-    main_cmd += format_shell_args(extend_with_db_options)
-    execute_with_status(main_cmd, :hidden_patterns => [configuration['password']])
-  end
-
   def env_content_ids_with_null_content
     sql = <<-SQL
       SELECT ec.id
@@ -87,12 +73,4 @@ class Features::CandlepinDatabase < ForemanMaintain::Feature
     output = /#{key_name}=([^&]*)?/.match(query_string)
     output[1] if output
   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/instance.rb

@@ -28,6 +28,14 @@ class Features::Instance < ForemanMaintain::Feature
     end
   end
 
+  def project_support_entity
+    if downstream
+      'Red Hat Technical Support'
+    else
+      'Foreman Community Support'
+    end
+  end
+
   def database_remote?(feature)
     !!feature(feature) && !feature(feature).local?
   end

data/definitions/scenarios/restore.rb

@@ -45,7 +45,9 @@ module ForemanMaintain::Scenarios
         add_step(Procedures::Service::Stop.new(:only => ['postgresql']))
       end
 
-      if feature(:instance).postgresql_local? && !backup.online_backup?
+      if feature(:instance).postgresql_local? &&
+         !backup.online_backup? &&
+         backup.different_source_os?
         add_step_with_context(Procedures::Restore::ReindexDatabases)
       end
 

data/lib/foreman_maintain/reporter/cli_reporter.rb

@@ -330,20 +330,20 @@ module ForemanMaintain
           whitelist_labels = steps_to_whitelist.map(&:label_dashed).join(',')
           unless whitelist_labels.empty?
             recommend << if scenario.detector.feature(:instance).downstream
-                           format(<<-MESSAGE.strip_heredoc, whitelist_labels)
+                           format(<<-MESSAGE.strip_heredoc)
               Resolve the failed steps and rerun the command.
 
               If the situation persists and, you are unclear what to do next,
-              contact Red Hat Technical Support.
+              contact #{scenario.detector.feature(:instance).project_support_entity}.
 
               In case the failures are false positives, use
-              --whitelist="%s"
+              --whitelist="#{whitelist_labels}"
                            MESSAGE
                          else
-                           format(<<-MESSAGE.strip_heredoc, whitelist_labels)
+                           format(<<-MESSAGE.strip_heredoc)
               Resolve the failed steps and rerun the command.
               In case the failures are false positives, use
-              --whitelist="%s"
+              --whitelist="#{whitelist_labels}"
                            MESSAGE
                          end
           end

data/lib/foreman_maintain/utils/backup.rb

@@ -265,6 +265,14 @@ module ForemanMaintain
       def with_qpidd?
         installed_rpms.any? { |rpm| rpm.start_with?('qpid-cpp-server-') }
       end
+
+      def source_os_version
+        metadata.fetch('os_version', 'unknown')
+      end
+
+      def different_source_os?
+        source_os_version != "#{os_name} #{os_version}"
+      end
     end
   end
 end

data/lib/foreman_maintain/version.rb

@@ -1,3 +1,3 @@
 module ForemanMaintain
-  VERSION = '1.3.3'.freeze
+  VERSION = '1.3.4'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Ivan Nečas
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-12 00:00:00.000000000 Z
+date: 2023-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: clamp
@@ -160,7 +160,6 @@ files:
 - definitions/checks/backup/certs_tar_exist.rb
 - definitions/checks/backup/directory_ready.rb
 - definitions/checks/candlepin/db_up.rb
-- definitions/checks/candlepin/validate_db.rb
 - definitions/checks/check_for_newer_packages.rb
 - definitions/checks/check_hotfix_installed.rb
 - definitions/checks/check_tmout.rb

data/definitions/checks/candlepin/validate_db.rb

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