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

Align che-theia with upstream changes #20366

Closed
vitaliy-guliy opened this issue Aug 26, 2021 · 8 comments
Closed

Align che-theia with upstream changes #20366

vitaliy-guliy opened this issue Aug 26, 2021 · 8 comments
Assignees
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

Comments

@vitaliy-guliy
Copy link
Contributor

Describe the bug

It seems the commit to the upstream broke che-theia.

che-theia image can be built successfully (no any issues, as tests are turned off), but the container fails with the following output

TypeError: Class constructor MessagingContribution cannot be invoked without 'new'
    at new CheMessagingContribution (/home/theia/node_modules/@eclipse-che/theia-messaging/lib/node/messaging/che-messaging-contribution.js:78:47)
    at _createInstance (/home/theia/node_modules/inversify/lib/resolution/instantiation.js:27:12)
    at Object.resolveInstance (/home/theia/node_modules/inversify/lib/resolution/instantiation.js:47:18)
    at /home/theia/node_modules/inversify/lib/resolution/resolver.js:75:42
    at Object.resolve (/home/theia/node_modules/inversify/lib/resolution/resolver.js:99:12)
    at /home/theia/node_modules/inversify/lib/container/container.js:335:37
    at Container._get (/home/theia/node_modules/inversify/lib/container/container.js:326:44)
    at Container.get (/home/theia/node_modules/inversify/lib/container/container.js:243:21)
    at Binding.dynamicValue (/home/theia/node_modules/inversify/lib/syntax/binding_to_syntax.js:76:75)
    at /home/theia/node_modules/inversify/lib/resolution/resolver.js:66:118
Theia app listening on http://127.0.0.1:3100.

Che version

next (development version)

Steps to reproduce

a) Since build of eclipse/che-theia:next is broken (the image must be published on each change), it needs to build the image by hands.

  1. clone che-theia, build the image
./build.sh --build-args:GITHUB_TOKEN=${GITHUB_TOKEN} --skip-tests --dockerfile:Dockerfile.alpine
  1. then use this image when creating a workspace

b) Create a workspace using the devfile

apiVersion: 1.0.0
metadata:
  name: che-theia-next
projects:
  - name: web-java-spring-petclinic
    source:
      location: 'https://github.com/che-samples/java-spring-petclinic'
      type: git
components:
  - type: cheEditor
    alias: che-theia
    reference: https://gist.githubusercontent.com/vitaliy-guliy/2bb6318a84a2911e334cba6d7dfbf423/raw/b52df74920ace02d0c11d35c01bc621eadb5d3bd/editor.meta.yaml

Expected behavior

Working editor.

Runtime

OpenShift, minikube

Screenshots

Screenshot from 2021-08-26 17-51-49

Screenshot from 2021-08-26 17-51-55

Installation method

chectl/next

Environment

Linux, Dev Sandbox (workspaces.openshift.com)

Eclipse Che Logs

No response

Additional context

To build che-theia use eclipse-che/che-theia/main and eclipse-theia/theia/master

@vitaliy-guliy vitaliy-guliy added kind/bug Outline of a bug - must adhere to the bug report template. severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. labels Aug 26, 2021
@ibuziuk ibuziuk added the area/editor/theia Issues related to the che-theia IDE of Che label Aug 30, 2021
@azatsarynnyy azatsarynnyy added this to the 7.36 milestone Aug 30, 2021
@azatsarynnyy azatsarynnyy self-assigned this Aug 30, 2021
@RomanNikitenko
Copy link
Member

I detected one more breaking change:

activation of extensions fails (Language Support for Java(TM) by Red Hat and Debugger for Java):

extensions_activation

@azatsarynnyy
Copy link
Member

Probably, it's related #20360

@RomanNikitenko
Copy link
Member

RomanNikitenko commented Aug 31, 2021

Probably, it's related #20360

I guess it's related to https://github.com/eclipse-theia/theia/pull/9660
I pinned che-theia before those changes within my PR eclipse-che/che-theia#1205 and it works well.

@azatsarynnyy
Copy link
Member

Cool! Now, we can stabilize Che-Theia build and continue the investigation. Thanks @RomanNikitenko!

@azatsarynnyy
Copy link
Member

azatsarynnyy commented Sep 2, 2021

I'm lowering the severity from blocker to P1, as we've pinned Che-Theia to a specific upstream Theia commit SHA.
Currently, Che-Theia build is stabilized and we continue working on adapting to the upstream breaking changes.

@azatsarynnyy azatsarynnyy added severity/P1 Has a major impact to usage or development of the system. sprint/next and removed severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. labels Sep 2, 2021
@tsmaeder
Copy link
Contributor

tsmaeder commented Sep 7, 2021

Looking at the compiled javascript of the stack trace given in the description makes me think it is a problem with compiling to ES2017.

@azatsarynnyy
Copy link
Member

Looking at the compiled javascript of the stack trace given in the description makes me think it is a problem with compiling to ES2017.

@tsmaeder your guesses are correct. See the more details in the draft PR eclipse-che/che-theia#1213 (comment)

@RomanNikitenko RomanNikitenko added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 24, 2021
@azatsarynnyy
Copy link
Member

It should be already fixed by eclipse-che/che-theia#1213

@azatsarynnyy azatsarynnyy modified the milestones: 7.37, 7.38 Sep 29, 2021
@azatsarynnyy azatsarynnyy removed status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. sprint/current labels Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

No branches or pull requests

5 participants