X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/ee347878e1ad3b6f82787ac27b6d3b6510401ecd..606957c578a59510a4fafac6568477c06de43606:/src/views/process-panel/process-details-attributes.tsx diff --git a/src/views/process-panel/process-details-attributes.tsx b/src/views/process-panel/process-details-attributes.tsx index 6f3277cbf6..b0d8039f93 100644 --- a/src/views/process-panel/process-details-attributes.tsx +++ b/src/views/process-panel/process-details-attributes.tsx @@ -5,7 +5,7 @@ import React from "react"; import { Grid, StyleRulesCallback, withStyles } from "@material-ui/core"; import { Dispatch } from 'redux'; -import { formatDate } from "common/formatters"; +import { formatContainerCost, formatDate } from "common/formatters"; import { resourceLabel } from "common/labels"; import { DetailsAttribute } from "components/details-attribute/details-attribute"; import { ResourceKind } from "models/resource"; @@ -19,6 +19,8 @@ import { navigateToOutput, openWorkflow } from "store/process-panel/process-pane import { ArvadosTheme } from "common/custom-theme"; import { ProcessRuntimeStatus } from "views-components/process-runtime-status/process-runtime-status"; import { getPropertyChip } from "views-components/resource-properties-form/property-chip"; +import { ContainerRequestResource } from "models/container-request"; +import { filterResources } from "store/resources/resources"; type CssRules = 'link' | 'propertyTag'; @@ -37,8 +39,13 @@ const styles: StyleRulesCallback = (theme: ArvadosTheme) => ({ }); const mapStateToProps = (state: RootState, props: { request: ProcessResource }) => { + const process = getProcess(props.request.uuid)(state.resources); return { - container: getProcess(props.request.uuid)(state.resources)?.container, + container: process?.container, + subprocesses: filterResources((resource: ContainerRequestResource) => + resource.kind === ResourceKind.CONTAINER_REQUEST && + resource.requestingContainerUuid === process?.containerRequest.containerUuid + )(state.resources), }; }; @@ -54,11 +61,16 @@ const mapDispatchToProps = (dispatch: Dispatch): ProcessDetailsAttributesActionP export const ProcessDetailsAttributes = withStyles(styles, { withTheme: true })( connect(mapStateToProps, mapDispatchToProps)( - (props: { request: ProcessResource, container?: ContainerResource, twoCol?: boolean, hideProcessPanelRedundantFields?: boolean, classes: Record } & ProcessDetailsAttributesActionProps) => { + (props: { request: ProcessResource, container?: ContainerResource, subprocesses: ContainerRequestResource[], twoCol?: boolean, hideProcessPanelRedundantFields?: boolean, classes: Record } & ProcessDetailsAttributesActionProps) => { const containerRequest = props.request; const container = props.container; + const subprocesses = props.subprocesses; const classes = props.classes; const mdSize = props.twoCol ? 6 : 12; + const filteredPropertyKeys = Object.keys(containerRequest.properties) + .filter(k => (typeof containerRequest.properties[k] !== 'object')); + const hasTotalCost = containerRequest && containerRequest.cumulativeCost > 0; + const totalCostNotReady = container && container.cost > 0 && container.state === "Running" && containerRequest && containerRequest.cumulativeCost === 0 && subprocesses.length > 0; return @@ -67,10 +79,10 @@ export const ProcessDetailsAttributes = withStyles(styles, { withTheme: true })( } - + - @@ -98,15 +110,30 @@ export const ProcessDetailsAttributes = withStyles(styles, { withTheme: true })( + {(containerRequest && containerRequest.modifiedByUserUuid) && + } /> + } + {(container && container.runtimeUserUuid && container.runtimeUserUuid !== containerRequest.modifiedByUserUuid) && + } /> + } - + - + {containerRequest.outputUuid && props.navigateToOutput(containerRequest.outputUuid!)}> } + {container && + 0 ? formatContainerCost(container.cost) : 'not available'} for this container` + } /> + } {containerRequest.properties.template_uuid && props.openWorkflow(containerRequest.properties.template_uuid)}> @@ -118,13 +145,13 @@ export const ProcessDetailsAttributes = withStyles(styles, { withTheme: true })( {/* - NOTE: The property list should be kept at the bottom, because it spans - the entire available width, without regards of the twoCol prop. - */} + NOTE: The property list should be kept at the bottom, because it spans + the entire available width, without regards of the twoCol prop. + */} - {Object.keys(containerRequest.properties).length > 0 - ? Object.keys(containerRequest.properties).map(k => + {filteredPropertyKeys.length > 0 + ? filteredPropertyKeys.map(k => Array.isArray(containerRequest.properties[k]) ? containerRequest.properties[k].map((v: string) => getPropertyChip(k, v, undefined, classes.propertyTag))