Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-8533

Timeout collision in Client Nodes test suite

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.7
    • None

    Attachments

      Issue Links

        Activity

          githubbot ASF GitHub Bot added a comment -

          GitHub user andrewmed opened a pull request:

          https://github.com/apache/ignite/pull/4030

          IGNITE-8533 Fix timeout collision in Client Nodes test suite introduc…

          …ed by IGNITE-8085

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/andrewmed/ignite ignite-8533

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/ignite/pull/4030.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #4030


          commit 212a9fc4d525203e8912775730b61193fde25b9a
          Author: AMedvedev <amedvedev@...>
          Date: 2018-05-20T22:07:47Z

          IGNITE-8533 Fix timeout collision in Client Nodes test suite introduced by IGNITE-8085


          githubbot ASF GitHub Bot added a comment - GitHub user andrewmed opened a pull request: https://github.com/apache/ignite/pull/4030 IGNITE-8533 Fix timeout collision in Client Nodes test suite introduc… …ed by IGNITE-8085 You can merge this pull request into a Git repository by running: $ git pull https://github.com/andrewmed/ignite ignite-8533 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/ignite/pull/4030.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #4030 commit 212a9fc4d525203e8912775730b61193fde25b9a Author: AMedvedev <amedvedev@...> Date: 2018-05-20T22:07:47Z IGNITE-8533 Fix timeout collision in Client Nodes test suite introduced by IGNITE-8085
          andmed Andrew Medvedev added a comment - PR https://github.com/apache/ignite/pull/4030

          TC: https://ci.ignite.apache.org/viewLog.html?buildId=1315393&

          I can see one flaky test again there, remote node could not start: https://ci.ignite.apache.org/viewLog.html?buildId=1315393&buildTypeId=IgniteTests24Java8_StartNodes&tab=buildResultsDiv#testNameId6814497542781613621

          not sure if this means that any longer timeout  would help (now we have 30sec)

           

          Here is clean run on TC

          https://ci.ignite.apache.org/viewQueued.html?itemId=1315677:0:153

          andmed Andrew Medvedev added a comment - TC:  https://ci.ignite.apache.org/viewLog.html?buildId=1315393& I can see one flaky test again there, remote node could not start:  https://ci.ignite.apache.org/viewLog.html?buildId=1315393&buildTypeId=IgniteTests24Java8_StartNodes&tab=buildResultsDiv#testNameId6814497542781613621 not sure if this means that any longer timeout  would help (now we have 30sec)   Here is clean run on TC https://ci.ignite.apache.org/viewQueued.html?itemId=1315677:0:153
          dpavlov Dmitry Pavlov added a comment -

          Hi andmed, patch is looking good for me, and I agree to merge.

          In the same time I hope ivanan.fed will share his opinion on patch and future of IGNITE-8085

          dpavlov Dmitry Pavlov added a comment - Hi andmed , patch is looking good for me, and I agree to merge. In the same time I hope ivanan.fed will share his opinion on patch and future of IGNITE-8085
          dpavlov Dmitry Pavlov added a comment -

          Merged to master, andmed thank you for noticing that issue and providing fix.

          Let's discuss IGNITE-8085 future in that ticket comments.

          dpavlov Dmitry Pavlov added a comment - Merged to master, andmed thank you for noticing that issue and providing fix. Let's discuss IGNITE-8085 future in that ticket comments.
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/ignite/pull/4030

          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/ignite/pull/4030

          People

            andmed Andrew Medvedev
            andmed Andrew Medvedev
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: