foreman-tasks 13.0.0 → 13.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9fa716da28a9b6bf38240aa02e47f9ab4a4ab6ba82f98c5d26f7d6066f3149e4
-  data.tar.gz: eb7ec125870236abf803a65c132b042fd4360f4de5d7d9a5b2bc718ba8672235
+  metadata.gz: 270e537ec64af9327b2598da48e8422a56adc1716e45ee5105cb57c9bd2d1078
+  data.tar.gz: da3627e82bdecfcd63516cf3e973461b22762023374ee58ac105e7f9e66c2cf7
 SHA512:
-  metadata.gz: 0547b82e39eabef4f7205100733272e5bdb7b619721ea1a9b94259403c1be370a4ee34564825ccbca83ed16767e72634dd2a070f288ceed28627c3c58671d726
-  data.tar.gz: c27122956334571f9b1cc287e8ec6cf322f5f2931be69780539cc58a7f37cbf7f7c4c81800807fc81a79ca8213d1510c127552492ee942c21a1067eb09258a9e
+  metadata.gz: dc37e5ebbdb2aa62a131d2828a619f00c68b1c6c262a78df83671801b7047eeafd2bd31fd43e52fff79d7a9b2bb69267a950bd2e71babc6569c20d55a90caab2
+  data.tar.gz: ba180fe7ad221035c14474ed9b1cc62795d2d90ff585fdc0304bda52432e43917829d016abd03e18e68ffcb9654f1a73864695d93fe1219214f7a14d43de0033

data/app/services/foreman_tasks/troubleshooting_help_generator.rb

@@ -67,7 +67,7 @@ module ForemanTasks
         i.add_line _('A paused task represents a process that has not finished properly. '\
                         'Any task in paused state can lead to potential inconsistency '\
                         'and needs to be resolved.')
-        i.add_line _("The recommended approach is to investigate the error messages below and in 'errors' tab, "\
+        i.add_line _("The recommended approach is to investigate the error messages below and in 'Execution details' tab, "\
                      'address the primary cause of the issue and resume the task.')
         if (link = troubleshooting_link)
           i.add_link(link)

data/lib/foreman_tasks/version.rb

@@ -1,3 +1,3 @@
 module ForemanTasks
-  VERSION = '13.0.0'.freeze
+  VERSION = '13.1.0'.freeze
 end

data/webpack/ForemanTasks/Components/TaskDetails/Components/Task.js

@@ -1,75 +1,13 @@
-import React, { useState } from 'react';
-import PropTypes from 'prop-types';
+import React from 'react';
+import { Stack, StackItem } from '@patternfly/react-core';
 import TaskInfo from './TaskInfo';
-import {
-  ForceUnlockConfirmationModal,
-  UnlockConfirmationModal,
-} from '../../common/ClickConfirmation';
-import { TaskButtons } from './TaskButtons';
 
-const Task = props => {
-  const {
-    taskReload,
-    id,
-    forceCancelTaskRequest,
-    unlockTaskRequest,
-    action,
-    taskReloadStart,
-  } = props;
-  const forceUnlock = () => {
-    if (!taskReload) {
-      taskReloadStart(id);
-    }
-    forceCancelTaskRequest(id, action);
-  };
-  const unlock = () => {
-    if (!taskReload) {
-      taskReloadStart(id);
-    }
-    unlockTaskRequest(id, action);
-  };
-  const [unlockModalOpen, setUnlockModalOpen] = useState(false);
-  const [forceUnlockModalOpen, setForceUnlockModalOpen] = useState(false);
-
-  return (
-    <React.Fragment>
-      <UnlockConfirmationModal
-        onClick={unlock}
-        isOpen={unlockModalOpen}
-        setModalClosed={() => setUnlockModalOpen(false)}
-      />
-      <ForceUnlockConfirmationModal
-        onClick={forceUnlock}
-        isOpen={forceUnlockModalOpen}
-        setModalClosed={() => setForceUnlockModalOpen(false)}
-      />
-      <TaskButtons
-        taskReloadStart={taskReloadStart}
-        setUnlockModalOpen={setUnlockModalOpen}
-        setForceUnlockModalOpen={setForceUnlockModalOpen}
-        {...props}
-      />
+const Task = props => (
+  <Stack hasGutter>
+    <StackItem>
       <TaskInfo {...props} />
-    </React.Fragment>
-  );
-};
-
-Task.propTypes = {
-  taskReload: PropTypes.bool,
-  id: PropTypes.string,
-  forceCancelTaskRequest: PropTypes.func,
-  unlockTaskRequest: PropTypes.func,
-  action: PropTypes.string,
-  taskReloadStart: PropTypes.func,
-};
-
-Task.defaultProps = {
-  taskReload: false,
-  id: '',
-  forceCancelTaskRequest: () => null,
-  unlockTaskRequest: () => null,
-  action: '',
-  taskReloadStart: () => null,
-};
+    </StackItem>
+  </Stack>
+);
 
 export default Task;

data/webpack/ForemanTasks/Components/TaskDetails/Components/TaskButtons.js

@@ -1,9 +1,12 @@
 import React from 'react';
 import PropTypes from 'prop-types';
-import { Button, Icon } from '@patternfly/react-core';
-import { SyncAltIcon } from '@patternfly/react-icons';
+import { Button, Flex, FlexItem, Icon } from '@patternfly/react-core';
+import { SimpleDropdown } from '@patternfly/react-templates';
+import { EllipsisVIcon, SyncAltIcon } from '@patternfly/react-icons';
 import { translate as __ } from 'foremanReact/common/I18n';
 
+import './TaskButtons.scss';
+
 export const TaskButtons = ({
   canEdit,
   dynflowEnableConsole,
@@ -30,120 +33,128 @@ export const TaskButtons = ({
     ? undefined
     : `dynflow_enable_console ${__('Setting is off')}`;
 
+  const overflowItems = [
+    {
+      value: 'reload',
+      ouiaId: 'task-buttons-reload-button',
+      content: (
+        <>
+          <Icon>
+            <SyncAltIcon className={taskReload ? 'spin' : ''} />
+          </Icon>
+          &nbsp;
+          {taskReload ? __('Stop auto-reloading') : __('Start auto-reloading')}
+        </>
+      ),
+      onClick: taskProgressToggle,
+    },
+    {
+      value: 'resume',
+      ouiaId: 'task-buttons-resume-button',
+      content: __('Resume'),
+      onClick: () => {
+        if (!taskReload) {
+          taskReloadStart(id);
+        }
+
+        resumeTaskRequest(id, action);
+      },
+      isDisabled: !canEdit || !resumable,
+      tooltip: editActionsTitle,
+    },
+    ...(parentTask
+      ? [
+          {
+            value: 'parent',
+            ouiaId: 'task-buttons-parent-button',
+            content: __('Parent task'),
+            to: `/foreman_tasks/tasks/${parentTask}`,
+          },
+        ]
+      : []),
+    ...(hasSubTasks
+      ? [
+          {
+            value: 'subtasks',
+            ouiaId: 'task-buttons-subtask-button',
+            content: __('Sub tasks'),
+            to: `/foreman_tasks/tasks/${id}/sub_tasks`,
+          },
+        ]
+      : []),
+    {
+      value: 'unlock',
+      ouiaId: 'task-buttons-unlock-button',
+      content: __('Unlock'),
+      onClick: () => setUnlockModalOpen(true),
+      isDisabled: !canEdit || state !== 'paused',
+      tooltip: editActionsTitle,
+    },
+    {
+      value: 'force-unlock',
+      ouiaId: 'task-buttons-force-unlock-button',
+      content: __('Force Unlock'),
+      onClick: () => setForceUnlockModalOpen(true),
+      isDisabled: !canEdit || state === 'stopped',
+      tooltip: editActionsTitle,
+    },
+  ];
+
   return (
-    <>
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-reload-button"
-        className="reload-button"
-        size="sm"
-        onClick={taskProgressToggle}
-      >
-        <Icon className={taskReload ? 'spin' : ''}>
-          <SyncAltIcon />
-        </Icon>
-        {taskReload ? __('Stop auto-reloading') : __('Start auto-reloading')}
-      </Button>
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-dynflow-button"
-        className="dynflow-button"
-        size="sm"
-        component="a"
-        href={`/foreman_tasks/dynflow/${externalId}`}
-        isDisabled={!dynflowEnableConsole}
-        target="_blank"
-        rel="noopener noreferrer"
-      >
-        <span title={dynflowTitle} data-original-title={dynflowTitle}>
-          {__('Dynflow console')}
-        </span>
-      </Button>
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-resume-button"
-        className="resume-button"
-        size="sm"
-        title={editActionsTitle}
-        data-original-title={editActionsTitle}
-        isDisabled={!canEdit || !resumable}
-        onClick={() => {
-          if (!taskReload) {
-            taskReloadStart(id);
-          }
-          resumeTaskRequest(id, action);
-        }}
-      >
-        {__('Resume')}
-      </Button>
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-cancel-button"
-        className="cancel-button"
-        size="sm"
-        title={editActionsTitle}
-        data-original-title={editActionsTitle}
-        isDisabled={!canEdit || !cancellable}
-        onClick={() => {
-          if (!taskReload) {
-            taskReloadStart(id);
-          }
-          cancelTaskRequest(id, action);
-        }}
-      >
-        {__('Cancel')}
-      </Button>
-      {parentTask && (
+    <Flex
+      className="task-buttons"
+      display={{ default: 'inlineFlex' }}
+      alignItems={{ default: 'alignItemsCenter' }}
+      spaceItems={{ default: 'spaceItemsSm' }}
+    >
+      <FlexItem>
         <Button
           variant="secondary"
-          ouiaId="task-buttons-parent-button"
-          className="parent-button"
+          ouiaId="task-buttons-cancel-button"
+          className="cancel-button"
           size="sm"
-          href={`/foreman_tasks/tasks/${parentTask}`}
-          component="a"
+          title={editActionsTitle}
+          data-original-title={editActionsTitle}
+          isDisabled={!canEdit || !cancellable}
+          onClick={() => {
+            if (!taskReload) {
+              taskReloadStart(id);
+            }
+
+            cancelTaskRequest(id, action);
+          }}
         >
-          {__('Parent task')}
+          {__('Cancel')}
         </Button>
-      )}
-      {hasSubTasks && (
+      </FlexItem>
+      <FlexItem>
         <Button
           variant="secondary"
-          ouiaId="task-buttons-subtask-button"
-          className="subtask-button"
+          ouiaId="task-buttons-dynflow-button"
+          className="dynflow-button"
           size="sm"
-          href={`/foreman_tasks/tasks/${id}/sub_tasks`}
           component="a"
+          href={`/foreman_tasks/dynflow/${externalId}`}
+          isDisabled={!dynflowEnableConsole}
+          target="_blank"
+          rel="noopener noreferrer"
         >
-          {__('Sub tasks')}
+          <span title={dynflowTitle} data-original-title={dynflowTitle}>
+            {__('Dynflow console')}
+          </span>
         </Button>
-      )}
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-unlock-button"
-        className="unlock-button"
-        size="sm"
-        isDisabled={!canEdit || state !== 'paused'}
-        onClick={() => {
-          setUnlockModalOpen(true);
-        }}
-        title={editActionsTitle}
-        data-original-title={editActionsTitle}
-      >
-        {__('Unlock')}
-      </Button>
-      <Button
-        variant="secondary"
-        ouiaId="task-buttons-force-unlock-button"
-        className="force-unlock-button"
-        size="sm"
-        isDisabled={!canEdit || state === 'stopped'}
-        onClick={() => setForceUnlockModalOpen(true)}
-        title={editActionsTitle}
-        data-original-title={editActionsTitle}
-      >
-        {__('Force Unlock')}
-      </Button>
-    </>
+      </FlexItem>
+      <FlexItem>
+        <SimpleDropdown
+          ouiaId="task-buttons-overflow-dropdown"
+          toggleVariant="plain"
+          toggleContent={<EllipsisVIcon aria-hidden="true" />}
+          popperProps={{ position: 'right' }}
+          initialItems={overflowItems}
+          toggleProps={{ 'aria-label': __('Task actions') }}
+        />
+      </FlexItem>
+    </Flex>
   );
 };
 

data/webpack/ForemanTasks/Components/TaskDetails/Components/TaskButtons.scss

@@ -0,0 +1,45 @@
+.task-buttons {
+  .spin {
+    -webkit-animation: spin 1s infinite linear;
+    -moz-animation: spin 1s infinite linear;
+    -o-animation: spin 1s infinite linear;
+    animation: spin 1s infinite linear;
+    -webkit-transform-origin: 50% 50%;
+    transform-origin: 50% 50%;
+    -ms-transform-origin: 50% 50%; /* IE 9 */
+  }
+
+  @-moz-keyframes spin {
+    from {
+      -moz-transform: rotate(0deg);
+    }
+
+    to {
+      -moz-transform: rotate(360deg);
+    }
+  }
+
+  @-webkit-keyframes spin {
+    from {
+      -webkit-transform: rotate(0deg);
+    }
+
+    to {
+      -webkit-transform: rotate(360deg);
+    }
+  }
+
+  @keyframes spin {
+    from {
+      transform: rotate(0deg);
+    }
+
+    to {
+      transform: rotate(360deg);
+    }
+  }
+
+  .dynflow-button > span {
+    pointer-events: auto;
+  }
+}

data/webpack/ForemanTasks/Components/TaskDetails/Components/TaskHelper.js

@@ -22,3 +22,57 @@ export const durationInWords = (
 
 const numberWithDelimiter = x =>
   x.toString().replace(/\B(?=(\d{3})+(?!\d))/g, ',');
+
+export const isDelayed = ({ startAt, startedAt }) => {
+  if (
+    startAt == null ||
+    startedAt == null ||
+    startAt === '' ||
+    startedAt === ''
+  ) {
+    return false;
+  }
+
+  const a = new Date(startAt);
+  const b = new Date(startedAt);
+
+  if (Number.isNaN(a.getTime()) || Number.isNaN(b.getTime())) {
+    return false;
+  }
+
+  a.setMilliseconds(0);
+  b.setMilliseconds(0);
+
+  return a.getTime() !== b.getTime();
+};
+
+export const parseUsernameLinkHref = usernamePath => {
+  if (!usernamePath || typeof usernamePath !== 'string') {
+    return null;
+  }
+
+  const match = usernamePath.match(/href=(["'])(.*?)\1/i);
+
+  return match ? match[2] : null;
+};
+
+export const formatTaskDuration = (startedAtStr, endedAtStr, state) => {
+  if (state !== 'stopped' || !startedAtStr || !endedAtStr) {
+    return __('N/A');
+  }
+
+  const startMs = new Date(startedAtStr).getTime();
+  const endMs = new Date(endedAtStr).getTime();
+
+  if (Number.isNaN(startMs) || Number.isNaN(endMs) || endMs < startMs) {
+    return __('N/A');
+  }
+
+  const duration = durationInWords(startedAtStr, endedAtStr);
+
+  if (typeof duration === 'string') {
+    return duration;
+  }
+
+  return duration.text;
+};