foreman_rh_cloud 12.2.1 → 12.2.3

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModal.js

@@ -1,5 +1,6 @@
 /* eslint-disable react-hooks/exhaustive-deps */
 import React, { useEffect } from 'react';
+import Immutable from 'seamless-immutable';
 import PropTypes from 'prop-types';
 import {
   Table,
@@ -7,7 +8,7 @@ import {
   TableBody,
 } from '@patternfly/react-table/deprecated';
 import { Modal, ModalVariant, Button } from '@patternfly/react-core';
-import { isEmpty } from 'lodash';
+import { isEmpty, noop } from 'lodash';
 import { STATUS } from 'foremanReact/constants';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { columns } from './RemediationTableConstants';
@@ -15,8 +16,48 @@ import { modifyRows } from './RemediationHelpers';
 import ModalFooter from './RemediationModalFooter';
 import TableEmptyState from '../../../common/table/EmptyState';
 import './RemediationModal.scss';
+import { useAdvisorEngineConfig } from '../../../common/Hooks/ConfigHooks';
+
+/* eslint-disable spellcheck/spell-checker */
+
+// Sample iopData:
+// const iopTestData = Immutable([
+//   {
+//     hostid: 'c7c6727e-2966-4f7c-87f1-20ef14db7a2d',
+//     host_name: 'advisor-test.local',
+//     rulename: 'hardening_cryptopol_krb5|NO_CPOL_KRB5',
+//     resolutions: [
+//       {
+//         description: 'Remove manual crypto-policies',
+//         id: 'fix',
+//         needs_reboot: true,
+//         resolution_risk: 1,
+//       },
+//     ],
+//     rebootable: true,
+//     description: 'Decreased security: krb5 crypto-policies overridden',
+//   },
+//   {
+//     hostid: 'c7c6727e-2966-4f7c-87f1-20ef14db7a2d',
+//     host_name: 'advisor-test.local',
+//     rulename: 'hardening_logging_auditd|HARDENING_LOGGING_5_AUDITD',
+//     resolutions: [
+//       {
+//         description: 'Install and enable auditd',
+//         id: 'fix',
+//         needs_reboot: false,
+//         resolution_risk: 1,
+//       },
+//     ],
+//     rebootable: false,
+//     description: 'Decreased security: auditd not running',
+//   },
+// ]);
+
+/* eslint-enable spellcheck/spell-checker */
 
 const RemediationModal = ({
+  iopData,
   selectedIds,
   fetchRemediations,
   remediations,
@@ -24,31 +65,49 @@ const RemediationModal = ({
   error,
   isAllSelected,
   query,
+  isDisabled,
 }) => {
-  const [rows, setRows] = React.useState([]);
+  const iopRows = Immutable(iopData ?? []).map(recommendation => ({
+    id: recommendation.rulename,
+    host_id: recommendation.hostid,
+    hostname: recommendation.host_name,
+    title: recommendation.description,
+    resolutions: recommendation.resolutions ?? [],
+    reboot: recommendation.rebootable,
+  }));
+
   const [open, setOpen] = React.useState(false);
   const [resolutions, setResolutions] = React.useState([]);
   const [hostsIds, setHostsIds] = React.useState([]);
+  const [rows, setRows] = React.useState([]);
   const toggleModal = () => setOpen(prevValue => !prevValue);
 
+  const isIop = useAdvisorEngineConfig();
   useEffect(() => {
-    if (open) fetchRemediations({ selectedIds, isAllSelected, query });
+    // only fetch for Hosted. IoP provides via props.
+    if (!isIop && open)
+      fetchRemediations({ selectedIds, isAllSelected, query });
   }, [open]);
 
   useEffect(() => {
-    const modifiedRows =
-      status === STATUS.PENDING
-        ? []
-        : modifyRows(remediations, setResolutions, setHostsIds);
+    let modifiedRows;
+    if (isIop) {
+      modifiedRows = modifyRows(iopRows, setResolutions, setHostsIds, true);
+    } else {
+      modifiedRows =
+        status === STATUS.PENDING
+          ? []
+          : modifyRows(remediations, setResolutions, setHostsIds, false);
+    }
     setRows(modifiedRows);
-  }, [remediations, status]);
+  }, [remediations, status, iopData, isIop]);
 
   return (
     <React.Fragment>
       <Button
         ouiaId="button-remediate"
         variant="primary"
-        isDisabled={isEmpty(selectedIds)}
+        isDisabled={isDisabled || isEmpty(selectedIds)}
         onClick={() => {
           toggleModal();
         }}
@@ -68,6 +127,7 @@ const RemediationModal = ({
             toggleModal={toggleModal}
             resolutions={resolutions}
             hostsIds={hostsIds}
+            isIop={isIop}
           />
         }
       >
@@ -92,22 +152,36 @@ const RemediationModal = ({
 };
 
 RemediationModal.propTypes = {
-  selectedIds: PropTypes.object,
-  fetchRemediations: PropTypes.func.isRequired,
+  iopData: PropTypes.arrayOf(
+    PropTypes.shape({
+      hostid: PropTypes.string,
+      host_name: PropTypes.string,
+      rulename: PropTypes.string,
+      resolutions: PropTypes.array,
+      rebootable: PropTypes.bool,
+      description: PropTypes.string,
+    })
+  ),
+  selectedIds: PropTypes.shape({}),
+  fetchRemediations: PropTypes.func,
   remediations: PropTypes.array,
   status: PropTypes.string,
   error: PropTypes.string,
   isAllSelected: PropTypes.bool,
   query: PropTypes.string,
+  isDisabled: PropTypes.bool,
 };
 
 RemediationModal.defaultProps = {
+  iopData: null,
   selectedIds: {},
+  fetchRemediations: noop,
   remediations: [],
   status: null,
   error: null,
   isAllSelected: false,
   query: null,
+  isDisabled: false,
 };
 
 export default RemediationModal;

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModalFooter.js

@@ -2,18 +2,44 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Button } from '@patternfly/react-core';
 import { translate as __ } from 'foremanReact/common/I18n';
+import { useBulkSelect } from 'foremanReact/components/PF4/TableIndexPage/Table/TableHooks';
 import { JOB_INVOCATION_PATH } from './RemediationTableConstants';
 
-const ModalFooter = ({ toggleModal, resolutions, hostsIds }) => {
+const ModalFooter = ({ toggleModal, resolutions, hostsIds, isIop }) => {
   let token = document.querySelector('meta[name="csrf-token"]');
   token = token?.content || '';
+
+  const [jobInProgress, setJobInProgress] = React.useState(false);
+  const formRef = React.useRef(null);
+
+  const { fetchBulkParams } = useBulkSelect({
+    initialArry: hostsIds,
+    idColumn: 'insights_uuid',
+  });
+
+  const handleSubmit = e => {
+    e.preventDefault();
+    setJobInProgress(true);
+
+    setTimeout(() => {
+      // eslint-disable-next-line no-unused-expressions
+      formRef.current?.submit?.();
+    }, 100);
+  };
   return (
-    <form action={JOB_INVOCATION_PATH} method="post">
+    <form
+      action={JOB_INVOCATION_PATH}
+      method="post"
+      ref={formRef}
+      onSubmit={handleSubmit}
+    >
       <Button
         type="submit"
         ouiaId="button-confirm"
         key="confirm"
         variant="primary"
+        isDisabled={jobInProgress}
+        isLoading={jobInProgress}
       >
         {__('Remediate')}
       </Button>
@@ -32,9 +58,15 @@ const ModalFooter = ({ toggleModal, resolutions, hostsIds }) => {
         name="inputs[hit_remediation_pairs]"
         value={JSON.stringify(resolutions)}
       />
-      {hostsIds.map(id => (
-        <input type="hidden" name="host_ids[]" key={id} value={id} />
-      ))}
+      {!isIop &&
+        hostsIds.map(id => (
+          <input type="hidden" name="host_ids[]" key={id} value={id} />
+        ))}
+      {isIop && (
+        <>
+          <input type="hidden" name="search" value={fetchBulkParams()} />
+        </>
+      )}
     </form>
   );
 };
@@ -43,11 +75,13 @@ ModalFooter.propTypes = {
   toggleModal: PropTypes.func.isRequired,
   resolutions: PropTypes.array,
   hostsIds: PropTypes.array,
+  isIop: PropTypes.bool,
 };
 
 ModalFooter.defaultProps = {
   resolutions: [],
   hostsIds: [],
+  isIop: false,
 };
 
 export default ModalFooter;

data/webpack/InsightsCloudSync/Components/RemediationModal/Resolutions.js

@@ -2,12 +2,14 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { Radio } from '@patternfly/react-core';
+import { getResolutionId } from './RemediationHelpers';
 
 const Resolutions = ({
   resolutions,
   setResolutions,
   selectedResolution,
   hit_id,
+  isIop,
 }) => {
   const [checkedID, setCheckedID] = React.useState(selectedResolution);
 
@@ -27,6 +29,15 @@ const Resolutions = ({
               stateRes.map(res => {
                 if (hit_id === res.hit_id) {
                   setCheckedID(resolution_id);
+                  if (isIop)
+                    return {
+                      ...res,
+                      resolution_id: getResolutionId(
+                        resolution_id,
+                        res.rule_id
+                      ),
+                      resolution_type: resolution_id,
+                    };
                   return { ...res, resolution_id };
                 }
                 return res;
@@ -45,12 +56,14 @@ Resolutions.propTypes = {
   resolutions: PropTypes.array,
   hit_id: PropTypes.number,
   selectedResolution: PropTypes.number,
+  isIop: PropTypes.bool,
 };
 
 Resolutions.defaultProps = {
   resolutions: [],
   hit_id: null,
   selectedResolution: null,
+  isIop: false,
 };
 
 export default Resolutions;

data/webpack/InsightsCloudSync/InsightsCloudSync.js

@@ -67,13 +67,15 @@ export const generateRuleUrl = ruleId =>
   foremanUrl(`/foreman_rh_cloud/recommendations/${ruleId}`);
 
 const IopRecommendationsPage = props => (
-  <ScalprumComponent
-    scope={scope}
-    module={module}
-    IopRemediationModal={RemediationModal}
-    generateRuleUrl={generateRuleUrl}
-    {...props}
-  />
+  <div className="advisor">
+    <ScalprumComponent
+      scope={scope}
+      module={module}
+      IopRemediationModal={RemediationModal}
+      generateRuleUrl={generateRuleUrl}
+      {...props}
+    />
+  </div>
 );
 
 const IopRecommendationsPageWrapped = props => (

data/webpack/InsightsHostDetailsTab/NewHostDetailsTab.js

@@ -110,16 +110,18 @@ NewHostDetailsTab.defaultProps = {
 
 // Local Insights advisor
 const scope = 'advisor';
-const module = './SystemDetailsWrapped';
+const module = './SystemDetailWrapped';
 
 const IopInsightsTab = props => (
-  <ScalprumComponent
-    scope={scope}
-    module={module}
-    IopRemediationModal={RemediationModal}
-    generateRuleUrl={generateRuleUrl}
-    {...props}
-  />
+  <div className="advisor">
+    <ScalprumComponent
+      scope={scope}
+      module={module}
+      IopRemediationModal={RemediationModal}
+      generateRuleUrl={generateRuleUrl}
+      {...props}
+    />
+  </div>
 );
 
 const IopInsightsTabWrapped = props => (
@@ -132,7 +134,7 @@ const InsightsTab = props => {
   const { response } = props;
   const isLocalAdvisorEngine =
     // eslint-disable-next-line camelcase
-    response?.insights_attributes?.use_local_advisor_engine;
+    response?.insights_attributes?.use_iop_mode;
 
   return isLocalAdvisorEngine ? (
     <IopInsightsTabWrapped {...props} />
@@ -144,7 +146,7 @@ const InsightsTab = props => {
 InsightsTab.propTypes = {
   response: PropTypes.shape({
     insights_attributes: {
-      use_local_advisor_engine: PropTypes.bool,
+      use_iop_mode: PropTypes.bool,
     },
   }),
 };

data/webpack/InsightsVulnerability/InsightsVulnerabilityListPage.js

@@ -6,7 +6,7 @@ const InsightsVulnerabilityListPage = () => {
   const scope = 'vulnerability';
   const module = './CveListPage';
   return (
-    <div className="rh-cloud-insights-vulnerability-page">
+    <div className="rh-cloud-insights-vulnerability-page vulnerability">
       <ScalprumComponent scope={scope} module={module} />
     </div>
   );

data/webpack/IopRecommendationDetails/IopRecommendationDetails.js

@@ -13,7 +13,7 @@ const IopRecommendationDetails = props => {
   // eslint-disable-next-line camelcase
   const ruleId = urlParams?.params?.rule_id;
   return (
-    <div className="iop-recommendation-details-scalprum">
+    <div className="iop-recommendation-details-scalprum advisor">
       <ScalprumComponent
         scope={scope}
         module={module}

data/webpack/common/Hooks/ConfigHooks.js

@@ -14,6 +14,5 @@ export const useAdvisorEngineConfig = () => {
   );
 
   // eslint-disable-next-line camelcase
-  const isLocalAdvisorEngine = advisorEngineConfig?.use_local_advisor_engine;
-  return isLocalAdvisorEngine;
+  return advisorEngineConfig?.use_iop_mode;
 };

data/webpack/common/styles.scss

@@ -0,0 +1,7 @@
+// overwrite bootstrap style which added unintended bottom margins to chips
+// and dropdowns in Advisor and Vulnerability
+.vulnerability, .advisor {
+  ul, ol {
+    margin-bottom: 0px;
+  }
+}

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 12.2.1
+  version: 12.2.3
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
@@ -98,6 +98,7 @@ files:
 - app/controllers/api/v2/rh_cloud/advisor_engine_config_controller.rb
 - app/controllers/api/v2/rh_cloud/cloud_request_controller.rb
 - app/controllers/api/v2/rh_cloud/inventory_controller.rb
+- app/controllers/concerns/foreman_rh_cloud/iop_smart_proxy_access.rb
 - app/controllers/concerns/insights_cloud/candlepin_cache.rb
 - app/controllers/concerns/insights_cloud/client_authentication.rb
 - app/controllers/concerns/insights_cloud/package_profile_upload_extensions.rb
@@ -110,6 +111,7 @@ files:
 - app/controllers/foreman_inventory_upload/tasks_controller.rb
 - app/controllers/foreman_inventory_upload/uploads_controller.rb
 - app/controllers/foreman_inventory_upload/uploads_settings_controller.rb
+- app/controllers/foreman_rh_cloud/foreman_rh_cloud_controller.rb
 - app/controllers/insights_cloud/api/machine_telemetries_controller.rb
 - app/controllers/insights_cloud/hits_controller.rb
 - app/controllers/insights_cloud/settings_controller.rb
@@ -119,6 +121,7 @@ files:
 - app/helpers/foreman_inventory_upload_helper.rb
 - app/helpers/foreman_inventory_upload_host_helper.rb
 - app/models/concerns/rh_cloud_host.rb
+- app/models/foreman_rh_cloud/ping.rb
 - app/models/insights_client_report_status.rb
 - app/models/insights_facet.rb
 - app/models/insights_hit.rb
@@ -662,6 +665,7 @@ files:
 - webpack/common/Switcher/__tests__/__snapshots__/HelpLabel.test.js.snap
 - webpack/common/Switcher/__tests__/__snapshots__/SwitcherPF4.test.js.snap
 - webpack/common/Switcher/index.js
+- webpack/common/styles.scss
 - webpack/common/table/EmptyState.js
 - webpack/common/table/helpers.js
 - webpack/global_index.js