katello 3.16.0.rc3.1 → 3.16.0.rc4

data/webpack/components/Content/ContentTable.js

@@ -34,6 +34,8 @@ const ContentTable = ({ content, tableSchema, onPaginationChange }) => {
 ContentTable.propTypes = {
   content: PropTypes.shape({
     loading: PropTypes.bool,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
     pagination: PropTypes.shape({}),
     itemCount: PropTypes.number,

data/webpack/components/Content/Details/ContentDetails.js

@@ -59,9 +59,12 @@ ContentDetails.propTypes = {
   contentDetails: PropTypes.shape({
     loading: PropTypes.bool,
     name: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     profiles: PropTypes.array,
     repositories: PropTypes.array,
     artifacts: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
     stream: PropTypes.string,
   }).isRequired,
   schema: PropTypes.arrayOf(PropTypes.shape({})).isRequired,

data/webpack/scenes/AnsibleCollections/Details/AnsibleCollectionDetails.js

@@ -83,8 +83,11 @@ AnsibleCollectionDetails.propTypes = {
     name: PropTypes.string,
     namespace: PropTypes.string,
     version: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     repositories: PropTypes.array,
     tags: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
   }).isRequired,
 };
 

data/webpack/scenes/ModuleStreams/Details/ModuleStreamDetails.js

@@ -81,9 +81,12 @@ ModuleStreamDetails.propTypes = {
   moduleStreamDetails: PropTypes.shape({
     loading: PropTypes.bool,
     name: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     profiles: PropTypes.array,
     repositories: PropTypes.array,
     artifacts: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
     stream: PropTypes.string,
   }).isRequired,
 };

data/webpack/scenes/RedHatRepositories/RedHatRepositoriesPage.js

@@ -111,6 +111,8 @@ RedHatRepositoriesPage.propTypes = {
     recommended: PropTypes.bool,
     loading: PropTypes.bool,
     search: PropTypes.shape({}),
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     missingPermissions: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/RedHatRepositories/components/EnabledRepository/EnabledRepository.js

@@ -94,6 +94,8 @@ EnabledRepository.propTypes = {
   search: PropTypes.shape({
     query: PropTypes.string,
     searchList: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     filters: PropTypes.array,
   }),
   pagination: PropTypes.shape({

data/webpack/scenes/RedHatRepositories/components/RepositorySetRepository/RepositorySetRepository.js

@@ -141,6 +141,8 @@ RepositorySetRepository.propTypes = {
   enabledSearch: PropTypes.shape({
     query: PropTypes.string,
     searchList: PropTypes.string,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     filters: PropTypes.array,
   }),
   enabledPagination: PropTypes.shape({

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailAssociations.js

@@ -45,6 +45,8 @@ SubscriptionDetailAssociations.propTypes = {
   subscriptionDetails: PropTypes.shape({
     id: PropTypes.number,
     host_count: PropTypes.number,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     activation_keys: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailProductContent.js

@@ -50,6 +50,8 @@ const SubscriptionDetailProductContent = ({ productContent }) => {
 
 SubscriptionDetailProductContent.propTypes = {
   productContent: PropTypes.shape({
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Details/SubscriptionDetailProducts.js

@@ -18,6 +18,8 @@ const SubscriptionDetailProducts = ({ subscriptionDetails }) => (
 
 SubscriptionDetailProducts.propTypes = {
   subscriptionDetails: PropTypes.shape({
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     provided_products: PropTypes.array,
   }).isRequired,
 };

data/webpack/scenes/Subscriptions/Manifest/ManageManifestModal.js

@@ -352,6 +352,8 @@ ManageManifestModal.propTypes = {
   simpleContentAccess: PropTypes.bool,
   manifestHistory: PropTypes.shape({
     loading: PropTypes.bool,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
   }).isRequired,
   setModalClosed: PropTypes.func.isRequired,

data/webpack/scenes/Subscriptions/SubscriptionActions.js

@@ -28,8 +28,9 @@ import {
   SUBSCRIPTIONS_ENABLE_DELETE_BUTTON,
   BLOCKING_FOREMAN_TASK_TYPES,
   SUBSCRIPTIONS_RESET_TASKS,
+  MANIFEST_DELETE_TASK_LABEL,
 } from './SubscriptionConstants';
-import { filterRHSubscriptions, selectSubscriptionsQuantitiesFromResponse } from './SubscriptionHelpers.js';
+import { selectSubscriptionsQuantitiesFromResponse } from './SubscriptionHelpers.js';
 import { apiError } from '../../move_to_foreman/common/helpers.js';
 import {
   startPollingTask,
@@ -39,6 +40,7 @@ import {
   toastTaskFinished,
 } from '../Tasks/TaskActions';
 import { selectIsPollingTasks } from '../Tasks/TaskSelectors';
+import { pingUpstreamSubscriptions } from './UpstreamSubscriptions/UpstreamSubscriptionsActions';
 
 export const createSubscriptionParams = (extendedParams = {}) => ({
   ...{
@@ -51,10 +53,8 @@ export const createSubscriptionParams = (extendedParams = {}) => ({
 export const loadAvailableQuantities = (extendedParams = {}) => async (dispatch) => {
   dispatch({ type: SUBSCRIPTIONS_QUANTITIES_REQUEST });
 
-  const params = createSubscriptionParams(extendedParams);
-
   try {
-    const { data } = await api.get(`/organizations/${orgId()}/upstream_subscriptions`, {}, params);
+    const { data } = await api.get(`/organizations/${orgId()}/upstream_subscriptions`, {}, propsToSnakeCase(extendedParams));
     return dispatch({
       type: SUBSCRIPTIONS_QUANTITIES_SUCCESS,
       payload: selectSubscriptionsQuantitiesFromResponse(data),
@@ -76,10 +76,6 @@ export const loadSubscriptions = (extendedParams = {}) => async (dispatch) => {
       response: data,
       search: extendedParams.search,
     });
-    const poolIds = filterRHSubscriptions(data.results).map(subs => subs.id);
-    if (poolIds.length > 0) {
-      dispatch(loadAvailableQuantities({ poolIds }));
-    }
     return result;
   } catch (error) {
     return dispatch(apiError(SUBSCRIPTIONS_FAILURE, error));
@@ -110,6 +106,10 @@ export const handleFinishedTask = task => (dispatch) => {
   dispatch(resetTasks());
   dispatch(pollTasks());
   dispatch(loadSubscriptions());
+
+  if (task.label !== MANIFEST_DELETE_TASK_LABEL) {
+    dispatch(pingUpstreamSubscriptions());
+  }
 };
 
 export const handleStartTask = task => (dispatch) => {

data/webpack/scenes/Subscriptions/SubscriptionConstants.js

@@ -37,10 +37,12 @@ export const SUBSCRIPTIONS_CLOSE_DELETE_MODAL = 'SUBSCRIPTIONS_CLOSE_DELETE_MODA
 export const SUBSCRIPTIONS_DISABLE_DELETE_BUTTON = 'SUBSCRIPTIONS_DISABLE_DELETE_BUTTON';
 export const SUBSCRIPTIONS_ENABLE_DELETE_BUTTON = 'SUBSCRIPTIONS_ENABLE_DELETE_BUTTON';
 
+export const MANIFEST_DELETE_TASK_LABEL = 'Actions::Katello::Organization::ManifestDelete';
+
 export const BLOCKING_FOREMAN_TASK_TYPES = [
   'Actions::Katello::Organization::ManifestImport',
   'Actions::Katello::Organization::ManifestRefresh',
-  'Actions::Katello::Organization::ManifestDelete',
+  MANIFEST_DELETE_TASK_LABEL,
   'Actions::Katello::UpstreamSubscriptions::BindEntitlements',
   'Actions::Katello::UpstreamSubscriptions::UpdateEntitlement',
   'Actions::Katello::UpstreamSubscriptions::RemoveEntitlements',

data/webpack/scenes/Subscriptions/SubscriptionReducer.js

@@ -32,9 +32,13 @@ import {
   REFRESH_MANIFEST_SUCCESS,
 } from './Manifest/ManifestConstants';
 
+import {
+  PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS,
+  PING_UPSTREAM_SUBSCRIPTIONS_FAILURE,
+} from './UpstreamSubscriptions/UpstreamSubscriptionsConstants';
+
 const initialState = Immutable({
   ...initialApiState,
-  disconnected: false,
   searchQuery: '',
   deleteModalOpened: false,
   deleteButtonDisabled: true,
@@ -43,10 +47,15 @@ const initialState = Immutable({
   task: null,
   tableColumns: [],
   selectedTableColumns: [],
+  hasUpstreamConnection: false,
 });
 
 export default (state = initialState, action) => {
   switch (action.type) {
+    case PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS:
+      return state.set('hasUpstreamConnection', true);
+    case PING_UPSTREAM_SUBSCRIPTIONS_FAILURE:
+      return state.set('hasUpstreamConnection', false);
     case SUBSCRIPTIONS_REQUEST:
       return state.set('loading', true);
     case SUBSCRIPTIONS_COLUMNS_REQUEST:
@@ -130,6 +139,11 @@ export default (state = initialState, action) => {
     }
 
     case DELETE_MANIFEST_SUCCESS:
+      return state.merge({
+        task: action.response,
+        hasUpstreamConnection: false,
+      });
+
     case UPLOAD_MANIFEST_SUCCESS:
     case REFRESH_MANIFEST_SUCCESS:
     case UPDATE_QUANTITY_SUCCESS:

data/webpack/scenes/Subscriptions/SubscriptionsPage.js

@@ -11,7 +11,7 @@ import ManageManifestModal from './Manifest/';
 import { MANAGE_MANIFEST_MODAL_ID } from './Manifest/ManifestConstants';
 import { SubscriptionsTable } from './components/SubscriptionsTable';
 import SubscriptionsToolbar from './components/SubscriptionsToolbar';
-import { manifestExists } from './SubscriptionHelpers';
+import { filterRHSubscriptions, manifestExists } from './SubscriptionHelpers';
 import api, { orgId } from '../../services/api';
 
 import { createSubscriptionParams } from './SubscriptionActions.js';
@@ -25,6 +25,7 @@ class SubscriptionsPage extends Component {
     super(props);
     this.state = {
       selectedRows: [],
+      availableQuantitiesLoaded: false,
     };
   }
 
@@ -39,8 +40,20 @@ class SubscriptionsPage extends Component {
 
   componentDidUpdate(prevProps) {
     const {
-      organization, task, handleStartTask, handleFinishedTask, isTaskPending, isPollingTask,
+      handleStartTask,
+      handleFinishedTask,
+      isTaskPending,
+      isPollingTask,
+      hasUpstreamConnection,
+      loadAvailableQuantities,
+      organization,
+      pingUpstreamSubscriptions,
+      subscriptions,
+      task,
     } = this.props;
+
+    const { disconnected } = subscriptions;
+
     if (task) {
       if (isPollingTask) {
         if (prevProps.isTaskPending && !isTaskPending) {
@@ -55,6 +68,24 @@ class SubscriptionsPage extends Component {
       if (!prevProps.organization || prevProps.organization.id !== organization.id) {
         this.loadData();
       }
+
+      if (disconnected === false && disconnected !== prevProps.subscriptions.disconnected) {
+        if (manifestExists(organization)) {
+          pingUpstreamSubscriptions();
+          this.state.availableQuantitiesLoaded = false;
+        }
+      }
+    }
+
+    if (hasUpstreamConnection) {
+      const subscriptionsChanged = subscriptions.results !== prevProps.subscriptions.results;
+      if (subscriptionsChanged || !this.state.availableQuantitiesLoaded) {
+        const poolIds = filterRHSubscriptions(subscriptions.results).map(subs => subs.id);
+        if (poolIds.length > 0) {
+          loadAvailableQuantities({ poolIds });
+          this.state.availableQuantitiesLoaded = true;
+        }
+      }
     }
   }
 
@@ -63,7 +94,12 @@ class SubscriptionsPage extends Component {
   }
 
   getDisabledReason(deleteButton) {
-    const { task, subscriptions, organization } = this.props;
+    const {
+      hasUpstreamConnection,
+      task,
+      subscriptions,
+      organization,
+    } = this.props;
     const { disconnected } = subscriptions;
     let disabledReason = null;
 
@@ -75,6 +111,8 @@ class SubscriptionsPage extends Component {
       disabledReason = __('This is disabled because no subscriptions are selected.');
     } else if (!manifestExists(organization)) {
       disabledReason = __('This is disabled because no manifest has been uploaded.');
+    } else if (!hasUpstreamConnection) {
+      disabledReason = __('This is disabled because no connection could be made to the upstream Subscription Allocation.');
     }
 
     return disabledReason;
@@ -106,8 +144,8 @@ class SubscriptionsPage extends Component {
     const {
       deleteModalOpened, openDeleteModal, closeDeleteModal,
       deleteButtonDisabled, disableDeleteButton, enableDeleteButton,
-      searchQuery, updateSearchQuery, simpleContentAccess,
-      task, activePermissions, subscriptions, organization, subscriptionTableSettings,
+      searchQuery, updateSearchQuery, simpleContentAccess, hasUpstreamConnection,
+      task, activePermissions, subscriptions, subscriptionTableSettings,
     } = this.props;
     // Basic permissions - should we even show this page?
     if (subscriptions.missingPermissions && subscriptions.missingPermissions.length > 0) {
@@ -122,7 +160,7 @@ class SubscriptionsPage extends Component {
       canEditOrganizations,
     } = permissions;
     const { disconnected } = subscriptions;
-    const disableManifestActions = !!task || disconnected;
+    const disableManifestActions = !!task || disconnected || !hasUpstreamConnection;
 
     const openManageManifestModal = () => this.props.setModalOpen({ id: MANAGE_MANIFEST_MODAL_ID });
 
@@ -199,7 +237,7 @@ class SubscriptionsPage extends Component {
               disableManifestReason={this.getDisabledReason()}
               disableDeleteButton={deleteButtonDisabled}
               disableDeleteReason={this.getDisabledReason(true)}
-              disableAddButton={!manifestExists(organization)}
+              disableAddButton={disableManifestActions}
               getAutoCompleteParams={getAutoCompleteParams}
               updateSearchQuery={updateSearchQuery}
               onDeleteButtonClick={openDeleteModal}
@@ -245,6 +283,7 @@ class SubscriptionsPage extends Component {
                 task={task}
                 selectedRows={this.state.selectedRows}
                 onSelectedRowsChange={this.handleSelectedRowsChange}
+                selectionEnabled={!disableManifestActions}
               />
               <ModalProgressBar
                 show={!!task}
@@ -261,7 +300,9 @@ class SubscriptionsPage extends Component {
 }
 
 SubscriptionsPage.propTypes = {
+  pingUpstreamSubscriptions: PropTypes.func.isRequired,
   loadSubscriptions: PropTypes.func.isRequired,
+  loadAvailableQuantities: PropTypes.func.isRequired,
   uploadManifest: PropTypes.func.isRequired,
   deleteManifest: PropTypes.func.isRequired,
   resetTasks: PropTypes.func.isRequired,
@@ -270,9 +311,13 @@ SubscriptionsPage.propTypes = {
   simpleContentAccess: PropTypes.bool,
   subscriptions: PropTypes.shape({
     disconnected: PropTypes.bool,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    /* eslint-disable react/forbid-prop-types */
     tableColumns: PropTypes.array,
     selectedTableColumns: PropTypes.array,
     missingPermissions: PropTypes.array,
+    results: PropTypes.array,
+    /* eslint-enable react/forbid-prop-types */
   }).isRequired,
   activePermissions: PropTypes.shape({
     can_delete_manifest: PropTypes.bool,
@@ -302,6 +347,7 @@ SubscriptionsPage.propTypes = {
   cancelPollTasks: PropTypes.func.isRequired,
   handleStartTask: PropTypes.func.isRequired,
   handleFinishedTask: PropTypes.func.isRequired,
+  hasUpstreamConnection: PropTypes.bool,
   loadSetting: PropTypes.func.isRequired,
   loadTables: PropTypes.func.isRequired,
   createColumns: PropTypes.func.isRequired,
@@ -330,6 +376,7 @@ SubscriptionsPage.defaultProps = {
   deleteButtonDisabled: true,
   subscriptionTableSettings: {},
   simpleContentAccess: false,
+  hasUpstreamConnection: false,
   activePermissions: {
     can_import_manifest: false,
     can_manage_subscription_allocations: false,

data/webpack/scenes/Subscriptions/SubscriptionsSelectors.js

@@ -26,3 +26,6 @@ export const selectIsTaskPending = (state) => {
 
 export const selectTableSettings = (state, tableName) =>
   state.katello.settings.tables[tableName] || undefined;
+
+export const selectHasUpstreamConnection = state =>
+  selectSubscriptionsState(state).hasUpstreamConnection;

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/UpstreamSubscriptionsActions.js

@@ -10,7 +10,21 @@ import {
   SAVE_UPSTREAM_SUBSCRIPTIONS_REQUEST,
   SAVE_UPSTREAM_SUBSCRIPTIONS_SUCCESS,
   SAVE_UPSTREAM_SUBSCRIPTIONS_FAILURE,
-} from './UpstreamSubscriptionsContstants';
+  PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS,
+  PING_UPSTREAM_SUBSCRIPTIONS_FAILURE,
+} from './UpstreamSubscriptionsConstants';
+
+export const pingUpstreamSubscriptions = () => async (dispatch) => {
+  try {
+    const { data } = await api.get(`/organizations/${orgId()}/upstream_subscriptions/ping`);
+    return dispatch({
+      type: PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS,
+      payload: data,
+    });
+  } catch (error) {
+    return dispatch(apiError(PING_UPSTREAM_SUBSCRIPTIONS_FAILURE, error));
+  }
+};
 
 export const loadUpstreamSubscriptions = (extendedParams = {}) => async (dispatch) => {
   dispatch({ type: UPSTREAM_SUBSCRIPTIONS_REQUEST });

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/{UpstreamSubscriptionsContstants.js → UpstreamSubscriptionsConstants.js}

@@ -5,3 +5,6 @@ export const UPSTREAM_SUBSCRIPTIONS_FAILURE = 'UPSTREAM_SUBSCRIPTIONS_FAILURE';
 export const SAVE_UPSTREAM_SUBSCRIPTIONS_REQUEST = 'SAVE_UPSTREAM_SUBSCRIPTIONS_REQUEST';
 export const SAVE_UPSTREAM_SUBSCRIPTIONS_SUCCESS = 'SAVE_UPSTREAM_SUBSCRIPTIONS_SUCCESS';
 export const SAVE_UPSTREAM_SUBSCRIPTIONS_FAILURE = 'SAVE_UPSTREAM_SUBSCRIPTIONS_FAILURE';
+
+export const PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS = 'PING_UPSTREAM_SUBSCRIPTIONS_SUCCESS';
+export const PING_UPSTREAM_SUBSCRIPTIONS_FAILURE = 'PING_UPSTREAM_SUBSCRIPTIONS_FAILURE';

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/UpstreamSubscriptionsPage.js

@@ -262,6 +262,8 @@ UpstreamSubscriptionsPage.propTypes = {
   upstreamSubscriptions: PropTypes.shape({
     loading: PropTypes.bool,
     itemCount: PropTypes.number,
+    // Disabling rule as existing code failed due to an eslint-plugin-react update
+    // eslint-disable-next-line react/forbid-prop-types
     results: PropTypes.array,
     pagination: PropTypes.shape({}),
     task: PropTypes.shape({

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/UpstreamSubscriptionsReducer.js

@@ -7,7 +7,7 @@ import {
   SAVE_UPSTREAM_SUBSCRIPTIONS_REQUEST,
   SAVE_UPSTREAM_SUBSCRIPTIONS_SUCCESS,
   SAVE_UPSTREAM_SUBSCRIPTIONS_FAILURE,
-} from './UpstreamSubscriptionsContstants';
+} from './UpstreamSubscriptionsConstants';
 
 const initialState = initialApiState;
 

data/webpack/scenes/Subscriptions/UpstreamSubscriptions/__tests__/UpstreamSubscriptionsReducer.test.js

@@ -1,4 +1,4 @@
-import * as types from '../UpstreamSubscriptionsContstants';
+import * as types from '../UpstreamSubscriptionsConstants';
 
 import {
   initialState,