Skip to content

Make container status check less restrictive#1523

Merged
dkwon17 merged 1 commit intodevfile:mainfrom
dkwon17:workspace-restart
Oct 15, 2025
Merged

Make container status check less restrictive#1523
dkwon17 merged 1 commit intodevfile:mainfrom
dkwon17:workspace-restart

Conversation

@dkwon17
Copy link
Copy Markdown
Collaborator

@dkwon17 dkwon17 commented Oct 14, 2025

What does this PR do?

Fixes issue with workspace restarts.

The issue with workspace restarts happens when the new CDE pod is starting when the previous CDE pod has not yet terminated. For example:
image

In this case, there is a temporary error with the new pod. The error has no error message, but looks like the following:

Container 'universal-developer-image' terminated with Error: 

This error seems to be occurring for only a few seconds, and dissappears in seconds. Although, I'm not too sure what is the root cause of this error.

Nevertheless, this PR reverts an error check that was done as part of #1440 to allow the DevWorkspace startup to not fail when restarting the DevWorkspace.

What issues does this PR fix or reference?

eclipse-che/che#23551

Is it tested? How?

  1. Build and deploy the DWO with Eclipse Che.

The catalog source I used for testing is:

apiVersion: operators.coreos.com/v1alpha1
kind: CatalogSource
metadata:
  name: devworkspace-operator-testing-catalog
  namespace: openshift-marketplace
spec:
  displayName: DevWorkspace Operator Catalog
  image: 'quay.io/dkwon17/devworkspace-operator-index:workspace-restart'
  publisher: DWO Testing
  sourceType: grpc
  1. Create an empty workspace.
  2. Wait until the workspace is running.
  3. From the Che Dashboard, restart the running workspace.
  4. Verify that the issue mentioned in Error at restarting workspace eclipse-che/che#23551 is not reproduced.
  5. Verify that the post start timeout feature still works by following steps 2 and onwards from postStart hook commands timeout #1440.

PR Checklist

  • E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)
    • v8-devworkspace-operator-e2e: DevWorkspace e2e test
    • v8-che-happy-path: Happy path for verification integration with Che

Signed-off-by: David Kwon <dakwon@redhat.com>
@rohanKanojia
Copy link
Copy Markdown
Member

rohanKanojia commented Oct 15, 2025

This PR should also fix #1508

The issue was intermittent within the E2E pipeline. I had initially suspected a configuration or stability problem with the CI infrastructure.

@rohanKanojia
Copy link
Copy Markdown
Member

I tested this PR, and it seems to work as expected.

  1. Restarted the empty workspace multiple times, and it restarted without throwing any errors.
  2. Created a problematic workspace that timed out after the specified postStartTimeout of 30s

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Oct 15, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akurinnoy, dkwon17, rohanKanojia

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

The pull request process is described 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

@dkwon17 dkwon17 merged commit 8d196fb into devfile:main Oct 15, 2025
11 checks passed
@rohanKanojia rohanKanojia modified the milestone: v0.38.x Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants