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

fix(plugin-remote): upstream theia is now requiring EnvVariables #682

Merged
merged 1 commit into from
Mar 20, 2020

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Mar 20, 2020

What does this PR do?

upstream theia introduce a new required dependency in plugin-ext to EnvVariablesServer

What issues does this PR fix or reference?

Fix eclipse-che/che#16412

Change-Id: Ia6f06ec4591b78e08263330b8c47457d04e3c2ab
Signed-off-by: Florent Benoit fbenoit@redhat.com

@azatsarynnyy
Copy link
Member

The same nsfw: Command failed on CI as it was yesterday with #676
Interesting that it appears occasionally

@azatsarynnyy
Copy link
Member

It's weird but it didn't fix
No matching bindings found for serviceIdentifier: Symbol(EnvVariablesServer)

Change-Id: Ia6f06ec4591b78e08263330b8c47457d04e3c2ab
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Mar 20, 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:682
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:682

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

@che-bot
Copy link
Contributor

che-bot commented Mar 20, 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:682
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:682

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

@azatsarynnyy azatsarynnyy merged commit 985d5df into eclipse-che:master Mar 20, 2020
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
Signed-off-by: Esteban Mañaricua <emanaricua@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Che Theia remote plugins aren't started
3 participants