Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theia terminal extension must propagate error from che-machine-exec #16593

Closed
4 of 24 tasks
sleshchenko opened this issue Apr 14, 2020 · 7 comments
Closed
4 of 24 tasks
Assignees
Labels
area/machine-exec kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@sleshchenko
Copy link
Member

Describe the bug

After something went wrong with my workspace (f?url=https://github.com/eclipse/che-dashboard) I got Theia with empty tabs for tasks and terminal. Refreshing a page does not help.
Empty tabs make impression that terminal/tasks are loading but actually they failed, if I check websocket connections I have the following errors:
Screenshot_20200414_110958

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

It can't be reproduced stable but I got it after.

  1. Create che-dashboard workspace: f?url=https://github.com/eclipse/che-dashboard
  2. Start a terminal
  3. Start Install Dependencies task and wait until it's finished.
  4. Start Build task.
    Actual behavior: Build task is empty forever, it's not possible to use already initialized terminal.

Expected behavior

The progress of Build task is shown. It's possible to use already initialized terminal.
Or at least theia propagates errors which happens during communication with che-machine-exec via websockets.

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube (include output of minikube version and kubectl version)
  • minishift (include output of minishift version and oc version)
  • crc 1.8.0+0a318dc
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

Installation method

  • chectl server:start --chenamespace=vetal --che-operator-cr-patch-yaml ~/che-single-user.yaml --platform=openshift --installer=operator
    where che-single-user.yaml is
spec:
  server:
    selfSignedCert: false
    tlsSupport: false
    customCheProperties:
      CHE_MULTIUSER: "false"
      CHE_LIMITS_USER_WORKSPACES_RAM: "-1"
      CHE_LIMITS_WORKSPACE_IDLE_TIMEOUT: "-1"
      CHE_LOGGER_CONFIG: "export CHE_LOGGER_CONFIG=org.eclipse.che.api.workspace=DEBUG,org.eclipse.che.workspace.infrastructure.kubernetes=DEBUG,org.eclipse.che.workspace.infrastructure.openshift=DEBUG"
  • che-operator
  • minishift-addon
  • I don't know

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify

Eclipse Che Logs

Additional context

@sleshchenko sleshchenko added kind/bug Outline of a bug - must adhere to the bug report template. area/editor/theia Issues related to the che-theia IDE of Che area/machine-exec labels Apr 14, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Apr 14, 2020
@metlos metlos added severity/P2 Has a minor but important impact to the usage or development of the system. team/deploy and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Apr 14, 2020
@che-bot
Copy link
Contributor

che-bot commented Oct 28, 2020

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Oct 28, 2020
@sleshchenko
Copy link
Member Author

/remove-lifecycle stale

@che-bot che-bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 6, 2020
@sleshchenko sleshchenko added severity/P1 Has a major impact to usage or development of the system. and removed severity/P2 Has a minor but important impact to the usage or development of the system. labels Feb 22, 2021
@sleshchenko
Copy link
Member Author

That should be pretty easy fix(websocket already has error message, it just needs to be displayed) but that's so confusing. So, set P1

@azatsarynnyy azatsarynnyy added this to the 7.28 milestone Feb 24, 2021
@tolusha tolusha modified the milestone: 7.28 Feb 25, 2021
@azatsarynnyy azatsarynnyy mentioned this issue Feb 25, 2021
11 tasks
@tolusha tolusha mentioned this issue Feb 25, 2021
70 tasks
@azatsarynnyy
Copy link
Member

We've discussed it with @AndrienkoAleksandr. And it is not clear ATM if it's the problem on Che-Theia side or on machine-exec side. Che-Theia already has an error watcher https://github.com/eclipse/che-theia/blob/9a76d18382e3b90858b1fd63e713eb908facbb70/extensions/eclipse-che-theia-terminal/src/browser/terminal-widget/remote-terminal-widget.ts#L109
So, it requires more investigation on the machine-exec side first.

@tolusha
Copy link
Contributor

tolusha commented Apr 2, 2021

we are waiting for a release of https://github.com/kubernetes/client-go which is expected to be soon
kubernetes/api#36

@tolusha tolusha modified the milestones: 7.29, 7.30 Apr 12, 2021
@tolusha tolusha mentioned this issue Apr 12, 2021
63 tasks
@tolusha tolusha mentioned this issue Apr 29, 2021
41 tasks
@tolusha tolusha modified the milestones: 7.30, 7.31 May 5, 2021
@tolusha tolusha mentioned this issue May 24, 2021
47 tasks
@tolusha tolusha modified the milestones: 7.31, 7.32 May 24, 2021
@tolusha tolusha closed this as completed May 27, 2021
@tolusha tolusha modified the milestones: 7.32, 7.31 May 27, 2021
@tolusha
Copy link
Contributor

tolusha commented May 27, 2021

Switching to a new go client fixes the problem
eclipse-che/che-machine-exec#148

@sleshchenko
Copy link
Member Author

That's awesome if it's true 🎉
I'll let you know if I face it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/machine-exec kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

6 participants