foreman_discovery 22.0.2 → 23.0.0

data/locale/zh_TW/foreman_discovery.po

@@ -5,7 +5,7 @@
 # Translators:
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_discovery 21.0.0\n"
+"Project-Id-Version: foreman_discovery v22.0.4\n"
 "Report-Msgid-Bugs-To: \n"
 "PO-Revision-Date: 2013-11-15 17:25+0000\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
@@ -86,9 +86,6 @@ msgstr "複製 %s"
 msgid "Collapse All"
 msgstr "全部收回"
 
-msgid "Command line options for kexec during PXE-less provisioning."
-msgstr ""
-
 msgid "Could not get facts from proxy %{url}: %{error}"
 msgstr ""
 
@@ -175,7 +172,7 @@ msgstr ""
 msgid "Discovered hosts summary"
 msgstr ""
 
-msgid "Discovery Kexec template"
+msgid "Discovery"
 msgstr ""
 
 msgid "Discovery Proxy"
@@ -259,7 +256,7 @@ msgstr ""
 msgid "Expected discovery_fact '%s' is missing, unable to detect primary interface and set hostname"
 msgstr ""
 
-msgid "Extra facter columns to show in host lists (separate by comma)"
+msgid "Extra facter columns to show in host lists"
 msgstr ""
 
 msgid "Fact + prefix"
@@ -304,6 +301,9 @@ msgstr ""
 msgid "Force DNS entries creation when provisioning discovered host"
 msgstr ""
 
+msgid "Foreman Discovery"
+msgstr ""
+
 msgid "Foreman discovered hosts summary"
 msgstr ""
 
@@ -446,7 +446,7 @@ msgstr ""
 msgid "List all fact values of a given discovered host"
 msgstr ""
 
-msgid "List of facts to use for the hostname (separated by comma, first wins)"
+msgid "List of facts to use for the hostname (first wins)"
 msgstr ""
 
 msgid "Location"
@@ -548,6 +548,18 @@ msgstr ""
 msgid "One or more hosts have been discovered"
 msgstr ""
 
+msgid "One or more hosts with failed discovery due to error: %s"
+msgstr ""
+
+msgid "Option 'url' must be provided"
+msgstr ""
+
+msgid "Option 'url' must be valid URI: %s"
+msgstr ""
+
+msgid "Options must be hash"
+msgstr ""
+
 msgid "Organization"
 msgstr "組織"
 

data/package.json

@@ -27,7 +27,6 @@
     "@babel/core": "^7.7.0",
     "@theforeman/builder": ">= 0",
     "@theforeman/eslint-plugin-foreman": ">= 0",
-    "@theforeman/stories": ">= 0",
     "@theforeman/test": ">= 0",
     "eslint": "^6.7.2",
     "prettier": "^1.19.1",

data/test/functional/api/v2/fact_value_extensions_test.rb

@@ -19,7 +19,7 @@ module Api
             "memorysize_mb"     => "42000.42",
             "discovery_version" => "3.0.0",
         }
-        Setting['discovery_hostname'] = 'discovery_bootif'
+        Setting['discovery_hostname'] = ['discovery_bootif']
       end
 
 

data/test/functional/discovered_hosts_controller_test.rb

@@ -28,7 +28,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
   end
 
   def test_index_with_custom_column
-    Setting['discovery_fact_column'] = "bios_vendor"
+    Setting['discovery_fact_column'] = ["bios_vendor"]
     facts = @facts.merge({"bios_vendor" => "QEMU"})
     discover_host_from_facts(facts)
     get :index, params: {}, session: set_session_user_default_reader

data/test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb

@@ -0,0 +1,75 @@
+require_relative '../test_plugin_helper'
+require ForemanDiscovery::Engine.root.join('db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array')
+
+class MigrateDiscoveryHostnameAndFactColumnToArrayTest < ActiveSupport::TestCase
+  let(:migrations_paths) { ActiveRecord::Migrator.migrations_paths + [ForemanDiscovery::Engine.root.join('db/migrate/').to_s] }
+
+  let(:previous_version) { '20221102065954'.to_i }
+  let(:current_version) { '20221102075151'.to_i }
+
+  #only load the two migrations we care about (previous one and current one)
+  let(:migrations) do
+    [
+      ActiveRecord::MigrationProxy.new("FixDiscoverySettingsCategoryToDsl", previous_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102065954_fix_discovery_settings_category_to_dsl.rb", ""),
+      ActiveRecord::MigrationProxy.new("MigrateDiscoveryHostnameAndFactColumnToArray", current_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array.rb", "")
+    ]
+  end
+
+  def migrate_up
+    ActiveRecord::Migrator.new(:up, migrations, ActiveRecord::SchemaMigration, current_version).migrate
+  end
+
+  def setup
+    ActiveRecord::Migration.suppress_messages do
+      ActiveRecord::Migrator.new(:down, migrations, ActiveRecord::SchemaMigration, previous_version).migrate
+    end
+  end
+
+  def setup_setting(name, value)
+    setting = Setting.find_or_create_by(name: name)
+    setting.value = value
+    setting.save(validate: false)
+  end
+
+  def test_discovery_hostname_string
+    setup_setting('discovery_hostname', 'discovery_bootif')
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_multistring
+    setup_setting('discovery_hostname', 'discovery_bootif, fqdn')
+    migrate_up
+    assert_equal ['discovery_bootif', 'fqdn'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_array
+    Setting['discovery_hostname'] = ['discovery_bootif']
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_fact_column_empty
+    setup_setting('discovery_fact_column', '')
+    migrate_up
+    assert_equal [], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_string
+    setup_setting('discovery_fact_column', 'bios_vendor')
+    migrate_up
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_multistring
+    setup_setting('discovery_fact_column', 'bios_vendor, fqdn')
+    migrate_up
+    assert_equal ['bios_vendor', 'fqdn'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_array
+    Setting['discovery_fact_column'] = ['bios_vendor']
+    migrate_up
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+end

data/test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig

@@ -0,0 +1,81 @@
+require_relative '../test_plugin_helper'
+require ForemanDiscovery::Engine.root.join('db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array')
+
+class MigrateDiscoveryHostnameAndFactColumnToArrayTest < ActiveSupport::TestCase
+  let(:migrations_paths) { ActiveRecord::Migrator.migrations_paths + [ForemanDiscovery::Engine.root.join('db/migrate/').to_s] }
+
+  let(:previous_version) { '20221102065954'.to_i }
+  let(:current_version) { '20221102075151'.to_i }
+
+  #only load the two migrations we care about (previous one and current one)
+  let(:migrations) do
+    [
+      ActiveRecord::MigrationProxy.new("FixDiscoverySettingsCategoryToDsl", previous_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102065954_fix_discovery_settings_category_to_dsl.rb", ""),
+      ActiveRecord::MigrationProxy.new("MigrateDiscoveryHostnameAndFactColumnToArray", current_version, "#{ForemanDiscovery::Engine.root}/db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array.rb", "")
+    ]
+  end
+
+  def migrate_up
+    ActiveRecord::Migrator.new(:up, migrations, ActiveRecord::SchemaMigration, current_version).migrate
+  end
+
+  def setup
+    ActiveRecord::Migration.suppress_messages do
+      ActiveRecord::Migrator.new(:down, migrations, ActiveRecord::SchemaMigration, previous_version).migrate
+    end
+  end
+
+  def setup_setting(name, value)
+    setting = Setting.find_or_create_by(name: name)
+    setting.value = value
+    setting.save(validate: false)
+  end
+
+  def test_discovery_hostname_string
+    setup_setting('discovery_hostname', 'discovery_bootif')
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_multistring
+    setup_setting('discovery_hostname', 'discovery_bootif, fqdn')
+    migrate_up
+    assert_equal ['discovery_bootif', 'fqdn'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_hostname_array
+    Setting['discovery_hostname'] = ['discovery_bootif']
+    migrate_up
+    assert_equal ['discovery_bootif'], Setting['discovery_hostname']
+  end
+
+  def test_discovery_fact_column_empty
+    setup_setting('discovery_fact_column', '')
+    migrate_up
+    assert_equal [], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_string
+    setup_setting('discovery_fact_column', 'bios_vendor')
+    migrate_up
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_multistring
+    setup_setting('discovery_fact_column', 'bios_vendor, fqdn')
+    migrate_up
+    assert_equal ['bios_vendor', 'fqdn'], Setting['discovery_fact_column']
+  end
+
+  def test_discovery_fact_column_array
+    Setting['discovery_fact_column'] = ['bios_vendor']
+<<<<<<< HEAD
+    migrate_up
+=======
+    
+    migrate_up
+    
+>>>>>>> fd3524f (Fix tests for array settings migrations)
+    assert_equal ['bios_vendor'], Setting['discovery_fact_column']
+  end
+end

data/test/test_helper_discovery.rb

@@ -53,7 +53,7 @@ end
 
 def set_default_settings
   Setting['discovery_fact'] = 'discovery_bootif'
-  Setting['discovery_hostname'] = 'discovery_bootif'
+  Setting['discovery_hostname'] = ['discovery_bootif']
   Setting['discovery_auto'] = true
   Setting['discovery_reboot'] = true
   Setting['discovery_organization'] = "Organization 1"

data/test/unit/discovery_attribute_set_test.rb

@@ -6,7 +6,7 @@ class DiscoveryAttributeSetTest < ActiveSupport::TestCase
 
   setup do
     @facts = parse_json_fixture('regular_host', true)
-    Setting['discovery_hostname'] = 'discovery_bootif'
+    Setting['discovery_hostname'] = ['discovery_bootif']
     Setting['discovery_prefix'] = 'mac'
     ::ForemanDiscovery::HostConverter.stubs(:unused_ip_for_host)
   end

data/test/unit/host_discovered_test.rb

@@ -181,7 +181,7 @@ class HostDiscoveredTest < ActiveSupport::TestCase
   end
 
   test "should create discovered host with hostname if a fact was supplied" do
-    Setting[:discovery_hostname] = 'somefact'
+    Setting[:discovery_hostname] = ['somefact']
     facts = @facts.merge({"somefact" => "somename"})
     host = discover_host_from_facts(facts)
     assert_equal 'macsomename', host.name
@@ -219,7 +219,7 @@ class HostDiscoveredTest < ActiveSupport::TestCase
 
   test "should create discovered host with fact_name as a name if it is a valid mac" do
     Setting[:discovery_fact] = 'somefact'
-    Setting[:discovery_hostname] = 'somefact'
+    Setting[:discovery_hostname] = ['somefact']
     facts = @facts.merge({"somefact" => "E4:1F:13:CC:36:5A"})
     host = discover_host_from_facts(facts)
     assert_equal 'mace41f13cc365a', host.name
@@ -277,7 +277,7 @@ class HostDiscoveredTest < ActiveSupport::TestCase
   end
 
   test "should raise when hostname fact cannot be found" do
-    Setting[:discovery_hostname] = 'macaddress_foo'
+    Setting[:discovery_hostname] = ['macaddress_foo']
     exception = assert_raises(::Foreman::Exception) do
       discover_host_from_facts(@facts)
     end

data/webpack/src/ForemanDiscovery/DiscoveryRules/Components/EmptyState/EmptyState.js

@@ -17,7 +17,7 @@ const EmptyState = props => {
   };
   return (
     <ForemanEmptyState
-      header="Discovery Rules"
+      header={__('Discovery Rules')}
       description={description}
       icon="gavel"
       iconType="fa"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_discovery
 version: !ruby/object:Gem::Version
-  version: 22.0.2
+  version: 23.0.0
 platform: ruby
 authors:
 - Aditi Puntambekar
@@ -76,7 +76,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-11-03 00:00:00.000000000 Z
+date: 2023-11-22 00:00:00.000000000 Z
 dependencies: []
 description: MaaS Discovery Plugin engine for Foreman
 email: gsutclif@redhat.com
@@ -88,6 +88,23 @@ extra_rdoc_files:
 files:
 - LICENSE
 - README.md
+- app/assets/javascripts/foreman_discovery/locale/ca/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/cs_CZ/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/de/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/en/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/en_GB/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/es/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/fr/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/gl/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/it/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ja/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ka/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ko/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/pt_BR/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/ru/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/sv_SE/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/zh_CN/foreman_discovery.js
+- app/assets/javascripts/foreman_discovery/locale/zh_TW/foreman_discovery.js
 - app/controllers/api/v2/discovered_hosts_controller.rb
 - app/controllers/api/v2/discovery_rules_controller.rb
 - app/controllers/api/v2/fact_values_controller_extensions.rb
@@ -109,7 +126,6 @@ files:
 - app/models/host/managed_extensions.rb
 - app/models/hostgroup_extensions.rb
 - app/models/nic/managed_extensions.rb
-- app/models/setting/discovered.rb
 - app/services/discovery_fact_importer.rb
 - app/services/foreman_discovery/fact_parser.rb
 - app/services/foreman_discovery/fact_to_category_resolver.rb
@@ -190,6 +206,7 @@ files:
 - db/migrate/20171222120314_add_constraints_on_discovery_rules_hostgroups.rb
 - db/migrate/20180412124505_add_priority_score_to_discovery_rules.rb
 - db/migrate/20221102065954_fix_discovery_settings_category_to_dsl.rb
+- db/migrate/20221102075151_migrate_discovery_hostname_and_fact_column_to_array.rb
 - db/seeds.d/60_discovery_proxy_feature.rb
 - db/seeds.d/70_discovery_mail_notification.rb
 - db/seeds.d/80_discovery_ui_notification.rb
@@ -201,35 +218,56 @@ files:
 - lib/foreman_discovery/engine.rb
 - lib/foreman_discovery/version.rb
 - locale/ca/LC_MESSAGES/foreman_discovery.mo
+- locale/ca/foreman_discovery.edit.po
 - locale/ca/foreman_discovery.po
+- locale/cs_CZ/LC_MESSAGES/foreman_discovery.mo
+- locale/cs_CZ/foreman_discovery.edit.po
+- locale/cs_CZ/foreman_discovery.po
 - locale/de/LC_MESSAGES/foreman_discovery.mo
+- locale/de/foreman_discovery.edit.po
 - locale/de/foreman_discovery.po
 - locale/en/LC_MESSAGES/foreman_discovery.mo
+- locale/en/foreman_discovery.edit.po
 - locale/en/foreman_discovery.po
 - locale/en_GB/LC_MESSAGES/foreman_discovery.mo
+- locale/en_GB/foreman_discovery.edit.po
 - locale/en_GB/foreman_discovery.po
 - locale/es/LC_MESSAGES/foreman_discovery.mo
+- locale/es/foreman_discovery.edit.po
 - locale/es/foreman_discovery.po
 - locale/foreman_discovery.pot
 - locale/fr/LC_MESSAGES/foreman_discovery.mo
+- locale/fr/foreman_discovery.edit.po
 - locale/fr/foreman_discovery.po
 - locale/gl/LC_MESSAGES/foreman_discovery.mo
+- locale/gl/foreman_discovery.edit.po
 - locale/gl/foreman_discovery.po
 - locale/it/LC_MESSAGES/foreman_discovery.mo
+- locale/it/foreman_discovery.edit.po
 - locale/it/foreman_discovery.po
 - locale/ja/LC_MESSAGES/foreman_discovery.mo
+- locale/ja/foreman_discovery.edit.po
 - locale/ja/foreman_discovery.po
+- locale/ka/LC_MESSAGES/foreman_discovery.mo
+- locale/ka/foreman_discovery.edit.po
+- locale/ka/foreman_discovery.po
 - locale/ko/LC_MESSAGES/foreman_discovery.mo
+- locale/ko/foreman_discovery.edit.po
 - locale/ko/foreman_discovery.po
 - locale/pt_BR/LC_MESSAGES/foreman_discovery.mo
+- locale/pt_BR/foreman_discovery.edit.po
 - locale/pt_BR/foreman_discovery.po
 - locale/ru/LC_MESSAGES/foreman_discovery.mo
+- locale/ru/foreman_discovery.edit.po
 - locale/ru/foreman_discovery.po
 - locale/sv_SE/LC_MESSAGES/foreman_discovery.mo
+- locale/sv_SE/foreman_discovery.edit.po
 - locale/sv_SE/foreman_discovery.po
 - locale/zh_CN/LC_MESSAGES/foreman_discovery.mo
+- locale/zh_CN/foreman_discovery.edit.po
 - locale/zh_CN/foreman_discovery.po
 - locale/zh_TW/LC_MESSAGES/foreman_discovery.mo
+- locale/zh_TW/foreman_discovery.edit.po
 - locale/zh_TW/foreman_discovery.po
 - package.json
 - test/factories/discovery_host_related.rb
@@ -257,6 +295,8 @@ files:
 - test/functional/discovery_rules_controller_test.rb
 - test/functional/foreman_discovery/concerns/hosts_controller_extensions_test.rb
 - test/integration/discovered_hosts_test.rb
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig
 - test/static_fixtures/redhat_kexec.erb
 - test/test_helper_discovery.rb
 - test/test_plugin_helper.rb
@@ -315,8 +355,8 @@ signing_key:
 specification_version: 4
 summary: MaaS Discovery Plugin for Foreman
 test_files:
-- test/factories/discovery_host_related.rb
 - test/factories/discovery_rule_related.rb
+- test/factories/discovery_host_related.rb
 - test/facts/bond0-eth0-eth1-active-passive.json
 - test/facts/default.json
 - test/facts/dell_npars.json
@@ -337,21 +377,23 @@ test_files:
 - test/functional/api/v2/discovery_rules_controller_test.rb
 - test/functional/api/v2/fact_value_extensions_test.rb
 - test/functional/foreman_discovery/concerns/hosts_controller_extensions_test.rb
-- test/functional/discovery_rules_controller_test.rb
 - test/functional/discovered_hosts_controller_test.rb
+- test/functional/discovery_rules_controller_test.rb
 - test/integration/discovered_hosts_test.rb
 - test/unit/ui_notifications/destroy_host_test.rb
 - test/unit/ui_notifications/new_host_test.rb
 - test/unit/discovered_mailer_test.rb
+- test/unit/discovered_extensions_test.rb
 - test/unit/discovery_attribute_set_test.rb
 - test/unit/discovery_rule_test.rb
 - test/unit/discovery_taxonomy_extensions_test.rb
 - test/unit/fact_parser_test.rb
 - test/unit/fact_to_category_resolver_test.rb
 - test/unit/lldp_neighbors_test.rb
-- test/unit/discovered_extensions_test.rb
-- test/unit/host_discovered_test.rb
 - test/unit/managed_extensions_test.rb
-- test/test_helper_discovery.rb
+- test/unit/host_discovered_test.rb
 - test/test_plugin_helper.rb
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb.orig
+- test/migrations/20221102075151_migrate_discovery_hostname_and_fact_column_to_array_test.rb
 - test/static_fixtures/redhat_kexec.erb
+- test/test_helper_discovery.rb

data/app/models/setting/discovered.rb

@@ -1,22 +0,0 @@
-class Setting::Discovered
-  def self.discovery_fact_column_array
-    from_array(Setting['discovery_fact_column'])
-  end
-
-  def self.discovery_hostname_fact_array
-    return [] if !Setting['discovery_hostname'].present?
-    from_array Setting['discovery_hostname']
-  end
-
-  def self.discovery_lock?
-    Foreman::Cast.to_bool(Setting['discovery_lock'])
-  end
-
-  def self.from_array(setting)
-    return [] unless setting.present?
-    setting.to_s.split(",").map(&:strip)
-  rescue Exception => e
-    logger.warn "Failed to parse discovery_fact_column, ignoring: #{e}"
-    []
-  end
-end