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

Fix JavaSpringboot build task timeout #18487

Merged

Conversation

ScrewTSW
Copy link
Member

@ScrewTSW ScrewTSW commented Nov 30, 2020

What does this PR do?

  • Adds Logger.error output if a test case fails with exact timeout value
  • Changes Terminal.waitIconSuccess logic to use polling instead of driverHelper.wait
  • Increases timeout for running JavaSpringBoot built application

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

redhat-developer/rh-che#2008

How to test this PR?

  • npm run test-all-devfiles

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Tibor Dancs <tdancs@redhat.com>
@ScrewTSW ScrewTSW added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. team/che-qe area/qe labels Nov 30, 2020
@ScrewTSW ScrewTSW added this to the 7.23 milestone Nov 30, 2020
@che-bot che-bot added kind/bug Outline of a bug - must adhere to the bug report template. kind/periodic-e2e-test labels Nov 30, 2020
@che-bot
Copy link
Contributor

che-bot commented Nov 30, 2020

✅ E2E Happy path tests succeed 🎉

See Details

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

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

Signed-off-by: Tibor Dancs <tdancs@redhat.com>
@ScrewTSW ScrewTSW force-pushed the rhche2008-feature-fix-run-task-timeout branch from 12c8101 to 6575e84 Compare November 30, 2020 13:58
@che-bot
Copy link
Contributor

che-bot commented Nov 30, 2020

✅ E2E Happy path tests succeed 🎉

See Details

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

  • Use comment "[crw-ci-test]" to rerun happy path E2E test.
  • Use comment "[crw-ci-test --rebuild]" to re-build the images and rerun happy path E2E test.

@ScrewTSW ScrewTSW merged commit 7cf011b into eclipse-che:master Dec 4, 2020
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/qe kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants