You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/10/26 21:50:21 UTC

[GitHub] [geode] nonbinaryprogrammer opened a new pull request #7048: GEODE-9692: enable HashesAndCrashesDUnitTest

nonbinaryprogrammer opened a new pull request #7048:
URL: https://github.com/apache/geode/pull/7048


   When I previously checked which Ignored tests pass, I found that this one did not. Since then someone must have fixed whatever issue was causing the failures because now they all pass. I simply removed the @Ignore.
   
   <!-- Thank you for submitting a contribution to Apache Geode. -->
   
   <!-- In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken: 
   -->
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
   
   - [ ] Has your PR been rebased against the latest commit within the target branch (typically `develop`)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   - [ ] Does `gradlew build` run cleanly?
   
   - [ ] Have you written or updated unit tests to verify your changes?
   
   - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   <!-- Note:
   Please ensure that once the PR is submitted, check Concourse for build issues and
   submit an update to your PR as soon as possible. If you need help, please send an
   email to dev@geode.apache.org.
   -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] nonbinaryprogrammer commented on pull request #7048: GEODE-9692: enable HashesAndCrashesDUnitTest

Posted by GitBox <gi...@apache.org>.
nonbinaryprogrammer commented on pull request #7048:
URL: https://github.com/apache/geode/pull/7048#issuecomment-952406534


   stress new test failures are due to connection issues, which are an existing issue not within the scope of this PR


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] dschneider-pivotal closed pull request #7048: GEODE-9692: enable HashesAndCrashesDUnitTest

Posted by GitBox <gi...@apache.org>.
dschneider-pivotal closed pull request #7048:
URL: https://github.com/apache/geode/pull/7048


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] DonalEvans commented on pull request #7048: GEODE-9692: enable HashesAndCrashesDUnitTest

Posted by GitBox <gi...@apache.org>.
DonalEvans commented on pull request #7048:
URL: https://github.com/apache/geode/pull/7048#issuecomment-952927952


   The stress-new-test-openjdk8 failure is due to a hang, as the job took over 10 hours to run instead of the usual 40 minutes. Could this job be reran to make sure this isn't a repeat occurrence?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] dschneider-pivotal commented on pull request #7048: GEODE-9692: enable HashesAndCrashesDUnitTest

Posted by GitBox <gi...@apache.org>.
dschneider-pivotal commented on pull request #7048:
URL: https://github.com/apache/geode/pull/7048#issuecomment-953036801


   Since the only test you changed was HashesAndCrashesDUnitTest it will be the only one run by stressNewTest. It seems like this test can't handle being run by stressNewTest. Maybe it leaves some stuff around between repeats?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org