Skip to content
This repository was archived by the owner on Apr 4, 2023. It is now read-only.

Update to the latest Theia #823

Merged
merged 10 commits into from
Sep 24, 2020
Merged

Update to the latest Theia #823

merged 10 commits into from
Sep 24, 2020

Conversation

azatsarynnyy
Copy link
Member

@azatsarynnyy azatsarynnyy commented Aug 3, 2020

Signed-off-by: Artem Zatsarynnyi azatsary@redhat.com

What does this PR do?

Updates to the latest Theia.

What issues does this PR fix or reference?

closes eclipse-che/che#17357
closes eclipse-che/che#17550
closes eclipse-che/che#17812
closes eclipse-che/che#16751
probably, should help in solving eclipse-che/che#16352

Depends on

eclipse-che/che#17933

Release Notes

Docs PR

Hapy Path Channel

HAPPY_PATH_CHANNEL=next

@eclipse-che eclipse-che deleted a comment from che-bot Sep 1, 2020
@RomanNikitenko RomanNikitenko force-pushed the theia-upgrade branch 2 times, most recently from 64f412f to 5b6813f Compare September 7, 2020 09:52
@eclipse-che eclipse-che deleted a comment from che-bot Sep 7, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 7, 2020
@eclipse-che eclipse-che deleted a comment from azatsarynnyy Sep 7, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 7, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 7, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 9, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 9, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 10, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 10, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 10, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 10, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 10, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 15, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 16, 2020
@eclipse-che eclipse-che deleted a comment from che-bot Sep 16, 2020
@RomanNikitenko
Copy link
Member

@benoitf
I see some resolutions related logic here. Do you mean this place?

@benoitf
Copy link
Contributor

benoitf commented Sep 23, 2020

well it would work only in case of Docker build (not for che-theia inside che-theia development)
but let's resolve that later, priority is to merge this PR :)

@eclipse-che eclipse-che deleted a comment from che-bot Sep 23, 2020
@dmytro-ndp
Copy link
Contributor

[crw-ci-test]

@che-bot
Copy link
Contributor

che-bot commented Sep 23, 2020

❌ E2E Happy path tests failed ❗

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:823
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:823

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

ℹ️ Use comment "[crw-ci-test]" to rerun happy path E2E test.

@RomanNikitenko
Copy link
Member

@che-bot
Copy link
Contributor

che-bot commented Sep 23, 2020

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:823
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:823

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

ℹ️ Use comment "[crw-ci-test]" to rerun happy path E2E test.

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Sep 23, 2020

@RomanNikitenko: please ignore test results #823 (comment) - that was at the middle of Try It buttons links fixup.

azatsarynnyy and others added 10 commits September 24, 2020 15:29
Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
…s language features'

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
@RomanNikitenko
Copy link
Member

RomanNikitenko commented Sep 24, 2020

I did the following:

Now I'm waiting on result of happy path tests and going to merge the PR.

@benoitf
Copy link
Contributor

benoitf commented Sep 24, 2020

cool, lot of commits to handle since August 3 ! 🎉

@RomanNikitenko
Copy link
Member

Looks like after alignment with current master of theia and che-theia we have a new problem:

 1) TypeScript
       Check Invalid Syntax:
     AssertionError: Timed out retrying: Expected to find content: 'typescripttestTIVgt' within the element: <div#files.p-Widget.theia-Tree.theia-FileTree.theia-Files.ps.p-mod-hidden> but never did.
  AssertionError: Expected to find element: `:not(script,style):cy-contains('typescripttestTIVgt'), [type='submit'][value~='typescripttestTIVgt']`, but never found it.
      at ensureExistence (http://localhost:3100/__cypress/runner/cypress_runner.js:137673:32)
      at Object.ensureElExistence (http://localhost:3100/__cypress/runner/cypress_runner.js:137691:12)
      at ensureExistence (http://localhost:3100/__cypress/runner/cypress_runner.js:127867:23)
      at onFailFn (http://localhost:3100/__cypress/runner/cypress_runner.js:127905:9)
      at tryCatcher (http://localhost:3100/__cypress/runner/cypress_runner.js:9065:23)
      at Promise._settlePromiseFromHandler (http://localhost:3100/__cypress/runner/cypress_runner.js:7000:31)
      at Promise._settlePromise (http://localhost:3100/__cypress/runner/cypress_runner.js:7057:18)
      at Promise._settlePromise0 (http://localhost:3100/__cypress/runner/cypress_runner.js:7102:10)
      at Promise._settlePromises (http://localhost:3100/__cypress/runner/cypress_runner.js:7178:18)
      at _drainQueueStep (http://localhost:3100/__cypress/runner/cypress_runner.js:3772:12)
      at _drainQueue (http://localhost:3100/__cypress/runner/cypress_runner.js:3765:9)
      at Async.../../node_modules/bluebird/js/release/async.js.Async._drainQueues (http://localhost:3100/__cypress/runner/cypress_runner.js:3781:5)
      at Async.drainQueues (http://localhost:3100/__cypress/runner/cypress_runner.js:3651:14)

Investigating...

@che-bot
Copy link
Contributor

che-bot commented Sep 24, 2020

✅ E2E Happy path tests succeed 🎉

Try Che-Theia editor only Try Che-Theia with Java/maven example Try Che-Theia with NodeJs example

See Details

name link
che-theia docker.io/maxura/che-theia:823
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:823

Tested with Eclipse Che Single User on K8S (minikube v1.1.1)

ℹ️ Use comment "[crw-ci-test]" to rerun happy path E2E test.

@RomanNikitenko
Copy link
Member

Checked locally build of images:
cypress_tests

Jobs also look happy after restarting.

@RomanNikitenko RomanNikitenko merged commit 680a3ee into master Sep 24, 2020
@RomanNikitenko RomanNikitenko deleted the theia-upgrade branch September 24, 2020 13:54
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
* Temporarily remove some plugins where vsix files are missing

Part of eclipse-che/che#18933

Signed-off-by: Eric Williams <ericwill@redhat.com>

* Update bash-debug location instead of removing

Signed-off-by: Eric Williams <ericwill@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants