foreman_rh_cloud 4.0.24.1 → 4.0.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5de90e40ec6d14cf5483a2f92e46c61488c01a921a89feaa8194015f09c6fd0d
-  data.tar.gz: 3ac8ba217d2356211e25c1f1de144ad72578ba0be29116ce1d611817302fc097
+  metadata.gz: 4ac06b57bc84d2c31a7a2e5ad8ec8874053bf4419f018c94b95192ba7281ccf4
+  data.tar.gz: 1e8dd2dce4252a247123e0662f6b33621fadf34e3d03a008335d1902dc7d69db
 SHA512:
-  metadata.gz: 051caa1c8e5d69a8dad5aa77570b79854854d5ae10f810afe2f57dc6d0f2be0a8de1090caab8ceb0a3faea06b2cb42bad3beda18097792b62e191caa6bb3b41d
-  data.tar.gz: 5f53fcc4bfdcc134afe56d19ac8a6c46822b2a6aeab832296eb0c7ffbe462d3a0510f2aaf2d0548f4b0f212f2b44e893fb12a04dace6c35fdc039071a7dd70b2
+  metadata.gz: e7f0324f7854d483ce3c8836fcc5a70af02ed5f3f1bb64e3e1fcb68603eb4daa078b7cb7090ba4b9bcda6c4003639d5cd7265432f5d0b60b89cd5bccaf521fe3
+  data.tar.gz: f8e8b5317c59dc110e7315ba8bf665d7bdeda836e899dc81c1c7bcecdcd81a4568b0c8ad9849e69b9f74dd4cf7127ca097f234001b1492992d677d4cd5b44939

data/app/services/foreman_rh_cloud/cloud_auth.rb

@@ -4,6 +4,10 @@ module ForemanRhCloud
 
     include CloudRequest
 
+    def cloud_auth_available?
+      Setting[:rh_cloud_token].present?
+    end
+
     def rh_credentials
       @rh_credentials ||= query_refresh_token
     end

data/app/services/foreman_rh_cloud/remediations_retriever.rb

@@ -12,6 +12,11 @@ module ForemanRhCloud
     end
 
     def create_playbook
+      unless cloud_auth_available?
+        logger.debug('Cloud authentication is not available, cannot continue')
+        return
+      end
+
       response = query_playbook
 
       logger.debug("Got playbook response: #{response.body}")

data/lib/foreman_inventory_upload/generators/tags.rb

@@ -19,7 +19,9 @@ module ForemanInventoryUpload
       def generate_parameters
         return [] unless Setting[:include_parameter_tags]
 
-        (@host.host_inherited_params_objects || []).map { |item| [item.name, item.value] }
+        (@host.host_inherited_params_objects || [])
+          .map { |item| [item.name, item.value] }
+          .select { |_name, value| value.present? || value.is_a?(FalseClass) }
       end
 
       private

data/lib/foreman_rh_cloud/version.rb

@@ -1,3 +1,3 @@
 module ForemanRhCloud
-  VERSION = '4.0.24.1'.freeze
+  VERSION = '4.0.25'.freeze
 end

data/lib/insights_cloud/async/insights_full_sync.rb

@@ -6,6 +6,11 @@ module InsightsCloud
       include ::ForemanRhCloud::CloudAuth
 
       def plan
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping insights sync')
+          return
+        end
+
         sequence do
           # This can be turned off when we enable automatic status syncs
           # This step will query cloud inventory to retrieve inventory uuids for each host

data/lib/insights_cloud/async/insights_resolutions_sync.rb

@@ -7,6 +7,15 @@ module InsightsCloud
 
       RULE_ID_REGEX = /[^:]*:(?<id>.*)/
 
+      def plan
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping resolutions sync')
+          return
+        end
+
+        plan_self
+      end
+
       def run
         InsightsResolution.transaction do
           InsightsResolution.delete_all

data/lib/insights_cloud/async/insights_rules_sync.rb

@@ -6,6 +6,11 @@ module InsightsCloud
       include ::ForemanRhCloud::CloudAuth
 
       def plan
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping rules sync')
+          return
+        end
+
         plan_self
         plan_resolutions
       end

data/lib/inventory_sync/async/inventory_full_sync.rb

@@ -5,6 +5,11 @@ module InventorySync
       set_callback :step, :around, :update_statuses_batch
 
       def plan(organization)
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping inventory hosts sync')
+          return
+        end
+
         plan_self(organization_id: organization.id)
       end
 

data/lib/inventory_sync/async/inventory_hosts_sync.rb

@@ -5,6 +5,11 @@ module InventorySync
       set_callback :step, :around, :create_facets
 
       def plan
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping inventory hosts sync')
+          return
+        end
+
         # by default the tasks will be executed concurrently
         plan_self
         plan_self_host_sync

data/lib/inventory_sync/async/inventory_self_host_sync.rb

@@ -3,6 +3,15 @@ module InventorySync
     class InventorySelfHostSync < QueryInventoryJob
       set_callback :step, :around, :create_facets
 
+      def plan
+        unless cloud_auth_available?
+          logger.debug('Cloud authentication is not available, skipping self host sync')
+          return
+        end
+
+        plan_self
+      end
+
       def create_facets
         # get the results from the event
         results = yield

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "foreman_rh_cloud",
-  "version": "4.0.24.1",
+  "version": "4.0.25",
   "description": "Inventory Upload =============",
   "main": "index.js",
   "scripts": {

data/test/jobs/insights_full_sync_test.rb

@@ -7,6 +7,7 @@ class InsightsFullSyncTest < ActiveSupport::TestCase
   setup do
     InsightsCloud::Async::InsightsFullSync.any_instance.stubs(:plan_rules_sync)
     InsightsCloud::Async::InsightsFullSync.any_instance.stubs(:plan_notifications)
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
 
     uuid1 = 'accdf444-5628-451d-bf3e-cf909ad72756'
     @host1 = FactoryBot.create(:host, :managed, name: 'host1')

data/test/jobs/insights_resolutions_sync_test.rb

@@ -63,6 +63,7 @@ class InsightsResolutionsSyncTest < ActiveSupport::TestCase
     }
 
     @rule = FactoryBot.create(:insights_rule, rule_id: 'network_tcp_connection_hang|NETWORK_TCP_CONNECTION_HANG_WARN')
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
   end
 
   test 'Resolutions data is replaced with data from cloud' do

data/test/jobs/insights_rules_sync_test.rb

@@ -112,6 +112,7 @@ class InsightsRulesSyncTest < ActiveSupport::TestCase
     @hit = FactoryBot.create(:insights_hit, host_id: @host.id)
 
     InsightsCloud::Async::InsightsRulesSync.any_instance.stubs(:plan_resolutions)
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
   end
 
   test 'Hits data is replaced with data from cloud' do

data/test/jobs/inventory_full_sync_test.rb

@@ -242,6 +242,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
   end
 
   test 'Host status should be SYNC for inventory hosts' do
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'TEST TOKEN')
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
 
     ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host2.organization)
@@ -253,6 +254,7 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
   end
 
   test 'Host status should be DISCONNECT for hosts that are not returned from cloud' do
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'TEST TOKEN')
     InventorySync::Async::InventoryFullSync.any_instance.expects(:query_inventory).returns(@inventory)
     FactoryBot.create(:fact_value, fact_name: fact_names['virt::uuid'], value: '1234', host: @host2)
 
@@ -261,4 +263,12 @@ class InventoryFullSyncTest < ActiveSupport::TestCase
 
     assert_equal InventorySync::InventoryStatus::DISCONNECT, InventorySync::InventoryStatus.where(host_id: @host1.id).first.status
   end
+
+  test 'Task should be aborted if token is not present' do
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: '')
+
+    InventorySync::Async::InventoryFullSync.any_instance.expects(:plan_self).never
+
+    ForemanTasks.sync_task(InventorySync::Async::InventoryFullSync, @host1.organization)
+  end
 end

data/test/jobs/inventory_hosts_sync_test.rb

@@ -6,6 +6,7 @@ class InventoryHostsSyncTest < ActiveSupport::TestCase
 
   setup do
     User.current = User.find_by(login: 'secret_admin')
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
 
     env = FactoryBot.create(:katello_k_t_environment)
     cv = env.content_views << FactoryBot.create(:katello_content_view, organization: env.organization)

data/test/jobs/inventory_self_host_sync_test.rb

@@ -6,6 +6,7 @@ class InventorySelfHostSyncTest < ActiveSupport::TestCase
 
   setup do
     User.current = User.find_by(login: 'secret_admin')
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
 
     # this host would pass our plugin queries, so it could be uploaded to the cloud.
     @host1 = FactoryBot.create(:host)

data/test/unit/services/foreman_rh_cloud/template_renderer_helper_test.rb

@@ -8,6 +8,7 @@ class TemplateRendererHelperTest < ActiveSupport::TestCase
     response.stubs(:body).returns('TEST PLAYBOOK')
     ForemanRhCloud::RemediationsRetriever.any_instance.stubs(:query_playbook).returns(response)
     @host1 = FactoryBot.create(:host)
+    FactoryBot.create(:setting, name: 'rh_cloud_token', value: 'MOCK_TOKEN')
   end
 
   test 'Generates a playbook for hit and remediation' do

data/test/unit/tags_generator_test.rb

@@ -62,6 +62,47 @@ class TagsGeneratorTest < ActiveSupport::TestCase
     assert_equal false, actual.key?('content_view')
   end
 
+  test 'generates parameter tags' do
+    FactoryBot.create(:setting, :name => 'include_parameter_tags', :settings_type => "boolean", :category => "Setting::RhCloud", :default => false, :value => true)
+
+    @host.stubs(:host_inherited_params_objects).returns(
+      [
+        OpenStruct.new(name: 'bool_param', value: true),
+        OpenStruct.new(name: 'false_param', value: false),
+        OpenStruct.new(name: 'int_param', value: 1),
+        OpenStruct.new(name: 'empty_param', value: nil),
+        OpenStruct.new(name: 'empty_str_param', value: ''),
+      ]
+    )
+
+    generator = create_generator
+    actual = Hash[generator.generate_parameters]
+
+    assert_equal 3, actual.count
+    assert_equal true, actual['bool_param']
+    assert_equal false, actual['false_param']
+    assert_equal 1, actual['int_param']
+  end
+
+  test 'skips parameter tags if include_parameter_tags setting is off' do
+    FactoryBot.create(:setting, :name => 'include_parameter_tags', :settings_type => "boolean", :category => "Setting::RhCloud", :default => false, :value => false)
+
+    @host.stubs(:host_inherited_params_objects).returns(
+      [
+        OpenStruct.new(name: 'bool_param', value: true),
+        OpenStruct.new(name: 'false_param', value: false),
+        OpenStruct.new(name: 'int_param', value: 1),
+        OpenStruct.new(name: 'empty_param', value: nil),
+        OpenStruct.new(name: 'empty_str_param', value: ''),
+      ]
+    )
+
+    generator = create_generator
+    actual = generator.generate_parameters.group_by { |key, value| key }
+
+    assert_equal 0, actual.count
+  end
+
   private
 
   def create_generator

data/webpack/ForemanInventoryUpload/Components/AccountList/Components/EmptyState/EmptyState.js

@@ -9,7 +9,7 @@ const inventoryEmptyState = () => (
     <EmptyState.Title>
       {__('Fetching data about your accounts')}
     </EmptyState.Title>
-    <EmptyState.Info>{__('Loading')}...</EmptyState.Info>
+    <EmptyState.Info>{__('Loading...')}</EmptyState.Info>
   </EmptyState>
 );
 

data/webpack/ForemanInventoryUpload/Components/AccountList/Components/EmptyState/__tests__/__snapshots__/EmptyState.test.js.snap

@@ -19,8 +19,7 @@ exports[`EmptyState rendering render without Props 1`] = `
   <EmptyStateInfo
     className=""
   >
-    Loading
-    ...
+    Loading...
   </EmptyStateInfo>
 </EmptyState>
 `;

data/webpack/ForemanInventoryUpload/Components/FileDownload/FileDownload.js

