katello 4.5.0 → 4.6.0.rc1

data/webpack/components/extensions/HostDetails/Cards/SystemPurposeCard/SystemPurposeSelectors.js

@@ -0,0 +1,25 @@
+import {
+  selectAPIStatus,
+  selectAPIError,
+  selectAPIResponse,
+} from 'foremanReact/redux/API/APISelectors';
+import { STATUS } from 'foremanReact/constants';
+
+export const selectOrganization = (state, orgId) =>
+  selectAPIResponse(state, `ORGANIZATION_${orgId}`) ?? {};
+
+export const selectOrganizationStatus = (state, orgId) =>
+  selectAPIStatus(state, `ORGANIZATION_${orgId}`) ?? STATUS.PENDING;
+
+export const selectOrganizationError = (state, orgId) =>
+  selectAPIError(state, `ORGANIZATION_${orgId}`);
+
+export const selectAvailableReleaseVersions = (state, hostId) =>
+  selectAPIResponse(state, `AVAILABLE_RELEASE_VERSIONS_${hostId}`) ?? {};
+
+export const selectAvailableReleaseVersionsStatus = (state, hostId) =>
+  selectAPIStatus(state, `AVAILABLE_RELEASE_VERSIONS_${hostId}`) ??
+  STATUS.PENDING;
+
+export const selectAvailableReleaseVersionsError = (state, hostId) =>
+  selectAPIError(state, `AVAILABLE_RELEASE_VERSIONS_${hostId}`);

data/webpack/components/extensions/HostDetails/Cards/SystemPurposeCard/__tests__/SystemPurposeCard.test.js

@@ -0,0 +1,108 @@
+import React from 'react';
+import { renderWithRedux, act } from 'react-testing-lib-wrapper';
+import HOST_DETAILS from '../../../HostDetailsConstants';
+import SystemPurposeCard from '../SystemPurposeCard';
+import katelloApi, { foremanApi } from '../../../../../../services/api';
+import { assertNockRequest, nockInstance } from '../../../../../../test-utils/nockWrapper';
+
+const organizationDetails = katelloApi.getApiUrl('/organizations/1');
+const availableReleaseVersions = foremanApi.getApiUrl('/hosts/1/subscriptions/available_release_versions');
+
+const baseHostDetails = {
+  id: 1,
+  organization_id: 1,
+  permissions: {
+    edit_hosts: true,
+  },
+  subscription_facet_attributes: {
+    purpose_addons: ['Addon1', 'Addon2'],
+    purpose_role: 'Red Hat Enterprise Linux Server',
+    purpose_usage: 'Production',
+    service_level: 'Premium',
+    release_version: '8',
+  },
+};
+
+const renderOptions = () => ({
+  apiNamespace: HOST_DETAILS,
+  initialState: {
+    API: {
+      HOST_DETAILS: {
+        response: {
+          id: 1,
+          name: 'test-host',
+          ...baseHostDetails,
+        },
+        status: 'RESOLVED',
+      },
+    },
+  },
+});
+
+test('shows system purpose details', async (done) => {
+  const orgScope = nockInstance
+    .get(organizationDetails)
+    .reply(200, {
+      id: 1,
+    });
+  const availableReleaseVersionsScope = nockInstance
+    .get(availableReleaseVersions)
+    .reply(200, []);
+
+  const { getByText }
+    = renderWithRedux(<SystemPurposeCard hostDetails={baseHostDetails} />, renderOptions());
+  expect(getByText('Red Hat Enterprise Linux Server')).toBeInTheDocument();
+  expect(getByText('Production')).toBeInTheDocument();
+  expect(getByText('Premium')).toBeInTheDocument();
+  expect(getByText('Addon1')).toBeInTheDocument();
+  expect(getByText('Addon2')).toBeInTheDocument();
+  expect(getByText('8')).toBeInTheDocument();
+
+  assertNockRequest(orgScope);
+  assertNockRequest(availableReleaseVersionsScope, done);
+});
+
+
+test('shows edit button for a user with edit_hosts permission', async (done) => {
+  const orgScope = nockInstance
+    .get(organizationDetails)
+    .reply(200, {
+      id: 1,
+    });
+  const availableReleaseVersionsScope = nockInstance
+    .get(availableReleaseVersions)
+    .reply(200, []);
+
+  const { queryByText }
+    = renderWithRedux(<SystemPurposeCard hostDetails={baseHostDetails} />, renderOptions());
+  expect(queryByText('Edit')).toBeInTheDocument();
+
+  assertNockRequest(orgScope);
+  assertNockRequest(availableReleaseVersionsScope, done);
+});
+
+test('does not show edit button for a user without edit_hosts permission', (done) => {
+  const orgScope = nockInstance
+    .get(organizationDetails)
+    .reply(200, {
+      id: 1,
+    });
+  const availableReleaseVersionsScope = nockInstance
+    .get(availableReleaseVersions)
+    .reply(200, []);
+
+  const limitedUser = {
+    ...baseHostDetails,
+    permissions: {
+      edit_hosts: false,
+    },
+  };
+
+  const { queryByText }
+    = renderWithRedux(<SystemPurposeCard hostDetails={limitedUser} />);
+  expect(queryByText('Edit')).not.toBeInTheDocument();
+
+  assertNockRequest(orgScope);
+  assertNockRequest(availableReleaseVersionsScope, done);
+  act(done);
+});

