<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
<META NAME="GENERATOR" CONTENT="GtkHTML/3.32.2">
</HEAD>
<BODY>
There's a pull request for this:<BR>
<BR>
<A HREF="https://github.com/jbossas/jboss-as/pull/534">https://github.com/jbossas/jboss-as/pull/534</A><BR>
<BR>
Ondra<BR>
<BR>
<BR>
On Thu, 2011-10-27 at 17:24 +0100, Kabir Khan wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
One thing we talked about internally is that lthough not much thought has gone into what makes up the smoke tests, at least they are good for picking up problems in parsing and management layer which is very useful as a pre-pull request requirement.
We had a look around and found: <A HREF="http://maven.apache.org/guides/introduction/introduction-to-profiles.html">http://maven.apache.org/guides/introduction/introduction-to-profiles.html</A> which mentions: "As of Maven 3.0, profiles in the POM can also be activated based on properties from active profiles from the setttings.xml.".
Here is the rest of the discussion:
[27/10/2011 14:29:01] Stuart Douglas: but that requires a specific settings.xml
[27/10/2011 14:29:06] Kabir Khan: Oh
[27/10/2011 14:29:11] Stuart Douglas: although
[27/10/2011 14:29:31] Stuart Douglas: we coud just require all the QA people to have a specific settings.xml
[27/10/2011 14:31:43] Kabir Khan: Yeah, I've not really looked at the current setup so I don't know if it would work or not but if it does I'm all for it
[27/10/2011 14:36:42] Jason T. Greene: we bundle maven with as though
[27/10/2011 14:36:58] Jason T. Greene: and there is a settings.xml in there
[27/10/2011 14:37:41] Jason T. Greene: so we could put it all in there
[27/10/2011 14:38:07] Stuart Douglas: but really we need to settings.xml
[27/10/2011 14:38:16] Stuart Douglas: the the default developer one
[27/10/2011 14:38:21] Stuart Douglas: and the QE one
[27/10/2011 14:38:25] Jason T. Greene: sure
[27/10/2011 14:38:30] Jason T. Greene: thats reaosnable imo
[27/10/2011 14:38:32] Kabir Khan: qe-build.sh :-)
[27/10/2011 14:38:45] Stuart Douglas: and hopefully that will work better
On 27 Oct 2011, at 17:20, Ondrej Zizka wrote:
> Ok, seems like QA will have to carry the burden of disabling smoke tests whenever they want to run anything else - because Maven's profile activation logic really sucks.
> Fixing.
>
> Ondra
>
>
>
> On Thu, 2011-10-27 at 11:36 +0100, Kabir Khan wrote:
>> The smoke tests must be run as part of a standard build. Please fix.
>>
>> If QE have other requirements, which are not possible to combine with the above requirement maybe they should look into using alternate poms or the antrun plugin or something.
>> On 27 Oct 2011, at 09:48, Thomas Diesler wrote:
>>
>> > Currently, the smoke tests don't run as part of the build. Is this really what you want?
>> >
>> >
>> <A HREF="https://hudson.qa.jboss.com/hudson/view/JBossOSGi/job/jboss-as7-master/389/">https://hudson.qa.jboss.com/hudson/view/JBossOSGi/job/jboss-as7-master/389/</A>
>>
>> >
>> > -thomas
>> >
>> > On 10/27/2011 06:17 AM, Stuart Douglas wrote:
>> >> I just just merged some changes to the test suite that QE requires for EAP (see below for a link to their requirements). As a result the manner in which the tests are run has changed slightly.
>> >>
>> >> Due to limitations in maven it is no longer possible to activate all tests using mvn install -DallTests, instead you should use either:
>> >>
>> >> ./build.sh install -DallTests - To do a full build with all tests
>> >> ./integration-tests.sh -DallTests - To just run all integration tests
>> >>
>> >> Running individual tests from an IDE or via mvn test -Dtest=MyTest should work as normal.
>> >>
>> >> Ondej Zizka (in CC) has been the person working on these changes, so if you have any questions or comments he is the person to talk to.
>> >>
>> >> Stuart
>> >>
>> >> *
>> <A HREF="https://docspace.corp.redhat.com/docs/DOC-74146">https://docspace.corp.redhat.com/docs/DOC-74146</A>
>>
>> >> *
>> <A HREF="https://docspace.corp.redhat.com/docs/DOC-69049">https://docspace.corp.redhat.com/docs/DOC-69049</A>
>>
>> >> *
>> <A HREF="https://github.com/jbossas/jboss-as/pull/491">https://github.com/jbossas/jboss-as/pull/491</A>
>>
>> >>
>> >>
>> >>
>> >> _______________________________________________
>> >> jboss-as7-dev mailing list
>> >>
>> >>
>> <A HREF="mailto:jboss-as7-dev@lists.jboss.org">jboss-as7-dev@lists.jboss.org</A>
>>
>> >>
>> <A HREF="https://lists.jboss.org/mailman/listinfo/jboss-as7-dev">https://lists.jboss.org/mailman/listinfo/jboss-as7-dev</A>
>>
>> >
>> > --
>> > xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>> > Thomas Diesler
>> > JBoss OSGi Lead
>> > JBoss, a division of Red Hat
>> > xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>> >
>> > _______________________________________________
>> > jboss-as7-dev mailing list
>> >
>> <A HREF="mailto:jboss-as7-dev@lists.jboss.org">jboss-as7-dev@lists.jboss.org</A>
>>
>> >
>> <A HREF="https://lists.jboss.org/mailman/listinfo/jboss-as7-dev">https://lists.jboss.org/mailman/listinfo/jboss-as7-dev</A>
>>
>>
>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>>
>> <A HREF="mailto:jboss-as7-dev@lists.jboss.org">jboss-as7-dev@lists.jboss.org</A>
>> <A HREF="https://lists.jboss.org/mailman/listinfo/jboss-as7-dev">https://lists.jboss.org/mailman/listinfo/jboss-as7-dev</A>
>
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>