@@ -2,12 +2,14 @@ import React from 'react';
 import PropTypes from 'prop-types';
 import { Grid, Button, Icon } from 'patternfly-react';
 import { noop } from 'foremanReact/common/helpers';
+import { translate as __ } from 'foremanReact/common/I18n';
+
 import './fileDownload.scss';
 
 const FileDownload = ({ onClick }) => (
   <Grid.Col sm={12}>
     <Button onClick={onClick} className="download-button">
-      Download Report <Icon name="download" />
+      {__('Download Report')} <Icon name="download" />
     </Button>
   </Grid.Col>
 );

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

@@ -15,7 +15,8 @@ exports[`FileDownload rendering render without Props 1`] = `
     disabled={false}
     onClick={[Function]}
   >
-    Download Report 
+    Download Report
+     
     <Icon
       name="download"
       type="fa"

data/webpack/ForemanInventoryUpload/Components/InventoryFilter/InventoryFilter.js

@@ -15,8 +15,7 @@ const InventoryFilter = ({
   organization,
 }) => {
   useEffect(() => {
-    const initialTerm =
-      organization === __(ANY_ORGANIZATION) ? '' : organization;
+    const initialTerm = organization === ANY_ORGANIZATION ? '' : organization;
     handleFilterChange(initialTerm);
   }, []);
 

data/webpack/ForemanInventoryUpload/Components/InventoryFilter/InventoryFilterConstants.js

@@ -1,3 +1,5 @@
+import { translate as __ } from 'foremanReact/common/I18n';
+
 export const INVENTORY_FILTER_UPDATE = 'INVENTORY_FILTER_UPDATE';
 export const INVENTORY_FILTER_CLEAR = 'INVENTORY_FILTER_CLEAR';
-export const ANY_ORGANIZATION = 'Any Organization';
+export const ANY_ORGANIZATION = __('Any Organization');

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

@@ -28,7 +28,7 @@ export const PageDescription = () => (
         target="_blank"
         rel="noopener noreferrer"
       >
-        About subscription watch
+        {__('About subscription watch')}
       </a>
     </p>
     <p>
@@ -39,7 +39,7 @@ export const PageDescription = () => (
         target="_blank"
         rel="noopener noreferrer"
       >
-        Red Hat Insights Data and Application Security
+        {__('Red Hat Insights Data and Application Security')}
       </a>
     </p>
   </div>

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

@@ -48,7 +48,7 @@ const SyncModal = ({ show, toggleModal }) => (
           )}
         </p>
         <Button bsStyle="primary" bsSize="lg" onClick={toggleModal}>
-          Close
+          {__('Close')}
         </Button>
       </Grid>
     </Modal.Body>

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

@@ -20,13 +20,13 @@ const Toast = ({ syncHosts, disconnectHosts }) => {
         <strong>{disconnectHosts}</strong>
       </p>
       <p>
-        For more info, please visit the{' '}
+        {__('For more info, please visit the')}{' '}
         <a
           href={foremanUrl('/hosts')}
           target="_blank"
           rel="noopener noreferrer"
         >
-          hosts page
+          {__('hosts page')}
         </a>
       </p>
     </span>

data/webpack/ForemanInventoryUpload/Components/StatusChart/StatusChart.js

@@ -1,13 +1,14 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { Grid, DonutChart } from 'patternfly-react';
+import { translate as __ } from 'foremanReact/common/I18n';
 import './statusChart.scss';
 
 const StatusChart = ({ completed }) => {
   const donutConfigData = {
     columns: [
-      ['Completed', completed],
-      ['Remain', 100 - completed],
+      [__('Completed'), completed],
+      [__('Remain'), 100 - completed],
     ],
     order: null,
   };
@@ -37,7 +38,7 @@ const StatusChart = ({ completed }) => {
           title={{
             type: 'percent',
             primary: `${completed}%`,
-            secondary: 'Completed',
+            secondary: __('Completed'),
           }}
         />
       </div>

data/webpack/ForemanInventoryUpload/SubscriptionsPageExtension/InventoryAutoUpload/InventoryAutoUpload.js

@@ -41,7 +41,9 @@ const InventoryAutoUploadSwitcher = ({
 
           <Grid.Col sm={5}>
             <Popover
-              headerContent={<strong>Advanced Inventory Settings</strong>}
+              headerContent={
+                <strong>{__('Advanced Inventory Settings')}</strong>
+              }
               bodyContent={
                 <AdvancedSettings
                   autoUploadEnabled={autoUploadEnabled}

data/webpack/InsightsCloudSync/Components/InsightsSyncSwitcher/InsightsSyncSwitcher.js

@@ -17,7 +17,7 @@ class InsightsSyncSwitcher extends Component {
 
     return (
       <div className="insights_sync_switcher">
-        <span>Synchronize Automatically</span>
+        <span>{__('Synchronize Automatically')}</span>
         <FieldLevelHelp
           content={__(
             'Enable automatic synchronization of Insights recommendations from the Red Hat cloud'

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

@@ -5,7 +5,7 @@ import { Pagination, PaginationVariant } from '@patternfly/react-core';
 import { Table, TableHeader, TableBody } from '@patternfly/react-table';
 import { useForemanSettings } from 'foremanReact/Root/Context/ForemanContext';
 import SelectAllAlert from './SelectAllAlert';
-import { columns } from './InsightsTableConstants';
+import { columns, paginationTitles } from './InsightsTableConstants';
 import TableEmptyState from '../../../common/table/EmptyState';
 import {
   modifySelectedRows,
@@ -82,6 +82,7 @@ const InsightsTable = ({
         onSetPage={onTableSetPage}
         onPerPageSelect={onTablePerPageSelect}
         perPageOptions={getPerPageOptions(urlPerPage, appPerPage)}
+        titles={paginationTitles}
       />
     </React.Fragment>
   );

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

@@ -45,6 +45,20 @@ export const columns = [
   },
 ];
 
+export const paginationTitles = {
+  items: __('items'),
+  page: __('page'),
+  itemsPerPage: __('Items per page'),
+  perPageSuffix: __('per page'),
+  toFirstPage: __('Go to first page'),
+  toPreviousPage: __('Go to previous page'),
+  toLastPage: __('Go to last page'),
+  toNextPage: __('Go to next page'),
+  optionsToggle: __('Items per page'),
+  currPage: __('Current page'),
+  paginationTitle: __('Pagination'),
+};
+
 export const INSIGHTS_HITS_PATH = foremanUrl('/insights_cloud/hits');
 
 export const INSIGHTS_HITS_API_KEY = 'INSIGHTS_HITS';

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

@@ -18,7 +18,7 @@ const SelectAllAlert = ({
       <Alert
         isInline
         variant="info"
-        title={sprintf('Recommendations selected: %s.', selectedCount)}
+        title={sprintf(__('Recommendations selected: %s.'), selectedCount)}
         actionLinks={
           <AlertActionLink onClick={selectAll}>
             {__('Select recommendations from all pages')}

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

@@ -135,10 +135,10 @@ exports[`InsightsTable rendering render with Props 1`] = `
     titles={
       Object {
         "currPage": "Current page",
-        "items": "",
+        "items": "items",
         "itemsPerPage": "Items per page",
         "optionsToggle": "Items per page",
-        "page": "",
+        "page": "page",
         "paginationTitle": "Pagination",
         "perPageSuffix": "per page",
         "toFirstPage": "Go to first page",

data/webpack/InsightsCloudSync/Components/RemediationModal/RemediationModal.scss

@@ -6,4 +6,18 @@
       margin-top: 5px;
     }
   }
