Skip to content

Change the loading flow by always wait the runtime before open IDE#10263

Merged
ashumilova merged 2 commits intomasterfrom
loader_for_wn
Jul 9, 2018
Merged

Change the loading flow by always wait the runtime before open IDE#10263
ashumilova merged 2 commits intomasterfrom
loader_for_wn

Conversation

@ashumilova
Copy link
Copy Markdown
Contributor

Signed-off-by: Anna Shumilova ashumilo@redhat.com

What does this PR do?

Changes the flow of the workspace loading application:

  1. we used to check only workspace config for existing IDE server -> now we are waiting for runtime to appear and check the runtime.
  2. GWT IDE used to show all loading sequence immediately -> now we wait for runtime and only then redirect to GWT IDE if no IDE server was found.
  3. Added errors handling and displaying in workspace loader.

What issues does this PR fix or reference?

#9870

Release Notes

Docs PR

@ashumilova ashumilova added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Jul 4, 2018
@ashumilova ashumilova self-assigned this Jul 4, 2018
@ashumilova ashumilova requested a review from vparfonov as a code owner July 4, 2018 11:31
Copy link
Copy Markdown
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

@ashumilova do we have all workspace events displayed on the loader ?

@ashumilova
Copy link
Copy Markdown
Contributor Author

@benoitf We are tracking only 2: workspace status event (for further actions) and environment output - for displaying logs.

@benoitf
Copy link
Copy Markdown
Contributor

benoitf commented Jul 4, 2018

@ashumilova so it will include #10228 events as well ?

…d add error handling

Signed-off-by: Anna Shumilova <ashumilo@redhat.com>
@ashumilova ashumilova merged commit 2f7a5a4 into master Jul 9, 2018
@ashumilova ashumilova deleted the loader_for_wn branch July 9, 2018 08:57
@benoitf benoitf added this to the 6.8.0 milestone Jul 9, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jul 9, 2018
@ashumilova
Copy link
Copy Markdown
Contributor Author

@benoitf perhaps it should, if are published in machine/log channel.

hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
…clipse-che#10263)

* Change the loading flow by always wait the runtime before open IDE and add error handling

Signed-off-by: Anna Shumilova <ashumilo@redhat.com>
skabashnyuk pushed a commit to skabashnyuk/che that referenced this pull request Mar 11, 2020
…clipse-che#10263)

* Change the loading flow by always wait the runtime before open IDE and add error handling

Signed-off-by: Anna Shumilova <ashumilo@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Outline of a bug - must adhere to the bug report template.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants