Skip to content

Commit

Permalink
Merge branch 'main' into tedi/download
Browse files Browse the repository at this point in the history
  • Loading branch information
tedim52 authored Nov 9, 2023
2 parents 1af09ed + 8f631ae commit 6de3475
Show file tree
Hide file tree
Showing 10 changed files with 35 additions and 24 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,17 +8,22 @@ export const CreateEnclaveButton = () => {
return (
<>
<Menu matchWidth>
<MenuButton as={Button} colorScheme={"kurtosisGreen"} leftIcon={<FiPlus />} size={"md"}>
<MenuButton
as={Button}
colorScheme={"kurtosisGreen"}
leftIcon={<FiPlus />} size={"md"}
onClick={() => navigate(`#${KURTOSIS_CREATE_ENCLAVE_URL_ARG}`)}
>
New Enclave
</MenuButton>
<MenuList>
<MenuItem onClick={() => navigate(`#${KURTOSIS_CREATE_ENCLAVE_URL_ARG}`)} icon={<FiSettings />}>
Manual
</MenuItem>
<MenuItem onClick={() => navigate("/catalog")} icon={<FiPackage />}>
Catalog
</MenuItem>
</MenuList>
{/*<MenuList>*/}
{/* <MenuItem onClick={() => navigate(`#${KURTOSIS_CREATE_ENCLAVE_URL_ARG}`)} icon={<FiSettings />}>*/}
{/* Manual*/}
{/* </MenuItem>*/}
{/* <MenuItem onClick={() => navigate("/catalog")} icon={<FiPackage />}>*/}
{/* Catalog*/}
{/* </MenuItem>*/}
{/*</MenuList>*/}
</Menu>
</>
);
Expand Down
9 changes: 6 additions & 3 deletions enclave-manager/web/src/components/enclaves/logs/LogLine.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ export const LogLine = ({ timestamp, message, status }: LogLineProps) => {
};

return (
<Flex borderBottom={"1px solid #444444"} p={"14px 0"} m={"0 16px"} gap={"8px"} alignItems={"center"}>
<Flex p={"2px 0"} m={"0 16px"} gap={"8px"} alignItems={"top"}>
{isDefined(timestamp) && (
<Box
as={"pre"}
Expand All @@ -35,8 +35,11 @@ export const LogLine = ({ timestamp, message, status }: LogLineProps) => {
fontWeight={600}
fontFamily={logFontFamily}
color={"grey"}
minW={"200px"}
>
{timestamp.toLocal().toFormat("yyyy-MM-dd HH:MM:ss.SSS ZZZZ")} |
<>
{timestamp.toLocal().toFormat("yyyy-MM-dd HH:mm:ss.SSS ZZZZ")}
</>
</Box>
)}
<Box
Expand All @@ -48,7 +51,7 @@ export const LogLine = ({ timestamp, message, status }: LogLineProps) => {
fontFamily={logFontFamily}
color={statusToColor(status)}
>
{message || <i>No message</i>}
{message}
</Box>
</Flex>
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ export const LogViewer = ({

return (
<Flex flexDirection={"column"} gap={"32px"}>
<Flex flexDirection={"column"} position={"relative"} bg={"gray.800"} borderRadius={"8px"}>
<Flex flexDirection={"column"} position={"relative"} bg={"gray.800"}>
{isDefined(ProgressWidget) && (
<Box
display={"inline-flex"}
Expand Down Expand Up @@ -84,7 +84,7 @@ export const LogViewer = ({
isScrolling={setUserIsScrolling}
style={{ height: "660px" }}
data={logLines.filter(({ message }) => isDefined(message))}
itemContent={(index, line) => <LogLine {...line} />}
itemContent={(_, line) => <LogLine {...line} />}
/>
{isDefined(progressPercent) && (
<Progress
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,7 @@ export const ManualCreateEnclaveModal = ({ isOpen, onClose, onConfirm }: ManualC
disabled: isLoading,
required: true,
})}
autoFocus={true}
/>
</InputGroup>
<FormErrorMessage>{errors.url?.message}</FormErrorMessage>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,8 @@ export const ServiceLogs = ({ enclave, service }: ServiceLogsProps) => {
if (cancelled) {
return;
}
setLogLines((logLines) => [...logLines, { message: `Error: ${stringifyError(error)}`, status: "error" }]);
// TODO: We need a way to show this error
alert (`Error: ${stringifyError(error)}`)
}
})();
return () => {
Expand All @@ -54,5 +55,6 @@ export const ServiceLogs = ({ enclave, service }: ServiceLogsProps) => {
};
}, [enclave, service, kurtosisClient]);

return <LogViewer logLines={logLines} />;
const logsFileName = `${enclave.name}--${service.name}-logs.txt`
return <LogViewer logLines={logLines} logsFileName={logsFileName}/>;
};
6 changes: 3 additions & 3 deletions engine/server/webapp/asset-manifest.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
{
"files": {
"main.js": "./static/js/main.9849104e.js",
"main.js": "./static/js/main.f42abf2e.js",
"index.html": "./index.html",
"main.9849104e.js.map": "./static/js/main.9849104e.js.map"
"main.f42abf2e.js.map": "./static/js/main.f42abf2e.js.map"
},
"entrypoints": [
"static/js/main.9849104e.js"
"static/js/main.f42abf2e.js"
]
}
2 changes: 1 addition & 1 deletion engine/server/webapp/index.html
Original file line number Diff line number Diff line change
@@ -1 +1 @@
<!doctype html><html lang="en"><head><meta charset="utf-8"/><link rel="icon" href="./favicon.ico"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><meta name="description" content="Kurtosis Enclave Manager"/><title>Kurtosis Enclave Manager</title><script defer="defer" src="./static/js/main.9849104e.js"></script></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>
<!doctype html><html lang="en"><head><meta charset="utf-8"/><link rel="icon" href="./favicon.ico"/><meta name="viewport" content="width=device-width,initial-scale=1"/><meta name="theme-color" content="#000000"/><meta name="description" content="Kurtosis Enclave Manager"/><title>Kurtosis Enclave Manager</title><script defer="defer" src="./static/js/main.f42abf2e.js"></script></head><body><noscript>You need to enable JavaScript to run this app.</noscript><div id="root"></div></body></html>

Large diffs are not rendered by default.

Large diffs are not rendered by default.

0 comments on commit 6de3475

Please sign in to comment.