Skip to content

fix: let workspace pages download partial logs for unhealthy workspaces #13761

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Jul 3, 2024
Merged

fix: let workspace pages download partial logs for unhealthy workspaces #13761

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
f1f5f88
fix: get basic fix in for preventing download logs from blowing up UI
Parkreiner Jul 1, 2024
edd6362
fix: make sure blob units can't go out of bounds
Parkreiner Jul 1, 2024
901aa39
fix: make sure timeout is cleared on component unmount
Parkreiner Jul 1, 2024
e1d8113
fix: reduce risk of shared cache state breaking useAgentLogs
Parkreiner Jul 2, 2024
87f57aa
fix: allow partial downloading of logs
Parkreiner Jul 2, 2024
5bf2baf
fix: make sure useMemo cache is used properly
Parkreiner Jul 2, 2024
5c2316b
wip: commit current progress on updated logs functionality
Parkreiner Jul 2, 2024
edd6569
docs: rewrite comment for clarity
Parkreiner Jul 2, 2024
2eb7d6e
refactor: clean up current code
Parkreiner Jul 2, 2024
fa06515
fix: update styles for unavailable logs
Parkreiner Jul 2, 2024
8fb4496
fix: resolve linter violations
Parkreiner Jul 2, 2024
6935f50
fix: update type signature of getErrorDetail
Parkreiner Jul 2, 2024
f74eda4
fix: revert log/enabled logic for useAgentLogs
Parkreiner Jul 2, 2024
7b76eb7
fix: remove memoization from DownloadLogsDialog
Parkreiner Jul 2, 2024
07ff536
fix: update name of timeout state
Parkreiner Jul 2, 2024
460aa53
refactor: make log web sockets logic more clear
Parkreiner Jul 2, 2024
5a449b4
docs: reword comment for clarity
Parkreiner Jul 2, 2024
eb4f88c
fix: commit current style update progress
Parkreiner Jul 2, 2024
71692b6
fix: finish style updates
Parkreiner Jul 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: make sure timeout is cleared on component unmount
  • Loading branch information
@Parkreiner
Parkreiner committed Jul 1, 2024
commit 901aa39c9a220d47618e20a713afd5dd90ca0497
15 changes: 13 additions & 2 deletions site/src/pages/WorkspacePage/WorkspaceActions/DownloadLogsDialog.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { useTheme, type Interpolation, type Theme } from "@emotion/react";
import Skeleton from "@mui/material/Skeleton";
import { saveAs } from "file-saver";
import JSZip from "jszip";
import { useMemo, useState, type FC } from "react";
import { useMemo, useState, type FC, useRef, useEffect } from "react";
import { useQueries, useQuery } from "react-query";
import { agentLogs, buildLogs } from "api/queries/workspaces";
import type { Workspace, WorkspaceAgent } from "api/typesGenerated";
Expand Down Expand Up @@ -79,6 +79,15 @@ export const DownloadLogsDialog: FC<DownloadLogsDialogProps> = ({
const isLoadingFiles = downloadableFiles.some((f) => f.blob === undefined);
const [isDownloading, setIsDownloading] = useState(false);

const timeoutIdRef = useRef<number | undefined>(undefined);
useEffect(() => {
const clearTimeoutOnUnmount = () => {
window.clearTimeout(timeoutIdRef.current);
};

return clearTimeoutOnUnmount;
}, []);

return (
<ConfirmDialog
{...dialogProps}
Expand All @@ -96,10 +105,12 @@ export const DownloadLogsDialog: FC<DownloadLogsDialogProps> = ({
zip.file(f.name, f.blob);
}
});

const content = await zip.generateAsync({ type: "blob" });
download(content, `${workspace.name}-logs.zip`);
dialogProps.onClose();
setTimeout(() => {

timeoutIdRef.current = window.setTimeout(() => {
setIsDownloading(false);
}, theme.transitions.duration.leavingScreen);
} catch (error) {
Expand Down