You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Anthony Baker <ab...@pivotal.io> on 2016/03/23 17:56:13 UTC
GEODE-857 (was Re: [VOTE] RC2: Apache Geode (Incubating) first Milestone release - v1.0.0-incubating.M1)
I think we’ve finally found and fixed this issue. Thanks again for the report Brian!
Anthony
> On Feb 1, 2016, at 8:38 AM, Anthony Baker <ab...@pivotal.io> wrote:
>
> Sounds good, and thanks for the feedback. It’s really helpful to get these reports of test failures from a broader range of systems than ASF Jenkins.
>
> Anthony
>
>
>> On Jan 31, 2016, at 9:36 PM, Brian Dunlap <bd...@gmail.com> wrote:
>>
>> Thanks Anthony -
>>
>> To potentially help with tricky sleuthing...
>> I ran into this test exception on develop too.
>>
>> :gemfire-core:test
>>
>> com.gemstone.gemfire.SystemFailureJUnitTest > testStopThreads FAILED
>> java.lang.AssertionError
>> at org.junit.Assert.fail(Assert.java:86)
>> at org.junit.Assert.assertTrue(Assert.java:41)
>> at org.junit.Assert.assertTrue(Assert.java:52)
>> at
>> com.gemstone.gemfire.SystemFailureJUnitTest.testStopThreads(SystemFailureJUnitTest.java:51)
>>
>> 1974 tests completed, 1 failed, 34 skipped
>> :gemfire-core:test FAILED
>>
>> I'm all for getting additional early feedback with a M1 release - just
>> wanting to share my first build experience.
>> :)
>>
>> Once this tricky test issue is worked out...
>> For new Geode developers, it would be great to have the online docs,
>> README.md, and COMPILING.txt suggest running the same thing - either with
>> or without the unit tests.
>>
>>
>> Pro-Getting-Started-With-Geode!
>> Brian -
>>
>> On Sun, Jan 31, 2016 at 9:41 PM, Anthony Baker <ab...@pivotal.io> wrote:
>>
>>> Originally the -Dskip.tests=true flag was helpful if you didn’t want to
>>> run 12h worth of tests :-)
>>>
>>> Since then we’ve updated the build target to only run unit tests, so it
>>> should in under 10min depending on your hardware. You can run precheckin
>>> to get full test coverage:
>>>
>>> Anthony-Baker-MacBook-Pro:incubator-geode abaker$ gradle tasks | grep pre
>>> precheckin - Run this task before checking in code to validate changes.
>>> This task combines the following tasks: build, integrationTest, and
>>> distributedTest
>>>
>>> We do expect to get reliable test results for `gradle build`. I wasn’t
>>> able to reproduce the failure you saw with either idk 1.8.0_45-b14 or
>>> 1.8.0_71-b15. You might have encountered GEODE-824 (an intermittent
>>> failure) which has been fixed on develop but not on the release branch,
>>>
>>> Anthony
>>>
>>>
>>>> On Jan 31, 2016, at 6:14 PM, Brian Dunlap <bd...@gmail.com> wrote:
>>>>
>>>> Thanks Udo!
>>>>
>>>> If possible, I'd like to add -Dskip.tests=true onto the README.md so that
>>>> build instructions for newbies are consistent with:
>>>>
>>> http://geode.docs.pivotal.io/docs/getting_started/installation/install_standalone.html
>>>>
>>>> I logged into Geode's Jira, but I don't have permissions to create an
>>> issue.
>>>> If possible, would someone mind creating an issue for me to do the pull
>>>> request against?
>>>>
>>>> This is an easy softball - I'd like to go through Geode's pull request
>>>> process:
>>>> https://cwiki.apache.org/confluence/display/GEODE/Code+contributions
>>>>
>>>> Anthony, I saw the flip to gradle vs. gradlew - I'll be sure to keep
>>> gradle
>>>> in the README.md.
>>>>
>>>> Thanks,
>>>> Brian -
>>>>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> On Sun, Jan 31, 2016 at 5:36 PM, Udo Kohlmeyer <uk...@pivotal.io>
>>>> wrote:
>>>>
>>>>> Hi there Brian,
>>>>>
>>>>> Have you tried building with ./gradlew build installDist
>>> -Dskip.tests=true
>>>>> This will build geode without running tests.
>>>>>
>>>>> --Udo
>>>>>
>>>>>
>>>>> On 1/02/2016 10:14 am, Brian Dunlap wrote:
>>>>>
>>>>>> Hi everyone -
>>>>>>
>>>>>> I'm new to the Geode community - not sure if this impacts the M1 RC2
>>>>>> decision.
>>>>>>
>>>>>> Today, while trying a clean build against rel/v1.0.0-incubating.M1.RC2,
>>>>>> I'm
>>>>>> getting this failure:
>>>>>>
>>>>>> :gemfire-core:test
>>>>>>
>>>>>>
>>>>>>
>>> com.gemstone.gemfire.distributed.internal.membership.gms.messenger.JGroupsMessengerJUnitTest
>>>>>>
>>>>>>> testMulticastTest FAILED
>>>>>>>
>>>>>> java.lang.AssertionError
>>>>>> at org.junit.Assert.fail(Assert.java:86)
>>>>>> at org.junit.Assert.assertTrue(Assert.java:41)
>>>>>> at org.junit.Assert.assertFalse(Assert.java:64)
>>>>>> at org.junit.Assert.assertFalse(Assert.java:74)
>>>>>> at
>>>>>>
>>>>>>
>>> com.gemstone.gemfire.distributed.internal.membership.gms.messenger.JGroupsMessengerJUnitTest.testMulticastTest(JGroupsMessengerJUnitTest.java:818)
>>>>>>
>>>>>> 1850 tests completed, 1 failed, 35 skipped
>>>>>>
>>>>>> I was following the build instructions from github and running these
>>>>>> targets:
>>>>>> gradle build installDist
>>>>>>
>>>>>> Since the build fails, no worky-worky on kicking the tires.
>>>>>>
>>>>>> Any suggestions?
>>>>>>
>>>>>> I'm on the latest Java 8 JDK (71) and running on El Capitan (10.11.3)
>>>>>>
>>>>>>
>>>>>> Thanks!
>>>>>> Brian -
>>>>>>
>>>>>>
>>>>>> On Sun, Jan 31, 2016 at 10:44 AM, Niall Pemberton <
>>>>>> niall.pemberton@gmail.com
>>>>>>
>>>>>>> wrote:
>>>>>>> +1 from me for this release.
>>>>>>>
>>>>>>> * I checked the DISCLAIMER, LICENSE & NOTICE files
>>>>>>> * I check the sigs & checksums
>>>>>>> * I ran a build on the source distro
>>>>>>>
>>>>>>> Niall
>>>>>>>
>>>>>>> On Wed, Jan 27, 2016 at 8:21 PM, Nitin Lamba <ni...@ampool.io> wrote:
>>>>>>>
>>>>>>> All,
>>>>>>>>
>>>>>>>> This is the second release candidate of the first release for Apache
>>>>>>>> Geode, version 1.0.0-incubating.M1. Thanks to all the community
>>> members
>>>>>>>>
>>>>>>> to
>>>>>>>
>>>>>>>> drive towards this milestone!
>>>>>>>>
>>>>>>>> It fixes the following issues:
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12318420&version=12334248
>>>>>>>
>>>>>>>> Compared to first release candidate (RC1), it fixes the following
>>>>>>>> issues:
>>>>>>>> [GEODE-610] - Review Geode's LICENSE & NOTICE File(s)
>>>>>>>> [GEODE-769] - improve README.md file
>>>>>>>> [GEODE-815] - RC Feedback: Fix LICENSE and NOTICE files
>>>>>>>>
>>>>>>>> *** Please download, test and vote by Monday, February 01, 0800 hrs
>>> US
>>>>>>>> Pacific.
>>>>>>>>
>>>>>>>> Note that we are voting upon the source (tag):
>>>>>>>> rel/1.0.0-incubating.M1.RC2
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;a=tag;h=refs/tags/rel/v1.0.0-incubating.M1.RC2
>>>>>>>
>>>>>>>> <
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;a=tag;h=refs/tags/rel/v1.0.0-incubating.M1.RC1
>>>>>>>
>>>>>>>> Commit ID: c26334542fa9e90daff5f58467d6e174b3e0f3e2
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://git-wip-us.apache.org/repos/asf?p=incubator-geode.git;a=commit;h=c26334542fa9e90daff5f58467d6e174b3e0f3e2
>>>>>>>
>>>>>>>> Source and binary files:
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://dist.apache.org/repos/dist/dev/incubator/geode/1.0.0-incubating.M1.RC2/
>>>>>>>
>>>>>>>> For the first release, the documentation on how to install and use
>>>>>>>> Apache
>>>>>>>> Geode are hosted on pivotal.io:
>>>>>>>> http://geode.docs.pivotal.io<http://geode.docs.pivotal.io/>
>>>>>>>>
>>>>>>>> Maven staging repo:
>>>>>>>>
>>>>>>>>
>>> https://repository.apache.org/content/repositories/orgapachegeode-1001/
>>>>>>>
>>>>>>>> Geode's KEYS file containing PGP keys we use to sign the release:
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>> https://github.com/apache/incubator-geode/blob/release/1.0.0-incubating.M1/KEYS
>>>>>>>
>>>>>>>> Release Key: pub 4096R/C72CFB64 2015-10-01
>>>>>>>> Fingerprint: 948E 8234 14BE 693A 7F74 ABBE 19DB CAEE C72C FB64
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Anthony & Nitin (on behalf of the Geode team)
>>>>>>>>
>>>>>>>>
>>>>>
>>>
>>>
>