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.

Sign up for GitHub

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

Jump to bottom

workspace welcome page fails to load in single-host #17694

Closed
5 tasks done
sparkoo opened this issue Aug 24, 2020 · 7 comments
Closed
5 tasks done

workspace welcome page fails to load in single-host #17694

sparkoo opened this issue Aug 24, 2020 · 7 comments
Assignees
@azatsarynnyy
Labels
area/editor/theia Issues related to the che-theia IDE of Che 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
7.23

Comments

@sparkoo
Copy link
Member

sparkoo commented Aug 24, 2020

Describe the bug

Having Che deployed in single-host, workspace welcome page fails to load. It just shows Cannot GET /serverj4587ysj-jwtproxy/server-4402/webview/index.html.

Che version

  • latest

Steps to reproduce

  1. deploy Che in single-host mode
  2. start any workspace
  3. see workspace welcome page error

Expected behavior

Welcome page should load correctly

Runtime

  • minikube (include output of minikube version and kubectl version)

Screenshots

20200824_104420_1650x941_scrot

Installation method

  • chectl

Environment

  • my computer
    • Linux

Eclipse Che Logs

Additional context
@sparkoo sparkoo 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/che-server labels Aug 24, 2020
@sparkoo
Copy link
Member Author

sparkoo commented Aug 24, 2020

not yet sure if it's wsmaster or che-theia issue. Adding both labels for now

@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 Aug 24, 2020
@ericwill ericwill added severity/P2 Has a minor but important impact to the usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Aug 24, 2020
@sparkoo
Copy link
Member Author

sparkoo commented Sep 14, 2020

I can see this 404 request in browser console:

GEThttps://192.168.39.51.nip.io/serverpkys5zyv-jwtproxy/webviews/serverpkys5zyv-jwtproxy/server-4402/webview/index.html?id=ded62160-10e0-4f45-991a-3a4a9e22a26f
[HTTP/2 404 Not Found 10ms]

from this it's clear that url is wrong as it is for some reason concatenating 2 server paths togetner with /serverpkys5zyv-jwtproxy/webviews/serverpkys5zyv-jwtproxy/server-4402/webview/.

I was trying to debug che-theia to figure out how this url is constructed, but I couldn't get anywhere. Leaving this to someone more skilled with che-theia development.

Please ping me for any help with setup.

@azatsarynnyy
Copy link
Member

Thanks @sparkoo for publishing your findings.
As we've discussed with @skabashnyuk, we'll try to help you with that during the next sprint.

@azatsarynnyy azatsarynnyy mentioned this issue Sep 17, 2020
Closed
18 tasks
@azatsarynnyy azatsarynnyy added this to the 7.20 milestone Sep 17, 2020
@azatsarynnyy
Copy link
Member

Moving to 7.22 as we don't have enough resources to look at it in 7.21

@azatsarynnyy azatsarynnyy modified the milestones: 7.20, 7.22 Oct 7, 2020
@azatsarynnyy
Copy link
Member

I'm setting P1, as a single host mode is going to be a default.

@azatsarynnyy azatsarynnyy 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 Oct 22, 2020
@nickboldt
Copy link
Contributor

Marking this critical in downstream for 2.6 (7.22): https://issues.redhat.com/browse/CRW-1336

@azatsarynnyy azatsarynnyy added status/in-progress This issue has been taken by an engineer and is under active development. and removed area/che-server team/editors labels Oct 30, 2020
@azatsarynnyy azatsarynnyy mentioned this issue Nov 2, 2020
Closed
16 tasks
@ibuziuk ibuziuk mentioned this issue Nov 6, 2020
Merged
@azatsarynnyy azatsarynnyy modified the milestones: 7.22, 7.23 Nov 12, 2020
This was referenced Nov 18, 2020
Closed
Merged
@azatsarynnyy azatsarynnyy removed the status/in-progress This issue has been taken by an engineer and is under active development. label Nov 24, 2020
@azatsarynnyy
Copy link
Member

Fixed by eclipse-che/che-theia#932

@azatsarynnyy azatsarynnyy mentioned this issue Nov 30, 2020
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azatsarynnyy azatsarynnyy

Labels
area/editor/theia Issues related to the che-theia IDE of Che 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
Milestone
7.23
Development

No branches or pull requests
6 participants
@nickboldt @sparkoo @skabashnyuk @azatsarynnyy @ericwill @che-bot