foreman_maintain 1.12.0 → 1.12.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dfa2280e285e63ea4e7b6269dc6761598c17034fc9febfde0aca731b19a684c3
-  data.tar.gz: b83257b1cefd0ed6e6ad12205442b1817a2b2395b315b08f59a34c8b09833517
+  metadata.gz: 841269b0b17944b3a732b728e7c51ba5c11bdec27b47c45ce6b13fb61ec8be06
+  data.tar.gz: 966dd22a328585cb68ecb248341b8b566787f7efa6f590eb8dc1877203fdc895
 SHA512:
-  metadata.gz: da581d87a81c7b38ab139245323e2ad3f9958fff5094c27b00ef28ff7e519741f18bb909388c660208ce2531ddc14bb39c7873deeacc1502d11185417f4bb321
-  data.tar.gz: 143cd40662be08087e1da5f2cb29c40dbe24bbecd19ea292c6c34394b218606aeb024ddd2757d7fbbbb0edfc5d1220b92dcde0596bd011be6e8ec2cc47274a3c
+  metadata.gz: f996ba857e7ad9598f6b4e26811710387d8592c2841bb26dbf63660d44774c388312ab217d795dbd24c258767b3c276b048f13ced797927703dd6c85dbde106f
+  data.tar.gz: 72cc8e83ee3ff2fb76fdcaa226695c05b90fe70ed63bb82b7d65e8307d82cbab6b6141b51a8f5ceaac410648d41f9308744998b255a9ab2e9f8bf43b0d50d8fa

data/definitions/features/iop.rb

@@ -10,7 +10,7 @@ class Features::Iop < ForemanMaintain::Feature
 
   def config_files
     [
-      '/var/lib/kafka',
+      '/var/lib/containers/storage/volumes/iop-core-kafka-data',
       '/var/lib/vmaas',
     ]
   end
@@ -39,7 +39,6 @@ class Features::Iop < ForemanMaintain::Feature
       system_service('iop-service-vuln-listener', 20),
       system_service('iop-service-vuln-manager', 20),
       system_service('iop-service-vuln-taskomatic', 20),
-      system_service('iop-service-vuln-vmaas-sync', 20),
     ]
   end
   # rubocop:enable Metrics/MethodLength

data/definitions/features/iop_advisor_database.rb

@@ -23,13 +23,19 @@ class Features::IopAdvisorDatabase < ForemanMaintain::Feature
 
   private
 
+  # rubocop:disable Metrics/MethodLength
   def load_configuration
-    podman_command = "podman exec iop-service-advisor-api bash -c 'env |grep DB_'"
-    podman_result = execute!(podman_command, merge_stderr: false).lines.map do |l|
-      l.strip.split('=')
-    end.to_h
+    podman_command = "podman exec iop-service-advisor-backend-api bash -c 'env |grep DB_'"
+    podman_result = begin
+      execute!(podman_command, merge_stderr: false).lines.map do |l|
+        l.strip.split('=')
+      end.to_h
+    rescue ForemanMaintain::Error::ExecutionError
+      {}
+    end
 
-    db_host = if podman_result['ADVISOR_DB_HOST'].start_with?('/var/run/postgresql')
+    db_host = if podman_result['ADVISOR_DB_HOST'].nil? ||
+                 podman_result['ADVISOR_DB_HOST'].start_with?('/var/run/postgresql')
                 'localhost'
               else
                 podman_result['ADVISOR_DB_HOST']
@@ -42,4 +48,5 @@ class Features::IopAdvisorDatabase < ForemanMaintain::Feature
       'username' => podman_result['ADVISOR_DB_USER'],
     }
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/definitions/features/iop_inventory_database.rb

@@ -23,13 +23,19 @@ class Features::IopInventoryDatabase < ForemanMaintain::Feature
 
   private
 
+  # rubocop:disable Metrics/MethodLength
   def load_configuration
     podman_command = "podman exec iop-core-host-inventory bash -c 'env |grep INVENTORY_DB_'"
-    podman_result = execute!(podman_command, merge_stderr: false).lines.map do |l|
-      l.strip.split('=')
-    end.to_h
+    podman_result = begin
+      execute!(podman_command, merge_stderr: false).lines.map do |l|
+        l.strip.split('=')
+      end.to_h
+    rescue ForemanMaintain::Error::ExecutionError
+      {}
+    end
 
-    db_host = if podman_result['INVENTORY_DB_HOST'].start_with?('/var/run/postgresql')
+    db_host = if podman_result['INVENTORY_DB_HOST'].nil? ||
+                 podman_result['INVENTORY_DB_HOST'].start_with?('/var/run/postgresql')
                 'localhost'
               else
                 podman_result['INVENTORY_DB_HOST']
@@ -42,4 +48,5 @@ class Features::IopInventoryDatabase < ForemanMaintain::Feature
       'username' => podman_result['INVENTORY_DB_USER'],
     }
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/definitions/features/iop_remediations_database.rb

@@ -23,13 +23,19 @@ class Features::IopRemediationsDatabase < ForemanMaintain::Feature
 
   private
 
+  # rubocop:disable Metrics/MethodLength
   def load_configuration
     podman_command = "podman exec iop-service-remediations-api bash -c 'env |grep DB_'"
-    podman_result = execute!(podman_command, merge_stderr: false).lines.map do |l|
-      l.strip.split('=')
-    end.to_h
+    podman_result = begin
+      execute!(podman_command, merge_stderr: false).lines.map do |l|
+        l.strip.split('=')
+      end.to_h
+    rescue ForemanMaintain::Error::ExecutionError
+      {}
+    end
 
-    db_host = if podman_result['DB_HOST'].start_with?('/var/run/postgresql')
+    db_host = if podman_result['DB_HOST'].nil? ||
+                 podman_result['DB_HOST'].start_with?('/var/run/postgresql')
                 'localhost'
               else
                 podman_result['DB_HOST']
@@ -42,4 +48,5 @@ class Features::IopRemediationsDatabase < ForemanMaintain::Feature
       'username' => podman_result['DB_USERNAME'],
     }
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/definitions/features/iop_vmaas_database.rb

@@ -23,13 +23,19 @@ class Features::IopVmaasDatabase < ForemanMaintain::Feature
 
   private
 
+  # rubocop:disable Metrics/MethodLength
   def load_configuration
     podman_command = "podman exec iop-service-vmaas-reposcan bash -c 'env |grep POSTGRESQL_'"
-    podman_result = execute!(podman_command, merge_stderr: false).lines.map do |l|
-      l.strip.split('=')
-    end.to_h
+    podman_result = begin
+      execute!(podman_command, merge_stderr: false).lines.map do |l|
+        l.strip.split('=')
+      end.to_h
+    rescue ForemanMaintain::Error::ExecutionError
+      {}
+    end
 
-    db_host = if podman_result['POSTGRESQL_HOST'].start_with?('/var/run/postgresql')
+    db_host = if podman_result['POSTGRESQL_HOST'].nil? ||
+                 podman_result['POSTGRESQL_HOST'].start_with?('/var/run/postgresql')
                 'localhost'
               else
                 podman_result['POSTGRESQL_HOST']
@@ -42,4 +48,5 @@ class Features::IopVmaasDatabase < ForemanMaintain::Feature
       'username' => podman_result['POSTGRESQL_USER'],
     }
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/definitions/features/iop_vulnerability_database.rb

@@ -23,13 +23,19 @@ class Features::IopVulnerabilityDatabase < ForemanMaintain::Feature
 
   private
 
+  # rubocop:disable Metrics/MethodLength
   def load_configuration
     podman_command = "podman exec iop-service-vuln-manager bash -c 'env |grep POSTGRES_'"
-    podman_result = execute!(podman_command, merge_stderr: false).lines.map do |l|
-      l.strip.split('=')
-    end.to_h
+    podman_result = begin
+      execute!(podman_command, merge_stderr: false).lines.map do |l|
+        l.strip.split('=')
+      end.to_h
+    rescue ForemanMaintain::Error::ExecutionError
+      {}
+    end
 
-    db_host = if podman_result['POSTGRES_HOST'].start_with?('/var/run/postgresql')
+    db_host = if podman_result['POSTGRES_HOST'].nil? ||
+                 podman_result['POSTGRES_HOST'].start_with?('/var/run/postgresql')
                 'localhost'
               else
                 podman_result['POSTGRES_HOST']
@@ -42,4 +48,5 @@ class Features::IopVulnerabilityDatabase < ForemanMaintain::Feature
       'username' => podman_result['POSTGRES_USER'],
     }
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/definitions/scenarios/foreman_upgrade.rb

@@ -90,10 +90,6 @@ module Scenarios::Foreman
 
     def compose
       add_step(Procedures::Repositories::Setup.new(:version => 'nightly'))
-      if el8?
-        modules_to_switch = ['postgresql:13']
-        add_step(Procedures::Packages::SwitchModules.new(:module_names => modules_to_switch))
-      end
 
       add_step(Procedures::Packages::Update.new(
         :assumeyes => true,

data/definitions/scenarios/satellite_upgrade.rb

@@ -92,12 +92,6 @@ module Scenarios::Satellite
 
     def compose
       add_step(Procedures::Repositories::Setup.new(:version => target_version))
-      if el8?
-        modules_to_switch = ['postgresql:13']
-        add_step(Procedures::Packages::SwitchModules.new(:module_names => modules_to_switch))
-        modules_to_enable = ["#{feature(:instance).downstream.module_name}:#{el_short_name}"]
-        add_step(Procedures::Packages::EnableModules.new(:module_names => modules_to_enable))
-      end
       add_steps(
         Procedures::Packages::Update.new(
           :assumeyes => true,

data/definitions/scenarios/self_upgrade.rb

@@ -74,8 +74,6 @@ module ForemanMaintain::Scenarios
         )
       end
 
-      ForemanMaintain.enable_maintenance_module
-
       add_step(Procedures::Packages::Update.new(packages: pkgs_to_update, assumeyes: true,
         enabled_repos: req_repos_to_update_pkgs))
     end

data/lib/foreman_maintain/cli/report_command.rb

@@ -20,8 +20,9 @@ module ForemanMaintain
         end
       end
 
-      option '--output', 'FILE', 'Output the generate report into FILE'
       subcommand 'generate', 'Generates the usage reports' do
+        option '--output', 'FILE', 'Output the generate report into FILE'
+
         def execute
           report_data = generate_report
           yaml = report_data.to_yaml
@@ -31,11 +32,12 @@ module ForemanMaintain
         end
       end
 
-      option '--input', 'FILE', 'Input the report from FILE'
-      option '--output', 'FILE', 'Output the condense report into FILE'
-      option '--max-age', 'HOURS', 'Max age of the report in hours'
       subcommand 'condense',
         'Generate a JSON formatted report with condensed data from the original report.' do
+        option '--input', 'FILE', 'Input the report from FILE'
+        option '--output', 'FILE', 'Output the condense report into FILE'
+        option '--max-age', 'HOURS', 'Max age of the report in hours'
+
         def execute
           data = if fresh_enough?(@input, @max_age)
                    YAML.load_file(@input)
@@ -43,76 +45,11 @@ module ForemanMaintain
                    generate_report
                  end
 
-          report = condense_report(data)
+          report = Utils::ReportCondenser.condense_report(data)
           report = prefix_keys(report)
           save_report(JSON.dump(report), @output)
         end
 
-        def condense_report(data)
-          result = {}
-          %w[advisor_on_prem_remediations_count rhel_ai_workload_host_count].each do |key|
-            result[key] = data[key] || 0
-          end
-          result.merge!(aggregate_host_count(data))
-          result.merge!(aggregate_image_mode_host_count(data))
-          result.merge!(aggregate_networking_metrics(data))
-          result
-        end
-
-        # Aggregates the host count numbers. The goal is to distinguish
-        # - RHEL hosts
-        # - RedHat family but not RHEL hosts
-        # - Other hosts
-        def aggregate_host_count(data)
-          result = {}
-          rhel_count = data['hosts_by_os_count|RedHat'] || 0
-          rh_count = data['hosts_by_family_count|Redhat'] || 0
-          result['host_rhel_count'] = rhel_count
-          result['host_redhat_count'] = rh_count - rhel_count
-          result['host_other_count'] = data.select do |k, _|
-            k.start_with?('hosts_by_os_count')
-          end.values.sum - rhel_count - rh_count
-          result
-        end
-
-        def aggregate_image_mode_host_count(data)
-          count = data.select { |k, _| k.start_with?('image_mode_hosts_by_os_count') }.values.sum
-          { 'image_mode_host_count' => count }
-        end
-
-        def aggregate_networking_metrics(data)
-          ipv6 = any_positive?(data, %w[subnet_ipv6_count hosts_with_ipv6only_interface_count
-                                        foreman_interfaces_ipv6only_count])
-          # Deployment is considered to run in dualstack mode if:
-          # - Foreman has both ipv6 and ipv4 addresses on a single interface
-          # - or if any host in Foreman has both ipv6 and ipv4 addresses on a single interface
-          dualstack = any_positive?(data, %w[hosts_with_dualstack_interface_count
-                                             foreman_interfaces_dualstack_count])
-
-          # - or if there are both ipv4 and ipv6 subnets defined
-          dualstack |= all_positive?(data, %w[subnet_ipv4_count subnet_ipv6_count])
-
-          # - or if any host in Foreman has an interface with only an ipv4 address
-          #   as well as another interface with ipv6 address
-          dualstack |= all_positive?(data, %w[hosts_with_ipv4only_interface_count
-                                              hosts_with_ipv6only_interface_count])
-
-          # - or if Foreman has an interface with only an ipv4 address
-          #   as well as another interface with ipv6 address
-          dualstack |= all_positive?(data,
-            %w[foreman_interfaces_ipv4only_count foreman_interfaces_ipv6only_count])
-
-          { 'use_dualstack' => dualstack, 'use_ipv6' => ipv6 }
-        end
-
-        def all_positive?(source, keys)
-          source.values_at(*keys).map { |x| x || 0 }.all?(&:positive?)
-        end
-
-        def any_positive?(source, keys)
-          source.values_at(*keys).map { |x| x || 0 }.any?(&:positive?)
-        end
-
         def prefix_keys(data)
           data.transform_keys { |key| 'foreman.' + key }
         end

data/lib/foreman_maintain/concerns/os_facts.rb

@@ -54,10 +54,6 @@ module ForemanMaintain
         File.exist?('/etc/redhat-release')
       end
 
-      def el8?
-        el_major_version == 8
-      end
-
       def debian?
         os_id == 'debian'
       end

data/lib/foreman_maintain/utils/report_condenser.rb

@@ -0,0 +1,72 @@
+module ForemanMaintain::Utils
+  module ReportCondenser
+    class << self
+      def condense_report(data)
+        result = {}
+        %w[advisor_on_prem_remediations_count rhel_ai_workload_host_count].each do |key|
+          result[key] = data[key] || 0
+        end
+        result.merge!(aggregate_host_count(data))
+        result.merge!(aggregate_image_mode_host_count(data))
+        result.merge!(aggregate_networking_metrics(data))
+        result
+      end
+
+      # Aggregates the host count numbers. The goal is to distinguish
+      # - RHEL hosts
+      # - RedHat family but not RHEL hosts
+      # - Other hosts
+      def aggregate_host_count(data)
+        result = {}
+        rhel_count = data['hosts_by_os_count|RedHat'] || 0
+        rh_count = data['hosts_by_family_count|Redhat'] || 0
+        result['host_rhel_count'] = rhel_count
+        result['host_redhat_without_rhel_count'] = rh_count - rhel_count
+        result['host_other_count'] = data.select do |k, _|
+          k.start_with?('hosts_by_os_count')
+        end.values.sum - rh_count
+        result
+      end
+
+      def aggregate_image_mode_host_count(data)
+        count = data.select { |k, _| k.start_with?('image_mode_hosts_by_os_count') }.values.sum
+        { 'image_mode_host_count' => count }
+      end
+
+      def aggregate_networking_metrics(data)
+        ipv6 = any_positive?(data, %w[subnet_ipv6_count hosts_with_ipv6only_interface_count
+                                      foreman_interfaces_ipv6only_count])
+        # Deployment is considered to run in dualstack mode if:
+        # - Foreman has both ipv6 and ipv4 addresses on a single interface
+        # - or if any host in Foreman has both ipv6 and ipv4 addresses on a single interface
+        dualstack = any_positive?(data, %w[hosts_with_dualstack_interface_count
+                                           foreman_interfaces_dualstack_count])
+
+        # - or if there are both ipv4 and ipv6 subnets defined
+        dualstack |= all_positive?(data, %w[subnet_ipv4_count subnet_ipv6_count])
+
+        # - or if any host in Foreman has an interface with only an ipv4 address
+        #   as well as another interface with ipv6 address
+        dualstack |= all_positive?(data, %w[hosts_with_ipv4only_interface_count
+                                            hosts_with_ipv6only_interface_count])
+
+        # - or if Foreman has an interface with only an ipv4 address
+        #   as well as another interface with ipv6 address
+        dualstack |= all_positive?(data,
+          %w[foreman_interfaces_ipv4only_count foreman_interfaces_ipv6only_count])
+
+        { 'use_dualstack' => dualstack, 'use_ipv6' => ipv6 }
+      end
+
+      private
+
+      def all_positive?(source, keys)
+        source.values_at(*keys).map { |x| x || 0 }.all?(&:positive?)
+      end
+
+      def any_positive?(source, keys)
+        source.values_at(*keys).map { |x| x || 0 }.any?(&:positive?)
+      end
+    end
+  end
+end

data/lib/foreman_maintain/utils.rb

@@ -4,5 +4,6 @@ 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/report_condenser'
 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.12.0'.freeze
+  VERSION = '1.12.2'.freeze
 end

data/lib/foreman_maintain.rb

@@ -191,7 +191,6 @@ module ForemanMaintain
 
       puts "Checking for new version of #{packages_to_update_str}..."
 
-      enable_maintenance_module
       package_manager = ForemanMaintain.package_manager
 
       if package_manager.update_available?(packages_to_update)
@@ -204,19 +203,6 @@ module ForemanMaintain
       puts "Nothing to update, can't find new version of #{packages_to_update_str}."
     end
 
-    def enable_maintenance_module
-      return unless el8?
-
-      maintenance_module = 'satellite-maintenance:el8'
-      package_manager = ForemanMaintain.package_manager
-
-      if package_manager.module_exists?(maintenance_module) &&
-         !package_manager.module_enabled?(maintenance_module)
-        puts "\nEnabling #{maintenance_module} module"
-        package_manager.enable_module(maintenance_module)
-      end
-    end
-
     def main_package_name
       el? ? 'rubygem-foreman_maintain' : 'ruby-foreman-maintain'
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_maintain
 version: !ruby/object:Gem::Version
-  version: 1.12.0
+  version: 1.12.2
 platform: ruby
 authors:
 - Ivan Nečas
@@ -445,6 +445,7 @@ 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/report_condenser.rb
 - lib/foreman_maintain/utils/response.rb
 - lib/foreman_maintain/utils/service.rb
 - lib/foreman_maintain/utils/service/abstract.rb