Skip to content

Commit d896b74

Browse files
BrunoQuaresmaBrunoQuaresma
authored
feat(site): display build logs on workspace transitioning statuses (#8397)
1 parent b7641b2 commit d896b74

File tree

20 files changed

+426
-133
lines changed

20 files changed

+426
-133
lines changed

‎coderd/apidoc/docs.go

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎coderd/apidoc/swagger.json

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎codersdk/deployment.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1763,6 +1763,7 @@ const (
17631763
// oidc.
17641764
ExperimentConvertToOIDC Experiment = "convert-to-oidc"
17651765

1766+
ExperimentWorkspaceBuildLogsUI Experiment = "workspace_build_logs_ui"
17661767
// Add new experiments here!
17671768
// ExperimentExample Experiment = "example"
17681769
)
@@ -1771,7 +1772,9 @@ const (
17711772
// users to opt-in to via --experimental='*'.
17721773
// Experiments that are not ready for consumption by all users should
17731774
// not be included here and will be essentially hidden.
1774-
var ExperimentsAll = Experiments{}
1775+
var ExperimentsAll = Experiments{
1776+
ExperimentWorkspaceBuildLogsUI,
1777+
}
17751778

17761779
// Experiments is a list of experiments that are enabled for the deployment.
17771780
// Multiple experiments may be enabled at the same time.

‎docs/api/schemas.md

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2531,12 +2531,13 @@ AuthorizationObject can represent a "set" of objects, such as: all workspaces in
25312531

25322532
#### Enumerated Values
25332533

2534-
| Value |
2535-
| ------------------------ |
2536-
| `moons` |
2537-
| `workspace_actions` |
2538-
| `tailnet_pg_coordinator` |
2539-
| `convert-to-oidc` |
2534+
| Value |
2535+
| ------------------------- |
2536+
| `moons` |
2537+
| `workspace_actions` |
2538+
| `tailnet_pg_coordinator` |
2539+
| `convert-to-oidc` |
2540+
| `workspace_build_logs_ui` |
25402541

25412542
## codersdk.Feature
25422543

‎site/.storybook/preview.jsx

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { HelmetProvider } from "react-helmet-async"
55
import { dark } from "../src/theme"
66
import "../src/theme/globalFonts"
77
import "../src/i18n"
8+
import { LocalPreferencesProvider } from "../src/contexts/LocalPreferencesContext"
89

910
export const decorators = [
1011
(Story) => (
@@ -23,6 +24,13 @@ export const decorators = [
2324
</HelmetProvider>
2425
)
2526
},
27+
(Story) => {
28+
return (
29+
<LocalPreferencesProvider>
30+
<Story />
31+
</LocalPreferencesProvider>
32+
)
33+
},
2634
]
2735

2836
export const parameters = {

‎site/src/api/typesGenerated.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1431,11 +1431,13 @@ export type Experiment =
14311431
| "moons"
14321432
| "tailnet_pg_coordinator"
14331433
| "workspace_actions"
1434+
| "workspace_build_logs_ui"
14341435
export const Experiments: Experiment[] = [
14351436
"convert-to-oidc",
14361437
"moons",
14371438
"tailnet_pg_coordinator",
14381439
"workspace_actions",
1440+
"workspace_build_logs_ui",
14391441
]
14401442

14411443
// From codersdk/deployment.go

‎site/src/app.tsx

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import { GlobalSnackbar } from "./components/GlobalSnackbar/GlobalSnackbar"
99
import { dark } from "./theme"
1010
import "./theme/globalFonts"
1111
import { StyledEngineProvider, ThemeProvider } from "@mui/material/styles"
12+
import { LocalPreferencesProvider } from "contexts/LocalPreferencesContext"
1213

1314
const queryClient = new QueryClient({
1415
defaultOptions: {
@@ -25,17 +26,19 @@ export const AppProviders: FC<PropsWithChildren> = ({ children }) => {
2526
return (
2627
<HelmetProvider>
2728
<StyledEngineProvider injectFirst>
28-
<ThemeProvider theme={dark}>
29-
<CssBaseline enableColorScheme />
30-
<ErrorBoundary>
31-
<QueryClientProvider client={queryClient}>
32-
<AuthProvider>
33-
{children}
34-
<GlobalSnackbar />
35-
</AuthProvider>
36-
</QueryClientProvider>
37-
</ErrorBoundary>
38-
</ThemeProvider>
29+
<LocalPreferencesProvider>
30+
<ThemeProvider theme={dark}>
31+
<CssBaseline enableColorScheme />
32+
<ErrorBoundary>
33+
<QueryClientProvider client={queryClient}>
34+
<AuthProvider>
35+
{children}
36+
<GlobalSnackbar />
37+
</AuthProvider>
38+
</QueryClientProvider>
39+
</ErrorBoundary>
40+
</ThemeProvider>
41+
</LocalPreferencesProvider>
3942
</StyledEngineProvider>
4043
</HelmetProvider>
4144
)

‎site/src/components/Logs/Logs.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,11 @@ const useStyles = makeStyles<
9797
>((theme) => ({
9898
root: {
9999
minHeight: 156,
100-
padding: theme.spacing(2, 0),
100+
padding: theme.spacing(1, 0),
101101
borderRadius: theme.shape.borderRadius,
102102
overflowX: "auto",
103103
background: theme.palette.background.default,
104+
borderBottom: `1px solid ${theme.palette.divider}`,
104105
},
105106
scrollWrapper: {
106107
minWidth: "fit-content",

‎site/src/components/TemplateVersionEditor/TemplateVersionEditor.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,7 @@ export const TemplateVersionEditor: FC<TemplateVersionEditorProps> = ({
379379

380380
{buildLogs && buildLogs.length > 0 && (
381381
<WorkspaceBuildLogs
382-
templateEditorPane
382+
sx={{ borderRadius: 0 }}
383383
hideTimestamps
384384
logs={buildLogs}
385385
/>

‎site/src/components/Workspace/Workspace.stories.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import { withReactContext } from "storybook-react-context"
88
import EventSource from "eventsourcemock"
99
import { ProxyContext, getPreferredProxy } from "contexts/ProxyContext"
1010
import { DashboardProviderContext } from "components/Dashboard/DashboardProvider"
11+
import { WorkspaceBuildLogsSection } from "pages/WorkspacePage/WorkspaceBuildLogsSection"
1112

1213
const MockedAppearance = {
1314
config: Mocks.MockAppearance,
@@ -152,7 +153,7 @@ export const FailedWithLogs: Story = {
152153
},
153154
},
154155
},
155-
failedBuildLogs: makeFailedBuildLogs(),
156+
buildLogs: <WorkspaceBuildLogsSection logs={makeFailedBuildLogs()} />,
156157
},
157158
}
158159

@@ -170,8 +171,8 @@ export const FailedWithRetry: Story = {
170171
},
171172
},
172173
},
173-
failedBuildLogs: makeFailedBuildLogs(),
174174
canRetryDebugMode: true,
175+
buildLogs: <WorkspaceBuildLogsSection logs={makeFailedBuildLogs()} />,
175176
},
176177
}
177178

@@ -229,6 +230,7 @@ export const CancellationError: Story = {
229230
message: "Job could not be canceled.",
230231
}),
231232
},
233+
buildLogs: <WorkspaceBuildLogsSection logs={makeFailedBuildLogs()} />,
232234
},
233235
}
234236

0 commit comments

Comments
 (0)