Ok, I'm changing my vote to +1!

I don't know the root cause is, but I changed the "latch.await" timeouts
from 5 to 60 seconds and the test pass.  :-D

This was the only system that built the 1.0.0-beta on the first try, the
only change since then is the Windows 10 Anniversary update last week.  And
the firewall is off.


On Sun, Aug 28, 2016 at 10:13 PM, Joe Percivall <
[email protected]> wrote:

> Ah sorry, misread your message. I see you ruled out an EOL issue.
>
> Any idea what would change in regards to the SMTP tests in Windows 8 vs 10
> (or your system vs mine)?
>
> Joe
>  - - - - - -
> Joseph Percivall
> linkedin.com/in/Percivall
> e: [email protected]
>
>
>
>
> On Sunday, August 28, 2016 10:10 PM, Joe Percivall <[email protected]>
> wrote:
> JoeS,
>
> I tested the build on Windows 8 and everything worked fine for me.
> Wouldn't an EOL issue show up in all Windows builds?
>
> Joe
> - - - - - -
> Joseph Percivall
> linkedin.com/in/Percivall
> e: [email protected]
>
>
>
>
>
> On Sunday, August 28, 2016 10:05 PM, Joe Skora <[email protected]> wrote:
> -1 (non-binding)
>
> It looks good in general, but Windows 10 unit tests consistently fail in
> TestListenSMTP.
>
> It builds fine on OS X and Ubuntu has one error described below.
>
> It runs fine on OS X and Ubuntu including AWS S3 and HandleHTTP processors.
>
> 1. signature verifies
> 2. hashes verify
> 3. build and passes RAT check
>     a. OS X - builds and runs ok
>     b. Ubuntu 15.10 - builds with one error but runs ok [1]
>     c. Windows 10 - *FAILS* unit tests:
>         - Failing tests are
>             TestListSMTP.validateSuccessfulInteraction
>             TestListSMTP.validateSuccessfulInteractionWithTls
>             TestListSMTP.validateTooLargeMessage
>        - I tried Git core.autocrlf set to "input" and "false", in case it
> was an EOL issue, but that made no difference.
> 4. LICENSE, NOTICE, and README.MD look good
> 5. RC commit ID is good
> 6. RC is branched off correct commit ID
> 7. binary runs as expected with a test flow include AWS *S3 and HandleHTTP*
> processors
>      (including FetchFile, FetchS3Object, GenerateFlowFile, GetFile,
>       HandleHttpRequest, HandleHttpResponse, InvokeHttp, ListS3,
>       LogAttribute, PutFile, RouteOnAttribute, TransformXML, and
>       UpdateAttribute processors)
>
>
> [1] The build fails on Ubuntu 15.10 like it did with 1.0.0-BETA, I have to
> build with "--fail-never" to get a binary.  The failing test is
> TestPutUDP.testInvalidIPAddress(), it could be a JVM bug or network issue
> on the box, but the system shows no other problems.  Tracking through the
> call stack, the native method Inet6AddressImpl.lookupAllHostAddr() throws
> UnknownHostException on other systems with stacktraces similar to the one
> below, but on this one there is no exception thrown using Java 8 and Java 7
> only throws the exception if the "-Djava.net.preferIPv4Stack=true"
> property
> is defined.  Since the Mock framework doesn't actually transfer these files
> to the bad IP the test expects the files to route to "failure" because fo
> the exception but without the exception they go to "success"and the test
> fails.
>         java.net.UnknownHostException: 300.300.300.300: unknown error
>         at java.net.Inet6AddressImpl.lookupAllHostAddr(Native Method)
>         at java.net.InetAddress$2.lookupAllHostAddr(InetAddress.java:928)
>         at
> java.net.InetAddress.getAddressesFromNameService(InetAddress.java:1323)
>         at java.net.InetAddress.getAllByName0(InetAddress.java:1276)
>         at java.net.InetAddress.getAllByName(InetAddress.java:1192)
>         at java.net.InetAddress.getAllByName(InetAddress.java:1126)
>         at java.net.InetAddress.getByName(InetAddress.java:1076)
>         at TestBadIpAddress.main(TestBadIpAddress.java:13)
>
>
> On Sun, Aug 28, 2016 at 9:25 PM, Bryan Rosander <[email protected]>
> wrote:
>
> > Downloaded source
> >
> > Verified checksums
> >
> > Ran build with contrib-check and empty local repo
> >
> > Ran build with –T2 concurrency and empty local repo
> >
> > Used tls-toolkit to generate 4 localhost keystore, truststore,
> > nifi.properties and a single client certificate
> >
> > Started all 4 instances verified no port conflicts, correct certificate
> > usage
> >
> > Imported CA certificate, client cert into browser, verified tls working
> > properly
> >
> > +1 (non-binding)
> >
> > Thanks,
> > Bryan
> >
> > On Aug 28, 2016 7:30 PM, "Tony Kurc" <[email protected]> wrote:
> >
> > > +1 (binding)
> > >
> > > Full build starting with empty maven repository on Ubuntu 14.04
> (x86_64)
> > > successfully.
> > >
> > > Verified signatures and hashes. Reviewed LICENSE/NOTICE.
> > >
> > > Ran a simple flow with no issues. Ran a few nifi-toolkit commands.
> > >
> > >
> > >
> > > On Sun, Aug 28, 2016 at 5:29 PM, Joe Witt <[email protected]> wrote:
> > >
> > > > +1 (binding)
> > > >
> > > > Full clean build w/contrib check good.
> > > >
> > > > Verified signatures (and strength), hashes, and LICENSE/NOTICE within
> > > > sources and convenience binaries.
> > > >
> > > > Verified the commit the release was based on.
> > > >
> > > > Ran single and multi-node clusters with favorable results.
> > > >
> > > > For future release please do move any necessary elements of the UnRar
> > > > item to NOTICE where category B items belong.  See here for LEGAL
> > > > discuss https://issues.apache.org/jira/browse/LEGAL-52 on this.
> > > >
> > > > Great job pulling the release together again JoeP.
> > > >
> > > > Thanks
> > > > Joe
> > > >
> > > > On Fri, Aug 26, 2016 at 12:25 PM, Joe Percivall
> > > > <[email protected]> wrote:
> > > > > Hello Apache NiFi Community,
> > > > >
> > > > > I am pleased to be calling this vote for the source release of
> Apache
> > > > NiFi,
> > > > > nifi-1.0.0.
> > > > >
> > > > > The source zip, including signatures, digests, etc. can be found
> at:
> > > > > https://repository.apache.org/content/repositories/
> > orgapachenifi-1090/
> > > > >
> > > > > Note: There is a second binary being distributed now, the NiFi
> > Toolkit.
> > > > It
> > > > > can be used to facilitate securing a NiFi instance.
> > > > >
> > > > > The Git tag is nifi-1.0.0-RC1
> > > > > The Git commit hash is 74d5224783dfdc513f6b3ad5ed96671d3c581707
> > > > > * https://git-wip-us.apache.org/repos/asf?p=nifi.git;a=commit;h=
> > > > 74d5224783dfdc513f6b3ad5ed96671d3c581707
> > > > > * https://github.com/apache/nifi/commit/
> > 74d5224783dfdc513f6b3ad5ed9667
> > > > 1d3c581707
> > > > >
> > > > > Checksums of nifi-1.0.0-source-release.zip:
> > > > > MD5: 8bdba49a73b94d036fad6c63b0ebe39d
> > > > > SHA1: 504c58f9b2fb305c41598a17f5b78f68f2b2fa3d
> > > > > SHA256: 22167ede5127683ca8de6dbd2fb911
> 2cb1de650b7cfff7e640c905521447
> > > af92
> > > > >
> > > > > Release artifacts are signed with the following key:
> > > > > https://people.apache.org/keys/committer/jpercivall
> > > > >
> > > > > KEYS file available here:
> > > > > https://dist.apache.org/repos/dist/release/nifi/KEYS
> > > > >
> > > > > 595 issues were closed/resolved for this release:
> > > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?
> > > > projectId=12316020&version=12332640
> > > > >
> > > > > Release note highlights can be found here:
> > > > > https://cwiki.apache.org/confluence/display/NIFI/
> > > > Release+Notes#ReleaseNotes-Version1.0.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-1.0.0
> > > > > [ ] +0 no opinion
> > > > > [ ] -1 Do not release this package because...
> > > > >
> > > > > Thanks!
> > > >
> > >
> >
>

Reply via email to