Skip to content

feat(site): warn on provisioner health during builds #15589

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 18 commits into from
Nov 28, 2024
Merged

feat(site): warn on provisioner health during builds #15589

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
7df3b27
add alert to the create workspace page
SasSwart Nov 19, 2024
1e1262f
improve error case documentation
SasSwart Nov 19, 2024
0f78afc
add provisioner health hook
SasSwart Nov 20, 2024
f0f7216
show provisioner health warnings for templates
SasSwart Nov 21, 2024
a3eeb9c
consistent formatting for alerts across pages
SasSwart Nov 21, 2024
3cf53ef
Finalise Provisioner Warnings for Templates and template versions
SasSwart Nov 22, 2024
e281d6b
linter fixes
SasSwart Nov 22, 2024
2baa81e
Merge remote-tracking branch 'origin/main' into jjs/15048-fe
SasSwart Nov 26, 2024
0cab768
use matched_provisioners instead of a second api call
SasSwart Nov 27, 2024
b228257
provide a key to provisioner tags to keep react happy
SasSwart Nov 27, 2024
8b91dc0
fix linting issues
SasSwart Nov 27, 2024
bec2913
make fmt
SasSwart Nov 27, 2024
4796a32
Copy updates
SasSwart Nov 28, 2024
b6f99a6
make fmt
SasSwart Nov 28, 2024
aec9cba
add loader back in to build logs drawer
SasSwart Nov 28, 2024
75e7394
simplify logic
SasSwart Nov 28, 2024
0aed543
fix logic
SasSwart Nov 28, 2024
0bd9478
make fmt
SasSwart Nov 28, 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
consistent formatting for alerts across pages
  • Loading branch information
@SasSwart
SasSwart committed Nov 21, 2024
commit a3eeb9c52d4ce16e8135dcbc1885b489327fe162
28 changes: 26 additions & 2 deletions site/src/pages/CreateTemplatePage/BuildLogsDrawer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@ import { WorkspaceBuildLogs } from "modules/workspaces/WorkspaceBuildLogs/Worksp
import { type FC, useLayoutEffect, useRef } from "react";
import { navHeight } from "theme/constants";
import { provisionersUnhealthy, useCompatibleProvisioners } from "modules/provisioners/useCompatibleProvisioners";
import { Alert, AlertTitle } from "@mui/material";
import { AlertDetail } from "components/Alert/Alert";

type BuildLogsDrawerProps = {
error: unknown;
Expand Down Expand Up @@ -74,11 +76,33 @@ export const BuildLogsDrawer: FC<BuildLogsDrawerProps> = ({

{ !compatibleProvisioners && !logs ? (
// If there are no compatible provisioners, warn that this job may be stuck
<>No compatible provisioners</>
<Alert
severity="warning"
css={{
borderRadius: 0,
border: 0,
// borderBottom: `1px solid ${theme.palette.divider}`,
// borderLeft: `2px solid ${theme.palette.error.main}`,
}}
>
<AlertTitle>Build stuck</AlertTitle>
<AlertDetail>No Compatible Provisioner Daemons have been configured</AlertDetail>
</Alert>
) : compatibleProvisionersUnhealthy && !logs && (
// If there are compatible provisioners in the db, but they have not reported recent health checks,
// warn that the job might be stuck
<>Compatible provisioners are potentially unhealthy. Your job might be delayed</>
<Alert
severity="warning"
css={{
borderRadius: 0,
border: 0,
// borderBottom: `1px solid ${theme.palette.divider}`,
// borderLeft: `2px solid ${theme.palette.error.main}`,
}}
>
<AlertTitle>Build may be delayed</AlertTitle>
<AlertDetail>Compatible Provisioner Daemons have been silent for a while. This may result in a delayed build</AlertDetail>
</Alert>
)}

{isMissingVariables ? (
Expand Down
15 changes: 14 additions & 1 deletion site/src/pages/TemplateVersionEditorPage/TemplateVersionEditor.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -603,6 +603,19 @@ export const TemplateVersionEditor: FC<TemplateVersionEditorProps> = ({
<AlertDetail>{templateVersion.job.error}</AlertDetail>
</Alert>
</div>
) : !compatibleProvisioners ? (
<Alert
severity="warning"
css={{
borderRadius: 0,
border: 0,
borderBottom: `1px solid ${theme.palette.divider}`,
borderLeft: `2px solid ${theme.palette.error.main}`,
}}
>
<AlertTitle>Build stuck</AlertTitle>
<AlertDetail>No Compatible Provisioner Daemons have been configured</AlertDetail>
</Alert>
) : compatibleProvisionersUnhealthy && (
<div>
<Alert
Expand All @@ -615,7 +628,7 @@ export const TemplateVersionEditor: FC<TemplateVersionEditorProps> = ({
}}
>
<AlertTitle>Build may be delayed</AlertTitle>
<AlertDetail>No Compatible Provisioner Daemons have been recently seen</AlertDetail>
<AlertDetail>Compatible Provisioner Daemons have been silent for a while. This may result in a delayed build</AlertDetail>
</Alert>
</div>
)}
Expand Down
Loading