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

Unexpected hanging of the Loader after actions #8300

Closed
SkorikSergey opened this issue Jan 15, 2018 · 7 comments
Closed

Unexpected hanging of the Loader after actions #8300

SkorikSergey opened this issue Jan 15, 2018 · 7 comments
Assignees
Labels
e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. 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.

Comments

@SkorikSergey
Copy link
Contributor

SkorikSergey commented Jan 15, 2018

Description

We have unexpected fail of FileStructureNodesTest selenium test. The test sometimes fails while is waiting for the File Structure form.

Reproduction Steps:

  • Open a java class in the Editor.
  • Press Ctrl+F12 or menu Assistant -> Navigate File Structure.

Expected behavior:
The File Structure form is opened.

Observed behavior:
The File Structure form is not appeared and the test fails because the Loader is still visible(after waiting for 180 second that it is closed).

Failed selenium test: FileStructureNodesTest, PreviewUrlIntoCommandsEditorTest

Log:

org.openqa.selenium.TimeoutException: 
Expected condition failed: waiting for element to no longer be visible: By.id: gwt-debug-loader-panel (tried for 180 second(s) with 500 MILLISECONDS interval)
Build info: version: '3.0.1', revision: '1969d75', time: '2016-10-18 09:49:13 -0700'
System info: host: 'slave4.codenvycorp.com', ip: '127.0.0.1', os.name: 'Linux', os.arch: 'amd64', os.version: '3.10.0-514.16.1.el7.x86_64', java.version: '1.8.0_102'
Driver info: org.eclipse.che.selenium.core.SeleniumWebDriver
	at org.eclipse.che.selenium.miscellaneous.FileStructureNodesTest.checkFileStructureNodes(FileStructureNodesTest.java:127)

Screenshot:
org eclipse che selenium miscellaneous filestructurenodestest checkfilestructurenodes_wh971kfh

Report with reproduced problem:
https://ci.codenvycorp.com/view/qa/job/che-integration-tests-master-docker/148/Selenium_tests_report/

@SkorikSergey SkorikSergey added kind/bug Outline of a bug - must adhere to the bug report template. e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. labels Jan 15, 2018
@SkorikSergey SkorikSergey changed the title Unexpected fail of FileStructureNodesTest selenium test Unexpected the hanging of the Loader after actions Jan 24, 2018
@SkorikSergey
Copy link
Contributor Author

Similar behavior with RenameParametersTest, TestAnnotationsTest:

Log:

org.openqa.selenium.TimeoutException: 
Expected condition failed: waiting for element to no longer be visible: By.id: gwt-debug-loader-panel (tried for 180 second(s) with 500 MILLISECONDS interval)
Build info: version: '3.0.1', revision: '1969d75', time: '2016-10-18 09:49:13 -0700'
System info: host: 'slave6.codenvycorp.com', ip: '127.0.0.1', os.name: 'Linux', os.arch: 'amd64', os.version: '3.10.0-514.16.1.el7.x86_64', java.version: '1.8.0_102'
Driver info: org.eclipse.che.selenium.core.SeleniumWebDriver
	at org.eclipse.che.selenium.refactor.parameters.RenameParametersTest.checkRenameParameters0(RenameParametersTest.java:75)

https://ci.codenvycorp.com/view/qa/job/che-integration-tests-master-docker/157/Selenium_tests_report/

Screenshot:
hung-loader

@vparfonov
Copy link
Contributor

I've add some messages to the loader it will help identify problems more accurate. Not close issue yet will wait on detail information

@SkorikSergey
Copy link
Contributor Author

Screenshot after adding messages to the Loader:
org eclipse che selenium miscellaneous filestructurenodestest checkfilestructurenodes_bfurhkq5

@Ohrimenko1988
Copy link
Contributor

Attachment for PreviewUrlIntoCommandsEditorTest

Log:

org.openqa.selenium.WebDriverException: 
unknown error: Element <button type="button" class="gwt-Button GEU2T3BBGJC" id="gwt-debug-command-editor-button-run">...</button> is not clickable at point (1869, 103). Other element would receive the click: <div id="gwt-debug-glass-panel" class="GEU2T3BBCCB">...</div>
  (Session info: chrome=56.0.2924.87)
  (Driver info: chromedriver=2.27.440175 (9bc1d90b8bfa4dd181fbbf769a5eb5e575574320),platform=Linux 3.10.0-514.16.1.el7.x86_64 x86_64) (WARNING: The server did not provide any stacktrace information)
Command duration or timeout: 165 milliseconds
Build info: version: '3.1.0', revision: '86a5d70', time: '2017-02-16 07:57:44 -0800'
System info: host: '251773b88214', ip: '172.22.0.5', os.name: 'Linux', os.arch: 'amd64', os.version: '3.10.0-514.16.1.el7.x86_64', java.version: '1.8.0_121'
Driver info: org.openqa.selenium.chrome.ChromeDriver
Capabilities [{applicationCacheEnabled=false, rotatable=false, mobileEmulationEnabled=false, networkConnectionEnabled=false, chrome={chromedriverVersion=2.27.440175 (9bc1d90b8bfa4dd181fbbf769a5eb5e575574320), userDataDir=/tmp/.org.chromium.Chromium.tJaSOB}, takesHeapSnapshot=true, pageLoadStrategy=normal, databaseEnabled=false, handlesAlerts=true, hasTouchScreen=false, version=56.0.2924.87, platform=LINUX, browserConnectionEnabled=false, nativeEvents=true, acceptSslCerts=true, locationContextEnabled=true, webStorageEnabled=true, browserName=chrome, takesScreenshot=true, javascriptEnabled=true, cssSelectorsEnabled=true, unexpectedAlertBehaviour=}]
Session ID: ee26f4cfc9871c324bc78137d1e5bc93
Command duration or timeout: 830 milliseconds
Build info: version: '3.0.1', revision: '1969d75', time: '2016-10-18 09:49:13 -0700'
System info: host: 'slave2.codenvycorp.com', ip: '127.0.0.1', os.name: 'Linux', os.arch: 'amd64', os.version: '3.10.0-514.16.1.el7.x86_64', java.version: '1.8.0_102'
Driver info: org.openqa.selenium.remote.RemoteWebDriver
Capabilities [{applicationCacheEnabled=false, rotatable=false, mobileEmulationEnabled=false, networkConnectionEnabled=true, chrome={chromedriverVersion=2.27.440175 (9bc1d90b8bfa4dd181fbbf769a5eb5e575574320), userDataDir=/tmp/.org.chromium.Chromium.tJaSOB}, takesHeapSnapshot=true, pageLoadStrategy=normal, databaseEnabled=false, handlesAlerts=true, hasTouchScreen=true, version=56.0.2924.87, platform=LINUX, browserConnectionEnabled=false, nativeEvents=true, acceptSslCerts=true, webdriver.remote.sessionid=9ca65426-648a-4895-9fa1-f2a2157b2463, locationContextEnabled=true, webStorageEnabled=true, browserName=chrome, takesScreenshot=true, javascriptEnabled=true, cssSelectorsEnabled=true, unexpectedAlertBehaviour=}]
Session ID: 9ca65426-648a-4895-9fa1-f2a2157b2463
	at org.eclipse.che.selenium.intelligencecommand.PreviewUrlIntoCommandsEditorTest.checkSavePreviewUrlIntoCommand(PreviewUrlIntoCommandsEditorTest.java:89)

Screenshot:
hfghghjg

hkolvenbach pushed a commit to hkolvenbach/che that referenced this issue Mar 2, 2018
…migration to master

* commit '47fe46a34fa1df988ed61d46e5a478e0f67675b7': (156 commits)
  Fixed build errors
  eclipse-che#6331 disable key binding handle when any 'Window' opened (eclipse-che#8473)
  changed info about known issue (eclipse-che#8483)
  Avoid removal of non-test workspaces at the cleanup environment stage of selenium testing (eclipse-che#8481)
  Add changes in the methods of checking git-marker in the page object 'CodenvyEditor'
  updated the command
  Use nightly stack by default and -Pdev for development
  Changes pvc name to a prefix + 8 random characters
  init and cli images should not use tag from CHE (eclipse-che#8474)
  CHE-7581 Improved keycloak initialization (eclipse-che#8425)
  CHE-8240: Switch to project explorer tab for reavel resources action. (eclipse-che#8460)
  Selenium: Create selenium tests to cover the authorization on GitHub (eclipse-che#8395)
  apply formating for 'CreateNewPackagesWithHelpCreationJavaClassTest' selenium test (eclipse-che#8464)
  remove read 'preferences' in try/catch block (eclipse-che#8463)
  add project build to CheckAutocompleteFeaturesInTheTestFolderTest (eclipse-che#8457)
  Selenium: Add try/catch block for displaying "Known issue" with eclipse-che#8300 bug (eclipse-che#8454)
  add info about known issue to FileStructureNodesTest selenium test (eclipse-che#8452)
  Remove "source" field from stack object (eclipse-che#8407)
  Selenium: Fix unexpected fail of MacrosCommandsEditorTest selenium test (eclipse-che#8441)
  Fix formatting (eclipse-che#8447)
  ...
@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Mar 6, 2018

It is reliably reproduced when CPU is overloaded, for example, when the number of test threads >= number of CPU cores.
For the qa-slaves it is --threads=4 and more: https://ci.codenvycorp.com/view/qa-experimental/job/experimental-che-integration-tests-master-docker/88/Selenium_tests_report/

Report from here:

  1. Webdriver log: org.eclipse.che.selenium.refactor.types.TestAnnotationsTest.testAnnotation1_kuf2.log.
  2. Ws agent logs:

According to the logs there was a problem with processing request:

POST http://172.30.3.252:43078/api/java/refactoring/rename/create

@SkorikSergey SkorikSergey changed the title Unexpected the hanging of the Loader after actions Unexpected hanging of the Loader after actions Apr 13, 2018
@SkorikSergey SkorikSergey added the severity/P2 Has a minor but important impact to the usage or development of the system. label May 7, 2018
@tsmaeder
Copy link
Contributor

The try/catch blocks for FileStructureNodesTest, at least are too broad:

   try {
      fileStructure.waitExpectedTextInFileStructure(ITEMS_CLASS);
    } catch (WebDriverException ex) {
      // remove try-catch block after issue has been resolved
      fail("Known issue https://github.com/eclipse/che/issues/8300");
    }

I have at least one case where the problem is not a hung loader, but the expected text is wrong (looking at the screenshot). If we need to preserve context, we should pass it into the FileStructure.wait...() methods.

@vparfonov vparfonov added severity/P1 Has a major impact to usage or development of the system. and removed severity/P2 Has a minor but important impact to the usage or development of the system. labels Oct 17, 2018
@vinokurig vinokurig assigned vinokurig and unassigned vinokurig Nov 13, 2018
@vzhukovs vzhukovs self-assigned this Nov 14, 2018
@vzhukovs vzhukovs added the status/in-progress This issue has been taken by an engineer and is under active development. label Nov 14, 2018
@vzhukovs vzhukovs removed the status/in-progress This issue has been taken by an engineer and is under active development. label Nov 19, 2018
@vzhukovs
Copy link
Contributor

vzhukovs commented Nov 19, 2018

Investigation has shown that the issue can't be reproduced neither on jenkins nor local machine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. 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

7 participants