foreman_rh_cloud 3.0.19 → 3.0.23

data/webpack/ForemanInventoryUpload/Components/InventorySettings/AdvancedSetting/AdvancedSettingsConstants.js

@@ -3,7 +3,7 @@ import { translate as __ } from 'foremanReact/common/I18n';
 export const settingsDict = {
   autoUploadEnabled: {
     name: 'allow_auto_inventory_upload',
-    label: __('Auto upload'),
+    label: __('Automatic inventory upload'),
     tooltip: __(
       'Enable automatic upload of your hosts inventory to the Red Hat cloud'
     ),
@@ -20,7 +20,9 @@ export const settingsDict = {
   },
   excludePackagesEnabled: {
     name: 'exclude_installed_packages',
-    label: __('Exclude Packages'),
-    tooltip: __('Exclude packages from being uploaded to the Red Hat cloud'),
+    label: __('Exclude installed Packages'),
+    tooltip: __(
+      'Exclude installed packages from being uploaded to the Red Hat cloud'
+    ),
   },
 };

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/PageDescription/PageDescription.js

@@ -10,14 +10,38 @@ export const PageDescription = () => (
     </p>
     <p>
       {__(
-        'In order to utilize these services, you can set the auto upload in the settings to "ON".'
+        'You can toggle the Auto upload switch to the ON position to enable Satellite to automatically upload your host inventory once a day.'
       )}
     </p>
     <p>
       {__(
-        'You can also trigger the upload manually by opening the relevant organization card, and clicking on the "Restart" button'
+        'Click Restart to upload your host inventory to Red Hat Insights. Perform this step for each organization from which you want to manually upload a host inventory.'
       )}
     </p>
+    <p>
+      {__(
+        'Enabling inventory uploads is required by subscription watch. For more information about subscription watch see link:'
+      )}
+      &nbsp;
+      <a
+        href="https://access.redhat.com/documentation/en-us/subscription_central/2020-04/html/getting_started_with_subscription_watch/assembly-about-subscriptionwatch"
+        target="_blank"
+        rel="noopener noreferrer"
+      >
+        About subscription watch
+      </a>
+    </p>
+    <p>
+      {__('For more information about Insights and Cloud Connector read')}
+      &nbsp;
+      <a
+        href="https://console.redhat.com/security/insights/"
+        target="_blank"
+        rel="noopener noreferrer"
+      >
+        Red Hat Insights Data and Application Security
+      </a>
+    </p>
   </div>
 );
 

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/PageDescription/__tests__/__snapshots__/PageDescription.test.js.snap

@@ -8,10 +8,32 @@ exports[`PageDescription rendering render without Props 1`] = `
     Red Hat Insights is a set of cloud services which provide unified subscription reporting, predictive analysis and remediation of issues through this Satellite instance.
   </p>
   <p>
-    In order to utilize these services, you can set the auto upload in the settings to "ON".
+    You can toggle the Auto upload switch to the ON position to enable Satellite to automatically upload your host inventory once a day.
   </p>
   <p>
-    You can also trigger the upload manually by opening the relevant organization card, and clicking on the "Restart" button
+    Click Restart to upload your host inventory to Red Hat Insights. Perform this step for each organization from which you want to manually upload a host inventory.
+  </p>
+  <p>
+    Enabling inventory uploads is required by subscription watch. For more information about subscription watch see link:
+     
+    <a
+      href="https://access.redhat.com/documentation/en-us/subscription_central/2020-04/html/getting_started_with_subscription_watch/assembly-about-subscriptionwatch"
+      rel="noopener noreferrer"
+      target="_blank"
+    >
+      About subscription watch
+    </a>
+  </p>
+  <p>
+    For more information about Insights and Cloud Connector read
+     
+    <a
+      href="https://console.redhat.com/security/insights/"
+      rel="noopener noreferrer"
+      target="_blank"
+    >
+      Red Hat Insights Data and Application Security
+    </a>
   </p>
 </div>
 `;

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/SyncButtonActions.js

@@ -1,6 +1,5 @@
 import React from 'react';
-import { get, post } from 'foremanReact/redux/API';
-import { withInterval } from 'foremanReact/redux/middlewares/IntervalMiddleware';
+import { post } from 'foremanReact/redux/API';
 import { addToast } from 'foremanReact/redux/actions/toasts';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { inventoryUrl } from '../../../../ForemanInventoryHelpers';
@@ -9,7 +8,10 @@ import {
   INVENTORY_SYNC,
   INVENTORY_SYNC_TASK_UPDATE,
 } from './SyncButtonConstants';
-import { foremanUrl } from '../../../../../ForemanRhCloudHelpers';
+import {
+  setupTaskPolling,
+  taskRelatedToast,
+} from '../../../../../common/ForemanTasks';
 
 export const handleSync = () => dispatch => {
   dispatch(
@@ -21,9 +23,9 @@ export const handleSync = () => dispatch => {
           task: { id },
         },
       }) => {
-        dispatch(getSyncTaskInterval(id));
+        dispatch(setupInventorySyncTaskPolling(id, dispatch));
         return dispatch(
-          taskPageRefererToast(id, 'info', __('Inventory sync has started:'))
+          taskRelatedToast(id, 'info', __('Inventory sync has started:'))
         );
       },
       errorToast: inventorySyncErrorToast,
@@ -31,62 +33,25 @@ export const handleSync = () => dispatch => {
   );
 };
 
-export const getSyncTaskInterval = id => dispatch => {
-  dispatch(
-    withInterval(
-      get({
-        key: INVENTORY_SYNC_TASK_UPDATE,
-        url: inventoryUrl(`tasks/${id}`),
-        handleSuccess: ({ data: { result, output } }, stopTaskInterval) => {
-          if (result === 'success') {
-            const {
-              host_statuses: { sync, disconnect },
-            } = output;
-            dispatch(
-              addToast({
-                sticky: true,
-                type: 'success',
-                message: (
-                  <Toast syncHosts={sync} disconnectHosts={disconnect} />
-                ),
-              })
-            );
-          }
-          if (result === 'error') {
-            dispatch(
-              taskPageRefererToast(
-                id,
-                'error',
-                __('Inventory sync has failed:'),
-                true
-              )
-            );
-          }
-          stopTaskInterval();
-        },
-        errorToast: inventorySyncErrorToast,
-      })
-    )
-  );
-};
-
-const inventorySyncErrorToast = ({ message, response }) =>
-  `${__('Inventory sync has failed: ')} ${response.data?.message || message}`;
-
-const taskPageRefererToast = (taskID, toastType, prefix, sticky = false) =>
-  addToast({
-    sticky,
-    type: toastType,
-    message: (
-      <span>
-        {prefix}{' '}
-        <a
-          target="_blank"
-          rel="noopener noreferrer"
-          href={foremanUrl(`/foreman_tasks/tasks/${taskID}`)}
-        >
-          {__('view the task page for more details')}
-        </a>
-      </span>
-    ),
+export const setupInventorySyncTaskPolling = (id, dispatch) =>
+  setupTaskPolling({
+    taskId: id,
+    key: INVENTORY_SYNC_TASK_UPDATE,
+    onTaskSuccess: ({
+      output: {
+        host_statuses: { sync, disconnect },
+      },
+    }) =>
+      dispatch(
+        addToast({
+          sticky: true,
+          type: 'success',
+          message: <Toast syncHosts={sync} disconnectHosts={disconnect} />,
+        })
+      ),
+    dispatch,
   });
+
+const inventorySyncErrorToast = message =>
+  `${__('Inventory sync has failed: ')} ${message.response?.data?.message ||
+    message}`;

data/webpack/ForemanInventoryUpload/Components/PageHeader/components/SyncButton/__tests__/__snapshots__/integrations.test.js.snap

@@ -22,7 +22,7 @@ Array [
       "errorToast": [Function],
       "interval": 3000,
       "type": "API_GET",
-      "url": "/foreman_inventory_upload/tasks/1",
+      "url": "/foreman_tasks/api/tasks/1/details?include_permissions",
     },
   ],
   Array [
@@ -31,7 +31,7 @@ Array [
         "message": Object {
           "message": <span>
             Inventory sync has started:
-             
+            <br />
             <a
               href="/foreman_tasks/tasks/1"
               rel="noopener noreferrer"
@@ -40,7 +40,6 @@ Array [
               view the task page for more details
             </a>
           </span>,
-          "sticky": false,
           "type": "info",
         },
       },

data/webpack/ForemanInventoryUpload/Components/Terminal/Terminal.js

@@ -72,7 +72,7 @@ class Terminal extends React.Component {
     return (
       <Grid.Col sm={12}>
         <div
-          className="terminal"
+          className="rh-cloud-inventory-terminal"
           ref={this.terminal}
           onScroll={this.handleScroll}
         >

data/webpack/ForemanInventoryUpload/Components/Terminal/__tests__/Terminal.test.js

@@ -29,6 +29,6 @@ describe('Terminal', () => {
     const text = 'some-string-log';
     const modifiedProps = { ...props, logs: text };
     const wrapper = mount(<Terminal {...modifiedProps} />);
-    expect(wrapper.find('.terminal p').text()).toEqual(text);
+    expect(wrapper.find('.rh-cloud-inventory-terminal p').text()).toEqual(text);
   });
 });

data/webpack/ForemanInventoryUpload/Components/Terminal/__tests__/__snapshots__/Terminal.test.js.snap

@@ -7,7 +7,7 @@ exports[`Terminal rendering render with props 1`] = `
   sm={12}
 >
   <div
-    className="terminal"
+    className="rh-cloud-inventory-terminal"
     onScroll={[Function]}
   >
     <Grid
@@ -65,7 +65,7 @@ exports[`Terminal rendering render without Props 1`] = `
   sm={12}
 >
   <div
-    className="terminal"
+    className="rh-cloud-inventory-terminal"
     onScroll={[Function]}
   >
     <Grid

data/webpack/ForemanInventoryUpload/Components/Terminal/terminal.scss

@@ -1,34 +1,32 @@
-.rh-cloud-inventory-page {
-  .terminal {
-    height: 200px;
-    background-color: #222;
-    padding: 10px 0;
-    margin-bottom: 20px;
-    overflow-y: scroll;
-    overflow-x: hidden;
+.rh-cloud-inventory-terminal {
+  height: 200px;
+  background-color: #222;
+  padding: 10px 0;
+  margin-bottom: 20px;
+  overflow-y: scroll;
+  overflow-x: hidden;
 
-    p {
-      font-family: monospace;
-      font-size: 16px;
-      color: #22da26;
-      overflow-wrap: anywhere;
+  p {
+    font-family: monospace;
+    font-size: 16px;
+    color: #22da26;
+    overflow-wrap: anywhere;
 
-      &.terminal_error {
-        color: #f00;
-      }
+    &.terminal_error {
+      color: #f00;
     }
+  }
 
-    &::-webkit-scrollbar {
-      width: 12px;
-      height: 12px;
-      background: #aaa;
-    }
+  &::-webkit-scrollbar {
+    width: 12px;
+    height: 12px;
+    background: #aaa;
+  }
 
-    &::-webkit-scrollbar-thumb {
-      background: #222;
-      border-radius: 6px;
-      border: 3px solid transparent;
-      background-clip: content-box;
-    }
+  &::-webkit-scrollbar-thumb {
+    background: #222;
+    border-radius: 6px;
+    border: 3px solid transparent;
+    background-clip: content-box;
   }
 }

data/webpack/InsightsCloudSync/Components/InsightsTable/InsightsTable.js

@@ -6,7 +6,7 @@ import { Table, TableHeader, TableBody } from '@patternfly/react-table';
 import { useForemanSettings } from 'foremanReact/Root/Context/ForemanContext';
 import SelectAllAlert from './SelectAllAlert';
 import { columns } from './InsightsTableConstants';
-import TableEmptyState from './components/EmptyState';
+import TableEmptyState from '../../../common/table/EmptyState';
 import {
   modifySelectedRows,
   getSortColumnIndex,

data/webpack/InsightsCloudSync/Components/InsightsTable/InsightsTableActions.js

@@ -19,25 +19,6 @@ export const fetchInsights = (queryParams = {}) => (dispatch, getState) => {
     ...queryParams,
   };
 
-  dispatch(
-    get({
-      key: INSIGHTS_HITS_API_KEY,
-      url: INSIGHTS_HITS_PATH,
-      params: {
-        page,
-        per_page: perPage,
-        search: query,
-        order: `${sortBy} ${sortOrder}`,
-      },
-      handleSuccess: response => {
-        if (isSelectAll) {
-          selectAllIds(dispatch, response.data.hits || []);
-          dispatch(selectAll());
-        }
-      },
-    })
-  );
-
   const uri = new URI();
   uri.search({
     page,
@@ -58,6 +39,25 @@ export const fetchInsights = (queryParams = {}) => (dispatch, getState) => {
   if (!isSelectAll) {
     dispatch(setSelectAllAlert(false));
   }
+
+  return dispatch(
+    get({
+      key: INSIGHTS_HITS_API_KEY,
+      url: INSIGHTS_HITS_PATH,
+      params: {
+        page,
+        per_page: perPage,
+        search: query,
+        order: `${sortBy} ${sortOrder}`,
+      },
+      handleSuccess: response => {
+        if (isSelectAll) {
+          selectAllIds(dispatch, response.data.hits || []);
+          dispatch(selectAll());
+        }
+      },
+    })
+  );
 };
 
 const selectAllIds = (dispatch, results, prevSelectedIds = {}) => {

data/webpack/InsightsCloudSync/Components/InsightsTable/InsightsTableSelectors.js

@@ -76,3 +76,6 @@ export const selectItemCount = state =>
 
 export const selectHasToken = state =>
   selectAPIResponse(state, INSIGHTS_HITS_API_KEY).hasToken;
+
+export const selectExperimental = state =>
+  selectAPIResponse(state, INSIGHTS_HITS_API_KEY).isExperimentalMode;

data/webpack/InsightsCloudSync/Components/InsightsTable/__tests__/__snapshots__/InsightsTableActions.test.js.snap

@@ -26,6 +26,20 @@ Array [
 
 exports[`insights table actions should fetchInsights 1`] = `
 Array [
+  Array [
+    Object {
+      "payload": Object {
+        "args": Array [
+          Object {
+            "pathname": "/foreman_rh_cloud/insights_cloud",
+            "search": "?page=2&per_page=7&search=&sort_by=&sort_order=&select_all=true",
+          },
+        ],
+        "method": "push",
+      },
+      "type": "@@router/CALL_HISTORY_METHOD",
+    },
+  ],
   Array [
     Object {
       "payload": Object {
@@ -61,20 +75,6 @@ Array [
       "url": "/insights_cloud/hits",
     },
   ],
-  Array [
-    Object {
-      "payload": Object {
-        "args": Array [
-          Object {
-            "pathname": "/foreman_rh_cloud/insights_cloud",
-            "search": "?page=2&per_page=7&search=&sort_by=&sort_order=&select_all=true",
-          },
-        ],
-        "method": "push",
-      },
-      "type": "@@router/CALL_HISTORY_METHOD",
-    },
-  ],
 ]
 `;