foreman_puppet 10.1.3 → 11.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cee4c4bc88e7e50cc6215f4d63bce66361304d94687507338c89333f3bd81e39
-  data.tar.gz: db7d58c859b13521f37097955a30da8f23d8437934f1a307907a6ab6c73c6f6c
+  metadata.gz: 5a50beabd870a36bed2dbf49653e80e1766178fe79922553a227ce8ea54563c2
+  data.tar.gz: d68b2c95b690b4d5b9e38100c14d06ea0911de97587660708a0ab0b68e325e45
 SHA512:
-  metadata.gz: 5ffcc0c52303eb7b4e402351d56536449396888f98038756acf318ec38b0dd9616d003747d6edc987ebd08a7b68f8c46328052a468eea0e5cd8dc31c617dc1af
-  data.tar.gz: '048e3527e272f20bba27026fd9d65cdfc00686cf8a4bdc6ad078f5e3fee6e874ee3eef042a9694369a282d5a7a201b324fb99d2d8b4c50069345dbd220fe620a'
+  metadata.gz: d26f5e28b2c205a3484811497f0e332fef3d0ec1afede88e2ab81716038d1a103b9d8a43c1ce65c6cd4b7e77d6723ae8d0eef2fb69e3a0fd415e3b1ded2e604f
+  data.tar.gz: 37ecf68758a85dedc51f3856156bce673675b9d22fc9acec8c75848a7d819a91c022cfd1da4b08c4106b8d46e6591e00b16d1cb97f386a90c2172560c008529a

data/app/models/concerns/foreman_puppet/extensions/host.rb

@@ -13,9 +13,9 @@ module ForemanPuppet
         include_in_clone puppet: %i[config_groups host_config_groups host_classes]
 
         scoped_search relation: :environment, on: :name, complete_value: true, rename: :environment
-        scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :puppetclass, only_explicit: true, operators: ['= ', '~ '],
+        scoped_search relation: :search_puppetclasses, on: :name, complete_value: true, rename: :puppetclass, only_explicit: true, operators: ['= ', '~ '],
           ext_method: :search_by_puppetclass
-        scoped_search relation: :config_groups, on: :name, complete_value: true, rename: :config_group, only_explicit: true, operators: ['= ', '~ '],
+        scoped_search relation: :search_config_groups, on: :name, complete_value: true, rename: :config_group, only_explicit: true, operators: ['= ', '~ '],
           ext_method: :search_by_config_group
       end
 

data/app/models/concerns/foreman_puppet/extensions/host_common.rb

@@ -3,6 +3,41 @@ module ForemanPuppet
     module HostCommon
       extend ActiveSupport::Concern
 
+      ASSIGNMENT_RELATIONS = %i[search_puppetclasses search_config_groups].freeze
+
+      included do
+        singleton_class.prepend CompletionMethods
+
+        has_many :search_puppetclasses, through: :puppet, source: :puppetclasses, class_name: '::ForemanPuppet::Puppetclass'
+        has_many :search_config_groups, through: :puppet, source: :config_groups, class_name: '::ForemanPuppet::ConfigGroup'
+      end
+
+      module CompletionMethods
+        def complete_for(query, options = {})
+          return super if query.nil?
+
+          completer = AssignmentCompleter.new(scoped_search_definition, query, options)
+          return super unless completer.assignment_field?
+
+          completer.build_autocomplete_options
+        end
+      end
+
+      class AssignmentCompleter < ScopedSearch::AutoCompleteBuilder
+        def assignment_field?
+          return false unless complete_options(last_node).include?(:value)
+
+          token = last_token_is(COMPARISON_OPERATORS) ? tokens[-2] : tokens[-3]
+          ASSIGNMENT_RELATIONS.include?(definition.field_by_name(token)&.relation)
+        end
+
+        def completer_scope(field)
+          return super unless ASSIGNMENT_RELATIONS.include?(field.relation)
+
+          field.klass.completer_scope(@options).reorder(Arel.sql(field.quoted_field))
+        end
+      end
+
       def all_puppetclasses(env = environment)
         return ForemanPuppet::Puppetclass.none unless puppet
         puppet.all_puppetclasses(env)

data/app/models/concerns/foreman_puppet/extensions/hostgroup.rb

@@ -16,13 +16,13 @@ module ForemanPuppet
 
         # will need through relation to work properly
         scoped_search relation: :environment, on: :name, complete_value: true, rename: :environment, only_explicit: true
-        scoped_search relation: :puppetclasses, on: :name,
+        scoped_search relation: :search_puppetclasses, on: :name,
           complete_value: true,
           rename: :puppetclass,
           only_explicit: true,
           operators: ['= ', '~ '],
           ext_method: :search_by_puppetclass
-        scoped_search relation: :config_groups, on: :name,
+        scoped_search relation: :search_config_groups, on: :name,
           complete_value: true,
           rename: :config_group,
           only_explicit: true,
@@ -45,9 +45,16 @@ module ForemanPuppet
         def search_by_puppetclass(_key, operator, value)
           conditions = sanitize_sql_for_conditions(["puppetclasses.name #{operator} ?", value_to_sql(operator, value)])
           hostgroup_ids = ::Hostgroup.joins(puppet: :puppetclasses).where(conditions).map(&:subtree_ids)
+          config_group_ids = ForemanPuppet::ConfigGroup.joins(:puppetclasses).where(conditions).pluck(:id)
+          if config_group_ids.any?
+            hostgroup_cg_ids = ForemanPuppet::HostgroupPuppetFacet.joins(:host_config_groups)
+                                                                  .where(host_config_groups: { config_group_id: config_group_ids })
+                                                                  .pluck(:hostgroup_id)
+            hostgroup_ids += ::Hostgroup.where(id: hostgroup_cg_ids).map(&:subtree_ids)
+          end
 
           conds = []
-          conds << "hostgroups.id IN (#{hostgroup_ids.join(',')})" if hostgroup_ids.present?
+          conds << "hostgroups.id IN (#{hostgroup_ids.uniq.join(',')})" if hostgroup_ids.present?
 
           { conditions: conds.join(' OR ').presence || 'hostgroups.id < 0' }
         end

data/app/models/foreman_puppet/config_group.rb

@@ -16,6 +16,8 @@ module ForemanPuppet
 
     validates :name, presence: true, uniqueness: true
 
+    scope :assigned_to_hosts, -> { where(id: ForemanPuppet::HostConfigGroup.select(:config_group_id)) }
+
     scoped_search on: :name, complete_value: true
     scoped_search relation: :puppetclasses, on: :name, complete_value: true, rename: :puppetclass, only_explicit: true, operators: ['= ', '~ ']
 
@@ -41,5 +43,9 @@ module ForemanPuppet
     def hostgroups_count
       ::Hostgroup.authorized.search_for(%(config_group="#{name}")).size
     end
+
+    def self.completer_scope(options)
+      super.merge(assigned_to_hosts)
+    end
   end
 end

data/app/models/foreman_puppet/puppetclass.rb

@@ -41,6 +41,16 @@ module ForemanPuppet
 
     default_scope -> { order(:name) }
 
+    scope :assigned_to_hosts, lambda {
+      direct = ForemanPuppet::HostClass.select(:puppetclass_id)
+      via_hostgroup = ForemanPuppet::HostgroupClass.select(:puppetclass_id)
+      via_config_group = ForemanPuppet::ConfigGroupClass
+                         .where(config_group_id: ForemanPuppet::HostConfigGroup.select(:config_group_id))
+                         .select(:puppetclass_id)
+
+      where(id: direct).or(where(id: via_hostgroup)).or(where(id: via_config_group))
+    }
+
     scoped_search on: :name, complete_value: true
     scoped_search relation: :environments, on: :name, complete_value: true, rename: 'environment'
     scoped_search relation: :organizations, on: :name, complete_value: true, rename: 'organization', only_explicit: true
@@ -142,5 +152,9 @@ module ForemanPuppet
       puppet_classes = (direct + indirect).uniq
       { conditions: "puppetclasses.id IN(#{puppet_classes.join(',')})" }
     end
+
+    def self.completer_scope(options)
+      super.merge(assigned_to_hosts)
+    end
   end
 end

data/lib/foreman_puppet/register.rb

@@ -1,5 +1,5 @@
 Foreman::Plugin.register :foreman_puppet do
-  requires_foreman '>= 3.19'
+  requires_foreman '>= 5.0'
   register_gettext
 
   # Add Global JS file for extending foreman-core components and routes

