RUNNING = 'Running',
WARNING = 'Warning',
UNKNOWN = 'Unknown',
+ REUSED = 'Reused',
}
export const getProcess = (uuid: string) => (resources: ResourcesState): Process | undefined => {
running = true;
break;
case ProcessStatus.COMPLETED:
+ case ProcessStatus.REUSED:
color = theme.customs.colors.green800;
break;
case ProcessStatus.WARNING:
// Set text color to status color when running, else use white text for solid button
color: running ? color : theme.palette.common.white,
// Set border color when running, else omit the style entirely
- ...(running ? {border: `2px solid ${color}`} : {}),
+ ...(running ? { border: `2px solid ${color}` } : {}),
};
};
case containerRequest.state === ContainerRequestState.UNCOMMITTED:
return ProcessStatus.DRAFT;
- case container?.state === ContainerState.COMPLETE:
+ case container && container.state === ContainerState.COMPLETE:
if (container?.exitCode === 0) {
+ if (containerRequest && container.finishedAt) {
+ // don't compare on createdAt because the container can
+ // have a slightly earlier creation time when it is created
+ // in the same transaction as the container request.
+ // use finishedAt because most people will assume "reused" means
+ // no additional work needed to be done, it's possible
+ // to share a running container but calling it "reused" in that case
+ // is more likely to just be confusing.
+ const finishedAt = new Date(container.finishedAt).getTime();
+ const createdAt = new Date(containerRequest.createdAt).getTime();
+ if (finishedAt < createdAt) {
+ return ProcessStatus.REUSED;
+ }
+ }
return ProcessStatus.COMPLETED;
}
return ProcessStatus.FAILED;
containerRequest.priority === 0 &&
// Don't show run button when container is present & running or cancelled
!(container && (container.state === ContainerState.RUNNING ||
- container.state === ContainerState.CANCELLED ||
- container.state === ContainerState.COMPLETE))
+ container.state === ContainerState.CANCELLED ||
+ container.state === ContainerState.COMPLETE))
);
export const isProcessCancelable = ({ containerRequest, container }: Process): boolean => (
containerRequest.priority !== null &&
containerRequest.priority > 0 &&
container !== undefined &&
- (container.state === ContainerState.QUEUED ||
+ (container.state === ContainerState.QUEUED ||
container.state === ContainerState.LOCKED ||
container.state === ContainerState.RUNNING)
);