foreman_salt 13.2.2 → 14.1.0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_salt
 version: !ruby/object:Gem::Version
-  version: 13.2.2
+  version: 14.1.0
 platform: ruby
 authors:
 - Stephen Benjamin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-21 00:00:00.000000000 Z
+date: 2021-11-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -25,47 +25,47 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: foreman-tasks
+  name: foreman_remote_execution
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: 2.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0.8'
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
-  name: foreman_remote_execution
+  name: foreman-tasks
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: '0.8'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: theforeman-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.71.0
+        version: 0.0.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.71.0
+        version: 0.0.6
 description: Foreman Plug-in for Salt
 email:
 - stephen@redhat.com
@@ -82,6 +82,7 @@ files:
 - app/controllers/foreman_salt/api/v2/jobs_controller.rb
 - app/controllers/foreman_salt/api/v2/salt_autosign_controller.rb
 - app/controllers/foreman_salt/api/v2/salt_environments_controller.rb
+- app/controllers/foreman_salt/api/v2/salt_hostgroups_controller.rb
 - app/controllers/foreman_salt/api/v2/salt_keys_controller.rb
 - app/controllers/foreman_salt/api/v2/salt_minions_controller.rb
 - app/controllers/foreman_salt/api/v2/salt_states_controller.rb
@@ -90,6 +91,7 @@ files:
 - app/controllers/foreman_salt/concerns/hostgroups_controller_extensions.rb
 - app/controllers/foreman_salt/concerns/hosts_controller_extensions.rb
 - app/controllers/foreman_salt/concerns/salt_environment_parameters.rb
+- app/controllers/foreman_salt/concerns/salt_hostgroup_parameters.rb
 - app/controllers/foreman_salt/concerns/salt_module_parameters.rb
 - app/controllers/foreman_salt/concerns/salt_override_value.rb
 - app/controllers/foreman_salt/concerns/salt_variable_parameters.rb
@@ -110,21 +112,19 @@ files:
 - app/lib/proxy_api/salt.rb
 - app/models/foreman_salt/concerns/host_managed_extensions.rb
 - app/models/foreman_salt/concerns/hostgroup_extensions.rb
-- app/models/foreman_salt/fact_name.rb
 - app/models/foreman_salt/host_salt_module.rb
 - app/models/foreman_salt/hostgroup_salt_module.rb
 - app/models/foreman_salt/salt_environment.rb
 - app/models/foreman_salt/salt_module.rb
 - app/models/foreman_salt/salt_module_environment.rb
 - app/models/foreman_salt/salt_provider.rb
+- app/models/foreman_salt/salt_status.rb
 - app/models/foreman_salt/salt_variable.rb
 - app/models/setting/salt.rb
 - app/overrides/salt_environment_host_selector.rb
 - app/overrides/salt_environment_hostgroup_selector.rb
 - app/overrides/salt_modules_selector.rb
 - app/overrides/salt_proxy_selector.rb
-- app/services/foreman_salt/fact_importer.rb
-- app/services/foreman_salt/fact_parser.rb
 - app/services/foreman_salt/report_importer.rb
 - app/services/foreman_salt/smart_proxies/salt_keys.rb
 - app/views/foreman_salt/api/v2/salt_autosign/base.json.rabl
@@ -136,6 +136,9 @@ files:
 - app/views/foreman_salt/api/v2/salt_environments/index.json.rabl
 - app/views/foreman_salt/api/v2/salt_environments/main.json.rabl
 - app/views/foreman_salt/api/v2/salt_environments/show.json.rabl
+- app/views/foreman_salt/api/v2/salt_hostgroups/base.json.rabl
+- app/views/foreman_salt/api/v2/salt_hostgroups/show.json.rabl
+- app/views/foreman_salt/api/v2/salt_hostgroups/update.json.rabl
 - app/views/foreman_salt/api/v2/salt_keys/base.json.rabl
 - app/views/foreman_salt/api/v2/salt_keys/index.json.rabl
 - app/views/foreman_salt/api/v2/salt_keys/main.json.rabl
@@ -179,6 +182,7 @@ files:
 - app/views/foreman_salt/salt_variables/new.html.erb
 - app/views/hosts/select_multiple_salt_environment.html.erb
 - app/views/hosts/select_multiple_salt_master.html.erb
