foreman_monitoring 2.1.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6bc195afe4c0af30313894eaf37dcf99c36625fe510998783df94932111c9891
-  data.tar.gz: 8d37c47d99c8d48163242ce490159c59a6a4e6c2588709daac0bac841a450c5a
+  metadata.gz: bba72edc0a48bb00f5818a0662ef643ea892cef4f457b474fd35332471bf1751
+  data.tar.gz: b62364ba84ac51878993cd2d6d7be654cfb10aa2c2a565b2ec71f6dc38f933a1
 SHA512:
-  metadata.gz: cf715511ef2835a64702b2f865c6380c178dd8c70b2038428ea3a0c9a72fefcef4715ec4c2bcca82fbf687c9ea5a463c5e041009fe7e795cb6aab6484020d6f3
-  data.tar.gz: 8c4c4cbaad0dffa2f7d338d19e116cc58c05272b262d36d5bf221137391577a414dcd3ba83c34d78c46207bf8298675529a988d42fd736961751687325b615b1
+  metadata.gz: d1d0834a1dc76d2009dc1bd1bbd35c0fbcfc166e1b1206c87a3ba1fd7d8543c693d22550cd899129b41d9dcbf76d0a379ab7c0ef6a7d83e2ffa1960c0b8ec3a2
+  data.tar.gz: 8715c42a9d55a3f491a860373c74dcda03351be2752a14e2dc798781f1ab807b55ff332822ead436637b1d479702920cbe9c19b6b30bff4844418acbe3d8611f

data/README.md

@@ -36,6 +36,7 @@ touch ~foreman/tmp/restart.txt
 | >= 1.17         | ~> 1.0         |
 | >= 2.0          | ~> 2.0         |
 | >= 2.2          | ~> 2.1         |
+| >= 3.0          | ~> 3.0         |
 
 # Usage
 

data/app/models/concerns/foreman_monitoring/host_extensions.rb

@@ -56,7 +56,7 @@ module ForemanMonitoring
     end
 
     def monitoring_attributes
-      {
+      attributes = {
         :ip => ip,
         :ip6 => ip6,
         :architecture => architecture.try(:name),
@@ -69,9 +69,12 @@ module ForemanMonitoring
         :organization => organization.try(:name),
         :location => location.try(:name),
         :comment => comment,
-        :environment => environment.try(:to_s),
         :owner_name => owner.try(:name)
       }
+
+      attributes[:environment] = environment.try(:to_s) if defined?(ForemanPuppet)
+
+      attributes
     end
 
     private

data/db/migrate/20221026065954_fix_monitoring_settings_category_to_dsl.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+class FixMonitoringSettingsCategoryToDsl < ActiveRecord::Migration[6.0]
+  def up
+    # rubocop:disable Rails/SkipsModelValidations
+    Setting.where(category: 'Setting::Monitoring').update_all(category: 'Setting')
+    # rubocop:enable Rails/SkipsModelValidations
+  end
+end

data/lib/foreman_monitoring/engine.rb

@@ -18,19 +18,31 @@ module ForemanMonitoring
       end
     end
 
-    initializer 'foreman_monitoring.load_default_settings',
-                :before => :load_config_initializers do |_app|
-      setting_table_exists = begin
-                               Setting.table_exists?
-                             rescue StandardError
-                               false
-                             end
-      require_dependency File.expand_path('../../app/models/setting/monitoring.rb', __dir__) if setting_table_exists
-    end
-
     initializer 'foreman_monitoring.register_plugin', :before => :finisher_hook do |_app|
       Foreman::Plugin.register :foreman_monitoring do
-        requires_foreman '>= 2.2'
+        requires_foreman '>= 3.0'
+
+        settings do
+          category(:monitoring, N_('Monitoring')) do
+            setting('monitoring_affect_global_status',
+                    type: :boolean,
+                    description: N_("Monitoring status will affect a host's global status when enabled"),
+                    default: true,
+                    full_name: N_('Monitoring status should affect global status'))
+            setting('monitoring_create_action',
+                    type: :string,
+                    description: N_('What action should be taken when a host is created'),
+                    default: 'create',
+                    full_name: N_('Host Create Action'),
+                    collection: proc { ::Monitoring::CREATE_ACTIONS })
+            setting('monitoring_delete_action',
+                    type: :string,
+                    description: N_('What action should be taken when a host is deleted'),
+                    default: 'delete',
+                    full_name: N_('Host Delete Action'),
+                    collection: proc { ::Monitoring::DELETE_ACTIONS })
+          end
+        end
 
         apipie_documented_controllers ["#{ForemanMonitoring::Engine.root}/app/controllers/api/v2/*.rb"]
 

data/lib/foreman_monitoring/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanMonitoring
-  VERSION = '2.1.0'
+  VERSION = '3.0.0'
 end

data/test/test_plugin_helper.rb

@@ -11,10 +11,6 @@ FactoryBot.reload
 # Foreman's setup doesn't handle cleaning up for Minitest::Spec
 DatabaseCleaner.strategy = :transaction
 
-def setup_settings
-  Setting::Monitoring.load_defaults
-end
-
 def disable_monitoring_orchestration
   ProxyAPI::Monitoring.any_instance.stubs(:query_host).returns({})
 end

data/test/unit/host_status/monitoring_status_test.rb

@@ -4,7 +4,6 @@ require 'test_plugin_helper'
 
 class MonitoringStatusTest < ActiveSupport::TestCase
   setup do
-    setup_settings
     disable_monitoring_orchestration
   end
 

data/test/unit/host_test.rb

@@ -5,7 +5,6 @@ require 'test_plugin_helper'
 class HostTest < ActiveSupport::TestCase
   setup do
     User.current = FactoryBot.build(:user, :admin)
-    setup_settings
     disable_orchestration
     disable_monitoring_orchestration
   end

data/test/unit/monitoring_result_test.rb

@@ -5,7 +5,6 @@ require 'test_plugin_helper'
 class MonitoringResultTest < ActiveSupport::TestCase
   setup do
     User.current = FactoryBot.build(:user, :admin)
-    setup_settings
     disable_orchestration
     disable_monitoring_orchestration
   end

data/test/unit/monitoring_test.rb

@@ -5,7 +5,6 @@ require 'test_plugin_helper'
 class MonitoringTest < ActiveSupport::TestCase
   setup do
     User.current = FactoryBot.build(:user, :admin)
-    setup_settings
     disable_monitoring_orchestration
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_monitoring
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Timo Goebel
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-01 00:00:00.000000000 Z
+date: 2022-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc
@@ -115,7 +115,6 @@ files:
 - app/models/concerns/orchestration/monitoring.rb
 - app/models/host_status/monitoring_status.rb
 - app/models/monitoring_result.rb
-- app/models/setting/monitoring.rb
 - app/overrides/add_host_monitoring_result_tab.rb
 - app/overrides/add_host_multiple_power_set_downtime_checkbox.rb
 - app/overrides/add_host_set_downtime_modal.rb
@@ -131,6 +130,7 @@ files:
 - db/migrate/20160817135723_create_monitoring_results.rb
 - db/migrate/20161220201510_add_monitoring_proxy_id_to_host_and_hostgroup.rb
 - db/migrate/201910180900_rename_downtime_host_permission.rb
+- db/migrate/20221026065954_fix_monitoring_settings_category_to_dsl.rb
 - db/seeds.d/60-monitoring_proxy_feature.rb
 - lib/foreman_monitoring.rb
 - lib/foreman_monitoring/engine.rb
@@ -156,7 +156,7 @@ homepage: https://github.com/theforeman/foreman_monitoring
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -171,20 +171,20 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Foreman plugin for monitoring system integration.
 test_files:
+- test/controllers/api/v2/downtime_controller_test.rb
+- test/factories/feature.rb
 - test/factories/host.rb
 - test/factories/monitoring_results.rb
 - test/factories/smart_proxy.rb
-- test/factories/feature.rb
-- test/lib/proxy_api/monitoring_test.rb
 - test/functional/hosts_controller_test.rb
-- test/controllers/api/v2/downtime_controller_test.rb
+- test/lib/proxy_api/monitoring_test.rb
 - test/test_plugin_helper.rb
-- test/unit/monitoring_result_test.rb
+- test/unit/host_status/monitoring_status_test.rb
 - test/unit/host_test.rb
+- test/unit/monitoring_result_test.rb
 - test/unit/monitoring_test.rb
-- test/unit/host_status/monitoring_status_test.rb

data/app/models/setting/monitoring.rb

@@ -1,34 +0,0 @@
-# frozen_string_literal: true
-
-class Setting
-  class Monitoring < ::Setting
-    def self.default_settings
-      [
-        set('monitoring_affect_global_status',
-            _("Monitoring status will affect a host's global status when enabled"),
-            true, N_('Monitoring status should affect global status')),
-        set('monitoring_create_action',
-            _('What action should be taken when a host is created'),
-            'create', N_('Host Create Action'), nil, :collection => proc { ::Monitoring::CREATE_ACTIONS }),
-        set('monitoring_delete_action',
-            _('What action should be taken when a host is deleted'),
-            'delete', N_('Host Delete Action'), nil, :collection => proc { ::Monitoring::DELETE_ACTIONS })
-      ]
-    end
-
-    def self.load_defaults
-      # Check the table exists
-      return unless super
-
-      self.transaction do
-        default_settings.each { |s| self.create! s.update(:category => 'Setting::Monitoring') }
-      end
-
-      true
-    end
-
-    def self.humanized_category
-      N_('Monitoring')
-    end
-  end
-end