data/lib/foreman_puppet/version.rb

@@ -1,3 +1,3 @@
 module ForemanPuppet
-  VERSION = '10.1.3'.freeze
+  VERSION = '11.0.0'.freeze
 end

data/test/models/foreman_puppet/host_test.rb

@@ -117,6 +117,72 @@ module ForemanPuppet
       end
     end
 
+    describe '#complete_for' do
+      let(:host) { FactoryBot.create(:host, :with_puppet_enc, :with_puppetclass) }
+
+      test 'can complete Puppet class values' do
+        puppetclass = host.puppet.puppetclasses.first
+        completions = ::Host::Managed.complete_for("puppetclass = #{puppetclass.name}")
+
+        assert_includes completions, %(puppetclass =  "#{puppetclass.name}")
+      end
+
+      test 'can complete config group values' do
+        config_group = FactoryBot.create(:config_group)
+        host.puppet.config_groups << config_group
+        completions = ::Host::Managed.complete_for("config_group = #{config_group.name}")
+
+        assert_includes completions, %(config_group =  "#{config_group.name}")
+      end
+
+      test 'can complete Puppet class values inherited from a hostgroup' do
+        hostgroup = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_puppetclass)
+        FactoryBot.create(:host, hostgroup: hostgroup)
+        puppetclass = hostgroup.puppet.puppetclasses.first
+        completions = ::Host::Managed.complete_for("puppetclass = #{puppetclass.name}")
+
+        assert_includes completions, %(puppetclass =  "#{puppetclass.name}")
+      end
+
+      test 'can complete Puppet class values assigned through a config group' do
+        config_group = FactoryBot.create(:config_group, :with_puppetclass)
+        host.puppet.config_groups << config_group
+        puppetclass = config_group.puppetclasses.first
+        completions = ::Host::Managed.complete_for("puppetclass = #{puppetclass.name}")
+
+        assert_includes completions, %(puppetclass =  "#{puppetclass.name}")
+      end
+
+      test 'can complete config group values inherited from a hostgroup' do
+        hostgroup = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_config_group)
+        FactoryBot.create(:host, hostgroup: hostgroup)
+        config_group = hostgroup.puppet.config_groups.first
+        completions = ::Host::Managed.complete_for("config_group = #{config_group.name}")
+
+        assert_includes completions, %(config_group =  "#{config_group.name}")
+      end
+
+      test 'does not complete unassigned Puppet class values' do
+        puppetclass = FactoryBot.create(:puppetclass)
+        completions = ::Host::Managed.complete_for("puppetclass = #{puppetclass.name}")
+
+        assert_not_includes completions, %(puppetclass =  "#{puppetclass.name}")
+      end
+
+      test 'does not complete unassigned config group values' do
+        config_group = FactoryBot.create(:config_group)
+        completions = ::Host::Managed.complete_for("config_group = #{config_group.name}")
+
+        assert_not_includes completions, %(config_group =  "#{config_group.name}")
+      end
+
+      test 'preserves core host value completions' do
+        completions = as_admin { ::Host::Managed.complete_for('user.login =') }
+
+        assert_includes completions, 'user.login = current_user'
+      end
+    end
+
     describe '#info puppet bits' do
       test 'ENC YAML omits environment if no puppet facet' do
         host = FactoryBot.build_stubbed(:host)

data/test/models/foreman_puppet/hostgroup_test.rb

@@ -22,5 +22,37 @@ module ForemanPuppet
       assert_includes hostgroup.puppet.puppetclasses, puppet_class
       assert_includes hostgroup.puppetclasses, puppet_class
     end
