katello 4.5.0.rc1 → 4.5.0.rc2

data/db/migrate/20220303160220_remove_duplicate_errata.rb

@@ -43,7 +43,7 @@ class RemoveDuplicateErrata < ActiveRecord::Migration[6.0]
         ::Katello::ModuleStreamErratumPackage.where(erratum_package_id: dup_err_package).each do |dup_mod_errata_package|
           if ::Katello::ModuleStreamErratumPackage.find_by(module_stream_id: dup_mod_errata_package.module_stream_id, erratum_package_id: erratum_package_to_keep&.id)
             dup_mod_errata_package.delete
-          else
+          elsif erratum_package_to_keep&.id
             begin
               dup_mod_errata_package.update(erratum_package_id: erratum_package_to_keep&.id)
             rescue

data/db/migrate/20220428203334_add_last_refreshed_to_katello_alternate_content_sources.rb

@@ -0,0 +1,5 @@
+class AddLastRefreshedToKatelloAlternateContentSources < ActiveRecord::Migration[6.0]
+  def change
+    add_column :katello_alternate_content_sources, :last_refreshed, :datetime
+  end
+end

data/engines/bastion_katello/app/assets/javascripts/bastion_katello/capsule-content/capsule-content.controller.js

@@ -109,7 +109,7 @@ angular.module('Bastion.capsule-content').controller('CapsuleContentController',
                         } else if (errorCount > 1) {
                             errorMessage += " " + translate("Plus 1 more error");
                         }
-                        Notification.setErrorMessage(errorMessage);
+                        Notification.setErrorMessage(translate('Last sync failed: ') + errorMessage);
                     }
                 }
                 $scope.syncState.set(stateFromTask(activeOrFailedTask));

data/lib/katello/permission_creator.rb

@@ -404,13 +404,15 @@ module Katello
                          :finder_scope => :editable
       @plugin.permission :edit_alternate_content_sources,
                          {
-                           'katello/api/v2/alternate_content_sources' => [:update]
+                           'katello/api/v2/alternate_content_sources' => [:update, :refresh],
+                           'katello/api/v2/alternate_content_sources_bulk_actions' => [:refresh_alternate_content_sources]
                          },
                          :resource_type => 'Katello::AlternateContentSource',
                          :finder_scope => :editable
       @plugin.permission :destroy_alternate_content_sources,
                          {
-                           'katello/api/v2/alternate_content_sources' => [:destroy]
+                           'katello/api/v2/alternate_content_sources' => [:destroy],
+                           'katello/api/v2/alternate_content_sources_bulk_actions' => [:destroy_alternate_content_sources]
                          },
                          :resource_type => 'Katello::AlternateContentSource',
                          :finder_scope => :deletable

data/lib/katello/tasks/refresh_alternate_content_sources.rake

@@ -0,0 +1,10 @@
+namespace :katello do
+  desc 'Refresh all alternate content sources'
+  task :refresh_alternate_content_sources => ["dynflow:client"] do
+    User.current = User.anonymous_admin
+    ::ForemanTasks.async_task(::Actions::BulkAction,
+                              ::Actions::Katello::AlternateContentSource::Refresh,
+                              ::Katello::AlternateContentSource.all)
+    puts _("Alternate content source refreshing started in the background.")
+  end
+end

data/lib/katello/version.rb

@@ -1,3 +1,3 @@
 module Katello
-  VERSION = "4.5.0.rc1".freeze
+  VERSION = "4.5.0.rc2".freeze
 end

data/webpack/components/Bookmark/index.js

@@ -4,14 +4,14 @@ import { useDispatch, useSelector, shallowEqual } from 'react-redux';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { STATUS } from 'foremanReact/constants';
 import { Dropdown, DropdownItem, DropdownToggle, DropdownSeparator } from '@patternfly/react-core';