+
+  // applies to the backdrop parent of the modal
+  @at-root .pf-c-backdrop {
+    width: calc(100% - 200px);
+    left: 200px;
+  }
+
+  // where the vertical nav breaks: https://github.com/theforeman/foreman/blob/3347fa49d500964f0209122d8d36c920d1feafcc/webpack/assets/javascripts/react_app/components/Layout/components/Toolbar/HeaderToolbar.scss#L26
+  @media (max-width: 768px) {
+    @at-root .pf-c-backdrop {
+      width: 100%;
+      left: 0;
+    }
+  }
 }

data/webpack/InsightsHostDetailsTab/InsightsTab.js

@@ -3,6 +3,7 @@ import PropTypes from 'prop-types';
 import { orderBy } from 'lodash';
 import { Grid, ListView } from 'patternfly-react';
 import { noop } from 'foremanReact/common/helpers';
+import { translate as __ } from 'foremanReact/common/I18n';
 import ListItem from './components/ListItem';
 import './InsightsTab.scss';
 
@@ -16,7 +17,7 @@ class InsightsHostDetailsTab extends React.Component {
     const { hits } = this.props;
 
     if (!hits.length) {
-      return <h2>No recommendations were found for this host!</h2>;
+      return <h2>{__('No recommendations were found for this host!')}</h2>;
     }
     const hitsSorted = orderBy(hits, ['total_risk'], ['desc']);
     const items = hitsSorted.map(
@@ -42,7 +43,7 @@ class InsightsHostDetailsTab extends React.Component {
       <div id="host_details_insights_tab">
         <Grid.Row>
           <Grid.Col xs={12}>
-            <h2>Recommendations</h2>
+            <h2>{__('Recommendations')}</h2>
             <ListView id="hits_list">{items}</ListView>
           </Grid.Col>
         </Grid.Row>

data/webpack/InsightsHostDetailsTab/InsightsTab.scss

@@ -48,19 +48,19 @@
           padding: 5px 8px;
           border-radius: 12px;
 
-          &.Low {
+          &.risk-1 {
             background-color: #e7f1fa;
           }
 
-          &.Moderate {
+          &.risk-2 {
             background-color: #fdf7e7;
           }
 
-          &.Important {
+          &.risk-3 {
             background-color: #f9dddd;
           }
 
-          &.Critical {
+          &.risk-4 {
             background-color: #ffecec;
           }
 

data/webpack/InsightsHostDetailsTab/components/ListItem/ListItem.js

@@ -1,12 +1,13 @@
 import React, { Fragment } from 'react';
 import { ListView, Icon } from 'patternfly-react';
 import PropTypes from 'prop-types';
+import { translate as __ } from 'foremanReact/common/I18n';
 
 const labelMapper = {
-  1: 'Low',
-  2: 'Moderate',
-  3: 'Important',
-  4: 'Critical',
+  1: __('Low'),
+  2: __('Moderate'),
+  3: __('Important'),
+  4: __('Critical'),
 };
 
 const ListItem = ({ title, totalRisk, resultsUrl, solutionUrl }) => {
@@ -18,7 +19,7 @@ const ListItem = ({ title, totalRisk, resultsUrl, solutionUrl }) => {
 
   const riskLabel = labelMapper[totalRisk];
   const additionalInfo = [
-    <span key={`risk-info-${title}`} className={`risk-label ${riskLabel}`}>
+    <span key={`risk-info-${title}`} className={`risk-label risk-${totalRisk}`}>
       <p>{riskLabel}</p>
     </span>,
   ];
@@ -26,7 +27,7 @@ const ListItem = ({ title, totalRisk, resultsUrl, solutionUrl }) => {
   const knowledgebaseLink = solutionUrl && (
     <p>
       <a href={solutionUrl} target="_blank" rel="noopener noreferrer">
-        Knowledgebase article <Icon name="external-link" />
+        {__('Knowledgebase article')} <Icon name="external-link" />
       </a>
     </p>
   );
@@ -34,7 +35,8 @@ const ListItem = ({ title, totalRisk, resultsUrl, solutionUrl }) => {
   const insightsCloudLink = resultsUrl && (
     <p>
       <a href={resultsUrl} target="_blank" rel="noopener noreferrer">
-        Read more about it in RH cloud insights <Icon name="external-link" />
+        {__('Read more about it in RH cloud insights')}{' '}
+        <Icon name="external-link" />
       </a>
     </p>
   );

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_rh_cloud
 version: !ruby/object:Gem::Version
-  version: 4.0.24.1
+  version: 4.0.25
 platform: ruby
 authors:
 - Foreman Red Hat Cloud team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-07-21 00:00:00.000000000 Z
+date: 2021-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: katello