+
+    test 'can complete Puppet class values' do
+      puppetclass = FactoryBot.create(:puppetclass)
+      hostgroup.puppet.puppetclasses << puppetclass
+      completions = ::Hostgroup.complete_for("puppetclass = #{puppetclass.name}")
+
+      assert_includes completions, %(puppetclass =  "#{puppetclass.name}")
+    end
+
+    test 'can complete config group values' do
+      config_group = hostgroup.puppet.config_groups.first
+      completions = ::Hostgroup.complete_for("config_group = #{config_group.name}")
+
+      assert_includes completions, %(config_group =  "#{config_group.name}")
+    end
+
+    test 'can complete Puppet class values inherited from a parent hostgroup' do
+      parent = FactoryBot.create(:hostgroup, :with_puppet_enc, :with_puppetclass)
+      FactoryBot.create(:hostgroup, parent: parent)
+      puppetclass = parent.puppet.puppetclasses.first
+      completions = ::Hostgroup.complete_for("puppetclass = #{puppetclass.name}")
+
+      assert_includes completions, %(puppetclass =  "#{puppetclass.name}")
+    end
+
+    test 'searches Puppet class values assigned through a config group' do
+      config_group = hostgroup.puppet.config_groups.first
+      puppetclass = FactoryBot.create(:puppetclass)
+      config_group.puppetclasses << puppetclass
+
+      assert_includes ::Hostgroup.search_for("puppetclass = #{puppetclass.name}"), hostgroup
+    end
   end
 end

data/webpack/src/Extends/Hosts/BulkActions/BulkChangeProxyCommon/index.js

@@ -15,17 +15,11 @@ import {
 } from '@patternfly/react-core';
 import { addToast } from 'foremanReact/components/ToastsList/slice';
 import { sprintf, translate as __ } from 'foremanReact/common/I18n';
-import { foremanUrl } from 'foremanReact/common/helpers';
-import { APIActions } from 'foremanReact/redux/API';
 import { STATUS } from 'foremanReact/constants';
 import {
   selectAPIStatus,
   selectAPIResponse,
 } from 'foremanReact/redux/API/APISelectors';
-import {
-  HOSTS_API_PATH,
-  API_REQUEST_KEY,
-} from 'foremanReact/routes/Hosts/constants';
 import {
   fetchSmartProxies,
   SMART_PROXY_KEY,
@@ -47,6 +41,7 @@ const BulkChangeProxyCommon = ({
   allHostsMessage,
   someHostsMessage,
   isCAProxy,
+  onSuccess: onSuccessCallback,
 }) => {
   const dispatch = useDispatch();
   const [smartProxyId, setSmartProxyId] = useState('');
@@ -82,6 +77,7 @@ const BulkChangeProxyCommon = ({
 
   const toggle = toggleRef => (
     <MenuToggle
+      ouiaId="bulk-change-proxy-common-select-toggle"
       ref={toggleRef}
       onClick={onToggleClick}
       isExpanded={smartProxySelectOpen}
@@ -112,13 +108,11 @@ const BulkChangeProxyCommon = ({
         message: response.data.message,
       })
     );
-    dispatch(
-      APIActions.get({
-        key: API_REQUEST_KEY,
-        url: foremanUrl(HOSTS_API_PATH),
-      })
-    );
-    handleModalClose();
+    try {
+      if (onSuccessCallback) onSuccessCallback();
+    } finally {
+      handleModalClose();
+    }
   };
 
   const handleConfirm = () => {
@@ -242,11 +236,13 @@ BulkChangeProxyCommon.propTypes = {
   allHostsMessage: PropTypes.string.isRequired,
   someHostsMessage: PropTypes.string.isRequired,
   isCAProxy: PropTypes.bool.isRequired,
+  onSuccess: PropTypes.func,
 };
 
 BulkChangeProxyCommon.defaultProps = {
   isOpen: false,
   closeModal: () => {},
+  onSuccess: undefined,
 };
 
 export default BulkChangeProxyCommon;

data/webpack/src/Extends/Hosts/BulkActions/BulkChangePuppetCAProxy/__tests__/index.test.js

@@ -18,11 +18,13 @@ jest.mock('../../BulkChangeProxyCommon', () => ({
 
 describe('BulkChangePuppetCAProxyScene', () => {
   const fetchBulkParams = jest.fn();
+  const refreshTableData = jest.fn();
   const contextValue = {
     selectAllHostsMode: false,
     selectedCount: 2,
     selectedResults: [1, 2],
     fetchBulkParams,
+    refreshTableData,
   };
 
   beforeEach(() => {
@@ -51,6 +53,7 @@ describe('BulkChangePuppetCAProxyScene', () => {
         selectAllHostsMode: false,
         isOpen: true,
         closeModal: expect.any(Function),
+        onSuccess: refreshTableData,
         selectMessage: 'Select a Puppet CA Proxy',
         handleErrorMessage: 'Failed to change Puppet CA Proxy',
         changeMessage: 'Change Puppet CA Proxy',

data/webpack/src/Extends/Hosts/BulkActions/BulkChangePuppetCAProxy/index.js

@@ -11,6 +11,7 @@ const BulkChangePuppetCAProxyScene = () => {
     selectedCount,
     selectedResults,
     fetchBulkParams,
+    refreshTableData,
   } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen(
     'bulk-change-puppet-ca-proxy'
@@ -24,6 +25,7 @@ const BulkChangePuppetCAProxyScene = () => {
       selectAllHostsMode={selectAllHostsMode}
       isOpen={isOpen}
       closeModal={closeModal}
+      onSuccess={refreshTableData}
       selectMessage={__('Select a Puppet CA Proxy')}
       handleErrorMessage={__('Failed to change Puppet CA Proxy')}
       changeMessage={__('Change Puppet CA Proxy')}

data/webpack/src/Extends/Hosts/BulkActions/BulkChangePuppetProxy/__tests__/index.test.js

@@ -18,11 +18,13 @@ jest.mock('../../BulkChangeProxyCommon', () => ({
 
 describe('BulkChangePuppetProxyScene', () => {
   const fetchBulkParams = jest.fn();
+  const refreshTableData = jest.fn();
   const contextValue = {
     selectAllHostsMode: false,
     selectedCount: 2,
     selectedResults: [1, 2],
     fetchBulkParams,
+    refreshTableData,
   };
 
   beforeEach(() => {
@@ -51,6 +53,7 @@ describe('BulkChangePuppetProxyScene', () => {
         selectAllHostsMode: false,
         isOpen: true,
         closeModal: expect.any(Function),
+        onSuccess: refreshTableData,
         selectMessage: 'Select a Puppet Proxy',
         handleErrorMessage: 'Failed to change Puppet Proxy',
         changeMessage: 'Change Puppet Proxy',

data/webpack/src/Extends/Hosts/BulkActions/BulkChangePuppetProxy/index.js

@@ -11,6 +11,7 @@ const BulkChangePuppetProxyScene = () => {
     selectedCount,
     selectedResults,
     fetchBulkParams,
+    refreshTableData,
   } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen(
     'bulk-change-puppet-proxy'
@@ -24,6 +25,7 @@ const BulkChangePuppetProxyScene = () => {
       selectAllHostsMode={selectAllHostsMode}
       isOpen={isOpen}
       closeModal={closeModal}
+      onSuccess={refreshTableData}
       selectMessage={__('Select a Puppet Proxy')}
       handleErrorMessage={__('Failed to change Puppet Proxy')}
       changeMessage={__('Change Puppet Proxy')}

data/webpack/src/Extends/Hosts/BulkActions/BulkRemoveProxyCommon/index.js

@@ -5,12 +5,6 @@ import { useDispatch } from 'react-redux';
 import { Modal, Button, TextContent, Text } from '@patternfly/react-core';
 import { addToast } from 'foremanReact/components/ToastsList/slice';
 import { translate as __ } from 'foremanReact/common/I18n';
-import { foremanUrl } from 'foremanReact/common/helpers';
-import { APIActions } from 'foremanReact/redux/API';
-import {
-  HOSTS_API_PATH,
-  API_REQUEST_KEY,
-} from 'foremanReact/routes/Hosts/constants';
 
 import {
   BULK_REMOVE_PUPPET_PROXY_KEY,
@@ -30,6 +24,7 @@ const BulkRemoveProxyCommon = ({
   removeMessage,
   allHostsMessage,
   someHostsMessage,
+  onSuccess: onSuccessCallback,
 }) => {
   const actionKey = isCAProxy
     ? BULK_REMOVE_PUPPET_CA_PROXY_KEY
@@ -57,13 +52,11 @@ const BulkRemoveProxyCommon = ({
         message: response.data.message,
       })
     );
-    dispatch(
-      APIActions.get({
-        key: API_REQUEST_KEY,
-        url: foremanUrl(HOSTS_API_PATH),
-      })
-    );
-    handleModalClose();
+    try {
+      if (onSuccessCallback) onSuccessCallback();
+    } finally {
+      handleModalClose();
+    }
   };
 
   const handleConfirm = () => {
@@ -153,12 +146,14 @@ BulkRemoveProxyCommon.propTypes = {
   removeMessage: PropTypes.string,
   allHostsMessage: PropTypes.string.isRequired,
   someHostsMessage: PropTypes.string.isRequired,
+  onSuccess: PropTypes.func,
 };
 
 BulkRemoveProxyCommon.defaultProps = {
   isOpen: false,
   closeModal: () => {},
   removeMessage: 'Remove Puppet (CA) Proxy',
+  onSuccess: undefined,
 };
 
 export default BulkRemoveProxyCommon;

data/webpack/src/Extends/Hosts/BulkActions/BulkRemovePuppetCAProxy/__tests__/index.test.js

@@ -18,11 +18,13 @@ jest.mock('../../BulkRemoveProxyCommon', () => ({
 
 describe('BulkRemovePuppetCAProxyScene', () => {
   const fetchBulkParams = jest.fn();
+  const refreshTableData = jest.fn();
   const contextValue = {
     selectAllHostsMode: false,
     selectedCount: 2,
     selectedResults: [1, 2],
     fetchBulkParams,
+    refreshTableData,
   };
 
   beforeEach(() => {
@@ -51,6 +53,7 @@ describe('BulkRemovePuppetCAProxyScene', () => {
         fetchBulkParams,
         isOpen: true,
         closeModal: expect.any(Function),
+        onSuccess: refreshTableData,
         handleErrorMessage: 'Failed to remove Puppet CA Proxy',
         allHostsMessage:
           'Removing the Puppet CA proxy will affect {boldCount} selected hosts. Warning: If a Puppet Proxy is still set, the Puppet CA Proxy will fall back to that value after removal!',

data/webpack/src/Extends/Hosts/BulkActions/BulkRemovePuppetCAProxy/index.js

@@ -10,6 +10,7 @@ const BulkRemovePuppetCAProxyScene = () => {
     selectedCount,
     selectedResults,
     fetchBulkParams,
+    refreshTableData,
   } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen(
     'bulk-remove-puppet-ca-proxy'
@@ -24,6 +25,7 @@ const BulkRemovePuppetCAProxyScene = () => {
       fetchBulkParams={fetchBulkParams}
       isOpen={isOpen}
       closeModal={closeModal}
+      onSuccess={refreshTableData}
       handleErrorMessage={__('Failed to remove Puppet CA Proxy')}
       allHostsMessage={__(
         'Removing the Puppet CA proxy will affect {boldCount} selected hosts. Warning: If a Puppet Proxy is still set, the Puppet CA Proxy will fall back to that value after removal!'

data/webpack/src/Extends/Hosts/BulkActions/BulkRemovePuppetProxy/__tests__/index.test.js

@@ -18,11 +18,13 @@ jest.mock('../../BulkRemoveProxyCommon', () => ({
 
 describe('BulkRemovePuppetProxyScene', () => {
   const fetchBulkParams = jest.fn();
+  const refreshTableData = jest.fn();
   const contextValue = {
     selectAllHostsMode: false,
     selectedCount: 2,
     selectedResults: [1, 2],
     fetchBulkParams,
+    refreshTableData,
   };
 
   beforeEach(() => {
@@ -51,6 +53,7 @@ describe('BulkRemovePuppetProxyScene', () => {
         fetchBulkParams,
         isOpen: true,
         closeModal: expect.any(Function),
+        onSuccess: refreshTableData,
         handleErrorMessage: 'Failed to remove Puppet Proxy',
         removeMessage: 'Remove Puppet Proxy',
       })

data/webpack/src/Extends/Hosts/BulkActions/BulkRemovePuppetProxy/index.js

@@ -10,6 +10,7 @@ const BulkRemovePuppetProxyScene = () => {
     selectedCount,
     selectedResults,
     fetchBulkParams,
+    refreshTableData,
   } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen(
     'bulk-remove-puppet-proxy'
@@ -24,6 +25,7 @@ const BulkRemovePuppetProxyScene = () => {
       fetchBulkParams={fetchBulkParams}
       isOpen={isOpen}
       closeModal={closeModal}
+      onSuccess={refreshTableData}
       handleErrorMessage={__('Failed to remove Puppet Proxy')}
       allHostsMessage={__(
         'Removing the Puppet proxy will affect {boldCount} selected hosts.'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_puppet
 version: !ruby/object:Gem::Version
-  version: 10.1.3
+  version: 11.0.0
 platform: ruby
 authors:
 - Ondřej Ezr