foreman_puppet 8.1.1 → 9.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3c393e0ec019e0b5a8b7083811a4d24b34b71db9f50f1935aca40dc034b8525e
-  data.tar.gz: d376a55c7a7582fc8aabbd70b244b1fd9b7a3b53188aedb706bbc8306f044836
+  metadata.gz: f592b421e0f61e5b8720f7e392368a711100d865cc089ac65682ddb7a6cd9434
+  data.tar.gz: 91a9a5b62eae0c8632da7cacc3c468e227b0ca997f98dc2c66a710ee8102ebc7
 SHA512:
-  metadata.gz: 8736e728664cb79bc455a73016d006cbd5ce1afae7617f8f59e70aea2bb9c1ad90fb5f51d824bf25ca729f0768bc455f61f3a1294039aa3848a323f859e6bb07
-  data.tar.gz: 66ffa66d6745c3e464023b70654bb5e357c692df02a553ca7f24e4f9a633b4752762c773805928006e1b9b87d11be705aaf04bfc203cb665aff4c5cb5927b34f
+  metadata.gz: 953d65c70056b2ba2fa6e7d4e8e8ecff19e90a5d2fdb14bff1c14b43ff480546250a7c286fa75f47277e8bd2aba4246916cc01319a502d7356b3121fcd905678
+  data.tar.gz: 17dd15952387f5664c6a9639b8c0f6b4686d7e8165ca851eb65736f3ca2e031f0d4964868a8a9993a6d6d71af8488f26618c2b657507be4f1988cabffb36cfaa

data/app/controllers/concerns/foreman_puppet/extensions/api_smart_proxies_controller.rb

@@ -140,7 +140,7 @@ module ForemanPuppet
       end
 
       def find_optional_environment
-        @environment = Environment.authorized(:view_environments).find(@env_id) if @env_id
+        @environment = resource_finder(Environment.authorized(:view_environments), @env_id) if @env_id
       rescue ActiveRecord::RecordNotFound => e
         Foreman::Logging.exception('Resource not found', e, level: :debug)
         nil

data/app/services/foreman_puppet/puppet_class_importer.rb

@@ -229,7 +229,17 @@ module ForemanPuppet
     end
 
     def load_ignored_file
-      File.exist?(ignored_file_path) ? YAML.load_file(ignored_file_path) : {}
+      if File.exist?(ignored_file_path)
+        # Ruby 3+
+        if YAML.respond_to(:safe_load_file)
+          YAML.safe_load_file(ignored_file_path, permitted_classes: [Symbol, Regexp])
+        else
+          # NOTE: Once we drop Ruby 2.7 support, you can drop the load_file call.
+          YAML.load_file(ignored_file_path)
+        end
+      else
+        {}
+      end
     end
 
     def ignored_file

data/lib/foreman_puppet/version.rb

@@ -1,3 +1,3 @@
 module ForemanPuppet
-  VERSION = '8.1.1'.freeze
+  VERSION = '9.0.0'.freeze
 end

data/test/integration/foreman_puppet/host_js_test.rb

@@ -23,7 +23,7 @@ module ForemanPuppet
         wait_for_ajax
         click_on_inherit('puppet_attributes_environment')
         select2(overridden_hostgroup.name, from: 'host_hostgroup_id')
-        assert page.find('#s2id_host_puppet_attributes_environment_id .select2-chosen').has_text? overridden_hostgroup.puppet.environment.name
+        assert page.find(select2_selector('host_puppet_attributes_environment_id')).has_text? overridden_hostgroup.puppet.environment.name
       end
 
       test 'sets fields to "inherit" when hostgroup is selected' do
@@ -137,15 +137,15 @@ module ForemanPuppet
         visit edit_host_path(host)
         select2(original_hostgroup.name, from: 'host_hostgroup_id')
 
-        assert_equal original_hostgroup.puppet_proxy.name, find('#s2id_host_puppet_proxy_id .select2-chosen').text
+        assert select2_chosen_selector('host_puppet_proxy_id').has_text? original_hostgroup.puppet_proxy.name
 
         click_on_inherit('puppet_proxy')
         select2(overridden_hostgroup.name, from: 'host_hostgroup_id')
 
-        assert find('#s2id_host_puppet_attributes_environment_id .select2-chosen').has_text? original_hostgroup.puppet.environment.name
+        assert select2_chosen_selector('host_puppet_attributes_environment_id').has_text? original_hostgroup.puppet.environment.name
 
         # On host group change, the disabled select will be reset to an empty value - disabled select2 is invisible on chrome
-        assert find('#s2id_host_puppet_proxy_id .select2-chosen', visible: :all).has_text? ''
+        assert find(select2_selector('host_puppet_proxy_id'), visible: :all).has_text? ''
       end
 
       context 'has inherited Puppetclasses' do
@@ -201,14 +201,14 @@ module ForemanPuppet
           switch_form_tab('Puppet ENC')
           assert puppetclass_params.has_selector?("a[data-tag='remove']", visible: :visible)
           assert puppetclass_params.has_selector?("a[data-tag='override']", visible: :hidden)
-          assert_equal('false', find("#s2id_host_lookup_values_attributes_#{lookup_key.id}_value .select2-chosen").text)
+          assert select2_chosen_selector("host_lookup_values_attributes_#{lookup_key.id}_value").has_text? 'false'
           select2 'true', from: "host_lookup_values_attributes_#{lookup_key.id}_value"
           click_button('Submit')
           find('h5', text: /#{host.name}/)
 
           visit edit_host_path(host)
           switch_form_tab('Puppet ENC')
-          assert_equal('true', find("#s2id_host_lookup_values_attributes_#{lookup_key.id}_value .select2-chosen").text)
+          assert select2_chosen_selector("host_lookup_values_attributes_#{lookup_key.id}_value").has_text? 'true'
         end
 
         test 'class parameters and overrides are displayed correctly for strings' do

data/webpack/src/Extends/Host/PuppetTab/SubTabs/Reports/components/ConfigStatusCard/index.js

@@ -10,13 +10,7 @@ import {
   DescriptionListGroup,
   DescriptionListDescription,
 } from '@patternfly/react-core';
-import {
-  TableComposable,
-  TableText,
-  Tr,
-  Tbody,
-  Td,
-} from '@patternfly/react-table';
+import { Table, TableText, Tr, Tbody, Td } from '@patternfly/react-table';
 import { useSelector, useDispatch } from 'react-redux';
 import { selectAPIResponse } from 'foremanReact/redux/API/APISelectors';
 import CardTemplate from 'foremanReact/components/HostDetails/Templates/CardItem/CardTemplate';
@@ -96,7 +90,7 @@ const createPuppetMetricsTableElement = (name, value = '--') => (
 );
 
 const createPuppetMetricsTable = (metrics = undefined) => (
-  <TableComposable
+  <Table
     aria-label="foreman puppet metrics table"
     variant="compact"
     borders="compactBorderless"
@@ -104,12 +98,12 @@ const createPuppetMetricsTable = (metrics = undefined) => (
     key="foreman-puppet-metrics-table"
   >
     <Tbody>
-      <Tr>
+      <Tr ouiaId="foreman-puppet-metrics-row-1">
         {createPuppetMetricsTableElement(__('Failed'), metrics.failed)}
         {createPuppetMetricsTableElement(__('Changed'), metrics.changed)}
         {createPuppetMetricsTableElement(__('Scheduled'), metrics.scheduled)}
       </Tr>
-      <Tr>
+      <Tr ouiaId="foreman-puppet-metrics-row-2">
         {createPuppetMetricsTableElement(
           __('Failed to start'),
           metrics.failed_to_start
@@ -120,7 +114,7 @@ const createPuppetMetricsTable = (metrics = undefined) => (
           metrics.corrective_change
         )}
       </Tr>
-      <Tr>
+      <Tr ouiaId="foreman-puppet-metrics-row-3">
         {createPuppetMetricsTableElement(__('Skipped'), metrics.skipped)}
         {createPuppetMetricsTableElement(
           __('Out of sync'),
@@ -129,7 +123,7 @@ const createPuppetMetricsTable = (metrics = undefined) => (
         {createPuppetMetricsTableElement(__('Total'), metrics.total)}
       </Tr>
     </Tbody>
-  </TableComposable>
+  </Table>
 );
 
 const ConfigStatusCard = ({ hostName, parentStatus }) => {

data/webpack/src/Extends/Host/PuppetTab/SubTabs/Reports/styles.scss

@@ -1,7 +1,7 @@
 .report-tab {
   padding: 16px 24px;
 
-  .pf-c-card {
+  .pf-v5-c-card {
     height: 100%;
   }
 }

data/webpack/src/Extends/Host/PuppetTab/styles.scss

@@ -1,5 +1,5 @@
 
-.pf-c-tabs.pf-m-secondary.foreman-puppet-host-details-tab {
+.pf-v5-c-tabs.pf-m-secondary.foreman-puppet-host-details-tab {
   overflow: visible;
 }
 

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_puppet
 version: !ruby/object:Gem::Version
-  version: 8.1.1
+  version: 9.0.0
 platform: ruby
 authors:
 - Ondřej Ezr
 - Shira Maximov
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-20 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies: []
 description: Allow assigning Puppet environments and classes to the Foreman Hosts.
 email:
@@ -238,83 +237,45 @@ files:
 - lib/tasks/foreman_puppet_tasks.rake
 - locale/Makefile
 - locale/ca/LC_MESSAGES/foreman_puppet.mo
-- locale/ca/foreman_puppet.edit.po
 - locale/ca/foreman_puppet.po
-- locale/ca/foreman_puppet.po.time_stamp
 - locale/cs_CZ/LC_MESSAGES/foreman_puppet.mo
-- locale/cs_CZ/foreman_puppet.edit.po
 - locale/cs_CZ/foreman_puppet.po
-- locale/cs_CZ/foreman_puppet.po.time_stamp
 - locale/de/LC_MESSAGES/foreman_puppet.mo
-- locale/de/foreman_puppet.edit.po
 - locale/de/foreman_puppet.po
-- locale/de/foreman_puppet.po.time_stamp
 - locale/en/LC_MESSAGES/foreman_puppet.mo
-- locale/en/foreman_puppet.edit.po
 - locale/en/foreman_puppet.po
-- locale/en/foreman_puppet.po.time_stamp
 - locale/en_GB/LC_MESSAGES/foreman_puppet.mo
-- locale/en_GB/foreman_puppet.edit.po
 - locale/en_GB/foreman_puppet.po
-- locale/en_GB/foreman_puppet.po.time_stamp
 - locale/es/LC_MESSAGES/foreman_puppet.mo
-- locale/es/foreman_puppet.edit.po
 - locale/es/foreman_puppet.po
-- locale/es/foreman_puppet.po.time_stamp
 - locale/foreman_puppet.pot
 - locale/fr/LC_MESSAGES/foreman_puppet.mo
-- locale/fr/foreman_puppet.edit.po
 - locale/fr/foreman_puppet.po
-- locale/fr/foreman_puppet.po.time_stamp
 - locale/gemspec.rb
 - locale/gl/LC_MESSAGES/foreman_puppet.mo
-- locale/gl/foreman_puppet.edit.po
 - locale/gl/foreman_puppet.po
-- locale/gl/foreman_puppet.po.time_stamp
 - locale/it/LC_MESSAGES/foreman_puppet.mo
-- locale/it/foreman_puppet.edit.po
 - locale/it/foreman_puppet.po
-- locale/it/foreman_puppet.po.time_stamp
 - locale/ja/LC_MESSAGES/foreman_puppet.mo
-- locale/ja/foreman_puppet.edit.po
 - locale/ja/foreman_puppet.po
-- locale/ja/foreman_puppet.po.time_stamp
 - locale/ka/LC_MESSAGES/foreman_puppet.mo
-- locale/ka/foreman_puppet.edit.po
 - locale/ka/foreman_puppet.po
-- locale/ka/foreman_puppet.po.time_stamp
 - locale/ko/LC_MESSAGES/foreman_puppet.mo
-- locale/ko/foreman_puppet.edit.po
 - locale/ko/foreman_puppet.po
-- locale/ko/foreman_puppet.po.time_stamp
 - locale/nl_NL/LC_MESSAGES/foreman_puppet.mo
-- locale/nl_NL/foreman_puppet.edit.po
 - locale/nl_NL/foreman_puppet.po
-- locale/nl_NL/foreman_puppet.po.time_stamp
 - locale/pl/LC_MESSAGES/foreman_puppet.mo
-- locale/pl/foreman_puppet.edit.po
 - locale/pl/foreman_puppet.po
-- locale/pl/foreman_puppet.po.time_stamp
 - locale/pt_BR/LC_MESSAGES/foreman_puppet.mo
-- locale/pt_BR/foreman_puppet.edit.po
 - locale/pt_BR/foreman_puppet.po
-- locale/pt_BR/foreman_puppet.po.time_stamp
 - locale/ru/LC_MESSAGES/foreman_puppet.mo
-- locale/ru/foreman_puppet.edit.po
 - locale/ru/foreman_puppet.po
-- locale/ru/foreman_puppet.po.time_stamp
 - locale/sv_SE/LC_MESSAGES/foreman_puppet.mo
-- locale/sv_SE/foreman_puppet.edit.po
 - locale/sv_SE/foreman_puppet.po
-- locale/sv_SE/foreman_puppet.po.time_stamp
 - locale/zh_CN/LC_MESSAGES/foreman_puppet.mo
-- locale/zh_CN/foreman_puppet.edit.po
 - locale/zh_CN/foreman_puppet.po
-- locale/zh_CN/foreman_puppet.po.time_stamp
 - locale/zh_TW/LC_MESSAGES/foreman_puppet.mo
-- locale/zh_TW/foreman_puppet.edit.po
 - locale/zh_TW/foreman_puppet.po
-- locale/zh_TW/foreman_puppet.po.time_stamp
 - package.json
 - test/controllers/foreman_puppet/api/v2/config_groups_controller_test.rb
 - test/controllers/foreman_puppet/api/v2/environments_controller_test.rb
@@ -426,7 +387,6 @@ homepage: https://github.com/theforeman/foreman_puppet
 licenses:
 - GPL-3.0
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -444,8 +404,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.23
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Add Puppet features to Foreman
 test_files: