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
linter fixes
  • Loading branch information
@SasSwart
SasSwart committed Nov 22, 2024
commit e281d6b7530ef2f2f695b4f736dee19d9a78f644
9 changes: 6 additions & 3 deletions site/src/modules/provisioners/ProvisionerAlert.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
import { Alert, AlertColor, AlertTitle } from "@mui/material";
import { Theme } from "@mui/material";
import Alert from "@mui/material/Alert";
import AlertTitle from "@mui/material/AlertTitle";
import type { AlertColor } from "@mui/material/Alert";
import { AlertDetail } from "components/Alert/Alert";
import { type FC } from "react";
import type { FC } from "react";

type ProvisionerAlertProps = {
title: string,
Expand All @@ -16,7 +19,7 @@ export const ProvisionerAlert : FC<ProvisionerAlertProps> = ({
return (
<Alert
severity={severity}
css={(theme) => ({
css={(theme: Theme) => ({
borderRadius: 0,
border: 0,
borderBottom: `1px solid ${theme.palette.divider}`,
Expand Down
4 changes: 2 additions & 2 deletions site/src/pages/CreateTemplatePage/BuildLogsDrawer.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { type Interpolation, type Theme } from "@emotion/react";
import type { Interpolation, Theme } from "@emotion/react";
import Close from "@mui/icons-material/Close";
import WarningOutlined from "@mui/icons-material/WarningOutlined";
import Button from "@mui/material/Button";
Expand Down Expand Up @@ -86,7 +86,7 @@ export const BuildLogsDrawer: FC<BuildLogsDrawerProps> = ({
title="Something went wrong"
detail="Could not determine provisioner status. Your template build may fail. If your template does not build, please contact your administrator"
/>
) : (!compatibleProvisioners || compatibleProvisioners.length == 0) ? (
) : (!compatibleProvisioners || compatibleProvisioners.length === 0) ? (
<ProvisionerAlert
severity="warning"
title="Template Creation Stuck"
Expand Down
2 changes: 1 addition & 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 @@ -612,7 +612,7 @@ export const TemplateVersionEditor: FC<TemplateVersionEditorProps> = ({
title="Something went wrong"
detail="Could not determine provisioner status. Your template build may fail. If your template does not build, please contact your administrator"
/>
) : !compatibleProvisioners || compatibleProvisioners.length == 0 ? (
) : !compatibleProvisioners || compatibleProvisioners.length === 0 ? (
<ProvisionerAlert
severity="warning"
title="Build Stuck"
Expand Down
Loading