-import { CaretDownIcon } from '@patternfly/react-icons';
+import { OutlinedBookmarkIcon } from '@patternfly/react-icons';
 import { getBookmarks } from './BookmarkActions';
 import { selectBookmarks, selectBookmarkStatus } from './BookmarkSelectors';
 import './Bookmark.scss';
 import AddBookmarkModal from './AddBookmarkModal';
 
 const Bookmark = ({
-  selectItem, selectedItem, controller = '', isDisabled,
+  selectItem, selectedItem, controller = '', isDisabled, readOnlyBookmarks,
 }) => {
   const dispatch = useDispatch();
   const [dropdownOpen, setDropdownOpen] = useState(false);
@@ -20,6 +20,7 @@ const Bookmark = ({
     useSelector(state => selectBookmarks(state, controller), shallowEqual);
   const status =
     useSelector(state => selectBookmarkStatus(state, controller), shallowEqual);
+  const showActions = !readOnlyBookmarks;
 
   useEffect(() => {
     dispatch(getBookmarks(controller));
@@ -31,6 +32,7 @@ const Bookmark = ({
     }
     setDropdownOpen(false);
   };
+  if (!results.length && readOnlyBookmarks) return null;
 
   const dropDownItems = [
     ...results.map(({ name, id, query }) => (
@@ -41,16 +43,21 @@ const Bookmark = ({
       >
         {name}
       </DropdownItem >)),
-    <DropdownSeparator key="separator" />,
-    <DropdownItem
-      onClick={() => {
-        setDropdownOpen(false);
-        setModalOpen(true);
-      }}
-      key="ADD_BOOKMARK"
-    >
-      {selectedItem ? __('Bookmark this search') : __('Add new bookmark')}
-    </DropdownItem >];
+  ];
+  if (showActions) {
+    dropDownItems.push(
+      <DropdownSeparator key="separator" />,
+      <DropdownItem
+        onClick={() => {
+          setDropdownOpen(false);
+          setModalOpen(true);
+        }}
+        key="ADD_BOOKMARK"
+      >
+        {selectedItem ? __('Bookmark this search') : __('Add new bookmark')}
+      </DropdownItem >,
+    );
+  }
 
 
   return (
@@ -62,9 +69,8 @@ const Bookmark = ({
             isDisabled={isDisabled || status !== STATUS.RESOLVED}
             onToggle={setDropdownOpen}
             id="toggle-id"
-            toggleIndicator={CaretDownIcon}
           >
-            <i className="fas fa-bookmark" />
+            <OutlinedBookmarkIcon />
           </DropdownToggle>
         }
         isOpen={dropdownOpen}
@@ -85,10 +91,12 @@ Bookmark.propTypes = {
   controller: PropTypes.string.isRequired,
   selectItem: PropTypes.func.isRequired,
   selectedItem: PropTypes.string.isRequired,
+  readOnlyBookmarks: PropTypes.bool,
 };
 
 Bookmark.defaultProps = {
   isDisabled: undefined,
+  readOnlyBookmarks: false,
 };
 
 export default Bookmark;

data/webpack/components/Search/Search.js

@@ -22,6 +22,7 @@ const Search = ({
   getAutoCompleteParams,
   foremanApiAutoComplete,
   bookmarkController,
+  readOnlyBookmarks,
   placeholder,
   isTextInput,
   setTextInputValue,
@@ -84,6 +85,7 @@ const Search = ({
       <TypeAhead
         autoSearchDelay={autoSearchDelay}
         bookmarkController={bookmarkController}
+        readOnlyBookmarks={readOnlyBookmarks}
         isDisabled={isDisabled}
         items={items}
         onInputUpdate={onInputUpdate}
@@ -112,6 +114,7 @@ Search.propTypes = {
     autoSearchDelay: PropTypes.number,
   }),
   bookmarkController: PropTypes.string,
+  readOnlyBookmarks: PropTypes.bool,
   placeholder: PropTypes.string,
   isTextInput: PropTypes.bool,
   setTextInputValue: PropTypes.func,
@@ -127,6 +130,7 @@ Search.defaultProps = {
   },
   isDisabled: undefined,
   bookmarkController: undefined,
+  readOnlyBookmarks: false,
   placeholder: undefined,
   isTextInput: false,
   setTextInputValue: undefined,

data/webpack/components/Table/MainTable.scss

@@ -14,4 +14,8 @@
 
 .pf-c-table tbody tr td {
   vertical-align: inherit;
-}
+}
+
+input[type="checkbox"][disabled] {
+  cursor: initial;
+}

data/webpack/components/Table/TableWrapper.js

@@ -43,6 +43,7 @@ const TableWrapper = ({
   disableSearch,
   nodesBelowSearch,
   bookmarkController,
+  readOnlyBookmarks,
   ...allTableProps
 }) => {
   const dispatch = useDispatch();
@@ -191,6 +192,7 @@ const TableWrapper = ({
               getAutoCompleteParams={getAutoCompleteParams}
               foremanApiAutoComplete={foremanApiAutoComplete}
               bookmarkController={bookmarkController}
+              readOnlyBookmarks={readOnlyBookmarks}
               placeholder={searchPlaceholderText}
             />
           </FlexItem>
@@ -272,7 +274,7 @@ TableWrapper.propTypes = {
   actionButtons: PropTypes.node,
   toggleGroup: PropTypes.node,
   children: PropTypes.node,
-  // additionalListeners are anything that can trigger another API call, e.g. a filter
+  // additionalListeners are anything that should trigger another API call, e.g. a filter
   additionalListeners: PropTypes.arrayOf(PropTypes.oneOfType([
     PropTypes.number,
     PropTypes.string,
@@ -300,6 +302,7 @@ TableWrapper.propTypes = {
   disableSearch: PropTypes.bool,
   nodesBelowSearch: PropTypes.node,
   bookmarkController: PropTypes.string,
+  readOnlyBookmarks: PropTypes.bool,
 };
 
 TableWrapper.defaultProps = {
@@ -326,6 +329,7 @@ TableWrapper.defaultProps = {
   disableSearch: false,
   nodesBelowSearch: null,
   bookmarkController: undefined,
+  readOnlyBookmarks: false,
 };
 
 export default TableWrapper;

data/webpack/components/TypeAhead/TypeAhead.js

@@ -22,6 +22,7 @@ const TypeAhead = ({
   autoSearchEnabled,
   autoSearchDelay,
   bookmarkController,
+  readOnlyBookmarks,
   placeholder,
   isTextInput,
   setTextInputValue,
@@ -69,6 +70,7 @@ const TypeAhead = ({
       }) => {
         const typeAheadProps = {
           bookmarkController,
+          readOnlyBookmarks,
           isDisabled,
           userInputValue: inputValue,
           clearSearch,
@@ -118,6 +120,7 @@ TypeAhead.propTypes = {
   autoSearchEnabled: PropTypes.bool.isRequired,
   autoSearchDelay: PropTypes.number,
   bookmarkController: PropTypes.string,
+  readOnlyBookmarks: PropTypes.bool,
   placeholder: PropTypes.string,
   isTextInput: PropTypes.bool,
   setTextInputValue: PropTypes.func,
@@ -130,6 +133,7 @@ TypeAhead.defaultProps = {
   isDisabled: undefined,
   autoSearchDelay: 500,
   bookmarkController: undefined,
+  readOnlyBookmarks: false,
   placeholder: undefined,
   isTextInput: false,
   setTextInputValue: undefined,

data/webpack/components/TypeAhead/pf4Search/TypeAheadSearch.js

@@ -13,6 +13,7 @@ const TypeAheadSearch = ({
   userInputValue, clearSearch, getInputProps, getItemProps, isOpen, highlightedIndex,
   selectedItem, selectItem, openMenu, onSearch, items, activeItems, shouldShowItems,
   autoSearchEnabled, isDisabled, bookmarkController, inputValue, placeholder, isTextInput,
+  readOnlyBookmarks,
 }) => (
   <>
     <InputGroup>
@@ -44,6 +45,7 @@ const TypeAheadSearch = ({
               isDisabled,
               selectedItem,
               selectItem,
+              readOnlyBookmarks,
             }}
             controller={bookmarkController}
           />}

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

@@ -9,7 +9,6 @@ import {
 import { translate as __ } from 'foremanReact/common/I18n';
 import { STATUS } from 'foremanReact/constants';
 import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
-import { HOST_DETAILS_KEY } from 'foremanReact/components/HostDetails/consts';
 import { selectAPIStatus } from 'foremanReact/redux/API/APISelectors';
 import EnvironmentPaths from '../../../../../scenes/ContentViews/components/EnvironmentPaths/EnvironmentPaths';
 import { ENVIRONMENT_PATHS_KEY } from '../../../../../scenes/ContentViews/components/EnvironmentPaths/EnvironmentPathConstants';
@@ -20,6 +19,7 @@ import { uniq } from '../../../../../utils/helpers';
 import ContentViewIcon from '../../../../../scenes/ContentViews/components/ContentViewIcon';
 import updateHostContentViewAndEnvironment from './HostContentViewActions';
 import HOST_CV_AND_ENV_KEY from './HostContentViewConstants';
+import { getHostDetails } from '../../HostDetailsActions';
 
 const ENV_PATH_OPTIONS = { key: ENVIRONMENT_PATHS_KEY };
 
@@ -110,13 +110,7 @@ const ChangeHostCVModal = ({
 
   const refreshHostDetails = () => {
     handleModalClose();
-    return dispatch({
-      type: 'API_GET',
-      payload: {
-        key: HOST_DETAILS_KEY,
-        url: `/api/hosts/${hostName}`,
-      },
-    });
+    return dispatch(getHostDetails({ hostname: hostName }));
   };
 
   const handleSave = () => {

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

@@ -20,12 +20,18 @@ import { translate as __ } from 'foremanReact/common/I18n';
 import { propsToCamelCase } from 'foremanReact/common/helpers';
 import PropTypes from 'prop-types';
 import ContentViewIcon from '../../../../../scenes/ContentViews/components/ContentViewIcon';
-import { hostIsRegistered } from '../../hostDetailsHelpers';
+import { hasRequiredPermissions, hostIsRegistered } from '../../hostDetailsHelpers';
 import ChangeHostCVModal from './ChangeHostCVModal';
 
+const requiredPermissions = [
+  'view_lifecycle_environments', 'view_content_views',
+  'promote_or_remove_content_views_to_environments',
+];
+
 const HostContentViewDetails = ({
   contentView, lifecycleEnvironment, contentViewVersionId, contentViewDefault,
   contentViewVersion, contentViewVersionLatest, hostId, hostName, orgId, hostEnvId,
+  hostPermissions, permissions,
 }) => {
   let versionLabel = `Version ${contentViewVersion}`;
   if (contentViewVersionLatest) {
@@ -41,6 +47,9 @@ const HostContentViewDetails = ({
     setIsModalOpen(true);
   };
 
+  const userPermissions = { ...hostPermissions, ...permissions };
+  const showKebab = hasRequiredPermissions(requiredPermissions, userPermissions);
+
   const dropdownItems = [
     <DropdownItem
       aria-label="change-host-content-view"
@@ -71,16 +80,18 @@ const HostContentViewDetails = ({
                 </FlexItem>
               </Flex>
             </FlexItem>
-            <FlexItem>
-              <Dropdown
-                toggle={<KebabToggle aria-label="change_content_view_hamburger" onToggle={toggleHamburger} />}
-                isOpen={isDropdownOpen}
-                isPlain
-                ouiaId="change-host-content-view-kebab"
-                position="right"
-                dropdownItems={dropdownItems}
-              />
-            </FlexItem>
+            {showKebab && (
+              <FlexItem>
+                <Dropdown
+                  toggle={<KebabToggle aria-label="change_content_view_hamburger" onToggle={toggleHamburger} />}
+                  isOpen={isDropdownOpen}
+                  isPlain
+                  ouiaId="change-host-content-view-kebab"
+                  position="right"
+                  dropdownItems={dropdownItems}
+                />
+              </FlexItem>
+            )}
           </Flex>
         </CardHeader>
         <CardBody>
@@ -146,6 +157,7 @@ const ContentViewDetailsCard = ({ hostDetails }) => {
       hostName={hostDetails.name}
       orgId={hostDetails.organization_id}
       hostEnvId={hostDetails.content_facet_attributes.lifecycle_environment_id}
+      hostPermissions={hostDetails.permissions}
       {...propsToCamelCase(hostDetails.content_facet_attributes)}
     />);
   }
@@ -172,6 +184,14 @@ HostContentViewDetails.propTypes = {
   hostName: PropTypes.string,
   orgId: PropTypes.number,
   hostEnvId: PropTypes.number,
+  hostPermissions: PropTypes.shape({
+    edit_hosts: PropTypes.bool,
+  }),
+  permissions: PropTypes.shape({
+    view_content_views: PropTypes.bool,
+    view_lifecycle_environments: PropTypes.bool,
+    promote_or_remove_content_views_to_environments: PropTypes.bool,
+  }),
 };
 
 HostContentViewDetails.defaultProps = {
@@ -182,6 +202,8 @@ HostContentViewDetails.defaultProps = {
   hostName: '',
   orgId: null,
   contentViewDefault: false,
+  hostPermissions: {},
+  permissions: {},
 };
 
 ContentViewDetailsCard.propTypes = {
@@ -191,6 +213,14 @@ ContentViewDetailsCard.propTypes = {
     organization_id: PropTypes.number,
     content_facet_attributes: PropTypes.shape({
       lifecycle_environment_id: PropTypes.number,
+      permissions: PropTypes.shape({
+        view_content_views: PropTypes.bool,
+        view_lifecycle_environments: PropTypes.bool,
+        promote_or_remove_content_views_to_environments: PropTypes.bool,
+      }),
+    }),
+    permissions: PropTypes.shape({
+      edit_hosts: PropTypes.bool,
     }),
   }),
 };

data/webpack/components/extensions/HostDetails/Cards/HostCollectionsCard/HostCollectionsActions.js

@@ -24,7 +24,7 @@ export const alterHostCollections = (hostId, params, refreshHostDetails) => put(
   url: foremanApi.getApiUrl(`/hosts/${hostId}/host_collections`),
   params,
   successToast: () => __('Host collections updated'),
-  errorToast: error => errorToast(error),
-  handleSuccess: () => refreshHostDetails(),
+  errorToast,
+  handleSuccess: refreshHostDetails,
 });
 

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

@@ -18,10 +18,13 @@ import { propsToCamelCase } from 'foremanReact/common/helpers';
 import PropTypes from 'prop-types';
 import { useSet } from '../../../../Table/TableHooks';
 import { HostCollectionsAddModal, HostCollectionsRemoveModal } from './HostCollectionsModal';
-import { hostIsRegistered } from '../../hostDetailsHelpers';
+import { hasRequiredPermissions, hostIsRegistered, userPermissionsFromHostDetails } from '../../hostDetailsHelpers';
+
+const requiredPermissions = ['edit_hosts', 'view_host_collections'];
 
 const HostCollectionsDetails = ({
   hostCollections, id: hostId, name: hostName,
+  showKebab,
 }) => {
   const [isDropdownOpen, setIsDropdownOpen] = useState(false);
   const toggleBulkAction = () => setIsDropdownOpen(prev => !prev);
@@ -85,15 +88,17 @@ const HostCollectionsDetails = ({
                 </FlexItem>
               </Flex>
             </FlexItem>
-            <FlexItem>
-              <Dropdown
-                toggle={<KebabToggle aria-label="host_collections_bulk_actions" onToggle={toggleBulkAction} />}
-                isOpen={isDropdownOpen}
-                isPlain
-                position="right"
-                dropdownItems={dropdownItems}
-              />
-            </FlexItem>
+            {showKebab && (
+              <FlexItem>
+                <Dropdown
+                  toggle={<KebabToggle aria-label="host_collections_bulk_actions" onToggle={toggleBulkAction} />}
+                  isOpen={isDropdownOpen}
+                  isPlain
+                  position="right"
+                  dropdownItems={dropdownItems}
+                />
+              </FlexItem>)
+            }
           </Flex>
         </CardHeader>
         <CardBody>
@@ -159,7 +164,9 @@ const HostCollectionsDetails = ({
 
 const HostCollectionsCard = ({ hostDetails }) => {
   if (hostIsRegistered({ hostDetails })) {
-    return <HostCollectionsDetails {...propsToCamelCase(hostDetails)} />;
+    const showKebab =
+      hasRequiredPermissions(requiredPermissions, userPermissionsFromHostDetails({ hostDetails }));
+    return <HostCollectionsDetails showKebab={showKebab} {...propsToCamelCase(hostDetails)} />;
   }
   return null;
 };
@@ -168,20 +175,32 @@ HostCollectionsDetails.propTypes = {
   hostCollections: PropTypes.arrayOf(PropTypes.shape({})),
   id: PropTypes.number,
   name: PropTypes.string,
+  showKebab: PropTypes.bool,
 };
 
 HostCollectionsDetails.defaultProps = {
   hostCollections: [],
   id: null,
   name: '',
+  showKebab: false,
 };
 
 HostCollectionsCard.propTypes = {
-  hostDetails: PropTypes.shape({}),
+  hostDetails: PropTypes.shape({
+    permissions: PropTypes.shape({}),
+    contentFacetAttributes: PropTypes.shape({
+      permissions: PropTypes.shape({}),
+    }),
+  }),
 };
 
 HostCollectionsCard.defaultProps = {
-  hostDetails: null,
+  hostDetails: {
+    permissions: {},
+    contentFacetAttributes: {
+      permissions: {},
+    },
+  },
 };
 
 export default HostCollectionsCard;

data/webpack/components/extensions/HostDetails/Cards/HostCollectionsCard/__tests__/hostCollectionsCard.test.js

@@ -41,6 +41,14 @@ const emptyHostDetails = {
   },
 };
 
+jest.mock('../../../hostDetailsHelpers', () => ({
+  ...jest.requireActual('../../../hostDetailsHelpers'),
+  userPermissionsFromHostDetails: () => ({
+    view_host_collections: true,
+    edit_hosts: true,
+  }),
+}));
+
 test('shows host collections and host limits when present', () => {
   const { getByText } = render(<HostCollectionsCard hostDetails={hostDetails} />);
   expect(getByText('Host collections')).toBeInTheDocument();

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

@@ -0,0 +1,37 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import RelativeDateTime from 'foremanReact/components/common/dates/RelativeDateTime';
+import {
+  DescriptionListGroup,
+  DescriptionListTerm,
+  DescriptionListDescription,
+} from '@patternfly/react-core';
+import { translate as __ } from 'foremanReact/common/I18n';
+
+const RecentCommunicationCardExtensions = ({ hostDetails }) => {
+  const { subscription_facet_attributes: subscriptionFacetAttributes } = hostDetails;
+  if (!Object.keys(subscriptionFacetAttributes ?? {}).includes('last_checkin')) return null;
+  const lastCheckin = subscriptionFacetAttributes?.last_checkin;
+  return (
+    <DescriptionListGroup>
+      <DescriptionListTerm>{__('Last check-in:')}</DescriptionListTerm>
+      <DescriptionListDescription>
+        <RelativeDateTime date={lastCheckin} defaultValue={__('Never')} />
+      </DescriptionListDescription>
+    </DescriptionListGroup>
+  );
+};
+
+RecentCommunicationCardExtensions.propTypes = {
+  hostDetails: PropTypes.shape({
+    subscription_facet_attributes: PropTypes.shape({
+      last_checkin: PropTypes.string,
+    }),
+  }),
+};
+
+RecentCommunicationCardExtensions.defaultProps = {
+  hostDetails: {},
+};
+
+export default RecentCommunicationCardExtensions;

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

@@ -1,2 +1,13 @@
+import HOST_DETAILS_KEY from './HostDetailsConstants';
+
 const hostIdNotReady = { type: 'NOOP_HOST_ID_NOT_READY' };
+
+export const getHostDetails = ({ hostname }) => ({
+  type: 'API_GET',
+  payload: {
+    key: HOST_DETAILS_KEY,
+    url: `/api/hosts/${hostname}`,
+  },
+});
+
 export default hostIdNotReady;

data/webpack/components/extensions/HostDetails/Tabs/ContentTab/SecondaryTabsRoutes.js

@@ -3,6 +3,7 @@ import { Route, Switch, Redirect } from 'react-router-dom';
 import { PackagesTab } from '../PackagesTab/PackagesTab.js';
 import { ErrataTab } from '../ErrataTab/ErrataTab.js';
 import { ModuleStreamsTab } from '../ModuleStreamsTab/ModuleStreamsTab';
+import RepositorySetsTab from '../RepositorySetsTab/RepositorySetsTab';
 import { route } from './helpers';
 
 const SecondaryTabRoutes = () => (
@@ -16,6 +17,9 @@ const SecondaryTabRoutes = () => (
     <Route path={route('module-streams')}>
       <ModuleStreamsTab />
     </Route>
+    <Route path={route('Repository sets')}>
+      <RepositorySetsTab />
+    </Route>
     <Redirect to={route('errata')} />
   </Switch>
 );

data/webpack/components/extensions/HostDetails/Tabs/ContentTab/constants.js

@@ -1,9 +1,11 @@
 import { translate as __ } from 'foremanReact/common/I18n';
+import { hideRepoSetsTab } from '../RepositorySetsTab/RepositorySetsTab';
 
 const SECONDARY_TABS = [
   { key: 'packages', title: __('Packages') },
   { key: 'errata', title: __('Errata') },
   { key: 'module-streams', title: __('Module streams') },
+  { key: 'Repository sets', hideTab: hideRepoSetsTab, title: __('Repository sets') },
 ];
 
 export default SECONDARY_TABS;

data/webpack/components/extensions/HostDetails/Tabs/ContentTab/index.js

@@ -2,12 +2,17 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { useHistory } from 'react-router-dom';
 import { Tabs, Tab, TabTitleText } from '@patternfly/react-core';
+import { useSelector } from 'react-redux';
+import { selectAPIResponse } from 'foremanReact/redux/API/APISelectors';
 import SecondaryTabRoutes from './SecondaryTabsRoutes';
 import { activeTab } from './helpers';
 import SECONDARY_TABS from './constants';
 
 const ContentTab = ({ location: { pathname } }) => {
   const hashHistory = useHistory();
+  const hostDetails = useSelector(state => selectAPIResponse(state, 'HOST_DETAILS'));
+  const filteredTabs =
+    SECONDARY_TABS?.filter(tab => !tab.hideTab?.({ hostDetails })) ?? [];
   return (
     <>
       <Tabs
@@ -16,7 +21,7 @@ const ContentTab = ({ location: { pathname } }) => {
         isSecondary
         activeKey={activeTab(pathname)}
       >
-        {SECONDARY_TABS.map(({ key, title }) => (
+        {filteredTabs.map(({ key, title }) => (
           <Tab
             key={key}
             eventKey={key}