+- config/api_routes.rb
 - config/routes.rb
 - db/migrate/20140813081913_add_salt_proxy_to_host_and_host_group.rb
 - db/migrate/20140817210214_create_salt_modules.rb
@@ -190,6 +194,8 @@ files:
 - db/migrate/20150509101505_add_primary_keys.rb
 - db/migrate/20161103104146_add_index_to_join_tables.rb
 - db/migrate/20190515112233_add_salt_module_id_to_lookup_keys.rb
+- db/migrate/20210312150333_add_salt_autosign_to_host.rb
+- db/migrate/20211108211312_add_missing_permissions.rb
 - db/seeds.d/75-salt_seeds.rb
 - db/seeds.d/76-job_templates.rb
 - lib/foreman_salt.rb
@@ -200,6 +206,10 @@ files:
 - lib/tasks/foreman_salt_tasks.rake
 - locale/Makefile
 - locale/action_names.rb
+- locale/ca/LC_MESSAGES/foreman_salt.mo
+- locale/ca/foreman_salt.po
+- locale/cs_CZ/LC_MESSAGES/foreman_salt.mo
+- locale/cs_CZ/foreman_salt.po
 - locale/de/LC_MESSAGES/foreman_salt.mo
 - locale/de/foreman_salt.edit.po
 - locale/de/foreman_salt.po
@@ -208,10 +218,39 @@ files:
 - locale/en/foreman_salt.edit.po
 - locale/en/foreman_salt.po
 - locale/en/foreman_salt.po.time_stamp
+- locale/en_GB/LC_MESSAGES/foreman_salt.mo
+- locale/en_GB/foreman_salt.po
+- locale/es/LC_MESSAGES/foreman_salt.mo
+- locale/es/foreman_salt.po
 - locale/foreman_salt.pot
+- locale/fr/LC_MESSAGES/foreman_salt.mo
+- locale/fr/foreman_salt.po
+- locale/gl/LC_MESSAGES/foreman_salt.mo
+- locale/gl/foreman_salt.po
+- locale/it/LC_MESSAGES/foreman_salt.mo
+- locale/it/foreman_salt.po
+- locale/ja/LC_MESSAGES/foreman_salt.mo
+- locale/ja/foreman_salt.po
+- locale/ko/LC_MESSAGES/foreman_salt.mo
+- locale/ko/foreman_salt.po
+- locale/nl_NL/LC_MESSAGES/foreman_salt.mo
+- locale/nl_NL/foreman_salt.po
+- locale/pl/LC_MESSAGES/foreman_salt.mo
+- locale/pl/foreman_salt.po
+- locale/pt_BR/LC_MESSAGES/foreman_salt.mo
+- locale/pt_BR/foreman_salt.po
+- locale/ru/LC_MESSAGES/foreman_salt.mo
+- locale/ru/foreman_salt.po
+- locale/sv_SE/LC_MESSAGES/foreman_salt.mo
+- locale/sv_SE/foreman_salt.po
+- locale/zh_CN/LC_MESSAGES/foreman_salt.mo
+- locale/zh_CN/foreman_salt.po
+- locale/zh_TW/LC_MESSAGES/foreman_salt.mo
+- locale/zh_TW/foreman_salt.po
 - test/factories/foreman_salt_factories.rb
 - test/functional/api/v2/salt_autosign_controller_test.rb
 - test/functional/api/v2/salt_environments_controller_test.rb
+- test/functional/api/v2/salt_hostgroups_controller_test.rb
 - test/functional/api/v2/salt_keys_controller_test.rb
 - test/functional/api/v2/salt_states_controller_test.rb
 - test/functional/api/v2/salt_variables_controller_test.rb
@@ -226,15 +265,12 @@ files:
 - test/integration/salt_module_test.rb
 - test/integration/salt_variable_test.rb
 - test/test_plugin_helper.rb
-- test/unit/grains_centos.json
 - test/unit/grains_importer_test.rb
 - test/unit/highstate.json
 - test/unit/highstate_pchanges.json
 - test/unit/host_extensions_test.rb
 - test/unit/hostgroup_extensions_test.rb
 - test/unit/report_importer_test.rb
-- test/unit/salt_fact_importer_test.rb
-- test/unit/salt_fact_parser_test.rb
 - test/unit/salt_keys_test.rb
 - test/unit/salt_modules_test.rb
 - test/unit/salt_variables_test.rb
@@ -274,10 +310,7 @@ test_files:
 - test/unit/highstate_pchanges.json
 - test/unit/hostgroup_extensions_test.rb
 - test/unit/highstate.json
-- test/unit/salt_fact_parser_test.rb
 - test/unit/salt_modules_test.rb
-- test/unit/grains_centos.json
-- test/unit/salt_fact_importer_test.rb
 - test/unit/report_importer_test.rb
 - test/unit/salt_variables_test.rb
 - test/unit/salt_keys_test.rb
@@ -289,6 +322,7 @@ test_files:
 - test/functional/api/v2/salt_variables_controller_test.rb
 - test/functional/api/v2/salt_states_controller_test.rb
 - test/functional/api/v2/salt_environments_controller_test.rb
+- test/functional/api/v2/salt_hostgroups_controller_test.rb
 - test/functional/api/v2/salt_keys_controller_test.rb
 - test/functional/salt_modules_controller_test.rb
 - test/functional/hosts_controller_test.rb

data/app/models/foreman_salt/fact_name.rb

@@ -1,14 +0,0 @@
-module ForemanSalt
-  # Define the class that fact names that come from Salt should have
-  # It allows us to filter facts by origin, and also to display the origin
-  # in the fact values table (/fact_values)
-  class FactName < ::FactName
-    def origin
-      'Salt'
-    end
-
-    def icon_path
-      'foreman_salt/Salt'
-    end
-  end
-end

data/app/services/foreman_salt/fact_importer.rb

@@ -1,15 +0,0 @@
-module ForemanSalt
-  class FactImporter < ::StructuredFactImporter
-    def fact_name_class
-      ForemanSalt::FactName
-    end
-
-    def self.support_background
-      true
-    end
-
-    def self.authorized_smart_proxy_features
-      'Salt'
-    end
-  end
-end

data/app/services/foreman_salt/fact_parser.rb

@@ -1,114 +0,0 @@
-module ForemanSalt
-  class FactParser < ::FactParser
-    attr_reader :facts
-
-    def operatingsystem
-      os = Operatingsystem.where(os_hash).first_or_initialize
-      if os.new_record?
-        os.deduce_family
-        os.release_name = facts[:lsb_distrib_codename]
-        os.save
-      end
-      os if os.persisted?
-    end
-
-    def architecture
-      name = facts[:osarch]
-      name = 'x86_64' if name == 'amd64'
-      Architecture.where(:name => name).first_or_create unless name.blank?
-    end
-
-    def environment
-      # Don't touch the Puppet environment field
-    end
-
-    def model
-      name = facts[:productname]
-      Model.where(:name => name.strip).first_or_create unless name.blank?
-    end
-
-    def domain
-      name = facts[:domain]
-      Domain.where(:name => name).first_or_create unless name.blank?
-    end
-
-    def ip
-      ip = facts.find { |fact, value| fact =~ /^fqdn_ip4/ && value && value != '127.0.0.1' }
-      ip[1] if ip
-    end
-
-    def primary_interface
-      interface = interfaces.find { |_, value| value[:ipaddress] == ip }
-      interface[0] if interface
-    end
-
-    def mac
-      interface = interfaces.find { |_, value| value[:ipaddress] == ip }
-      interface[1][:macaddress] if interface
-    end
-
-    def ipmi_interface
-      nil
-    end
-
-    def interfaces
-      interfaces = {}
-
-      facts.each do |fact, value|
-        next unless value && fact.to_s =~ /^ip_interfaces/
-
-        (_, interface_name) = fact.split(FactName::SEPARATOR)
-
-        next if (IPAddr.new('fe80::/10').include?(value) rescue false)
-
-        if !interface_name.blank? && interface_name != 'lo'
-          interface = interfaces.fetch(interface_name, {})
-          interface[:macaddress] = macs[interface_name]
-          if Net::Validations.validate_ip6(value)
-            interface[:ipaddress6] = value unless interface.include?(:ipaddress6)
-          else
-            interface[:ipaddress] = value unless interface.include?(:ipaddress)
-          end
-          interfaces[interface_name] = interface
-        end
-      end
-
-      interfaces.each do |name, interface|
-        set_additional_attributes(interface, name)
-      end
-
-      interfaces
-    end
-
-    def support_interfaces_parsing?
-      true
-    end
-
-    private
-
-    def os_hash
-      name = facts[:os]
-      (_, major, minor, sub) = /(\d+)\.?(\d+)?\.?(\d+)?/.match(facts[:osrelease]).to_a
-      if name == 'CentOS'
-        if sub
-          minor += '.' + sub
-        end
-      end
-      { :name => name, :major => major, :minor => minor }
-    end
-
-    def macs
-      unless @macs
-        @macs = {}
-        facts.each do |fact, value|
-          next unless value && fact.to_s =~ /^hwaddr_interfaces/
-
-          data = fact.split(FactName::SEPARATOR)
-          interface = data[1]
-          macs[interface] = value
-        end
-      end
-      @macs
-    end
-  end
-end