data/webpack/components/extensions/HostDetails/Cards/SystemPurposeCard/__tests__/SystemPurposeEditModal.test.js

@@ -0,0 +1,161 @@
+import React from 'react';
+import { propsToCamelCase } from 'foremanReact/common/helpers';
+import { renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
+import HOST_DETAILS from '../../../HostDetailsConstants';
+import SystemPurposeEditModal from '../SystemPurposeEditModal';
+import { assertNockRequest, nockInstance } from '../../../../../../test-utils/nockWrapper';
+import katelloApi, { foremanApi } from '../../../../../../services/api';
+
+const organizationDetails = katelloApi.getApiUrl('/organizations/1');
+const availableReleaseVersions = foremanApi.getApiUrl('/hosts/1/subscriptions/available_release_versions');
+const hostEditUrl = foremanApi.getApiUrl('/hosts/1');
+const hostDetailsGetUrl = '/api/hosts/test-host';
+
+const baseHostDetails = {
+  id: 1,
+  organization_id: 1,
+  permissions: {
+    edit_hosts: true,
+  },
+  subscription_facet_attributes: {
+    purpose_addons: ['Addon1', 'Addon2'],
+    purpose_role: 'Red Hat Enterprise Linux Server',
+    purpose_usage: 'Production',
+    service_level: 'Premium',
+    release_version: '8',
+  },
+};
+
+const facetAttributes = propsToCamelCase(baseHostDetails.subscription_facet_attributes);
+const baseAttributes = {
+  hostName: 'test-host',
+  closeModal: jest.fn(),
+  isOpen: true,
+  orgId: 1,
+  hostId: 1,
+};
+
+const renderOptions = () => ({
+  apiNamespace: HOST_DETAILS,
+  initialState: {
+    API: {
+      HOST_DETAILS: {
+        response: {
+          id: 1,
+          name: 'test-host',
+          ...baseHostDetails,
+        },
+        status: 'RESOLVED',
+      },
+    },
+  },
+});
+
+describe('SystemPurposeEditModal', () => {
+  test('Shows currently selected attributes as defaults', (done) => {
+    const orgScope = nockInstance
+      .get(organizationDetails)
+      .reply(200, {
+        id: 1,
+      });
+    const availableReleaseVersionsScope = nockInstance
+      .get(availableReleaseVersions)
+      .reply(200, []);
+
+    const { getByText }
+      = renderWithRedux(<SystemPurposeEditModal
+        {...baseAttributes}
+        {...facetAttributes}
+      />, renderOptions());
+
+    expect(getByText('Red Hat Enterprise Linux Server')).toBeInTheDocument();
+    expect(getByText('Production')).toBeInTheDocument();
+    expect(getByText('Premium')).toBeInTheDocument();
+    expect(getByText('Addon1')).toBeInTheDocument();
+    expect(getByText('Addon2')).toBeInTheDocument();
+    expect(getByText('8')).toBeInTheDocument();
+
+    assertNockRequest(orgScope);
+    assertNockRequest(availableReleaseVersionsScope, done);
+  });
+
+  test('Shows blank options as (unset)', () => {
+    const orgScope = nockInstance
+      .get(organizationDetails)
+      .reply(200, {
+        id: 1,
+      });
+    const availableReleaseVersionsScope = nockInstance
+      .get(availableReleaseVersions)
+      .reply(200, []);
+    const { getAllByText }
+      = renderWithRedux(<SystemPurposeEditModal
+        {...
+          {
+            ...baseAttributes,
+            purposeRole: '',
+          }}
+        {...facetAttributes}
+      />, renderOptions());
+
+    expect(getAllByText('(unset)')[0]).toBeVisible();
+
+    assertNockRequest(orgScope);
+    assertNockRequest(availableReleaseVersionsScope);
+  });
+  test('Calls API and changes syspurpose values', async (done) => {
+    const orgScope = nockInstance
+      .get(organizationDetails)
+      .reply(200, {
+        id: 1,
+      });
+    const availableReleaseVersionsScope = nockInstance
+      .get(availableReleaseVersions)
+      .reply(200, []);
+    const hostEditScope = nockInstance
+      .put(hostEditUrl, {
+        id: 1,
+        host: {
+          subscription_facet_attributes: {
+            autoheal: true,
+            // we're going to change role from 'Server' to 'Workstation'
+            purpose_role: 'Red Hat Enterprise Linux Workstation',
+            purpose_usage: 'Production',
+            purpose_addons: ['Addon1', 'Addon2'],
+            release_version: '8',
+            service_level: 'Premium',
+          },
+        },
+      })
+      .reply(200);
+    const hostDetailsScope = nockInstance
+      .get(hostDetailsGetUrl)
+      .reply(200);
+
+    const { getByLabelText, getByRole }
+      = renderWithRedux(<SystemPurposeEditModal
+        {...baseAttributes}
+        {...facetAttributes}
+      />, renderOptions());
+
+    const saveButton = getByRole('button', { name: 'Save' });
+    // Save button should be disabled if no values have been changed
+    expect(saveButton).toHaveAttribute('aria-disabled', 'true');
+
+    const roleDropdown = getByLabelText('Role');
+    fireEvent.change(roleDropdown, { target: { value: 'Red Hat Enterprise Linux Workstation' } });
+
+    // Save button should now be enabled
+    expect(saveButton).toHaveAttribute('aria-disabled', 'false');
+    fireEvent.click(saveButton);
+
+    await patientlyWaitFor(() => {
+      expect(baseAttributes.closeModal).toHaveBeenCalled();
+    });
+
+    [orgScope, availableReleaseVersionsScope, hostEditScope].forEach((scope) => {
+      assertNockRequest(scope);
+    });
+    assertNockRequest(hostDetailsScope, done);
+  });
+});

data/webpack/components/extensions/HostDetails/Cards/__tests__/errataOverviewCard.test.js

@@ -1,7 +1,6 @@
 import React from 'react';
-import { render } from 'react-testing-lib-wrapper';
+import { render, renderWithRedux } from 'react-testing-lib-wrapper';
 import ErrataOverviewCard from '../ErrataOverviewCard';
-import nock from '../../../../../test-utils/nockWrapper';
 
 const baseHostDetails = {
   id: 2,
@@ -9,12 +8,17 @@ const baseHostDetails = {
     uuid: '123',
   },
 };
-
+const renderOptions = {
+  initialState: {
+    // This is the API state that your tests depend on for their data
+    // You can cross reference the needed useSelectors from your tested components
+    // with the data found within the redux chrome add-on to help determine this fixture data.
+    katello: {
+      hostDetails: {},
+    },
+  },
+};
 describe('Without errata', () => {
-  afterEach(() => {
-    nock.cleanAll();
-  });
-
   test('shows zero counts when there are 0 installable errata', () => {
     const hostDetails = {
       ...baseHostDetails,
@@ -52,7 +56,7 @@ describe('Without errata', () => {
       },
     };
     /* eslint-disable max-len */
-    const { queryByLabelText, getByText } = render(<ErrataOverviewCard hostDetails={hostDetails} />);
+    const { queryByLabelText, getByText } = renderWithRedux(<ErrataOverviewCard hostDetails={hostDetails} />, renderOptions);
     /* eslint-enable max-len */
     expect(queryByLabelText('errataChart')).not.toBeInTheDocument();
     expect(getByText('All errata up-to-date')).toBeInTheDocument();
@@ -80,10 +84,6 @@ describe('Without errata', () => {
 });
 
 describe('With errata', () => {
-  afterEach(() => {
-    nock.cleanAll();
-  });
-
   test('shows links when there are errata', () => {
     const hostDetails = {
       ...baseHostDetails,

data/webpack/components/extensions/HostDetails/DetailsTabCards/HwPropertiesCard.js

@@ -0,0 +1,100 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { propsToCamelCase } from 'foremanReact/common/helpers';
+import { translate as __ } from 'foremanReact/common/I18n';
+import {
+  DescriptionList,
+  DescriptionListTerm,
+  DescriptionListGroup,
+  DescriptionListDescription,
+  Text,
+  TextVariants,
+} from '@patternfly/react-core';
+import CardTemplate from 'foremanReact/components/HostDetails/Templates/CardItem/CardTemplate';
+import { TranslatedPlural } from '../../../Table/components/TranslatedPlural';
+
+const HostDisks = ({ totalDisks }) => {
+  if (!totalDisks) return null;
+  return (
+    <>
+      <DescriptionListTerm>{__('Storage')}</DescriptionListTerm>
+      <Text component={TextVariants.h4}><TranslatedPlural count={totalDisks} singular={__('disk')} id="total-disks" /></Text>
+    </>
+  );
+};
+
+HostDisks.propTypes = {
+  totalDisks: PropTypes.number,
+};
+
+HostDisks.defaultProps = {
+  totalDisks: null,
+};
+
+const HwPropertiesCard = ({ isExpandedGlobal, hostDetails }) => {
+  const { facts } = hostDetails || {};
+  const model = facts?.['virt::host_type'];
+  const cpuCount = facts?.['cpu::cpu(s)'];
+  const cpuSockets = facts?.['cpu::cpu_socket(s)'];
+  const coreSocket = facts?.['cpu::core(s)_per_socket'];
+  const reportedFacts = propsToCamelCase(hostDetails?.reported_data || {});
+  const totalDisks = reportedFacts?.disksTotal;
+  const memory = facts?.['dmi::memory::maximum_capacity'];
+
+  return (
+    <CardTemplate
+      header={__('HW properties')}
+      expandable
+      isExpandedGlobal={isExpandedGlobal}
+    >
+      <DescriptionList isHorizontal>
+        <DescriptionListGroup>
+          <DescriptionListTerm>{__('Model')}</DescriptionListTerm>
+          <DescriptionListDescription>{model}</DescriptionListDescription>
+        </DescriptionListGroup>
+        <DescriptionListGroup>
+          <DescriptionListTerm>{__('Number of CPU(s)')}</DescriptionListTerm>
+          <DescriptionListDescription>{cpuCount}</DescriptionListDescription>
+        </DescriptionListGroup>
+        <DescriptionListGroup>
+          <DescriptionListTerm>{__('Sockets')}</DescriptionListTerm>
+          <DescriptionListDescription>{cpuSockets}</DescriptionListDescription>
+        </DescriptionListGroup>
+        <DescriptionListGroup>
+          <DescriptionListTerm>{__('Cores per socket')}</DescriptionListTerm>
+          <DescriptionListDescription>{coreSocket}</DescriptionListDescription>
+        </DescriptionListGroup>
+        <DescriptionListGroup>
+          <DescriptionListTerm>{__('RAM')}</DescriptionListTerm>
+          <DescriptionListDescription>{memory}</DescriptionListDescription>
+        </DescriptionListGroup>
+        <DescriptionListGroup>
+          <HostDisks totalDisks={totalDisks} />
+        </DescriptionListGroup>
+      </DescriptionList>
+    </CardTemplate>
+  );
+};
+
+HwPropertiesCard.propTypes = {
+  isExpandedGlobal: PropTypes.bool,
+  hostDetails: PropTypes.shape({
+    facts: PropTypes.shape({
+      model: PropTypes.string,
+      cpuCount: PropTypes.number,
+      cpuSockets: PropTypes.number,
+      coreSocket: PropTypes.number,
+      memory: PropTypes.string,
+    }),
+    reported_data: PropTypes.shape({
+      totalDisks: PropTypes.number,
+    }),
+  }),
+};
+
+HwPropertiesCard.defaultProps = {
+  isExpandedGlobal: false,
+  hostDetails: {},
+};
+
+export default HwPropertiesCard;

data/webpack/components/extensions/HostDetails/DetailsTabCards/SystemPropertiesCardExtensions.js

@@ -8,7 +8,7 @@ import {
 } from '@patternfly/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
 
-const SystemPropertiesCardExtensions = ({ hostDetails }) => {
+export const SystemPropertiesCardSubscription = ({ hostDetails }) => {
   const subscriptionUuid = hostDetails?.subscription_facet_attributes?.uuid;
   if (!subscriptionUuid) return null;
   return (
@@ -23,7 +23,7 @@ const SystemPropertiesCardExtensions = ({ hostDetails }) => {
   );
 };
 
-SystemPropertiesCardExtensions.propTypes = {
+SystemPropertiesCardSubscription.propTypes = {
   hostDetails: PropTypes.shape({
     subscription_facet_attributes: PropTypes.shape({
       uuid: PropTypes.string,
@@ -31,8 +31,30 @@ SystemPropertiesCardExtensions.propTypes = {
   }),
 };
 
-SystemPropertiesCardExtensions.defaultProps = {
+SystemPropertiesCardSubscription.defaultProps = {
   hostDetails: {},
 };
 
-export default SystemPropertiesCardExtensions;
+export const SystemPropertiesCardTracer = ({ hostDetails }) => {
+  const tracerStatus = hostDetails?.content_facet_attributes?.katello_tracer_installed;
+  return (
+    <DescriptionListGroup>
+      <DescriptionListTerm>{__('Tracer')}</DescriptionListTerm>
+      <DescriptionListDescription>
+        {tracerStatus ? __('Installed') : __('Not installed')}
+      </DescriptionListDescription>
+    </DescriptionListGroup>
+  );
+};
+
+SystemPropertiesCardTracer.propTypes = {
+  hostDetails: PropTypes.shape({
+    content_facet_attributes: PropTypes.shape({
+      katello_tracer_installed: PropTypes.bool,
+    }),
+  }),
+};
+
+SystemPropertiesCardTracer.defaultProps = {
+  hostDetails: {},
+};

data/webpack/components/extensions/HostDetails/HostDetailsConstants.js

@@ -1 +1,3 @@
-export default 'HOST_DETAILS';
+export const SET_CLEAR_SEARCH = 'SET_CLEAR_SEARCH';
+const HOST_DETAILS = 'HOST_DETAILS';
+export default HOST_DETAILS;

data/webpack/components/extensions/HostDetails/HostDetailsReducer.js

@@ -0,0 +1,14 @@
+import Immutable from 'seamless-immutable';
+
+import { SET_CLEAR_SEARCH } from './HostDetailsConstants';
+
+const initialState = Immutable({ clearSearch: undefined });
+
+export default (state = initialState, action) => {
+  switch (action.type) {
+  case SET_CLEAR_SEARCH:
+    return state.set('clearSearch', action.payload);
+  default:
+    return state;
+  }
+};

data/webpack/components/extensions/HostDetails/HostDetailsSelectors.js

@@ -7,10 +7,16 @@ import { STATUS } from 'foremanReact/constants';
 import HOST_DETAILS_KEY from './HostDetailsConstants';
 
 export const selectHostDetails = state =>
-  selectAPIResponse(state, HOST_DETAILS_KEY) || {};
+  selectAPIResponse(state, HOST_DETAILS_KEY) ?? {};
 
 export const selectHostDetailsStatus = state =>
-  selectAPIStatus(state, HOST_DETAILS_KEY) || STATUS.PENDING;
+  selectAPIStatus(state, HOST_DETAILS_KEY) ?? STATUS.PENDING;
 
 export const selectHostDetailsError = state =>
   selectAPIError(state, HOST_DETAILS_KEY);
+
+export const selectHostDetailsState = state =>
+  state.katello.hostDetails;
+
+export const selectHostDetailsClearSearch = state =>
+  selectHostDetailsState(state).clearSearch;

data/webpack/components/extensions/HostDetails/Tabs/ErrataTab/ErrataTab.js

@@ -231,6 +231,11 @@ export const ErrataTab = () => {
     }
   };
 
+  const resetFilters = () => {
+    setErrataTypeSelected(ERRATA_TYPE);
+    setErrataSeveritySelected(ERRATA_SEVERITY);
+  };
+
   const readOnlyBookmarks =
   cannot(createBookmarks, userPermissionsFromHostDetails({ hostDetails }));
 
@@ -336,7 +341,7 @@ export const ErrataTab = () => {
   ) : null;
 
   const hostIsNonLibrary = (
-    contentFacet?.contentViewDefault === false && contentFacet.lifecycleEnvironmentLibrary === false
+    contentFacet?.contentViewDefault === false || contentFacet.lifecycleEnvironmentLibrary === false
   );
   const toggleGroup = (
     <Split hasGutter>
@@ -406,6 +411,7 @@ export const ErrataTab = () => {
             selectedCount,
             selectNone,
             toggleGroup,
+            resetFilters,
           }
           }
           happyEmptyContent={allUpToDate}
@@ -423,6 +429,8 @@ export const ErrataTab = () => {
           variant={TableVariant.compact}
           {...selectAll}
           displaySelectAllCheckbox={showActions}
+          requestKey={HOST_ERRATA_KEY}
+          alwaysShowActionButtons={false}
         >
           <Thead>
             <Tr>

data/webpack/components/extensions/HostDetails/Tabs/ModuleStreamsTab/ModuleStreamsTab.js

@@ -7,7 +7,6 @@ import { Skeleton,
   Button,
   Split,
   SplitItem,
-  Spinner,
   Checkbox,
   Dropdown,
   Text,
@@ -36,7 +35,7 @@ import SortableColumnHeaders from '../../../../Table/components/SortableColumnHe
 import SelectableDropdown from '../../../../SelectableDropdown/SelectableDropdown';
 import {
   HOST_MODULE_STREAM_STATUSES, INSTALL_STATUS_PARAM_TO_FRIENDLY_NAME, INSTALLED_STATE,
-  STATUS_PARAM_TO_FRIENDLY_NAME,
+  STATUS_PARAM_TO_FRIENDLY_NAME, MODULE_STREAMS_KEY,
 } from './ModuleStreamsConstants';
 import { moduleStreamAction } from '../RemoteExecutionActions';
 import { katelloModuleStreamActionUrl } from '../customizedRexUrlHelpers';
@@ -213,9 +212,15 @@ export const ModuleStreamsTab = () => {
   const [hostModuleStreamAction, setHostModuleStreamAction] = useState(null);
 
   const emptyContentTitle = __('This host does not have any Module streams.');
-  const emptyContentBody = __('Module streams will appear here when available.');
+  const emptyContentBody = __('Module streams will appear here after enabling Red Hat repositories or creating custom products.');
   const emptySearchTitle = __('Your search returned no matching Module streams.');
   const emptySearchBody = __('Try changing your search criteria.');
+  const showPrimaryAction = true;
+  const showSecondaryAction = true;
+  const primaryActionTitle = __('Enable Red Hat repositories');
+  const secondaryActionTitle = __('Create a custom product');
+  const primaryActionLink = '/redhat_repositories';
+  const secondaryActionLink = '/products/new';
   const errorSearchTitle = __('Problem searching module streams');
   const {
     status: initialStatus,
@@ -324,6 +329,12 @@ export const ModuleStreamsTab = () => {
 
   const activeFilters = [statusSelected, installStatusSelected];
   const defaultFilters = [MODULE_STREAM_STATUS, MODULE_STREAM_INSTALLATION_STATUS];
+
+  const resetFilters = () => {
+    setStatusSelected(MODULE_STREAM_STATUS);
+    setInstallStatusSelected(MODULE_STREAM_INSTALLATION_STATUS);
+  };
+
   return (
     <div>
       <div id="modulestreams-tab">
@@ -342,6 +353,13 @@ export const ModuleStreamsTab = () => {
             activeFilters,
             defaultFilters,
             status,
+            showPrimaryAction,
+            showSecondaryAction,
+            primaryActionTitle,
+            secondaryActionTitle,
+            primaryActionLink,
+            secondaryActionLink,
+            resetFilters,
           }}
           ouiaId="host-module-stream-table"
           additionalListeners={[hostId, activeSortColumn, activeSortDirection,
@@ -354,6 +372,8 @@ export const ModuleStreamsTab = () => {
           foremanApiAutoComplete
           rowsCount={results?.length}
           variant={TableVariant.compact}
+          requestKey={MODULE_STREAMS_KEY}
+          alwaysShowActionButtons={false}
           actionButtons={
             <Split hasGutter>
               <SplitItem>
@@ -376,11 +396,6 @@ export const ModuleStreamsTab = () => {
                   setSelected={handleModuleStreamInstallationStatusSelected}
                 />
               </SplitItem>
-              {actionInProgress && (
-                <SplitItem style={{ alignSelf: 'center' }}>
-                  <Spinner size="lg" style={{ marginTop: '2px' }} />
-                </SplitItem>
-              )}
             </Split>
           }
         >