I am working with Michael on FreeBSD issues. I will try to fix one IT today and I will analyse the logs with next two ITs. Did we change something regarding SITE in Maven 3.5.0-SNAPSHOT ?
On Mon, Feb 13, 2017 at 6:32 PM, Tibor Digana-2 [via Maven] < ml-node+s40175n5898532...@n5.nabble.com> wrote: > The HEAD in Surefire has a Jira fix 1322 which is using the same IT which > failed on FreeBSD. So I simply included FreeBSD fix in the test > Surefire141PluggableProvidersIT. > I hope you will be happy with that. > The only last IT which fails and I did not have time to investigate is > Surefire772NoSurefireReportsIT. This is from your log you sent me: > > testOptionalSurefireReportGeneration(org.apache.maven.surefire.its.jiras. > Surefire772NoSurefireReportsIT) > Time elapsed: 0 sec <<< FAILURE! > java.lang.AssertionError: Expecting failsafe report file > at org.junit.Assert.fail(Assert.java:88) > at org.junit.Assert.assertTrue(Assert.java:41) > at > org.apache.maven.surefire.its.jiras.Surefire772NoSurefireReportsIT. > testOptionalSurefireReportGeneration(Surefire772NoSurefireReportsIT.java:75) > > > > On Mon, Feb 13, 2017 at 4:49 PM, Michael Osipov <[hidden email] > <http:///user/SendEmail.jtp?type=node&node=5898532&i=0>> wrote: > > > I am currently running a patched version of Maven with the offensive > > commit reverted against Surefire master on FreeBSD 10.3-STABLE. I'll see > > wether this is the cause in an hour or so. > > > > Michael > > > > > I’ve logged the change in precedence as https://issues.apache.org/ > > jira/browse/MNG-6172 since this has the potential to break CI builds > > which relied on the previous last-one-wins behaviour > > > > > > On Monday, 13 February 2017 at 06:54, Tibor Digana wrote: > > > > > > > Exactly this is the problem - forkMode and reuseForks in one > function. > > > > I will open Jira and push a fix. > > > > > > > > On Mon, Feb 13, 2017 at 2:44 AM, Stuart McCulloch [via Maven] < > > > > [hidden email] > <http:///user/SendEmail.jtp?type=node&node=5898532&i=1> (mailto:ml-node+ > > [hidden email] <http:///user/SendEmail.jtp?type=node&node=5898532&i=2>)> > wrote: > > > > > > > > > So I tweaked ForkedLauncher from maven-verifier to dump out the > > forked > > > > > command to log.txt and at least one of the failing ITs has a > > duplicated > > > > > system property on the command-line: > > > > > > > > > > mvn -e --batch-mode -Dmaven.repo.local=/tmp/maven- > > > > > surefire/surefire-integration-tests/../surefire-setup- > > > > > integration-tests/target/it-repo org.apache.maven.plugins: > > maven-clean-plugin:clean > > > > > -Dsurefire.version=2.19.2-SNAPSHOT -DtestNgVersion=5.7 > > > > > -DtestNgClassifier=jdk15 -DforkMode=perthread -DreuseForks=false > > > > > -DreuseForks=true -DthreadCount=1 -DtestProperty=testValue_${ > > > > > surefire.threadNumber}_${surefire.forkNumber} > > org.apache.maven.plugins. > > > > > surefire:maven-dump-pid-plugin:dump-pid test > > > > > > > > > > specifically: > > > > > > > > > > -DreuseForks=false -DreuseForks=true > > > > > > > > > > Looking at SurefireLauncher.java > > > > > > > > > > https://github.com/apache/maven-surefire/blob/master/ > > > > > surefire-integration-tests/src/test/java/org/apache/ > > > > > maven/surefire/its/fixture/SurefireLauncher.java > > > > > > > > > > 1. the reuseForks method adds a new system property argument each > > time > > > > > it’s called: -DreuseForks=<setting> > > > > > 2. the forkPerThread method calls forkMode( "perthread" > ).reuseForks( > > > > > false ) > > > > > 3. the forkOncePerThread calls forkPerThread().reuseForks( true ) > > > > > > > > > > So any test using SurefireLauncher’s “forkOncePerThread” method > will > > end > > > > > up with two “reuseForks” system property options on the > command-line > > - and > > > > > will therefore be affected by the change in system property option > > > > > precedence. > > > > > > > > > > This explains why ForkModeIT fails for me locally with master, but > > passes > > > > > with 3.3.9 - with current master the -DreuseForks=false option > wins, > > > > > whereas with 3.3.9 the -DreuseForks=true options wins. > > > > > > > > > > If I change SurefireLauncher.forkOncePerThread to avoid > duplicating > > this > > > > > system property, ie: > > > > > > > > > > return forkMode( "perthread" ).reuseForks( true ); > > > > > > > > > > then ForkModeIT passes with both current master and previous Maven > > > > > releases. > > > > > > > > > > Hope that helps... > > > > > > > > > > -- > > > > > Cheers, Stuart > > > > > > > > > > > > > > > On Monday, 13 February 2017 at 01:32, Stuart McCulloch wrote: > > > > > > > > > > > Ironically I got those results the wrong way round when > > cutting+pasting > > > > > :) > > > > > > > > > > > > To clarify, when testing > > > > > > > > > > > > mvn -Dmaven.repo.local=/tmp/aaa -Dmaven.repo.local=/tmp/zzz > > validate > > > > > > > > > > > > with previous Maven releases the last option wins: > > > > > > > > > > > > -Dmaven.repo.local=/tmp/zzz > > > > > > > > > > > > whereas with current master the first option wins: > > > > > > > > > > > > -Dmaven.repo.local=/tmp/aaa > > > > > > > > > > > > On Monday, 13 February 2017 at 00:19, Stuart McCulloch wrote: > > > > > > > > > > > > > Using the following command on a small test project: > > > > > > > > > > > > > > mvn -Dmaven.repo.local=/tmp/aaa -Dmaven.repo.local=/tmp/zzz > > validate > > > > > > > > > > > > > > and checking whether the “aaa” or “zzz” directory is created > > gives > > > > > these results for previous Maven releases: > > > > > > > > > > > > > > 2.0.11 aaa > > > > > > > 2.2.1 aaa > > > > > > > 3.0.5 aaa > > > > > > > 3.1.1 aaa > > > > > > > 3.2.5 aaa > > > > > > > 3.3.9 aaa > > > > > > > > > > > > > > whereas current master gives a different result: > > > > > > > > > > > > > > master zzz > > > > > > > > > > > > > > which confirms the precedence of CLI arguments is currently > > reversed > > > > > on master compared to previous releases > > > > > > > > > > > > > > -- > > > > > > > Cheers, Stuart > > > > > > > > > > > > > > > > > > > > > On Sunday, 12 February 2017 at 23:53, Stuart McCulloch wrote: > > > > > > > > > > > > > > > git bisect is pointing to the following commit: > > > > > > > > > > > > > > > > https://github.com/apache/maven/commit/ > > > > > ca4303031357a7decaee8de770b71fb2c2fedd28 > > > > > > > > > > > > > > > > if I revert this change then the wrong PID issue disappears > and > > > > > ForkModeIT passes again > > > > > > > > > > > > > > > > I suspect that reversing the whole array of system property > > > > > definitions, while solving MNG-6078, is breaking precedence of > > arguments on > > > > > the CLI (which then affects these tests) > > > > > > > > > > > > > > > > On Sunday, 12 February 2017 at 20:26, Tibor Digana wrote: > > > > > > > > > > > > > > > > > So this is a local build status of surefire running on the > > top of > > > > > certain > > > > > > > > > Maven Version. > > > > > > > > > Maven 3.3.9 OK on my side > > > > > > > > > Maven 3.5.0-SNAPSHOT failed. Wrong PIDs, other tests > failed > > on > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > freebsd and > > > > > > > > > not on Win7, so I asked Michael for logs which I do not > have. > > > > > > > > > So I am going to investigate. > > > > > > > > > > > > > > > > > > On Sun, Feb 12, 2017 at 9:05 PM, stephenconnolly [via > Maven] > > < > > > > > > > > > [hidden email] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > <http:///user/SendEmail.jtp?type=node&node=5898461&i=0> (mailto: > > [hidden > > > > > email] <http:///user/SendEmail.jtp?type=node&node=5898461&i=1>)> > > wrote: > > > > > > > > > > > > > > > > > > > Can you pop on HipChat with infra? > > > > > > > > > > > > > > > > > > > > You may need them to grab the files from the agent for > you > > (or > > > > > modify the > > > > > > > > > > Jenkinsfile temporarily to archive the bits you need) > > > > > > > > > > > > > > > > > > > > On Sun 12 Feb 2017 at 19:55, Tibor Digana <[hidden > email] > > > > > > > > > > <http:///user/SendEmail.jtp?type=node&node=5898384&i=0>> > > > wrote: > > > > > > > > > > > > > > > > > > > > > There is build process for surefire > > > > > > > > > > > > > > > > > > > > > > https://builds.apache.org/view/Maven/job/maven-master- > > > > > > > > > > release-status-test-surefire-linux > > > > > > > > > > > with Jenkins file. > > > > > > > > > > > This particular build fails with test > > > > > > > > > > > > > > > > > > > > > > Surefire141PluggableProvidersIT > > > > > > > > > > > > > > > > > > > > > > It is only one and different from your local build > > result. > > > > > > > > > > > > > > > > > > > > > > I asked Stephen to enable Workspace, because I need to > > have > > > > > files from > > > > > > > > > > > target folders for analysis to understand what is > going > > on the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > machine. > > > > > > > > > > > Stephen told me that I should trigger rebuild. So I > did > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > launched > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > "Build > > > > > > > > > > > with Parameters" but the build failed not running > Maven > > build, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > however > > > > > > > > > > > Workspace appears now. > > > > > > > > > > > > > > > > > > > > > > How can I force trigger the build? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://builds.apache.org/view/Maven/job/maven-master- > > > > > > > > > > release-status-test-surefire-linux/11/console > > > > > > > > > > > > > > > > > > > > > > There is another build process, old one actually, and > it > > is > > > > > successful. > > > > > > > > > > > The difference is because I think race condition when > JVM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > error once > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > goes > > > > > > > > > > > to std/err and other log in the first build with > Jenkins > > file. > > > > > > > > > > > This can, I think, be only one more assertion > statement > > in IT > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > but I need > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > to > > > > > > > > > > > see Workspace. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Sun, Feb 12, 2017 at 5:04 PM, Christian Schulte > [via > > Maven] > > > > > < > > > > > > > > > > > [hidden email] <http:///user/SendEmail.jtp? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > type=node&node=5898384&i=1>> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > @Stephen: Are there any Jenkins jobs left running > the > > plugin > > > > > ITs with > > > > > > > > > > > > current Maven master? Surefire has it's own git > > repository. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Do we have > > > > > > > > > > > > some Jenkins job for this as well? It's important to > > run all > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > those ITs > > > > > > > > > > > > (plugins from subversion and the ones with theire > own > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > repository) with > > > > > > > > > > > > what we are going to release as 3.5.0 before > releasing > > it. > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > -- > > > > > > > > > > > > Christian > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------ > > --------------------------------------- > > > > > > > > > > > > > > > > > To unsubscribe, e-mail: [hidden email] > > > > > > > > > > > > <http:///user/SendEmail.jtp? > > type=node&node=5898337&i=0> > > > > > > > > > > > > For additional commands, e-mail: [hidden email] > > > > > > > > > > > > <http:///user/SendEmail.jtp? > > type=node&node=5898337&i=1> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------ > > > > > > > > > > > > If you reply to this email, your message will be > added > > to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > discussion > > > > > > > > > > > > below: > > > > > > > > > > > > http://maven.40175.n5.nabble. > > com/I-think-we-are-ready-for- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3-5-0-alpha-1- > > > > > > > > > > > > tp5897626p5898337.html > > > > > > > > > > > > To start a new topic under Maven Developers, email > > > > > > > > > > > > [hidden email] <http:///user/SendEmail.jtp? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > type=node&node=5898384&i=2> > > > > > > > > > > > > To unsubscribe from Maven Developers, click here > > > > > > > > > > > > < > > > > > > > > > > > > > > > > > > > > > > > > . > > > > > > > > > > > > NAML > > > > > > > > > > > > < > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://maven.40175.n5.nabble. > > com/template/NamlServlet.jtp? > > > > > > > > > > > > > > > > > > > > macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml& > > > > > > > > > > > base=nabble.naml.namespaces.BasicNamespace-nabble.view. > > > > > > > > > > web.template.NabbleNamespace-nabble.view.web.template ( > > http://web.template.NabbleNamespace-nabble.view.web.template) ( > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://web.template.NabbleNamespace-nabble.view.web.template). > > > > > > > > > > NodeNamespace&breadcrumbs=notify_subscribers%21nabble% > > > > > > > > > > 3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_ > > > > > > > > > > instant_email%21nabble%3Aemail.naml > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > View this message in context: > > > > > > > > > > > http://maven.40175.n5.nabble. > > com/I-think-we-are-ready-for- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3-5-0-alpha-1- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > tp5897626p5898378.html > > > > > > > > > > > Sent from the Maven Developers mailing list archive at > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Nabble.com (http://Nabble.com). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > Sent from my phone > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------ > > > > > > > > > > If you reply to this email, your message will be added > to > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > discussion > > > > > > > > > > below: > > > > > > > > > > http://maven.40175.n5.nabble. > com/I-think-we-are-ready-for- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3-5-0-alpha-1- > > > > > > > > > > tp5897626p5898384.html > > > > > > > > > > To start a new topic under Maven Developers, email > > > > > > > > > > [hidden email] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > <http:///user/SendEmail.jtp?type=node&node=5898461&i=2> (mailto: > > [hidden > > > > > email] <http:///user/SendEmail.jtp?type=node&node=5898461&i=3>) > > > > > > > > > > To unsubscribe from Maven Developers, click here > > > > > > > > > > < > > > > > > > > > > . > > > > > > > > > > NAML > > > > > > > > > > <http://maven.40175.n5.nabble. > com/template/NamlServlet.jtp > > ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml& > > > > > base=nabble.naml.namespaces.BasicNamespace-nabble.view. > > > > > web.template.NabbleNamespace-nabble.view.web.template ( > > http://web.template.NabbleNamespace-nabble.view.web.template). > > > > > NodeNamespace&breadcrumbs=notify_subscribers%21nabble% > > > > > 3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_ > > > > > instant_email%21nabble%3Aemail.naml> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > View this message in context: http://maven.40175.n5.nabble. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > com/I-think-we-are-ready-for-3-5-0-alpha-1-tp5897626p5898389.html > > > > > > > > > Sent from the Maven Developers mailing list archive at > > Nabble.com (http://Nabble.com) ( > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://Nabble.com). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------ > > > > > If you reply to this email, your message will be added to the > > discussion > > > > > below: > > > > > http://maven.40175.n5.nabble.com/I-think-we-are-ready-for- > > 3-5-0-alpha-1- > > > > > tp5897626p5898461.html > > > > > To start a new topic under Maven Developers, email > > > > > [hidden email] > <http:///user/SendEmail.jtp?type=node&node=5898532&i=3> (mailto:ml-node+ > > [hidden email] <http:///user/SendEmail.jtp?type=node&node=5898532&i=4>) > > > > > To unsubscribe from Maven Developers, click here > > > > > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp? > > macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3 > > > wxNDIxNjZ8LTI4OTQ5MjEwMg==> > > > > > . > > > > > NAML > > > > > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp? > > macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml& > > base=nabble.naml.namespaces.BasicNamespace-nabble.view. > > web.template.NabbleNamespace-nabble.view.web.template. > > NodeNamespace&breadcrumbs=notify_subscribers%21nabble% > > 3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_ > > instant_email%21nabble%3Aemail.naml> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > View this message in context: http://maven.40175.n5.nabble. > > com/I-think-we-are-ready-for-3-5-0-alpha-1-tp5897626p5898468.html > > > > Sent from the Maven Developers mailing list archive at Nabble.com ( > > http://Nabble.com). > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [hidden email] > <http:///user/SendEmail.jtp?type=node&node=5898532&i=5> > > For additional commands, e-mail: [hidden email] > <http:///user/SendEmail.jtp?type=node&node=5898532&i=6> > > > > > > > -- > Cheers > Tibor > > > ------------------------------ > If you reply to this email, your message will be added to the discussion > below: > http://maven.40175.n5.nabble.com/I-think-we-are-ready-for-3-5-0-alpha-1- > tp5897626p5898532.html > To start a new topic under Maven Developers, email > ml-node+s40175n142166...@n5.nabble.com > To unsubscribe from Maven Developers, click here > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=142166&code=dGlib3JkaWdhbmFAYXBhY2hlLm9yZ3wxNDIxNjZ8LTI4OTQ5MjEwMg==> > . > NAML > <http://maven.40175.n5.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml> > -- View this message in context: http://maven.40175.n5.nabble.com/I-think-we-are-ready-for-3-5-0-alpha-1-tp5897626p5898552.html Sent from the Maven Developers mailing list archive at Nabble.com.
