foreman_remote_execution 15.0.1 → 16.0.0

data/locale/zh_TW/foreman_remote_execution.po

@@ -5,7 +5,7 @@
 # Translators:
 msgid ""
 msgstr ""
-"Project-Id-Version: foreman_remote_execution 15.0.0\n"
+"Project-Id-Version: foreman_remote_execution 15.0.1\n"
 "Report-Msgid-Bugs-To: \n"
 "PO-Revision-Date: 2016-02-15 13:54+0000\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
@@ -233,7 +233,7 @@ msgid "Cockpit URL"
 msgstr ""
 
 msgid "Command"
-msgstr ""
+msgstr "指令"
 
 msgid "Concurrency level"
 msgstr "同步等級"
@@ -323,7 +323,7 @@ msgid "Cronline"
 msgstr ""
 
 msgid "Current iteration"
-msgstr ""
+msgstr "現有重述"
 
 msgid "Current location %s is different from job's location %s. This job may run on different hosts than before."
 msgstr ""
@@ -674,7 +674,7 @@ msgid "Invocation type, one of %s"
 msgstr "祈願類型，%s 之一"
 
 msgid "Iteration limit"
-msgstr ""
+msgstr "重述限制"
 
 msgid "Job"
 msgstr "工作"
@@ -767,7 +767,7 @@ msgid "Last execution succeeded"
 msgstr "最後的執行成功"
 
 msgid "Last occurrence"
-msgstr ""
+msgstr "最後一次發生"
 
 msgid "Latest Jobs"
 msgstr ""
@@ -845,7 +845,7 @@ msgid "Next"
 msgstr "下一步"
 
 msgid "Next occurrence"
-msgstr ""
+msgstr "下次發生"
 
 msgid "No (override)"
 msgstr ""
@@ -1094,7 +1094,7 @@ msgid "Repeat amount can only be a positive number"
 msgstr ""
 
 msgid "Repeat until"
-msgstr ""
+msgstr "重複直到"
 
 msgid "Repeats"
 msgstr "重複"
@@ -1316,7 +1316,7 @@ msgid "Skip to review step"
 msgstr ""
 
 msgid "Smart proxy"
-msgstr ""
+msgstr "智慧代理"
 
 msgid "Snippet"
 msgstr "程式碼片段"
@@ -1418,7 +1418,7 @@ msgid "Task cancelled"
 msgstr ""
 
 msgid "Task count"
-msgstr ""
+msgstr "工作數量"
 
 msgid "Task for the host cancelled succesfully"
 msgstr ""
@@ -1603,7 +1603,7 @@ msgid "Unable to save template. Correct highlighted errors"
 msgstr ""
 
 msgid "Unknown"
-msgstr ""
+msgstr "不明"
 
 msgid "Unknown execution status"
 msgstr "未知的執行狀態"

data/package.json

@@ -19,20 +19,21 @@
   },
   "devDependencies": {
     "@babel/core": "^7.7.0",
-    "@theforeman/builder": ">= 12.0.1",
-    "@theforeman/eslint-plugin-foreman": ">= 12.0.1",
-    "@theforeman/eslint-plugin-rules": ">= 12.0.2",
-    "@theforeman/test": ">= 12.0.1",
-    "@theforeman/vendor-dev": ">= 12.0.1",
+    "@theforeman/builder": ">= 15.0.1",
+    "@theforeman/eslint-plugin-foreman": ">= 15.0.1",
+    "@theforeman/eslint-plugin-rules": ">= 15.0.1",
+    "@theforeman/test": ">= 15.0.1",
+    "@theforeman/vendor-dev": ">= 15.0.1",
     "babel-eslint": "^10.0.0",
     "eslint": "^6.8.0",
     "prettier": "^1.19.1",
     "redux-mock-store": "^1.2.2",
     "graphql-tag": "^2.11.0",
     "graphql": "^15.5.0",
-    "victory-core": "~36.8.6"
+    "victory-core": "~36.8.6",
+    "victory-pie": "~36.8.6"
   },
   "peerDependencies": {
-    "@theforeman/vendor": ">= 12.0.1"
+    "@theforeman/vendor": ">= 15.0.1"
   }
 }

data/webpack/JobInvocationDetail/JobAdditionInfo.js

@@ -69,7 +69,7 @@ const Schedule = ({ data }) => {
   return (
     <ExpandableSection
       toggleText={__('Schedule')}
-      onToggle={setIsExpanded}
+      onToggle={(_event, val) => setIsExpanded(val)}
       isExpanded={isExpanded}
     >
       <DataList isCompact>
@@ -107,7 +107,7 @@ const Recurring = ({ data }) => {
   return (
     <ExpandableSection
       toggleText={__('Recurring logic')}
-      onToggle={setIsExpanded}
+      onToggle={(_event, val) => setIsExpanded(val)}
       isExpanded={isExpanded}
     >
       <DataList isCompact>
@@ -149,7 +149,7 @@ const TargetHosts = ({ data }) => {
   return (
     <ExpandableSection
       toggleText={__('Target Hosts')}
-      onToggle={setIsExpanded}
+      onToggle={(_event, val) => setIsExpanded(val)}
       isExpanded={isExpanded}
     >
       <span>{targetingSelectioning}</span>{' '}
@@ -173,7 +173,7 @@ const Inputs = ({ data }) => {
   return (
     <ExpandableSection
       toggleText={__('User Inputs')}
-      onToggle={setIsExpanded}
+      onToggle={(_event, val) => setIsExpanded(val)}
       isExpanded={isExpanded}
     >
       <DataList isCompact>

data/webpack/JobInvocationDetail/JobInvocationDetail.scss

@@ -15,22 +15,22 @@
 
     .legend-title {
       font-weight: bold;
-      font-size: var(--pf-global--FontSize--sm);
+      font-size: var(--pf-v5-global--FontSize--sm);
       margin-left: 8px;
       margin-bottom: 0;
     }
 
-    .pf-c-description-list {
+    .pf-v5-c-description-list {
       margin-left: 8px;
       margin-top: 8px;
 
-      .pf-c-description-list__term .pf-c-description-list__text {
+      .pf-v5-c-description-list__term .pf-v5-c-description-list__text {
         font-weight: normal;
       }
     }
   }
 
-  .pf-c-divider {
+  .pf-v5-c-divider {
     max-height: $chart_size;
   }
 
@@ -104,7 +104,7 @@
     margin-top: 10px;
   }
 
-  .pf-c-toggle-group {
+  .pf-v5-c-toggle-group {
     margin-bottom: 10px;
   }
 }

data/webpack/JobInvocationDetail/JobInvocationHostTable.js

@@ -6,10 +6,10 @@ import { translate as __ } from 'foremanReact/common/I18n';
 import { FormattedMessage } from 'react-intl';
 import { Tr, Td, Tbody, ExpandableRowContent } from '@patternfly/react-table';
 import {
-  Title,
   EmptyState,
   EmptyStateVariant,
   EmptyStateBody,
+  EmptyStateHeader,
 } from '@patternfly/react-core';
 import { foremanUrl } from 'foremanReact/common/helpers';
 import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
@@ -163,9 +163,10 @@ const JobInvocationHostTable = ({
           <span className="empty-state-icon">
             <Icon name="add-circle-o" type="pf" size="2x" />
           </span>
-          <Title ouiaId="empty-state-header" headingLevel="h5" size="4xl">
-            {__('No Results')}
-          </Title>
+          <EmptyStateHeader
+            titleText={<>{__('No Results')}</>}
+            headingLevel="h5"
+          />
           <EmptyStateBody>
             <div className="empty-state-description">
               {targeting?.targeting_type === 'dynamic_query' ? (
@@ -217,10 +218,12 @@ const JobInvocationHostTable = ({
             setShowAlert={setShowAlert}
             results={results}
             id={id}
+            key="OpenAlInvocations"
           />,
           <JobInvocationHostTableToolbar
             dropdownFilter={selectedFilter}
             setDropdownFilter={wrapSetSelectedFilter}
+            key="JobInvocationHostTableToolbar"
           />,
         ]}
       >

data/webpack/JobInvocationDetail/JobInvocationHostTableToolbar.js

@@ -1,8 +1,13 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
-import { Select, SelectOption, SelectList } from '@patternfly/react-core/next'; // remove "/next" after switching to PF5
-import { MenuToggle, ToolbarItem } from '@patternfly/react-core';
+import {
+  MenuToggle,
+  ToolbarItem,
+  Select,
+  SelectOption,
+  SelectList,
+} from '@patternfly/react-core';
 import { STATUS_TITLES } from './JobInvocationConstants';
 
 const JobInvocationHostTableToolbar = ({
@@ -43,7 +48,7 @@ const JobInvocationHostTableToolbar = ({
           {Object.values(STATUS_TITLES).map(result => (
             <SelectOption
               key={result.id}
-              itemId={result.id}
+              value={result.id}
               isSelected={result.id === dropdownFilter}
             >
               {result.title}

data/webpack/JobInvocationDetail/JobInvocationToolbarButtons.js

@@ -1,16 +1,14 @@
 import PropTypes from 'prop-types';
 import React, { useEffect, useState } from 'react';
 import { useDispatch, useSelector } from 'react-redux';
+import { Button, Split, SplitItem } from '@patternfly/react-core';
 import {
-  Button,
   Dropdown,
   DropdownItem,
   DropdownPosition,
   DropdownSeparator,
   DropdownToggle,
-  Split,
-  SplitItem,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { foremanUrl } from 'foremanReact/common/helpers';
 import { STATUS as APIStatus } from 'foremanReact/constants';
@@ -242,7 +240,7 @@ const JobInvocationToolbarButtons = ({
                   </Button>,
                 ]}
                 splitButtonVariant="action"
-                onToggle={setIsActionOpen}
+                onToggle={(_event, val) => setIsActionOpen(val)}
               />
             }
             isOpen={isActionOpen}

data/webpack/JobInvocationDetail/TemplateInvocationComponents/OutputToggleGroup.js

@@ -97,7 +97,7 @@ export const OutputToggleGroup = ({
                 text={text}
                 buttonId={id}
                 isSelected={isSelected}
-                onChange={onClick}
+                onChange={(_event, val) => onClick(val)}
               />
             )
           )}

data/webpack/JobInvocationDetail/TemplateInvocationComponents/PreviewTemplate.js

@@ -1,18 +1,11 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import {
-  TableComposable,
-  Thead,
-  Tr,
-  Th,
-  Tbody,
-  Td,
-} from '@patternfly/react-table';
+import { Table, Thead, Tr, Th, Tbody, Td } from '@patternfly/react-table';
 import { translate as __ } from 'foremanReact/common/I18n';
 
 export const PreviewTemplate = ({ inputValues }) =>
   inputValues.length ? (
-    <TableComposable
+    <Table
       ouiaId="template-invocation-preview-table"
       isStriped
       variant="compact"
@@ -36,7 +29,7 @@ export const PreviewTemplate = ({ inputValues }) =>
           </Tr>
         ))}
       </Tbody>
-    </TableComposable>
+    </Table>
   ) : (
     <span>{__('No user input')}</span>
   );

data/webpack/JobInvocationDetail/TemplateInvocationComponents/TemplateActionButtons.js

@@ -7,6 +7,7 @@ import { APIActions } from 'foremanReact/redux/API';
 import { addToast } from 'foremanReact/components/ToastsList';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { selectTemplateInvocation } from '../JobInvocationSelectors';
+import './index.scss';
 
 const actions = ({
   taskID,
@@ -96,17 +97,15 @@ export const RowActions = ({ hostID, jobID }) => {
     permissions,
     dispatch,
   });
+
   const rowActions = Object.values(getActions)
-    .map(({ text, onClick, href, permission, isDisabled }) =>
+    .map(({ text, href, onClick, permission, isDisabled }) =>
       permission
         ? {
-            title: href ? (
-              <a href={href} target="_blank" rel="noreferrer">
-                {text}
-              </a>
-            ) : (
-              text
-            ),
+            title: text,
+            component: 'a',
+            className: 'jobs-table-action-item',
+            href,
             onClick,
             isDisabled,
           }
@@ -138,7 +137,7 @@ export const TemplateActionButtons = ({
       {rerun.permission && (
         <FlexItem spacer={{ default: 'spacerXs' }}>
           <Button
-            isSmall
+            size="sm"
             variant="secondary"
             isInline
             ouiaId={rerun.name}
@@ -153,7 +152,7 @@ export const TemplateActionButtons = ({
       {details.permission && (
         <FlexItem spacer={{ default: 'spacerXs' }}>
           <Button
-            isSmall
+            size="sm"
             variant="secondary"
             isInline
             ouiaId={details.name}
@@ -168,7 +167,7 @@ export const TemplateActionButtons = ({
       {cancel.permission && (
         <FlexItem spacer={{ default: 'spacerXs' }}>
           <Button
-            isSmall
+            size="sm"
             variant="danger"
             isInline
             ouiaId={cancel.name}
@@ -182,7 +181,7 @@ export const TemplateActionButtons = ({
       {abort.permission && (
         <FlexItem spacer={{ default: 'spacerXs' }}>
           <Button
-            isSmall
+            size="sm"
             variant="danger"
             isInline
             ouiaId={abort.name}

data/webpack/JobInvocationDetail/TemplateInvocationComponents/index.scss

@@ -0,0 +1,3 @@
+.jobs-table-action-item:hover {
+    color: var(--pf-v5-c-dropdown__menu-item--Color);
+}

data/webpack/JobInvocationDetail/__tests__/MainInformation.test.js

@@ -97,7 +97,7 @@ describe('JobInvocationDetailPage', () => {
 
     expect(screen.getByText('Description')).toBeInTheDocument();
     expect(
-      container.querySelector('.chart-donut .pf-c-chart')
+      container.querySelector('.chart-donut .pf-c-chart') // todo: change to pf5 once we update @patternfly/react-charts to v7
     ).toBeInTheDocument();
     expect(screen.getByText('2/6')).toBeInTheDocument();
     expect(screen.getByText('Systems')).toBeInTheDocument();
@@ -116,22 +116,22 @@ describe('JobInvocationDetailPage', () => {
 
     Object.entries(informationToCheck).forEach(([term, expectedText]) => {
       const termContainers = container.querySelectorAll(
-        '.pf-c-description-list__term .pf-c-description-list__text'
+        '.pf-v5-c-description-list__term .pf-v5-c-description-list__text'
       );
       termContainers.forEach(termContainer => {
         if (termContainer.textContent.includes(term)) {
           let descriptionContainer;
           if (term === 'SSH user:') {
             descriptionContainer = termContainer
-              .closest('.pf-c-description-list__group')
+              .closest('.pf-v5-c-description-list__group')
               .querySelector(
-                '.pf-c-description-list__description .pf-c-description-list__text .disabled-text'
+                '.pf-v5-c-description-list__description .pf-v5-c-description-list__text .disabled-text'
               );
           } else {
             descriptionContainer = termContainer
-              .closest('.pf-c-description-list__group')
+              .closest('.pf-v5-c-description-list__group')
               .querySelector(
-                '.pf-c-description-list__description .pf-c-description-list__text'
+                '.pf-v5-c-description-list__description .pf-v5-c-description-list__text'
               );
           }
           expect(descriptionContainer.textContent).toContain(expectedText);

data/webpack/JobInvocationDetail/__tests__/TemplateInvocation.test.js

@@ -126,6 +126,6 @@ describe('TemplateInvocation', () => {
       />
     );
 
-    expect(document.querySelectorAll('.pf-c-skeleton')).toHaveLength(1);
+    expect(document.querySelectorAll('.pf-v5-c-skeleton')).toHaveLength(1);
   });
 });

data/webpack/JobInvocationDetail/__tests__/fixtures.js

@@ -41,6 +41,7 @@ export const jobInvocationData = {
   task: {
     id: '37ad5ead-51de-4798-bc73-a17687c4d5aa',
     state: 'stopped',
+    started_at: '2024-01-01 12:34:56 +0100',
   },
   template_invocations: [
     {

data/webpack/JobInvocationDetail/index.js

@@ -1,31 +1,33 @@
-import PropTypes from 'prop-types';
-import React, { useEffect, useState } from 'react';
-import { useDispatch, useSelector } from 'react-redux';
 import {
   Divider,
   Flex,
   PageSection,
   PageSectionVariants,
 } from '@patternfly/react-core';
-import { translate as __, documentLocale } from 'foremanReact/common/I18n';
-import PageLayout from 'foremanReact/routes/common/PageLayout/PageLayout';
 import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
+import { translate as __, documentLocale } from 'foremanReact/common/I18n';
 import { stopInterval } from 'foremanReact/redux/middlewares/IntervalMiddleware';
+import PageLayout from 'foremanReact/routes/common/PageLayout/PageLayout';
+import PropTypes from 'prop-types';
+import React, { useEffect, useState } from 'react';
+import { useDispatch, useSelector } from 'react-redux';
+
+import { JobAdditionInfo } from './JobAdditionInfo';
 import { getJobInvocation, getTask } from './JobInvocationActions';
 import {
   CURRENT_PERMISSIONS,
+  currentPermissionsUrl,
   DATE_OPTIONS,
   JOB_INVOCATION_KEY,
   STATUS,
-  currentPermissionsUrl,
 } from './JobInvocationConstants';
-import './JobInvocationDetail.scss';
+import JobInvocationHostTable from './JobInvocationHostTable';
 import JobInvocationOverview from './JobInvocationOverview';
 import { selectItems } from './JobInvocationSelectors';
 import JobInvocationSystemStatusChart from './JobInvocationSystemStatusChart';
 import JobInvocationToolbarButtons from './JobInvocationToolbarButtons';
-import JobInvocationHostTable from './JobInvocationHostTable';
-import { JobAdditionInfo } from './JobAdditionInfo';
+
+import './JobInvocationDetail.scss';
 
 const JobInvocationDetailPage = ({
   match: {
@@ -60,10 +62,9 @@ const JobInvocationDetailPage = ({
   let isAlreadyStarted = false;
   let formattedStartDate;
   if (startAt) {
-    // Ensures date string compatibility across browsers
-    const convertedDate = new Date(startAt.replace(/[-.]/g, '/'));
-    isAlreadyStarted = convertedDate.getTime() <= new Date().getTime();
-    formattedStartDate = convertedDate.toLocaleString(
+    // eslint-disable-next-line camelcase
+    isAlreadyStarted = !!task?.started_at;
+    formattedStartDate = new Date(startAt).toLocaleString(
       documentLocale(),
       DATE_OPTIONS
     );

data/webpack/JobWizard/Footer.js

@@ -1,13 +1,11 @@
 import React from 'react';
 import { useSelector } from 'react-redux';
 import PropTypes from 'prop-types';
+import { Button, Tooltip, Spinner } from '@patternfly/react-core';
 import {
-  Button,
   WizardFooter,
   WizardContextConsumer,
-  Tooltip,
-  Spinner,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { WIZARD_TITLES } from './JobWizardConstants';
 import { selectIsSubmitting } from './JobWizardSelectors';

data/webpack/JobWizard/JobWizard.js

@@ -3,7 +3,7 @@
 import React, { useState, useEffect, useCallback } from 'react';
 import { useDispatch, useSelector } from 'react-redux';
 import PropTypes from 'prop-types';
-import { Wizard } from '@patternfly/react-core';
+import { Wizard } from '@patternfly/react-core/deprecated';
 import { get } from 'foremanReact/redux/API';
 import history from 'foremanReact/history';
 

data/webpack/JobWizard/JobWizard.scss

@@ -1,21 +1,21 @@
 .job-wizard {
-  font-size: var(--pf-global--FontSize--md);
+  font-size: var(--pf-v5-global--FontSize--md);
   .wizard-title {
     margin-bottom: 25px;
   }
 
-  .pf-c-wizard__nav.pf-m-expanded {
+  .pf-v5-c-wizard__nav.pf-m-expanded {
     z-index: calc(
-      var(--pf-c-wizard__toggle--ZIndex) + 2
+      var(--pf-v5-cwizard__toggle--ZIndex) + 2
     ); // So the small screen navigation can be shown above the select box and wizard body
   }
 
-  .pf-c-wizard__main {
+  .pf-v5-c-wizard__main {
     z-index: calc(
-      var(--pf-c-wizard__toggle--ZIndex) + 1
+      var(--pf-v5-cwizard__toggle--ZIndex) + 1
     ); // So the select box can be shown above the wizard footer and navigation toggle
   }
-  .pf-c-wizard__main-body {
+  .pf-v5-c-wizard__main-body {
     @media (max-width: 600px) {
       max-width: 100%;
     }
@@ -34,19 +34,19 @@
   }
 
   .gray-text {
-    color: var(--pf-global--Color--dark-200);
+    color: var(--pf-v5-global--Color--dark-200);
   }
 
   .target-hosts-and-inputs {
-    .pf-c-chip-group.pf-m-category {
+    .pf-v5-c-chip-group.pf-m-category {
       margin-bottom: 10px;
     }
-    .pf-c-select__toggle-typeahead {
+    .pf-v5-c-select__toggle-typeahead {
       border: 0px;
     }
 
     .target-method-select {
-      .pf-c-select__toggle-wrapper {
+      .pf-v5-c-select__toggle-wrapper {
         flex-wrap: nowrap;
       }
     }
@@ -63,8 +63,8 @@
       display: grid;
       grid-template-columns: repeat(7, 1fr);
     }
-    .pf-l-grid {
-      gap: var(--pf-c-form--GridGap);
+    .pf-v5-l-grid {
+      gap: var(--pf-v5-cform--GridGap);
     }
     #repeat-on-hourly {
       max-height: 300px;
@@ -99,14 +99,14 @@
     .clear-datetime-button {
       margin-left: 10px;
       align-self: center;
-      font-size: var(--pf-global--FontSize--md);
+      font-size: var(--pf-v5-global--FontSize--md);
     }
-    .pf-c-form__group-control {
+    .pf-v5-c-form__group-control {
       display: flex;
       flex-wrap: wrap;
     }
   }
-  .pf-c-date-picker {
+  .pf-v5-c-date-picker {
     vertical-align: top;
   }
 
@@ -123,7 +123,7 @@
     min-height: 40px;
     min-width: 100px;
   }
-  .pf-c-modal-box {
+  .pf-v5-c-modal-box {
     width: auto;
   }
 
@@ -132,15 +132,15 @@
       margin-left: 10px;
     }
   }
-  .pf-c-radio__body {
-    font-size: var(--pf-c-radio__label--FontSize);
+  .pf-v5-c-radio__body {
+    font-size: var(--pf-v5-cradio__label--FontSize);
   }
   .reset-default {
     padding-bottom: 0;
   }
 }
 
-.job-wizard-alert.pf-c-alert.pf-m-warning {
+.job-wizard-alert.pf-v5-c-alert.pf-m-warning {
   margin-bottom: 10px;
   margin-top: 10px;
 }