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

Pin Che Theia to upstream Theia before the file watchers related changes as it breaks Che#886

Merged
azatsarynnyy merged 2 commits intomasterfrom
watchers-fix
Oct 21, 2020
Merged

Pin Che Theia to upstream Theia before the file watchers related changes as it breaks Che#886
azatsarynnyy merged 2 commits intomasterfrom
watchers-fix

Conversation

@azatsarynnyy
Copy link
Member

@azatsarynnyy azatsarynnyy commented Oct 21, 2020

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

What does this PR do?

Pins Che Theia to upstream Theia before the file watchers related changes (eclipse-theia/theia#8546) as it breaks Che.
See eclipse-che/che#18151 for the details.

What issues does this PR fix or reference?

Release Notes

Docs PR

Happy Path Channel

HAPPY_PATH_CHANNEL=stable

…ges as it breaks Che

Signed-off-by: Artem Zatsarynnyi <azatsary@redhat.com>
@azatsarynnyy azatsarynnyy requested a review from benoitf as a code owner October 21, 2020 09:53
@che-bot
Copy link
Contributor

che-bot commented Oct 21, 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:886
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:886

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

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

Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Oct 21, 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:886
che-theia-endpoint-runtime-binary docker.io/maxura/che-theia-endpoint-runtime-binary:886

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

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

@paul-marechal
Copy link

This PR title scared me. The watcher changes are not the cause right?

@benoitf
Copy link
Contributor

benoitf commented Oct 23, 2020

@marechal-p no :-)

@benoitf
Copy link
Contributor

benoitf commented Oct 23, 2020

it was activation plug-in changes

@azatsarynnyy
Copy link
Member Author

This PR title scared me. The watcher changes are not the cause right?

sorry to scare you @marechal-p 😄
a bit later, we've figured out that the watchers changes have nothing to do with it

vinokurig pushed a commit that referenced this pull request Apr 6, 2021
… don't get sources twice; add missing --use-tmp-dir flag (#886)

Change-Id: I54a6a569ca964734296fd60b4db4c96fb080bc8c
Signed-off-by: nickboldt <nboldt@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

Development

Successfully merging this pull request may close these issues.

5 participants