foreman_discovery 15.1.0 → 16.1.3

data/locale/zh_TW/foreman_discovery.po

@@ -5,10 +5,10 @@
 # Translators:
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_discovery 15.0.0\n"
+"Project-Id-Version: foreman_discovery 16.0.1\n"
 "Report-Msgid-Bugs-To: \n"
-"PO-Revision-Date: 2019-04-18 19:36+0000\n"
-"Last-Translator: Lukáš Zapletal\n"
+"PO-Revision-Date: 2020-05-26 17:21+0000\n"
+"Last-Translator: Transifex Bot <>\n"
 "Language-Team: Chinese (Taiwan) (http://www.transifex.com/foreman/foreman/lang"
 "uage/zh_TW/)\n"
 "MIME-Version: 1.0\n"
@@ -32,6 +32,9 @@ msgstr ""
 msgid "A summary of discovered hosts"
 msgstr ""
 
+msgid "Action with sub plans"
+msgstr ""
+
 msgid "Actions"
 msgstr "動作"
 
@@ -134,6 +137,11 @@ msgstr "停用"
 msgid "Disable rule '%s'?"
 msgstr ""
 
+msgid "Discovered Host"
+msgid_plural "Discovered Hosts"
+msgstr[0] ""
+msgstr[1] ""
+
 msgid "Discovered Hosts"
 msgstr ""
 
@@ -345,6 +353,9 @@ msgstr "IP 位址"
 msgid "IP address"
 msgstr "IP 位址"
 
+msgid "IPAM must be configured for subnet '%s'"
+msgstr ""
+
 msgid "IPMI"
 msgstr "IPMI"
 
@@ -360,6 +371,12 @@ msgstr ""
 msgid "Image API returned HTTP/%{code} with '%{body}"
 msgstr ""
 
+msgid "Import Puppet classes"
+msgstr ""
+
+msgid "Import facts"
+msgstr ""
+
 msgid "In addition to @host attribute function rand for random integers is available. Examples:"
 msgstr ""
 
@@ -576,6 +593,9 @@ msgstr ""
 msgid "Reloading kernel on %s"
 msgstr ""
 
+msgid "Remote action:"
+msgstr ""
+
 msgid "Reported in the last 7 days"
 msgstr ""
 
@@ -675,13 +695,13 @@ msgstr ""
 msgid "Unable to find primary NIC with %{mac} specified via '%{fact}', NIC filter: %{filter}"
 msgstr ""
 
-msgid "Unable to perform kexec on %{name} via %{url}: %{msg}"
+msgid "Unable to perform kexec on %{name} (%{url}): %{msg}"
 msgstr ""
 
-msgid "Unable to provision %{host}: %{errors}"
+msgid "Unable to perform reboot on %{name} (%{url}): %{msg}"
 msgstr ""
 
-msgid "Unable to reboot %{name} via %{url}: %{msg}"
+msgid "Unable to provision %{host}: %{errors}"
 msgstr ""
 
 msgid "Update a rule"

data/test/facts/bond0-eth0-eth1-active-passive.json

@@ -0,0 +1,128 @@
+{
+  "uptime_seconds": 206,
+  "architecture": "x86_64",
+  "virtual": "kvm",
+  "is_virtual": true,
+  "kernel": "Linux",
+  "blockdevice_vda_size": 6442450944,
+  "blockdevice_vda_vendor": "0x1af4",
+  "blockdevices": "vda",
+  "dhcp_servers": {
+    "system": "192.168.122.1",
+    "bond0": "192.168.122.1"
+  },
+  "domain": "nat.lan",
+  "hardwaremodel": "x86_64",
+  "operatingsystem": "CentOS",
+  "os": {
+    "name": "CentOS",
+    "family": "RedHat",
+    "release": {
+      "major": "7",
+      "minor": "6",
+      "full": "7.6.1810"
+    }
+  },
+  "facterversion": "2.4.1",
+  "filesystems": "xfs",
+  "fqdn": "localhost.nat.lan",
+  "gid": "root",
+  "hardwareisa": "x86_64",
+  "hostname": "localhost",
+  "id": "root",
+  "interfaces": "bond0,eth0,eth1,lo",
+  "ipaddress_bond0": "192.168.122.7",
+  "macaddress_bond0": "52:54:00:fd:1e:62",
+  "netmask_bond0": "255.255.255.0",
+  "mtu_bond0": 1500,
+  "macaddress_eth0": "52:54:00:FD:1E:62",
+  "mtu_eth0": 1500,
+  "macaddress_eth1": "52:54:00:7C:84:DA",
+  "mtu_eth1": 1500,
+  "ipaddress_lo": "127.0.0.1",
+  "netmask_lo": "255.0.0.0",
+  "mtu_lo": 65536,
+  "ipaddress": "192.168.122.7",
+  "kernelmajversion": "3.10",
+  "kernelrelease": "3.10.0-957.el7.x86_64",
+  "kernelversion": "3.10.0",
+  "rubyplatform": "x86_64-linux",
+  "macaddress": "52:54:00:fd:1e:62",
+  "bios_vendor": "SeaBIOS",
+  "bios_version": "1.12.0-2.fc30",
+  "bios_release_date": "04/01/2014",
+  "manufacturer": "QEMU",
+  "productname": "Standard PC (Q35 + ICH9, 2009)",
+  "serialnumber": "Not Specified",
+  "uuid": "d1e61ef9-663e-4195-b256-f64b9f8cc8ae",
+  "type": "Other",
+  "memorysize": "8.41 GB",
+  "memoryfree": "8.20 GB",
+  "swapsize": "615.00 MB",
+  "swapfree": "615.00 MB",
+  "swapsize_mb": "615.00",
+  "swapfree_mb": "615.00",
+  "memorysize_mb": "8615.46",
+  "memoryfree_mb": "8395.15",
+  "netmask": "255.255.255.0",
+  "network_bond0": "192.168.122.0",
+  "network_lo": "127.0.0.0",
+  "uptime_hours": 0,
+  "operatingsystemmajrelease": "7",
+  "rubysitedir": "/usr/local/share/ruby/site_ruby/",
+  "operatingsystemrelease": "7.6.1810",
+  "osfamily": "RedHat",
+  "partitions": {
+    "vda1": {
+      "size": "2048"
+    },
+    "vda2": {
+      "uuid": "1f259325-5eb4-4334-81f3-699ace4a28ff",
+      "size": "2097152",
+      "mount": "/boot",
+      "filesystem": "xfs"
+    },
+    "vda3": {
+      "uuid": "6639b5c4-1ba1-41d6-a4a0-6253eeaaae05",
+      "size": "1259520",
+      "filesystem": "swap"
+    },
+    "vda4": {
+      "uuid": "09aced65-5b86-4efe-9da0-c86a3311e1b8",
+      "size": "9220096",
+      "mount": "/",
+      "filesystem": "xfs"
+    }
+  },
+  "path": "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/root/bin",
+  "rubyversion": "2.0.0",
+  "physicalprocessorcount": 2,
+  "processors": {
+    "models": [
+      "AMD EPYC Processor (with IBPB)",
+      "AMD EPYC Processor (with IBPB)"
+    ],
+    "count": 2,
+    "physicalcount": 2
+  },
+  "processor0": "AMD EPYC Processor (with IBPB)",
+  "processor1": "AMD EPYC Processor (with IBPB)",
+  "processorcount": 2,
+  "ps": "ps -ef",
+  "selinux": true,
+  "selinux_enforced": true,
+  "selinux_policyversion": "31",
+  "selinux_current_mode": "enforcing",
+  "selinux_config_mode": "enforcing",
+  "selinux_config_policy": "unknown",
+  "system_uptime": {
+    "seconds": 206,
+    "hours": 0,
+    "days": 0,
+    "uptime": "0:03 hours"
+  },
+  "timezone": "EDT",
+  "uniqueid": "007f0100",
+  "uptime": "0:03 hours",
+  "uptime_days": 0
+}

data/test/facts/facts_with_lldp_bond_candidate.json

@@ -6,14 +6,11 @@
     "kernelmajversion": "2.6",
     "kernelversion": "2.6.32",
     "macaddress_vnet0": "FE:54:00:1E:45:13",
-    "macaddress_br181": "E4:1F:13:CC:36:58",
     "macaddress_usb0": "E6:1F:13:D0:43:A3",
     "macaddress_vnet1": "FE:54:00:45:E5:3E",
     "rubysitedir": "/usr/lib/ruby/site_ruby/1.8",
-    "macaddress_br182": "E4:1F:13:CC:36:58",
     "processor4": "Intel(R) Xeon(R) CPU           E5620  @ 2.40GHz",
     "macaddress_vnet2": "FE:54:00:93:F5:56",
-    "macaddress_br183": "E4:1F:13:CC:36:58",
     "ps": "ps -ef",
     "processor5": "Intel(R) Xeon(R) CPU           E5620  @ 2.40GHz",
     "macaddress_vnet10": "FE:54:00:7D:F8:24",
@@ -28,16 +25,15 @@
     "macaddress_vnet4": "FE:54:00:62:53:67",
     "hostname": "h02",
     "osfamily": "RedHat",
-    "gateway_if": "br180",
+    "gateway_if": "eth0",
     "lsbrelease": ":core-4.0-amd64:core-4.0-noarch:graphics-4.0-amd64:graphics-4.0-noarch:printing-4.0-amd64:printing-4.0-noarch",
     "kernelrelease": "2.6.32-131.12.1.el6.x86_64",
     "kernel": "Linux",
     "facterversion": "1.6.2",
     "uptime_seconds": "13651752",
     "macaddress_vnet5": "FE:54:00:01:B9:9C",
-    "interfaces": "br180,br181,br182,br183,eth0,eth0_181,eth0_182,eth0_183,eth1,lo,usb0,vnet0,vnet1,vnet2,vnet3,vnet4,vnet5,vnet7,vnet10",
+    "interfaces": "eth0,eth0_181,eth0_182,eth0_183,eth1,lo,usb0,vnet0,vnet1,vnet2,vnet3,vnet4,vnet5,vnet7,vnet10",
     "macaddress_vnet7": "FE:54:00:D8:F9:38",
-    "netmask_br180": "255.255.255.192",
     "macaddress_eth0": "E4:1F:13:CC:36:58",
     "is_virtual": "false",
     "macaddress": "E4:1F:13:CC:36:58",
@@ -47,7 +43,6 @@
     "macaddress_eth1": "E4:1F:13:CC:36:5A",
     "timezone": "GMT",
     "fqdn": "a.server.b.domain",
-    "ipaddress_br180": "10.35.27.2",
     "ipaddress_eth0": "10.35.27.3",
     "ipaddress_eth1": "10.35.27.4",
     "puppetversion": "2.6.12",
@@ -69,12 +64,10 @@
     "network_lo": "127.0.0.0",
     "gateway": "10.35.27.62",
     "lsbdistid": "RedHatEnterpriseServer",
-    "network_br180": "10.35.27.0",
     "lsbdistdescription": "Red Hat Enterprise Linux Server release 6.2 (Santiago)",
     "swapsize": "17.70 GB",
     "processor0": "Intel(R) Xeon(R) CPU           E5620  @ 2.40GHz",
     "processor1": "Intel(R) Xeon(R) CPU           E5620  @ 2.40GHz",
-    "macaddress_br180": "E4:1F:13:CC:36:58",
     "selinux": "false",
     "domain": "server.b.domain",
     "architecture": "x86_64",

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

@@ -25,6 +25,7 @@ class Api::V2::DiscoveredHostsControllerTest < ActionController::TestCase
     }
     set_default_settings
     ::ForemanDiscovery::NodeAPI::PowerService.any_instance.stubs(:reboot).returns(true)
+    ::ForemanDiscovery::HostConverter.stubs(:unused_ip_for_host)
   end
 
   def test_get_index

data/test/functional/discovered_hosts_controller_test.rb

@@ -8,7 +8,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
   setup do
     assert discovered_notification_blueprint
     @request.env['HTTP_REFERER'] = '/discovery_rules'
-    FactoryBot.create(:subnet, :network => "192.168.100.1", :mask => "255.255.255.0", :locations => [location_one], :organizations => [organization_one])
+    FactoryBot.create(:subnet, :dhcp, :network => "192.168.100.1", :mask => "255.255.255.0", :locations => [location_one], :organizations => [organization_one])
     @facts = {
       "interfaces"             => "lo,eth0",
       "ipaddress"              => "192.168.100.42",
@@ -115,7 +115,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
       } }, session: set_session_user_default_manager
     # all inherit buttons are pressed
     assert_select('button[name=is_overridden_btn]') do |e|
-      e.attribute("class") =~ /active/
+      e.attribute("class").to_s =~ /active/
     end
     # particular fields are set
     assert_select '#host_hostgroup_id [selected]' do |e|
@@ -140,6 +140,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
 
   def test_update_inheritance
     host = discover_host_from_facts(@facts)
+    ForemanDiscovery::HostConverter.stubs(:unused_ip_for_subnet).returns(host.ip)
     hostgroup = setup_hostgroup(host)
     put :update, params: {
       commit: 'Update',
@@ -186,7 +187,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
     ::ForemanDiscovery::NodeAPI::PowerService.any_instance.expects(:reboot).returns(false)
     post :reboot, params: { :id => host.id }, session: set_session_user_default_manager
     assert_redirected_to discovered_hosts_url
-    assert_equal "Failed to reboot host #{host.name}", flash[:error]
+    assert_match(/ERF42-4036/, flash[:error])
   end
 
   def test_reboot_error
@@ -195,7 +196,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
     ::ForemanDiscovery::NodeAPI::PowerService.any_instance.expects(:reboot).raises("request failed")
     post :reboot, params: { :id => host.id }, session: set_session_user_default_manager
     assert_redirected_to discovered_hosts_url
-    assert_match(/ERF50-4973/, flash[:error])
+    assert_match(/ERF42-4036/, flash[:error])
   end
 
   def test_auto_provision_success
@@ -274,7 +275,7 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
     ::ForemanDiscovery::NodeAPI::PowerService.any_instance.expects(:reboot).returns(false)
     post :submit_multiple_reboot, params: {:host_ids => host.id}, session: set_session_user(User.current)
     assert_redirected_to discovered_hosts_url
-    assert_equal "Errors during reboot: #{host.name}: failed to reboot", flash[:error]
+    assert_match(/ERF42-4036/, flash[:error])
     assert_nil flash[:success]
   end
 
@@ -284,12 +285,15 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
     ::ForemanDiscovery::NodeAPI::PowerService.any_instance.expects(:reboot).raises("request failed")
     post :submit_multiple_reboot, params: {:host_ids => host.id}, session: set_session_user(User.current)
     assert_redirected_to discovered_hosts_url
-    assert_match(/ERF50-4973/, flash[:error])
+    assert_match(/ERF42-4036/, flash[:error])
     assert_nil flash[:success]
   end
 
   def test_no_dns_rebuild_if_dns_pending
     host = discover_host_from_facts(@facts)
+    ForemanDiscovery::HostConverter.stubs(:unused_ip_for_subnet).returns(host.ip)
+    Nic::Managed.any_instance.stubs(:dhcp_update_required?).returns(false)
+    Net::DHCP::Record.any_instance.stubs(:conflicting?).returns(false)
     hostgroup = prepare_hostgroup_for_dns_rebuild(host)
     Nic::Managed.any_instance.expects(:rebuild_dns).never
     Host::Managed.any_instance.stubs(:skip_orchestration?).returns(false)
@@ -306,6 +310,9 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
 
   def test_dns_rebuild
     host = prepare_host_for_dns_rebuild
+    ForemanDiscovery::HostConverter.stubs(:unused_ip_for_subnet).returns(host.ip)
+    Nic::Managed.any_instance.stubs(:dhcp_update_required?).returns(false)
+    Net::DHCP::Record.any_instance.stubs(:conflicting?).returns(false)
     hostgroup = prepare_hostgroup_for_dns_rebuild(host)
     Nic::Managed.any_instance.expects(:rebuild_dns)
     Host::Managed.any_instance.stubs(:skip_orchestration?).returns(false)
@@ -319,6 +326,8 @@ class DiscoveredHostsControllerTest < ActionController::TestCase
 
   def test_dns_rebuild_with_auto_provision
     host = prepare_host_for_dns_rebuild
+    ForemanDiscovery::HostConverter.stubs(:unused_ip_for_subnet).returns(host.ip)
+    Nic::Managed.any_instance.stubs(:dhcp_update_required?).returns(false)
     hostgroup = prepare_hostgroup_for_dns_rebuild(host)
     Nic::Managed.any_instance.expects(:rebuild_dns)
     Host::Managed.any_instance.stubs(:skip_orchestration?).returns(false)

data/test/integration/discovered_hosts_test.rb

@@ -10,6 +10,7 @@ class DiscoveredHostsTest < IntegrationTestWithJavascript
     discovered_host.save!
     visit discovered_hosts_path
     page.evaluate_script("document.getElementById('fixedPropertiesSelector-#{discovered_host.id}').classList.remove('fade')")
+    ::ForemanDiscovery::HostConverter.stubs(:unused_ip_for_host)
   end
 
   teardown do

data/test/test_helper_discovery.rb

@@ -133,6 +133,18 @@ def facts_simple_network100_42
   }
 end
 
+def facts_network_2001_db8
+  {
+    "interfaces"       => "lo,eth0,eth1",
+    "ipaddress6"        => "2001:db8::1",
+    "ipaddress6_eth0"   => "2001:db8::1",
+    "ipaddress6_eth1"   => "2001:db9::1",
+    "macaddress_eth0"  => "AA:BB:CC:DD:EE:FA",
+    "macaddress_eth1"  => "AA:BB:CC:DD:EE:FB",
+    "discovery_bootif" => "AA:BB:CC:DD:EE:FA",
+  }
+end
+
 def discover_host_from_facts(facts)
   User.as_anonymous_admin do
     Host::Discovered.import_host(facts)

data/test/unit/discovered_extensions_test.rb

@@ -7,7 +7,10 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
 
   setup do
     @facts = facts_simple_network100_42
+    @facts_ipv6 = facts_network_2001_db8
     set_default_settings
+    ProxyAPI::DHCP.any_instance.stubs(:record).returns(nil)
+    ::ForemanDiscovery::HostConverter.stubs(:unused_ip_for_host)
   end
 
   test "no rule is found for empty rule set" do
@@ -179,6 +182,57 @@ class DiscoveredExtensionsTest < ActiveSupport::TestCase
     assert_equal hostgroup.puppet_ca_proxy, managed_host.puppet_ca_proxy
   end
 
+  class StubIPAM
+    def suggest_ip
+      "192.168.101.13"
+    end
+  end
+
+  test "subnet is changed and unused_ip called" do
+    ::ForemanDiscovery::HostConverter.unstub(:unused_ip_for_host)
+    facts = @facts.merge({"somefact" => "abc"})
+    domain = FactoryBot.create(:domain)
+    subnet = FactoryBot.create(:subnet_ipv4, :tftp, :dhcp, :name => 'subnet_100', :network => '192.168.100.0', :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
+    subnet2 = FactoryBot.create(:subnet_ipv4, :tftp, :dhcp, :name => 'subnet_101', :network => '192.168.101.0', :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
+    host = discover_host_from_facts(facts)
+    assert_equal subnet, host.subnet
+    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet => subnet2, :domain => domain)
+    r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
+    Subnet.any_instance.expects(:unused_ip).with(host.mac).returns(StubIPAM.new)
+    host.primary_interface.stubs(:queue_tftp)
+    host.primary_interface.stubs(:queue_dhcp)
+    managed_host = perform_auto_provision(host, r1)
+    assert_empty host.errors
+    assert managed_host
+    assert_empty managed_host.errors
+    refute_nil hostgroup.subnet
+    refute_nil managed_host.subnet
+    assert_equal hostgroup.subnet, managed_host.subnet
+    assert_equal "192.168.101.13", managed_host.ip
+  end
+
+  test "subnet6 is changed and unused_ip called" do
+    ::ForemanDiscovery::HostConverter.unstub(:unused_ip_for_host)
+    facts = @facts_ipv6.merge({"somefact" => "abc"})
+    domain = FactoryBot.create(:domain)
+    subnet = FactoryBot.create(:subnet_ipv6, :tftp, :dhcp, :network => "2001:db8::/32", :mask => "ffff:ffff::", :name => "ipv6_discovered", :ipam => IPAM::MODES[:eui64], :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
+    subnet2 = FactoryBot.create(:subnet_ipv6, :tftp, :dhcp, :network => "2001:db9::/32", :mask => "ffff:ffff::", :name => "ipv6_provision", :ipam => IPAM::MODES[:eui64], :organizations => [Organization.find_by_name("Organization 1")], :locations => [Location.find_by_name("Location 1")])
+    host = discover_host_from_facts(facts)
+    assert_equal subnet, host.subnet6
+    hostgroup = FactoryBot.create(:hostgroup, :with_environment, :with_rootpass, :with_puppet_orchestration, :with_os, :pxe_loader => "PXELinux BIOS", :subnet6 => subnet2, :domain => domain)
+    r1 = FactoryBot.create(:discovery_rule, :priority => 1, :search => "facts.somefact = abc", :organizations => [host.organization], :locations => [host.location], :hostgroup => hostgroup)
+    host.primary_interface.stubs(:queue_tftp)
+    host.primary_interface.stubs(:queue_dhcp)
+    managed_host = perform_auto_provision(host, r1)
+    assert_empty host.errors
+    assert managed_host
+    assert_empty managed_host.errors
+    refute_nil hostgroup.subnet6
+    refute_nil managed_host.subnet6
+    assert_equal hostgroup.subnet6, managed_host.subnet6
+    assert_equal "2001:db9::a8bb:ccff:fedd:eefa", managed_host.ip6
+  end
+
   test "attributes from hostgroup are copied after auto provisioning for host without subnet detected" do
     facts = @facts.merge({"somefact" => "abc"})
     host = discover_host_from_facts(facts)