data/test/unit/grains_centos.json

@@ -1,113 +0,0 @@
-{
-  "facts": {
-    "kernelrelease": "2.6.32-431.23.3.el6.x86_64",
-    "ipv6::0": "::1",
-    "ipv6::1": "fe80::5054:ff:fe35:302a",
-    "saltversioninfo": [
-      2014,
-      1,
-      7
-    ],
-    "operatingsystem": "CentOS",
-    "lsb_distrib_id": "CentOS",
-    "pythonversion::2": 6,
-    "pythonversion::3": "final",
-    "pythonversion::0": 2,
-    "pythonversion::1": 6,
-    "cpu_model": "QEMU Virtual CPU version (cpu64-rhel6)",
-    "pythonversion::4": 0,
-    "oscodename": "Final",
-    "num_gpus": 1,
-    "productname": "KVM",
-    "osarch": "x86_64",
-    "biosversion": "0.5.1",
-    "kernel": "Linux",
-    "domain": "example.com",
-    "pythonpath::4": "/usr/lib64/python2.6/lib-tk",
-    "pythonpath::5": "/usr/lib64/python2.6/lib-old",
-    "pythonpath::6": "/usr/lib64/python2.6/lib-dynload",
-    "pythonpath::7": "/usr/lib64/python2.6/site-packages",
-    "pythonpath::0": "/usr/bin",
-    "mem_total": 742,
-    "pythonpath::2": "/usr/lib64/python2.6",
-    "pythonpath::3": "/usr/lib64/python2.6/plat-linux2",
-    "pythonpath::8": "/usr/lib/python2.6/site-packages",
-    "cpu_flags::25": "pni",
-    "cpu_flags::24": "unfair_spinlock",
-    "cpu_flags::27": "hypervisor",
-    "cpu_flags::26": "cx16",
-    "cpu_flags::21": "nx",
-    "cpu_flags::20": "syscall",
-    "cpu_flags::23": "up",
-    "cpu_flags::22": "lm",
-    "cpu_flags::28": "lahf_lm",
-    "defaultlanguage": "en_US",
-    "osfullname": "CentOS",
-    "localhost": "saltclient01.example.com",
-    "lsb_distrib_release": "6.5",
-    "saltpath": "/usr/lib/python2.6/site-packages/salt",
-    "biosreleasedate": "01/01/2007",
-    "host": "saltclient01",
-    "defaultencoding": "UTF8",
-    "path": "/sbin:/usr/sbin:/bin:/usr/bin",
-    "ip_interfaces::eth0::0": "10.7.13.141",
-    "ip_interfaces::eth0::1": "10.7.23.141",
-    "manufacturer": "Red Hat",
-    "fqdn": "saltclient01.example.com",
-    "os": "CentOS",
-    "osfinger": "CentOS-6",
-    "ps": "ps -efH",
-    "server_id": 654860635,
-    "zmqversion": "3.2.4",
-    "osmajorrelease::1": "5",
-    "osmajorrelease::0": "6",
-    "ip_interfaces::lo::0": "127.0.0.1",
-    "master": "saltstack.example.com",
-    "shell": "/bin/sh",
-    "saltversion": "2014.1.7",
-    "hwaddr_interfaces::lo": "00:00:00:00:00:00",
-    "_timestamp": "2014-08-28 10:02:16 ",
-    "cpu_flags::18": "sse",
-    "cpu_flags::19": "sse2",
-    "cpu_flags::14": "pse36",
-    "cpu_flags::15": "clflush",
-    "cpu_flags::16": "mmx",
-    "cpu_flags::17": "fxsr",
-    "cpu_flags::10": "mtrr",
-    "cpu_flags::11": "pge",
-    "cpu_flags::12": "mca",
-    "cpu_flags::13": "cmov",
-    "cpuarch": "x86_64",
-    "pythonpath::1": "/usr/lib64/python26.zip",
-    "cpu_flags::6": "mce",
-    "cpu_flags::7": "cx8",
-    "cpu_flags::4": "msr",
-    "cpu_flags::5": "pae",
-    "cpu_flags::2": "pse",
-    "cpu_flags::3": "tsc",
-    "cpu_flags::0": "fpu",
-    "cpu_flags::1": "de",
-    "serialnumber": "Not Specified",
-    "hwaddr_interfaces::eth0": "52:54:00:35:30:2a",
-    "cpu_flags::8": "apic",
-    "cpu_flags::9": "sep",
-    "gpus::0::model": "GD 5446",
-    "id": "saltclient01.example.com",
-    "osrelease": "6.5",
-    "num_cpus": 1,
-    "virtual": "kvm",
-    "ipv4::0": "10.7.23.141",
-    "ipv4::1": "10.7.13.141",
-    "ipv4::2": "127.0.0.1",
-    "_type": "foreman_salt",
-    "nodename": "saltclient01.example.com",
-    "os_family": "RedHat",
-    "operatingsystemrelease": "6.5",
-    "gpus::0::vendor": "unknown",
-    "lsb_distrib_codename": "Final",
-    "fqdn_ip4::0": "10.7.13.141",
-    "ip_interfaces::eth1::0": "1.2.3.4",
-    "hwaddr_interfaces::eth1": "DE:AD:BE:EF:07:13"
-  },
-  "name": "saltclient01.example.com"
-}

data/test/unit/salt_fact_importer_test.rb

@@ -1,24 +0,0 @@
-require 'test_plugin_helper'
-
-module ForemanSalt
-  class SaltFactImpoterTest < ActiveSupport::TestCase
-    def setup
-      @host = FactoryBot.build(:host)
-    end
-
-    test 'should have fact set' do
-      importer = FactImporter.new(@host, 'a' => 'b')
-      assert_equal({ 'a' => 'b' }, importer.send(:facts))
-    end
-
-    test 'should have Salt as origin' do
-      importer = FactImporter.new(@host, 'a' => 'b')
-      importer.stubs(:ensure_no_active_transaction).returns(true)
-      importer.import!
-      imported_fact = FactName.find_by_name('a')
-      assert_equal 'a', imported_fact.name
-      assert_equal 'Salt', imported_fact.origin
-      assert_equal 'foreman_salt/Salt', imported_fact.icon_path
-    end
-  end
-end

data/test/unit/salt_fact_parser_test.rb

@@ -1,44 +0,0 @@
-require 'test_plugin_helper'
-
-module ForemanSalt
-  class SaltFactsParserTest < ActiveSupport::TestCase
-    def setup
-      grains = JSON.parse(File.read(File.join(Engine.root, 'test', 'unit', 'grains_centos.json')))
-      @facts_parser = FactParser.new grains["facts"]
-      User.current = users :admin
-    end
-
-    test "should return list of interfaces" do
-      assert @facts_parser.interfaces.present?
-      assert_not_nil @facts_parser.suggested_primary_interface(FactoryBot.build(:host))
-      assert @facts_parser.interfaces.key?(@facts_parser.suggested_primary_interface(FactoryBot.build(:host)).first)
-    end
-
-    test "should set operatingsystem correctly" do
-      os = @facts_parser.operatingsystem
-      assert os.present?
-      assert_equal 'CentOS', os.name
-      assert_equal '6', os.major
-      assert_equal '5', os.minor
-      assert_equal 'CentOS 6.5', os.title
-    end
-
-    test "should set domain correctly" do
-      domain = @facts_parser.domain
-      assert domain.present?
-      assert_equal 'example.com', domain.name
-    end
-
-    test "should set ip correctly" do
-      assert_equal '10.7.13.141', @facts_parser.ip
-    end
-
-    test "should set primary_interface correctly" do
-      assert_equal 'eth0', @facts_parser.primary_interface
-    end
-
-    test "should set mac correctly" do
-      assert_equal '52:54:00:35:30:2a', @facts_parser.mac
-    end
-  end
-end