Skip to content

Get rid of non-working workaround to flakiness when Java LS server initialization failed#14945

Merged
dmytro-ndp merged 5 commits into
masterfrom
che-14944-log-workaround
Oct 22, 2019
Merged

Get rid of non-working workaround to flakiness when Java LS server initialization failed#14945
dmytro-ndp merged 5 commits into
masterfrom
che-14944-log-workaround

Conversation

@dmytro-ndp

@dmytro-ndp dmytro-ndp commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

What does this PR do?

It logs usage of workaround to failed check of LS server language initialization in E2E Happy path tests.

Update: it seems existed workaround doesn't work, and has just hidden failure details .
So, that workaround has been removed https://codeready-workspaces-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/che-pr-tests/view/K8S/job/Single-Che-PR-check-E2E-Happy-path-tests-against-k8s-on-codenvy-slave9/862/console:

        ▼ Ide.waitStatusBarContains "Starting Java Language Server"
Known flakiness https://github.com/eclipse/che/issues/14944
    1) Java LS initialization


  3 passing (2m)
  1 failing

  1) Language server validation
       Java LS initialization:
     TimeoutError: Wait timed out after 21008ms

What issues does this PR fix or reference?

#14944, #14283

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/bug Outline of a bug - must adhere to the bug report template. labels Oct 21, 2019
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@dmytro-ndp dmytro-ndp changed the title Log workaround failed Java LS server initialization Log workaround if Java LS server initialization failed Oct 21, 2019
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

2 similar comments
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 21, 2019

Copy link
Copy Markdown
Contributor

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@dmytro-ndp dmytro-ndp changed the title Log workaround if Java LS server initialization failed Get rid of non-working workaround to flakiness when Java LS server initialization failed Oct 22, 2019
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

1 similar comment
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

crw-ci-test

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E tests of Eclipse Che Multiuser on OCP has been successful:

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has failed:

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

ci-build

1 similar comment
@dmytro-ndp

Copy link
Copy Markdown
Contributor Author

ci-build

@che-bot

che-bot commented Oct 22, 2019

Copy link
Copy Markdown
Contributor

E2E tests of Eclipse Che Multiuser on OCP has failed:

@dmytro-ndp dmytro-ndp merged commit 2629f8a into master Oct 22, 2019
dmytro-ndp added a commit that referenced this pull request Oct 29, 2019
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
dmytro-ndp added a commit that referenced this pull request Oct 29, 2019
* Fix version of jar file in petclinic project of happy path tests (#14936)

* Log and fix workaround to Java LS server initialization failure (#14945)

Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
@dmytro-ndp dmytro-ndp deleted the che-14944-log-workaround branch October 31, 2019 13:18
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. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants