+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...
> >
>

Reply via email to