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
docs: rewrite comment for clarity
  • Loading branch information
@Parkreiner
Parkreiner committed Jul 2, 2024
commit edd6569b8d4f571343dc7424d5b2ec34808fb237
7 changes: 4 additions & 3 deletions site/src/modules/resources/AgentLogs/useAgentLogs.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,10 @@ export function useAgentLogs(

// One pitfall with the current approach: the enabled property does NOT
// prevent the useQuery call above from eventually having data. All it does
// is prevent it from getting data on its own. If a different useQuery call
// elsewhere in the app is enabled and gets data, the useQuery call here will
// re-render with that same new data, even if it's disabled. This can EASILY
// is prevent it from getting data on its own. Let's say a different useQuery
// call elsewhere in the app has the same query key and is enabled. When it
// gets data back from the server, the useQuery call here will re-render with
// that same new data, even though this state is "disabled". This can EASILY
// cause bugs.
const logs = enabled ? query.data : undefined;

Expand Down