fix: remove conditions before restart a workspace

Signed-off-by: Oleksii Orel <[email protected]>

packages/dashboard-frontend/src/components/WorkspaceProgress/__tests__/utils.spec.ts

@@ -0,0 +1,119 @@
+/*
+ * Copyright (c) 2018-2024 Red Hat, Inc.
+ * This program and the accompanying materials are made
+ * available under the terms of the Eclipse Public License 2.0
+ * which is available at https://www.eclipse.org/legal/epl-2.0/
+ *
+ * SPDX-License-Identifier: EPL-2.0
+ *
+ * Contributors:
+ *   Red Hat, Inc. - initial API and implementation
+ */
+
+import { getStartWorkspaceConditions } from '@/components/WorkspaceProgress/utils';
+import { DevWorkspaceBuilder } from '@/store/__mocks__/devWorkspaceBuilder';
+
+describe('WorkspaceProgress utils', () => {
+  beforeEach(() => {});
+
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  describe('getStartWorkspaceCondition', () => {
+    it('should return an empty array as a default value', () => {
+      const devWorkspace = new DevWorkspaceBuilder().build();
+
+      expect(devWorkspace.status?.conditions).toBeUndefined();
+
+      const conditions = getStartWorkspaceConditions(devWorkspace);
+
+      expect(conditions).toEqual([]);
+    });
+    it('should return conditions from the devWorkspace', () => {
+      const status = {
+        conditions: [
+          {
+            type: 'Stopped',
+            status: 'False',
+            reason: 'LimitReached',
+            message: 'Workspace stopped due to error.',
+          },
+        ],
+      };
+      const devWorkspace = new DevWorkspaceBuilder().withStatus(status).build();
+
+      const conditions = getStartWorkspaceConditions(devWorkspace);
+
+      expect(conditions).toEqual(status.conditions);
+    });
+  });
+  it('should filter conditions that are not related to the workspace start', () => {
+    const status = {
+      conditions: [
+        {
+          message: 'DevWorkspace is starting',
+          status: 'True',
+          type: 'Started',
+        },
+        {
+          message: 'Resolved plugins and parents from DevWorkspace',
+          status: 'True',
+          type: 'DevWorkspaceResolved',
+        },
+        {
+          message: 'Storage ready',
+          status: 'True',
+          type: 'StorageReady',
+        },
+        {
+          message: 'Networking ready',
+          status: 'True',
+          type: 'RoutingReady',
+        },
+        {
+          message: 'DevWorkspace serviceaccount ready',
+          status: 'True',
+          type: 'ServiceAccountReady',
+        },
+        {
+          message: 'Waiting for workspace deployment',
+          status: 'False',
+          type: 'DeploymentReady',
+        },
+      ],
+    };
+    const devWorkspace = new DevWorkspaceBuilder().withStatus(status).build();
+
+    const conditions = getStartWorkspaceConditions(devWorkspace);
+
+    expect(conditions).not.toEqual(status.conditions);
+    expect(conditions).toEqual([
+      {
+        message: 'DevWorkspace is starting',
+        status: 'True',
+        type: 'Started',
+      },
+      {
+        message: 'Resolved plugins and parents from DevWorkspace',
+        status: 'True',
+        type: 'DevWorkspaceResolved',
+      },
+      {
+        message: 'Storage ready',
+        status: 'True',
+        type: 'StorageReady',
+      },
+      {
+        message: 'Networking ready',
+        status: 'True',
+        type: 'RoutingReady',
+      },
+      {
+        message: 'DevWorkspace serviceaccount ready',
+        status: 'True',
+        type: 'ServiceAccountReady',
+      },
+    ]);
+  });
+});

packages/dashboard-frontend/src/components/WorkspaceProgress/index.tsx

@@ -30,7 +30,11 @@ import StartingStepInitialize from '@/components/WorkspaceProgress/StartingSteps
 import StartingStepOpenWorkspace from '@/components/WorkspaceProgress/StartingSteps/OpenWorkspace';
 import StartingStepStartWorkspace from '@/components/WorkspaceProgress/StartingSteps/StartWorkspace';
 import StartingStepWorkspaceConditions from '@/components/WorkspaceProgress/StartingSteps/WorkspaceConditions';
-import { ConditionType, isWorkspaceStatusCondition } from '@/components/WorkspaceProgress/utils';
+import {
+  ConditionType,
+  getStartWorkspaceConditions,
+  isWorkspaceStatusCondition,
+} from '@/components/WorkspaceProgress/utils';
 import WorkspaceProgressWizard, {
   WorkspaceProgressWizardStep,
 } from '@/components/WorkspaceProgress/Wizard';
@@ -184,7 +188,7 @@ class Progress extends React.Component<Props, State> {
         workspace.status === DevWorkspaceStatus.FAILING ||
         workspace.status === DevWorkspaceStatus.FAILED)
     ) {
-      const conditions = workspace.ref.status?.conditions || [];
+      const conditions = getStartWorkspaceConditions(workspace.ref);
 
       const lastScore = this.scoreConditions(this.state.conditions);
       const score = this.scoreConditions(conditions);

packages/dashboard-frontend/src/components/WorkspaceProgress/utils.ts

@@ -12,6 +12,8 @@
 
 import { V1alpha2DevWorkspaceStatusConditions } from '@devfile/api';
 
+import devfileApi from '@/services/devfileApi';
+
 export type ConditionType = V1alpha2DevWorkspaceStatusConditions & {
   status: 'True' | 'False' | 'Unknown';
 };
@@ -37,6 +39,24 @@ export function isConditionReady(
   );
 }
 
+export function getStartWorkspaceConditions(
+  workspace: devfileApi.DevWorkspace,
+): V1alpha2DevWorkspaceStatusConditions[] {
+  if (!workspace.status?.conditions || workspace.status.conditions.length === 0) {
+    return [];
+  }
+  const conditions: V1alpha2DevWorkspaceStatusConditions[] = [...workspace.status.conditions];
+  // remove all conditions that are not related to the workspace start
+  for (let i = conditions.length; i > 0; i--) {
+    if (conditions[i - 1].type === 'ServiceAccountReady') {
+      conditions.length = i;
+      break;
+    }
+  }
+
+  return conditions;
+}
+
 export function isConditionError(
   condition: ConditionType,
   prevCondition: ConditionType | undefined,