From 0e98c0e4560b8c898e7b666f190b4ad161529989 Mon Sep 17 00:00:00 2001
From: Sas Swart <sas.swart.cdk@gmail.com>
Date: Thu, 12 Dec 2024 17:25:10 +0200
Subject: [PATCH] fix(site): remove a misplaced warning banner in the frontend
 (#15837)

This PR fixes some faulty frontend logic that was introduced in #15686
---
 site/src/pages/WorkspacePage/Workspace.stories.tsx | 11 ++++++++++-
 site/src/pages/WorkspacePage/Workspace.tsx         |  8 ++++----
 2 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/site/src/pages/WorkspacePage/Workspace.stories.tsx b/site/src/pages/WorkspacePage/Workspace.stories.tsx
index 3dd05a398c..6efbeef76e 100644
--- a/site/src/pages/WorkspacePage/Workspace.stories.tsx
+++ b/site/src/pages/WorkspacePage/Workspace.stories.tsx
@@ -63,7 +63,16 @@ type Story = StoryObj<typeof Workspace>;
 
 export const Running: Story = {
 	args: {
-		workspace: Mocks.MockWorkspace,
+		workspace: {
+			...Mocks.MockWorkspace,
+			latest_build: {
+				...Mocks.MockWorkspace.latest_build,
+				matched_provisioners: {
+					count: 0,
+					available: 0,
+				},
+			},
+		},
 		handleStart: action("start"),
 		handleStop: action("stop"),
 		buildInfo: Mocks.MockBuildInfo,
diff --git a/site/src/pages/WorkspacePage/Workspace.tsx b/site/src/pages/WorkspacePage/Workspace.tsx
index 1d89e63d89..af4883e737 100644
--- a/site/src/pages/WorkspacePage/Workspace.tsx
+++ b/site/src/pages/WorkspacePage/Workspace.tsx
@@ -110,13 +110,13 @@ export const Workspace: FC<WorkspaceProps> = ({
 		(r) => resourceOptionValue(r) === resourcesNav.value,
 	);
 
-	const shouldDisplayBuildLogs =
-		(buildLogs ?? []).length > 0 && workspace.latest_build.status !== "running";
-
+	const workspaceRunning = workspace.latest_build.status === "running";
+	const haveBuildLogs = (buildLogs ?? []).length > 0;
 	const provisionersHealthy =
 		(workspace.latest_build.matched_provisioners?.available ?? 0) > 0;
+	const shouldDisplayBuildLogs = haveBuildLogs && !workspaceRunning;
 	const shouldShowProvisionerAlert =
-		!provisionersHealthy && (!buildLogs || buildLogs.length === 0);
+		!workspaceRunning && !haveBuildLogs && !provisionersHealthy;
 
 	return (
 		<div