katello 5.0.0.rc2 → 5.0.0

data/webpack/components/extensions/HostDetails/Cards/ContentViewDetailsCard/ContentViewDetailsCard.js

@@ -103,7 +103,7 @@ ContentViewEnvironmentDisplay.propTypes = {
   }).isRequired,
 };
 
-export const CVEDetailsBareCard = ({
+export const CvEnvDetailsBareCard = ({
   contentViewEnvironments, hostPermissions, permissions, dropdownItems,
   isDropdownOpen, toggleKebab, openModal, allowMultipleContentViews,
 }) => {
@@ -185,7 +185,7 @@ export const CVEDetailsBareCard = ({
   );
 };
 
-CVEDetailsBareCard.propTypes = {
+CvEnvDetailsBareCard.propTypes = {
   contentViewEnvironments: PropTypes.arrayOf(PropTypes.shape({
     content_view: PropTypes.shape({
       name: PropTypes.string,
@@ -212,7 +212,7 @@ CVEDetailsBareCard.propTypes = {
   allowMultipleContentViews: PropTypes.bool,
 };
 
-CVEDetailsBareCard.defaultProps = {
+CvEnvDetailsBareCard.defaultProps = {
   contentViewEnvironments: [],
   hostPermissions: {},
   permissions: {},
@@ -258,12 +258,12 @@ export const ContentViewEnvironmentDetails = ({
   const existingAssignments = contentViewEnvironments.map(env => ({
     contentView: env.content_view,
     environment: env.lifecycle_environment,
-    cveLabel: env.label, // Use the CVE label from the API
+    cvEnvLabel: env.label, // Use the content view environment label from the API
   }));
 
   return (
     <GridItem rowSpan={1} md={6} lg={4} xl2={3} >
-      <CVEDetailsBareCard
+      <CvEnvDetailsBareCard
         isDropdownOpen={isDropdownOpen}
         toggleKebab={toggleKebab}
         contentViewEnvironments={contentViewEnvironments}

data/webpack/components/extensions/HostDetails/Cards/ContentViewDetailsCard/OrderableAssignments.js

@@ -49,7 +49,7 @@ const AssignmentShape = PropTypes.shape({
   id: PropTypes.string.isRequired,
   contentView: ContentViewShape,
   environment: EnvironmentShape,
-  cveLabel: PropTypes.string,
+  cvEnvLabel: PropTypes.string,
   isExpanded: PropTypes.bool,
   cvSelectOpen: PropTypes.bool,
   selectedEnv: PropTypes.arrayOf(EnvironmentShape),
@@ -60,7 +60,7 @@ const AssignmentShape = PropTypes.shape({
 const ExistingAssignmentShape = PropTypes.shape({
   contentView: ContentViewShape,
   environment: EnvironmentShape,
-  cveLabel: PropTypes.string,
+  cvEnvLabel: PropTypes.string,
 });
 
 const AssignmentSection = ({
@@ -307,7 +307,7 @@ export const OrderableAssignmentList = ({
         id: `existing-${index}`,
         contentView: assignment.contentView,
         environment: assignment.environment,
-        cveLabel: assignment.cveLabel, // Preserve the CVE label from API
+        cvEnvLabel: assignment.cvEnvLabel, // Preserve the content view environment label from API
         isExpanded: !shouldCollapseByDefault,
         cvSelectOpen: false,
         selectedEnv: assignment.environment ? [assignment.environment] : [],
@@ -419,7 +419,7 @@ export const OrderableAssignmentList = ({
       selectedCV: null,
       environment: selection[0] || null,
       contentView: null,
-      cveLabel: null, // Clear CV environment label when environment changes
+      cvEnvLabel: null, // Clear CVEnv label when environment changes
     });
   };
 
@@ -451,7 +451,7 @@ export const OrderableAssignmentList = ({
       selectedCV: selection,
       cvSelectOpen: false,
       contentView: contentViewWithVersion,
-      cveLabel: null, // Clear CV environment label when CV changes
+      cvEnvLabel: null, // Clear CVEnv label when CV changes
     });
   };
 

data/webpack/components/extensions/HostDetails/Cards/ContentViewDetailsCard/__tests__/AssignHostCVModal.test.js

@@ -570,7 +570,7 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
       ];
 
@@ -602,12 +602,12 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
         {
           contentView: { id: 3, name: 'cv_2', label: 'cv_2' },
           environment: { id: 2, name: 'dev', label: 'dev' },
-          cveLabel: 'dev/cv_2',
+          cvEnvLabel: 'dev/cv_2',
         },
       ];
 
@@ -654,7 +654,7 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
       ];
 
@@ -706,12 +706,12 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
         {
           contentView: { id: 3, name: 'cv_2', label: 'cv_2' },
           environment: { id: 2, name: 'dev', label: 'dev' },
-          cveLabel: 'dev/cv_2',
+          cvEnvLabel: 'dev/cv_2',
         },
       ];
 
@@ -746,7 +746,7 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
       ];
 
@@ -805,7 +805,7 @@ describe('AssignHostCVModal', () => {
       jest.restoreAllMocks();
     });
 
-    test('Save with existing assignment preserves cveLabel', async () => {
+    test('Save with existing assignment preserves cvEnvLabel', async () => {
       const existingAssignments = [
         {
           contentView: {
@@ -821,7 +821,7 @@ describe('AssignHostCVModal', () => {
             label: 'Library',
             library: true,
           },
-          cveLabel: 'Library/cv_1', // Existing label from API
+          cvEnvLabel: 'Library/cv_1', // Existing label from API
         },
         {
           contentView: {
@@ -837,7 +837,7 @@ describe('AssignHostCVModal', () => {
             label: 'Library',
             library: true,
           },
-          cveLabel: 'Library', // Default CV in Library uses just env label
+          cvEnvLabel: 'Library', // Default CV in Library uses just env label
         },
       ];
 
@@ -877,7 +877,7 @@ describe('AssignHostCVModal', () => {
         userEvent.click(saveButton);
       });
 
-      // Verify API was called with existing cveLabel preserved for first assignment
+      // Verify API was called with existing cvEnvLabel preserved for first assignment
       await patientlyWaitFor(() => {
         expect(mockAssignHostCVEnvironments).toHaveBeenCalledWith(
           expect.objectContaining({
@@ -1048,17 +1048,17 @@ describe('AssignHostCVModal', () => {
         {
           contentView: { id: 2, name: 'cv_1', label: 'cv_1' },
           environment: { id: 1, name: 'Library', label: 'Library' },
-          cveLabel: 'Library/cv_1',
+          cvEnvLabel: 'Library/cv_1',
         },
         {
           contentView: { id: 3, name: 'composite_cv', label: 'composite_cv' },
           environment: { id: 2, name: 'dev', label: 'dev' },
-          cveLabel: 'dev/composite_cv',
+          cvEnvLabel: 'dev/composite_cv',
         },
         {
           contentView: { id: 4, name: 'cv_2', label: 'cv_2' },
           environment: { id: 3, name: 'prod', label: 'prod' },
-          cveLabel: 'prod/cv_2',
+          cvEnvLabel: 'prod/cv_2',
         },
       ];
 
@@ -1126,7 +1126,7 @@ describe('AssignHostCVModal', () => {
       });
     });
 
-    test('Changing CV in existing assignment builds new label instead of reusing old cveLabel', async () => {
+    test('Changing CV in existing assignment builds new label instead of reusing old cvEnvLabel', async () => {
       const existingAssignments = [
         {
           contentView: {
@@ -1142,7 +1142,7 @@ describe('AssignHostCVModal', () => {
             label: 'Library',
             library: true,
           },
-          cveLabel: 'Library/cv_1', // User currently has cv_1
+          cvEnvLabel: 'Library/cv_1', // User currently has cv_1
         },
       ];
 
@@ -1203,7 +1203,7 @@ describe('AssignHostCVModal', () => {
       });
 
       // Verify API was called with NEW label (Library/composite_cv),
-      // NOT the old cveLabel (Library/cv_1)
+      // NOT the old cvEnvLabel (Library/cv_1)
       await patientlyWaitFor(() => {
         expect(mockAssignHostCVEnvironments).toHaveBeenCalledWith(
           expect.objectContaining({

data/webpack/components/extensions/Hosts/BulkActions/BulkAssignCVEnvsModal/BulkAssignCVEnvsModal.js

@@ -6,9 +6,6 @@ import { Modal, Button, TextContent, Text, TextVariants } from '@patternfly/reac
 import { PlusCircleIcon } from '@patternfly/react-icons';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { STATUS } from 'foremanReact/constants';
-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 { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
 import { selectAPIStatus } from 'foremanReact/redux/API/APISelectors';
 import { ENVIRONMENT_PATHS_KEY } from '../../../../../scenes/ContentViews/components/EnvironmentPaths/EnvironmentPathConstants';
@@ -28,6 +25,7 @@ const BulkAssignCVEnvsModal = ({
   orgId,
   fetchBulkParams,
   allowMultipleContentViews,
+  refreshTableData,
 }) => {
   const [assignments, setAssignments] = useState([]);
   const dispatch = useDispatch();
@@ -42,11 +40,7 @@ const BulkAssignCVEnvsModal = ({
   };
 
   const handleSuccess = () => {
-    // Refresh the hosts table to show updated content view environments
-    dispatch(APIActions.get({
-      key: API_REQUEST_KEY,
-      url: foremanUrl(HOSTS_API_PATH),
-    }));
+    refreshTableData();
     handleModalClose();
   };
 
@@ -178,12 +172,14 @@ BulkAssignCVEnvsModal.propTypes = {
   orgId: PropTypes.number.isRequired,
   fetchBulkParams: PropTypes.func.isRequired,
   allowMultipleContentViews: PropTypes.bool,
+  refreshTableData: PropTypes.func,
 };
 
 BulkAssignCVEnvsModal.defaultProps = {
   isOpen: false,
   closeModal: () => {},
   allowMultipleContentViews: true,
+  refreshTableData: () => {},
 };
 
 export default BulkAssignCVEnvsModal;

data/webpack/components/extensions/Hosts/BulkActions/BulkAssignCVEnvsModal/__tests__/BulkAssignCVEnvsModal.test.js

@@ -272,7 +272,7 @@ test('builds correct payload for multiple CVEs preserving order', () => {
     },
   };
 
-  // Verify request payload contains all CVE labels in correct order
+  // Verify request payload contains all content view environment labels in correct order
   expect(requestBody).toEqual({
     content_view_environments: [
       'library_label/test_cv_1',

data/webpack/components/extensions/Hosts/BulkActions/BulkAssignCVEnvsModal/index.js

@@ -6,7 +6,7 @@ import BulkAssignCVEnvsModal from './BulkAssignCVEnvsModal';
 
 const BulkAssignCVEnvsModalScene = () => {
   const orgId = useForemanOrganization()?.id;
-  const { selectedCount, fetchBulkParams } = useContext(ForemanActionsBarContext);
+  const { selectedCount, fetchBulkParams, refreshTableData } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen('bulk-assign-cves-modal');
   const foremanContext = useForemanContext();
   const allowMultipleContentViews =
@@ -24,6 +24,7 @@ const BulkAssignCVEnvsModalScene = () => {
       closeModal={closeModal}
       orgId={orgId}
       allowMultipleContentViews={allowMultipleContentViews}
+      refreshTableData={refreshTableData}
     />
   );
 };

data/webpack/components/extensions/Hosts/BulkActions/BulkChangeHostCollectionsModal/BulkChangeHostCollectionsModal.js

@@ -22,12 +22,7 @@ import { ExclamationTriangleIcon, PlusCircleIcon } from '@patternfly/react-icons
 import { addToast } from 'foremanReact/components/ToastsList';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { STATUS } from 'foremanReact/constants';
-import { foremanUrl, noop } from 'foremanReact/common/helpers';
-import { APIActions } from 'foremanReact/redux/API';
-import {
-  HOSTS_API_PATH,
-  API_REQUEST_KEY,
-} from 'foremanReact/routes/Hosts/constants';
+import { noop } from 'foremanReact/common/helpers';
 import { useForemanOrganization } from 'foremanReact/Root/Context/ForemanContext';
 import { useTableIndexAPIResponse, useSetParamsAndApiAndSearch } from 'foremanReact/components/PF4/TableIndexPage/Table/TableIndexHooks';
 import { useBulkSelect } from 'foremanReact/components/PF4/TableIndexPage/Table/TableHooks';
@@ -46,6 +41,7 @@ const BulkChangeHostCollectionsModal = ({
   closeModal,
   fetchBulkParams,
   selectedCount: selectedHostsCount,
+  refreshTableData,
 }) => {
   const dispatch = useDispatch();
   const [addRadioChecked, setAddRadioChecked] = useState(true);
@@ -165,10 +161,7 @@ const BulkChangeHostCollectionsModal = ({
       : __('Host collections updated');
 
     dispatch(addToast({ type: 'success', message }));
-    dispatch(APIActions.get({
-      key: API_REQUEST_KEY,
-      url: foremanUrl(HOSTS_API_PATH),
-    }));
+    refreshTableData();
     handleModalClose();
   };
 
@@ -417,6 +410,7 @@ BulkChangeHostCollectionsModal.propTypes = {
   closeModal: PropTypes.func,
   fetchBulkParams: PropTypes.func,
   selectedCount: PropTypes.number,
+  refreshTableData: PropTypes.func,
 };
 
 BulkChangeHostCollectionsModal.defaultProps = {
@@ -424,6 +418,7 @@ BulkChangeHostCollectionsModal.defaultProps = {
   closeModal: () => {},
   fetchBulkParams: () => '',
   selectedCount: 0,
+  refreshTableData: () => {},
 };
 
 export default BulkChangeHostCollectionsModal;

data/webpack/components/extensions/Hosts/BulkActions/BulkChangeHostCollectionsModal/__tests__/BulkChangeHostCollectionsModal.test.js

@@ -64,6 +64,7 @@ const defaultProps = {
   closeModal: jest.fn(),
   fetchBulkParams: jest.fn(() => 'name ~ test'),
   selectedCount: 5,
+  refreshTableData: jest.fn(),
 };
 
 beforeEach(() => {
@@ -351,11 +352,6 @@ test('shows success toast notification on successful save', async () => {
     .put('/api/v2/hosts/bulk/add_host_collections')
     .reply(200, { displayMessages: ['Host collections updated'] });
 
-  const hostsRefreshScope = nockInstance
-    .get('/api/hosts')
-    .query(true)
-    .reply(200, { results: [] });
-
   const { getAllByRole, getByText } = renderWithRedux(
     <BulkChangeHostCollectionsModal {...defaultProps} />,
     renderOptions(),
@@ -387,10 +383,13 @@ test('shows success toast notification on successful save', async () => {
 
   await patientlyWaitFor(() => {
     expect(addToast).toHaveBeenCalledWith(expect.objectContaining({ type: 'success' }));
+    expect(defaultProps.refreshTableData).toHaveBeenCalledTimes(1);
+    expect(defaultProps.closeModal).toHaveBeenCalledTimes(1);
+    expect(defaultProps.refreshTableData.mock.invocationCallOrder[0])
+      .toBeLessThan(defaultProps.closeModal.mock.invocationCallOrder[0]);
   });
 
   assertNockRequest(saveScope, false);
-  assertNockRequest(hostsRefreshScope, false);
   assertNockRequest(hostCollectionsScope, false);
   assertNockRequest(autocompleteScope, false);
 });
@@ -444,6 +443,7 @@ test('shows error toast notification on failed save', async () => {
 
   await patientlyWaitFor(() => {
     expect(addToast).toHaveBeenCalledWith(expect.objectContaining({ type: 'danger' }));
+    expect(defaultProps.refreshTableData).not.toHaveBeenCalled();
   });
 
   assertNockRequest(saveScope, false);
@@ -469,11 +469,6 @@ test('makes correct API call when Add action is selected', async () => {
     .put('/api/v2/hosts/bulk/add_host_collections')
     .reply(200, { displayMessages: ['Host collections updated'] });
 
-  const hostsRefreshScope = nockInstance
-    .get('/api/hosts')
-    .query(true)
-    .reply(200, { results: [] });
-
   const { getAllByRole, getByText, getByLabelText } = renderWithRedux(
     <BulkChangeHostCollectionsModal {...defaultProps} />,
     renderOptions(),
@@ -510,7 +505,6 @@ test('makes correct API call when Add action is selected', async () => {
 
   assertNockRequest(hostCollectionsScope, false);
   assertNockRequest(autocompleteScope, false);
-  assertNockRequest(hostsRefreshScope, false);
 });
 
 // Test for Remove action API call
@@ -531,11 +525,6 @@ test('makes correct API call when Remove action is selected', async () => {
     .put('/api/v2/hosts/bulk/remove_host_collections')
     .reply(200, { displayMessages: ['Host collections updated'] });
 
-  const hostsRefreshScope = nockInstance
-    .get('/api/hosts')
-    .query(true)
-    .reply(200, { results: [] });
-
   const { getAllByRole, getByText, getByLabelText } = renderWithRedux(
     <BulkChangeHostCollectionsModal {...defaultProps} />,
     renderOptions(),
@@ -579,7 +568,6 @@ test('makes correct API call when Remove action is selected', async () => {
 
   assertNockRequest(hostCollectionsScope, false);
   assertNockRequest(autocompleteScope, false);
-  assertNockRequest(hostsRefreshScope, false);
 });
 
 // Test for multiple host collections selected
@@ -600,11 +588,6 @@ test('makes correct API call with multiple host collections', async () => {
     .put('/api/v2/hosts/bulk/add_host_collections')
     .reply(200, { displayMessages: ['Host collections updated'] });
 
-  const hostsRefreshScope = nockInstance
-    .get('/api/hosts')
-    .query(true)
-    .reply(200, { results: [] });
-
   const { getAllByRole, getByText } = renderWithRedux(
     <BulkChangeHostCollectionsModal {...defaultProps} />,
     renderOptions(),
@@ -639,7 +622,6 @@ test('makes correct API call with multiple host collections', async () => {
 
   assertNockRequest(hostCollectionsScope, false);
   assertNockRequest(autocompleteScope, false);
-  assertNockRequest(hostsRefreshScope, false);
 });
 
 const emptyHostCollections = {

data/webpack/components/extensions/Hosts/BulkActions/BulkChangeHostCollectionsModal/index.js

@@ -52,7 +52,7 @@ BulkChangeHostCollectionsMenuItem.defaultProps = {
 // This component renders only the modal (for the _all-hosts-modals slot)
 const BulkChangeHostCollectionsModalScene = () => {
   const orgId = useForemanOrganization()?.id;
-  const { selectedCount, fetchBulkParams } = useContext(ForemanActionsBarContext);
+  const { selectedCount, fetchBulkParams, refreshTableData } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen('bulk-change-host-collections-modal');
 
   if (!orgId) return null;
@@ -64,6 +64,7 @@ const BulkChangeHostCollectionsModalScene = () => {
       selectedCount={selectedCount}
       isOpen={isOpen}
       closeModal={closeModal}
+      refreshTableData={refreshTableData}
     />
   );
 };

data/webpack/components/extensions/Hosts/BulkActions/BulkErrataWizard/04_ReviewFooter.js

@@ -19,6 +19,7 @@ export const BulkErrataReviewFooter = () => {
     setFinishButtonLoading,
     selectedRexOption,
     closeModal,
+    refreshTableData,
     errataBulkSelect: {
       fetchBulkParams: getErrataBulkParams,
       selectedCount: errataSelectedCount,
@@ -55,6 +56,7 @@ export const BulkErrataReviewFooter = () => {
   const handleFinishButtonClick = () => {
     setFinishButtonLoading(true);
     triggerBulkErrataInstall();
+    refreshTableData();
     closeModal();
   };
 

data/webpack/components/extensions/Hosts/BulkActions/BulkErrataWizard/BulkErrataWizard.js

@@ -54,7 +54,7 @@ export const useErrataHostsBulkSelect = ({ initialSelectedHosts, modalIsOpen })
 export const ERRATA_URL = `${katelloApi.getApiUrl('/errata')}?per_page=7&include_permissions=true&errata_restrict_installable=true`;
 
 const BulkErrataWizard = ({ isOpen: modalOpen, closeModal }) => {
-  const { selectedCount: initialSelectedHostCount, fetchBulkParams }
+  const { selectedCount: initialSelectedHostCount, fetchBulkParams, refreshTableData }
     = useContext(ForemanActionsBarContext);
 
   const [shouldValidateStep2, setShouldValidateStep2] = useState(false);
@@ -112,6 +112,7 @@ const BulkErrataWizard = ({ isOpen: modalOpen, closeModal }) => {
     selectedRexOption,
     setSelectedRexOption,
     closeModal,
+    refreshTableData,
     errataBulkSelect,
     errataResults,
     errataMetadata,

data/webpack/components/extensions/Hosts/BulkActions/BulkManageTracesModal/BulkManageTracesModal.js

@@ -73,6 +73,7 @@ const BulkManageTracesModal = ({
   selectedCount: hostsSelectedCount,
   orgId,
   fetchBulkParams,
+  refreshTableData,
 }) => {
   const dispatch = useDispatch();
   const [isBulkActionOpen, setIsBulkActionOpen] = useState(false);
@@ -193,6 +194,7 @@ const BulkManageTracesModal = ({
         },
       },
     }));
+    refreshTableData();
     handleModalClose();
   };
 
@@ -410,11 +412,13 @@ BulkManageTracesModal.propTypes = {
   selectedCount: PropTypes.number.isRequired,
   orgId: PropTypes.number.isRequired,
   fetchBulkParams: PropTypes.func.isRequired,
+  refreshTableData: PropTypes.func,
 };
 
 BulkManageTracesModal.defaultProps = {
   isOpen: false,
   closeModal: () => {},
+  refreshTableData: () => {},
 };
 
 export default BulkManageTracesModal;

data/webpack/components/extensions/Hosts/BulkActions/BulkManageTracesModal/index.js

@@ -7,7 +7,7 @@ import BulkManageTracesModal from './BulkManageTracesModal';
 const BulkManageTracesModalScene = () => {
   const orgId = useForemanOrganization()?.id;
   const contextValue = useContext(ForemanActionsBarContext);
-  const { selectedCount, fetchBulkParams } = contextValue || {};
+  const { selectedCount, fetchBulkParams, refreshTableData } = contextValue || {};
   const { isOpen, close: closeModal } = useBulkModalOpen('bulk-manage-traces-modal');
 
   if (!orgId) return null;
@@ -25,6 +25,7 @@ const BulkManageTracesModalScene = () => {
       isOpen={isOpen}
       closeModal={closeModal}
       orgId={orgId}
+      refreshTableData={refreshTableData}
     />
   );
 };

data/webpack/components/extensions/Hosts/BulkActions/BulkPackagesWizard/04_ReviewFooter.js

@@ -19,6 +19,7 @@ export const BulkPackagesReviewFooter = () => {
     setFinishButtonLoading,
     selectedRexOption,
     closeModal,
+    refreshTableData,
     packagesBulkSelect: {
       fetchBulkParams: getPackagesBulkParams,
       selectedCount: packagesSelectedCount,
@@ -94,6 +95,7 @@ export const BulkPackagesReviewFooter = () => {
     if (selectedAction === REMOVE) {
       triggerBulkPackageRemove();
     }
+    refreshTableData();
     closeModal();
   };
 

data/webpack/components/extensions/Hosts/BulkActions/BulkPackagesWizard/BulkPackagesWizard.js

@@ -77,7 +77,7 @@ const BulkPackagesWizard = () => {
 
   const [selectedAction, setSelectedAction] = useState(UPGRADE_ALL);
 
-  const { selectedCount: initialSelectedHostCount, fetchBulkParams }
+  const { selectedCount: initialSelectedHostCount, fetchBulkParams, refreshTableData }
     = useContext(ForemanActionsBarContext);
 
   const [shouldValidateStep2, setShouldValidateStep2] = useState(false);
@@ -158,6 +158,7 @@ const BulkPackagesWizard = () => {
     selectedRexOption,
     setSelectedRexOption,
     closeModal,
+    refreshTableData,
     packagesBulkSelect,
     packagesResults,
     packagesMetadata,

data/webpack/components/extensions/Hosts/BulkActions/BulkRepositorySetsWizard/03_ReviewFooter.js

@@ -18,6 +18,7 @@ export const BulkRepositorySetsReviewFooter = () => {
     finishButtonLoading,
     setFinishButtonLoading,
     closeModal,
+    refreshTableData,
     hostsBulkSelect,
   } = useContext(BulkRepositorySetsWizardContext);
 
@@ -45,6 +46,7 @@ export const BulkRepositorySetsReviewFooter = () => {
   const handleFinishButtonClick = () => {
     setFinishButtonLoading(true);
     saveContentOverrides();
+    refreshTableData();
     closeModal();
   };
   return (

data/webpack/components/extensions/Hosts/BulkActions/BulkRepositorySetsWizard/BulkRepositorySetsWizard.js

@@ -43,7 +43,7 @@ const BulkRepositorySetsWizard = () => {
 
   const finishButtonText = __('Set content overrides');
   const replacementResponse = !modalOpen ? { response: {} } : false;
-  const { selectedCount: initialSelectedHostCount, fetchBulkParams }
+  const { selectedCount: initialSelectedHostCount, fetchBulkParams, refreshTableData }
       = useContext(ForemanActionsBarContext);
   const repoSetsResponse = useTableIndexAPIResponse({
     replacementResponse, // don't fetch data if modal is closed
@@ -109,6 +109,7 @@ const BulkRepositorySetsWizard = () => {
     finishButtonLoading,
     setFinishButtonLoading,
     closeModal,
+    refreshTableData,
     repoSetsBulkSelect,
     repoSetsResults,
     repoSetsMetadata,

data/webpack/components/extensions/Hosts/BulkActions/BulkSystemPurposeModal/BulkSystemPurposeModal.js

@@ -43,6 +43,7 @@ const BulkSystemPurposeModal = ({
   selectedCount,
   orgId,
   fetchBulkParams,
+  refreshTableData,
 }) => {
   const [selectedRole, setSelectedRole] = useState(NO_CHANGE);
   const [selectedUsage, setSelectedUsage] = useState(NO_CHANGE);
@@ -121,6 +122,7 @@ const BulkSystemPurposeModal = ({
       !releaseVersionDispatched || releaseVersionStatus !== STATUS.PENDING;
 
     if (sysPurposeComplete && releaseVersionComplete) {
+      refreshTableData();
       handleModalClose();
     }
   }, [
@@ -130,6 +132,7 @@ const BulkSystemPurposeModal = ({
     sysPurposeStatus,
     releaseVersionStatus,
     handleModalClose,
+    refreshTableData,
   ]);
 
   const roleOptions = buildSystemPurposeOptions(
@@ -348,10 +351,12 @@ BulkSystemPurposeModal.propTypes = {
   selectedCount: PropTypes.number.isRequired,
   orgId: PropTypes.number.isRequired,
   fetchBulkParams: PropTypes.func.isRequired,
+  refreshTableData: PropTypes.func,
 };
 
 BulkSystemPurposeModal.defaultProps = {
   isOpen: false,
+  refreshTableData: () => {},
 };
 
 export default BulkSystemPurposeModal;

data/webpack/components/extensions/Hosts/BulkActions/BulkSystemPurposeModal/index.js

@@ -6,7 +6,7 @@ import BulkSystemPurposeModal from './BulkSystemPurposeModal';
 
 const BulkSystemPurposeModalScene = () => {
   const orgId = useForemanOrganization()?.id;
-  const { selectedCount, fetchBulkParams } = useContext(ForemanActionsBarContext);
+  const { selectedCount, fetchBulkParams, refreshTableData } = useContext(ForemanActionsBarContext);
   const { isOpen, close: closeModal } = useBulkModalOpen('bulk-system-purpose-modal');
 
   if (!orgId) return null;
@@ -19,6 +19,7 @@ const BulkSystemPurposeModalScene = () => {
       isOpen={isOpen}
       closeModal={closeModal}
       orgId={orgId}
+      refreshTableData={refreshTableData}
     />
   );
 };