Skip to content

fix(ui): extend IDE URL timeout to 40 seconds - #1671

Open
olexii4 wants to merge 1 commit into
mainfrom
CRW-12569
Open

fix(ui): extend IDE URL timeout to 40 seconds#1671
olexii4 wants to merge 1 commit into
mainfrom
CRW-12569

Conversation

@olexii4

@olexii4 olexii4 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Increases the IDE URL timeout from 20 to 40 seconds during workspace startup.

Root Cause

When a workspace starts on a slow cluster or pulls a large editor image,
the IDE URL may not be available within 20 seconds. The TIMEOUT_TO_GET_URL_SEC
constant in WorkspaceProgress/const.ts controls how long the "Open IDE" step
waits before showing an error. 20 seconds is too aggressive for environments
with cold image pulls or high API latency.

Fix

-export const TIMEOUT_TO_GET_URL_SEC = 20;
+export const TIMEOUT_TO_GET_URL_SEC = 40;

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

fixes https://redhat.atlassian.net/browse/CRW-12569

Is it tested? How?

Unit tests in StartingSteps/OpenWorkspace/__tests__/index.spec.tsx (12 tests) cover the timeout value and error message.

Manual verification:

  1. Deploy Red Hat OpenShift Dev Spaces on an air-gapped cluster from changes from this PR
  2. Open the dashboard
  3. Select a desktop editor (e.g. Kiro)
  4. Select Empty Workspace
  5. Wait for the "Open IDE" step
  6. Workspace should start without any errors

Release Notes

Extended the IDE URL timeout from 20 to 40 seconds to prevent false startup errors on slower clusters.

Docs PR

N/A

Assisted-by: Claude Opus 4.6
Signed-off-by: Oleksii Orel <oorel@redhat.com>
@openshift-ci

openshift-ci Bot commented Sep 10, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: olexii4

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@olexii4
olexii4 requested a lite review from Copilot and removed request for akurinnoy and ibuziuk September 10, 2026 17:45
@che-bot

che-bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Click here to review and test in web IDE: Contribute

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

Change is narrowly scoped to a single constant used by the “Open IDE” step and aligns with the stated goal of avoiding premature timeout errors.

Pull request overview

This PR increases the workspace startup “Open IDE” step timeout by extending TIMEOUT_TO_GET_URL_SEC from 20s to 40s, reducing false failures on slow clusters (e.g., cold image pulls / higher latency).

Changes:

  • Increased TIMEOUT_TO_GET_URL_SEC from 20 to 40 seconds in WorkspaceProgress constants.
File summaries
File Description
packages/dashboard-frontend/src/components/WorkspaceProgress/const.ts Extends IDE URL wait timeout constant from 20s to 40s to better accommodate slow startup conditions.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1671 (linux/amd64, linux/arm64, linux/s390x)

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1671", name: che-dashboard}]}}]"

@olexii4

olexii4 commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.44%. Comparing base (d67917b) to head (7777308).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1671   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files         609      609           
  Lines       63875    63875           
  Branches     5015     5015           
=======================================
  Hits        59052    59052           
  Misses       4758     4758           
  Partials       65       65           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants