Corey Nolet 2013-03-21, 11:53
David Medinets 2013-03-21, 10:31
Keith Turner 2013-03-21, 12:20
Tim Reardon 2013-03-22, 01:59
David Medinets 2013-03-22, 02:42
Corey Nolet 2013-03-22, 02:51
-Re: Use pre-integration-test to start instamo, run tests in integration-test, then stop instamo in post-integration-test?
and making the recommended changes to pom.xml seems to work well. By
simply renaming ShellServerTest to ShellServerIT (and editing the
class name), my 'mvn clean test' was faster by two minutes. I hesitate
to commit such a change since I don't know how to change the jenkins
configuration and I don't want to break the build process. I'm
creating a JIRA ticket to move the discussion there.
On Thu, Mar 21, 2013 at 10:51 PM, Corey Nolet <[EMAIL PROTECTED]> wrote:
> Generally, you'd place it on the developer making changes to run their
> integration tests before checking in to keep jenkin's from whining about
> busted tests. The Proxy tests are quite slow right now and there's some
> duplication in the more detailed unit tests. The SimpleTest runs everything
> in a mini cluster while separate classes are unit testing the important
> methods. Would you guys be against throwing up a JIRA ticket to rename
> SimpleTest to SimpleIT?
> As Tim stated, it may be a good idea not to run those by default. 30+
> seconds for a single test is a little much.
> On Thu, Mar 21, 2013 at 10:42 PM, David Medinets
> <[EMAIL PROTECTED]>wrote:
>> Who is in charge of changing the Jenkins configuration? It would be
>> nice to know if the build process executes the integration-test phase.
>> By default, integration-test runs test classes named **/IT*.java,
>> **/*IT.java, and **/*ITCase.java. Once the Jenkins build is confirmed,
>> then renaming the long-running tests should not be difficult.
>> On Thu, Mar 21, 2013 at 9:59 PM, Tim Reardon <[EMAIL PROTECTED]> wrote:
>> > +1 for Corey. In other projects I've seen alot of mileage out of moving
>> > longer running tests to an "it" profile that Hudson/Jenkins runs by
>> > default, alleviating some of the burden for developers of running a basic
>> > mvn clean install while still maintaining a fallback via the CI build.
>> > On Thu, Mar 21, 2013 at 8:20 AM, Keith Turner <[EMAIL PROTECTED]> wrote:
>> >> On Thu, Mar 21, 2013 at 6:31 AM, David Medinets
>> >> <[EMAIL PROTECTED]> wrote:
>> >> > Running org.apache.accumulo.proxy.SimpleTest
>> >> > Tests run: 3, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 37.844
>> >> > sec <<< FAILURE!
>> >> >
>> >> > I am continued to be concerned by this unit test in the proxy module.
>> >> > And a few others that are long-running. I haven't found time to dig
>> >> > into the code but I assume it is the issue related to spinning up the
>> >> > Accumulo processes for the test. Could one instance of Instamo be spun
>> >> > up during the pre-integration-test and spin down during
>> >> > post-integration-test as stand-alone processes? Then any tests
>> >> > associated with Instamo can be run inside the integration-test phase.
>> >> Christopher has mentioned moving some test to integration test. He
>> >> may have created a ticket. There has also been discussion by Eric
>> >> and Christopher about moving alot of the existing functional test to
>> >> integration test. Some of the functional test restart processes, and
>> >> may not be suitable for integration test.
>> >> This would all be great for 1.6. Some test may need to spin up their
>> >> own Accumulo instance. Would need to evaluate each test and see if it
>> >> needs own instance or can use a shared instance (w/ tables names
>> >> unique to the test).
>> >> >
>> >> > On Thu, Mar 21, 2013 at 1:59 AM, Apache Jenkins Server
>> >> > <[EMAIL PROTECTED]> wrote:
>> >> >> See <
>> >> https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/150/changes>
>> >> >>
> Corey Nolet
> Senior Software Engineer
> TexelTek, inc.
> [Office] 301.880.7123
> [Cell] 410-903-2110