+1 (binding) Ran through the steps in the release helper guide. Noticed a few release helper instructions that can be improved for next time (no mention of bootstrap.sh), but was able to figure it out by using the README file included in the source release.
Hashes & sig all looked good. Was able to build, including the docker image. Tests all passed. Tested integration with a NiFi cluster with the 0.7.0 minifi sending flow files over s2s and everything worked as expected. Overall, seems to be a very solid, polished release. Nice work all! On Fri, Jan 10, 2020 at 3:35 PM Joe Witt <joe.w...@gmail.com> wrote: > > +1 (binding) > > Comments > - Notice file copyright year needs to be 2020 > - I did not enable python support but I get the minifi-python dir anyway > and log entry on startup such as > Caught Exception ModuleNotFoundError: No module named 'google' > > Great progress with this release! Nice work. Building and running tests > was easier than I ever remember. > > Thanks > > On Fri, Jan 10, 2020 at 8:27 AM Dániel Bakai <bakaid.apa...@gmail.com> > wrote: > > > +1, non-binding > > > > Verifications performed: > > > > SHA256 checksums downloaded: OK > > SHA512 checksum of nifi-minifi-cpp-0.7.0-source.tar.gz sent in mail: OK > > GPG signatures downloaded: OK > > Git tag in email matches sources in nifi-minifi-cpp-0.7.0-source.tar.gz: OK > > > > bootstrap.sh with default options && make && make package && sudo make test > > ARGS="-j4 --output-on-failure" && run package with a GenerateFlowFile -> > > LogAttribute flow: > > macOS 10.14.6: TailFileTests sometimes fail, otherwise OK > > ubuntu:bionic docker container: had to install 'tzdata' package for > > ExpressionLanguageTest to work, CTailFileChunkTests and > > CTailFileDelimitedTests consistently fail when ran in parallel, running > > them sequentially works, otherwise OK > > centos:7 docker container: had to install 'zlib-devel' and 'which', > > otherwise OK > > debian:stretch docker container: OK > > fedora:29 docker container: had to install 'zlib-devel' and 'which', > > otherwise OK > > ubuntu:xenial docker container: had to install 'tzdata' package for > > ExpressionLanguageTest to work, otherwise OK > > > > Run appropriate convenience binary with a GenerateFlowFile -> LogAttribute > > flow: > > macOS 10.14.6: OK > > ubuntu:bionic docker container: OK > > centos:7 docker container: OK > > debian:stretch docker container: OK > > fedora:29 docker container: OK > > ubuntu:xenial docker container: OK > > General note: there is a dynamic libpython dependency in most *nix > > convenience binaries, requiring manual installation from the native repos, > > in the case of CentOS 7 from EPEL. > > > > win_build_vs.bat build /64 /K /P && ctest -C Release && run package with a > > GenerateFlowFile -> LogAttribute flow: > > Windows 10: TailFileTests failed, but this is, as far as I know, a known > > issue, otherwise OK > > > > Install convenience msi on a completely clean Windows VM and run with a > > GenerateFlowFile -> LogAttribute flow both as a service and from cmd.exe: > > Windows 7: OK > > Windows 10: OK > > > > > > I think this release brings great improvements over 0.6.0. > > > > On Fri, 10 Jan 2020 at 02:32, Aldrin Piri <aldrinp...@gmail.com> wrote: > > > > > +1, binding > > > > > > Comments: > > > * signature and hashes looked good > > > * license and notification looked correct and comprehensive > > > * performed build and test successfully on Ubuntu 19.04, CentOS 7, and > > > MacOS 10.14 > > > * ran a variety of flows and achieved expected results > > > * we should update the helper to have folks make use of the bootstrap or > > > enumerate the needed dependencies. the prescribed approach worked for me > > > as all systems had the requisite tools and dependencies, but others may > > > experience trying to follow along. > > > > > > Thanks for acting as RM on this release, Arpad. Lots of great work since > > > the last release. Kudos to all the contributors. > > > > > > On Wed, Jan 8, 2020 at 11:56 AM Arpad Boda <ab...@apache.org> wrote: > > > > > > > Hello Apache NiFi community, > > > > > > > > I am pleased to call this vote for the source release of Apache NiFi > > > MiNiFi > > > > C++ 0.7.0 > > > > > > > > The source tar.gz, including signatures, digests, and convenience > > > binaries. > > > > can be found at: > > > > https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.7.0/ > > > > > > > > The Git tag is minifi-cpp-0.7.0-RC1 > > > > The Git commit ID is: aa42957a2e227df41510047cece3cd606dc1cb6a > > > > > > > > Checksum of nifi-minifi-cpp-0.7.0-source.tar.gz: > > > > SHA256: > > 2a3743e809652b429f3780e616373977fb1884330adabc1430449551b6e247bc > > > > SHA512: > > > > > > > > > cfa86cb3e34e1e2dc439ade2c0c389c70679a6403ebcc2befb6781995179cac3f4a8ec0e107204bcdd95f2cc6b5be67b2a7fcfe68eb87f561f209074b45fd507 > > > > > > > > Release artifacts are signed with the following key: > > > > https://people.apache.org/keys/committer/aboda.asc > > > > > > > > KEYS file available here: > > > > https://dist.apache.org/repos/dist/release/nifi/KEYS > > > > > > > > 144 issues were closed/resolved for this release: > > > > > > > > > > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12344866&projectId=12321520 > > > > > > > > Release note highlights can be found here: > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/MINIFI/Release+Notes#ReleaseNotes-Versioncpp-0.7.0 > > > > > > > > The vote will be open for 72 hours. > > > > Please download the release candidate and evaluate the necessary items > > > > including checking hashes, signatures, build > > > > from source, and test. Then please vote: > > > > > > > > [ ] +1 Release this package as nifi-minifi-cpp-0.7.0 > > > > [ ] +0 no opinion > > > > [ ] -1 Do not release this package because... > > > > > > > > >
