+1 Ran tests from source zip Installed and ran ExclamationTopology on single-node cluster from binary zip Verified no unexpected error logs. Clicked around a bit in UI, verified that logviewer works. Checked out tag and ran the license check plugin to generate DEPENDENCY-LICENSES. Verified that there are no category X licenses. A number of dependency versions have changed, but I don't think this should block the release. I'll see if we can automate generating the license files a bit more, so we don't forget to update them.
Derek, I'm not sure about a missing stormconf.ser, but there are a couple of open issues impacting tests stability. See https://issues.apache.org/jira/browse/STORM-3376 and https://issues.apache.org/jira/browse/STORM-3379. Maybe one of them fixes it? I've run the integration test and localcluster tests a bunch of times with these fixes, and the tests looks stable to me. If you get the errors occasionally, it would be nice if you could see if they are still there when applying the two fixes. Den tir. 30. apr. 2019 kl. 20.16 skrev Derek Dagit <[email protected] >: > OK ran the distribution from the tarball, and built and ran from the src > tarball on macOS with JDK8. Tested WordCountTopology on both and it seemed > fine. > > +1 > > On Tue, Apr 30, 2019 at 11:55 AM Derek Dagit <[email protected]> wrote: > > > Sent too soon: > > > > - I verified sigs and sums > > - Built the source from the tarball on macOS > > - I saw a couple of test errors because the local stormconf.ser was > gone > > when the supervisor tried to open it, and this resulted in a crash. I > > re-ran the tests and they passed. > > - I do not think this is a blocker, but it is probable worth looking > > into. > > > > I'll continue to evaluate the RC, but wanted to send this out in case > > anyone else has seen something similar. > > > > On Tue, Apr 30, 2019 at 11:52 AM Derek Dagit <[email protected]> wrote: > > > >> - I verified sigs and sums > >> > >> On Mon, Apr 29, 2019 at 5:49 PM P. Taylor Goetz <[email protected]> > >> wrote: > >> > >>> This is a call to vote on releasing Apache Storm 2.0.0 (rc7) > >>> > >>> Full list of changes in this release: > >>> > >>> > >>> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/RELEASE_NOTES.html > >>> > >>> The tag/commit to be voted upon is v2.0.0: > >>> > >>> > >>> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=007863edd95e838b3df414928c6fa3f28244ab49;hb=2ba95bbd1c911d4fc6363b1c4b9c4c6d86ac9aae > >>> > >>> The source archive being voted upon can be found here: > >>> > >>> > >>> > https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/apache-storm-2.0.0-src.tar.gz > >>> > >>> Other release files, signatures and digests can be found here: > >>> > >>> https://dist.apache.org/repos/dist/dev/storm/apache-storm-2.0.0-rc7/ > >>> > >>> The release artifacts are signed with the following key: > >>> > >>> > >>> > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd > >>> > >>> The Nexus staging repository for this release is: > >>> > >>> https://repository.apache.org/content/repositories/orgapachestorm-1079 > >>> > >>> Please vote on releasing this package as Apache Storm 2.0.0. > >>> > >>> When voting, please list the actions taken to verify the release. > >>> > >>> This vote will be open for at least 72 hours. > >>> > >>> [ ] +1 Release this package as Apache Storm 2.0.0 > >>> [ ] 0 No opinion > >>> [ ] -1 Do not release this package because... > >>> > >>> Thanks to everyone who contributed to this release. > >>> > >>> -Taylor > >> > >> >
