You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bookkeeper.apache.org by "Ivan Kelly (JIRA)" <ji...@apache.org> on 2012/11/19 13:28:59 UTC

[jira] [Created] (BOOKKEEPER-471) Add scripts for preCommit testing

Ivan Kelly created BOOKKEEPER-471:
-------------------------------------

             Summary: Add scripts for preCommit testing
                 Key: BOOKKEEPER-471
                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
             Project: Bookkeeper
          Issue Type: Bug
            Reporter: Ivan Kelly
            Assignee: Ivan Kelly
             Fix For: 4.2.0


We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500429#comment-13500429 ] 

Hadoop QA commented on BOOKKEEPER-471:
--------------------------------------

Testing JIRA BOOKKEEPER-471

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:red}-1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 120
.    {color:red}-1{color} the patch does not add/modify any testcase
{color:red}-1 RAT{color}
.    {color:red}-1{color} the patch seems to introduce 16 new RAT warning(s)
.    {color:red}WARNING: the current HEAD has 7 RAT warning(s), they should be addressed ASAP{color}
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings
.    {color:red}WARNING{color}: the current HEAD has 8 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac warnings
.    {color:red}WARNING{color}: the current HEAD has 9 javac warning(s)
{color:green}+1 FINDBUGS{color}
.    {color:green}+1{color} the patch does not seem to introduce new Findbugs warnings

----------------------------
{color:red}*-1 Overall result, please check the reported -1(s)*{color}

{color:red}.   There is at least one warning, please check{color}

The full output of the test-patch run is available at

.   https://builds.apache.org/job/bookkeeper-trunk-precommit-build/9/
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch

Did some cleanup on the scripts.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500628#comment-13500628 ] 

Hadoop QA commented on BOOKKEEPER-471:
--------------------------------------

Testing JIRA BOOKKEEPER-471

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:green}+1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 120
.    {color:green}+1{color} the patch does adds/modifies 1 testcase(s)
{color:green}+1 RAT{color}
.    {color:green}+1{color} the patch does not seem to introduce new RAT warnings
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings
.    {color:red}WARNING{color}: the current HEAD has 8 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac warnings
.    {color:red}WARNING{color}: the current HEAD has 9 javac warning(s)
{color:green}+1 FINDBUGS{color}
.    {color:green}+1{color} the patch does not seem to introduce new Findbugs warnings

----------------------------
{color:green}*+1 Overall result, good!, no -1s*{color}

{color:red}.   There is at least one warning, please check{color}

The full output of the test-patch run is available at

.   https://builds.apache.org/job/bookkeeper-trunk-precommit-build/12/
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: failTests.patch
    
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Uma Maheswara Rao G (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13502886#comment-13502886 ] 

Uma Maheswara Rao G commented on BOOKKEEPER-471:
------------------------------------------------

Hi Ivan, Thanks a lot for this Work.

one suggestion:

{code}
Testing JIRA BOOKKEEPER-471
{code}

Instead of just displaying JIRA ID, can we display the patch name on which the current results pasted? (Hadoop displays like that way)
So, that we can see which results for which file if at all Jenkins picks multiple patches one another. (Generally picks latest one), that will be useful when you look back the history of results in JIRA.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500540#comment-13500540 ] 

Hadoop QA commented on BOOKKEEPER-471:
--------------------------------------

Testing JIRA BOOKKEEPER-471

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:green}+1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 120
.    {color:green}+1{color} the patch does adds/modifies 1 testcase(s)
{color:green}+1 RAT{color}
.    {color:green}+1{color} the patch does not seem to introduce new RAT warnings
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings
.    {color:red}WARNING{color}: the current HEAD has 8 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac warnings
.    {color:red}WARNING{color}: the current HEAD has 15 javac warning(s)
{color:green}+1 FINDBUGS{color}
.    {color:green}+1{color} the patch does not seem to introduce new Findbugs warnings

----------------------------
{color:green}*+1 Overall result, good!, no -1s*{color}

{color:red}.   There is at least one warning, please check{color}

The full output of the test-patch run is available at

.   https://builds.apache.org/job/bookkeeper-trunk-precommit-build/11/
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Uma Maheswara Rao G (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500200#comment-13500200 ] 

Uma Maheswara Rao G commented on BOOKKEEPER-471:
------------------------------------------------

Big +1 for this task. Thanks a lot Ivan for adding the scripts.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: TriggerFindbugsFail.patch
    
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: TriggerFindbugsFail.patch
    
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500478#comment-13500478 ] 

Hadoop QA commented on BOOKKEEPER-471:
--------------------------------------

Testing JIRA BOOKKEEPER-471

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:red}-1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 120
.    {color:red}-1{color} the patch does not add/modify any testcase
{color:green}+1 RAT{color}
.    {color:green}+1{color} the patch does not seem to introduce new RAT warnings
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings
.    {color:red}WARNING{color}: the current HEAD has 8 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac warnings
.    {color:red}WARNING{color}: the current HEAD has 15 javac warning(s)
{color:red}-1 FINDBUGS{color}
.    {color:red}-1{color} the patch seems to introduce 1 new Findbugs warning(s) in module(s) [ bookkeeper-server]

----------------------------
{color:red}*-1 Overall result, please check the reported -1(s)*{color}

{color:red}.   There is at least one warning, please check{color}

The full output of the test-patch run is available at

.   https://builds.apache.org/job/bookkeeper-trunk-precommit-build/10/
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch

Ah, good catch Uma, new patch addresses this.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: failTests.patch
    
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504543#comment-13504543 ] 

Hudson commented on BOOKKEEPER-471:
-----------------------------------

Integrated in bookkeeper-trunk #821 (See [https://builds.apache.org/job/bookkeeper-trunk/821/])
    BOOKKEEPER-471: Add scripts for preCommit testing (ivank) (Revision 1414044)

     Result = FAILURE
ivank : 
Files : 
* /zookeeper/bookkeeper/trunk/CHANGES.txt
* /zookeeper/bookkeeper/trunk/bin
* /zookeeper/bookkeeper/trunk/bin/find-new-patch-available-jiras
* /zookeeper/bookkeeper/trunk/bin/test-patch
* /zookeeper/bookkeeper/trunk/bin/test-patch-00-clean
* /zookeeper/bookkeeper/trunk/bin/test-patch-05-patch-raw-analysis
* /zookeeper/bookkeeper/trunk/bin/test-patch-08-rat
* /zookeeper/bookkeeper/trunk/bin/test-patch-09-javadoc
* /zookeeper/bookkeeper/trunk/bin/test-patch-10-compile
* /zookeeper/bookkeeper/trunk/bin/test-patch-11-findbugs
* /zookeeper/bookkeeper/trunk/bin/test-patch-20-tests
* /zookeeper/bookkeeper/trunk/bin/test-patch-30-dist
* /zookeeper/bookkeeper/trunk/pom.xml

                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Uma Maheswara Rao G (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504000#comment-13504000 ] 

Uma Maheswara Rao G commented on BOOKKEEPER-471:
------------------------------------------------

Thanks a lot Ivan for the update! 
+1 for pushing this in. Nice work.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500366#comment-13500366 ] 

Hadoop QA commented on BOOKKEEPER-471:
--------------------------------------

Testing JIRA BOOKKEEPER-471

Cleaning local git workspace

----------------------------

{color:green}+1 PATCH_APPLIES{color}
{color:green}+1 CLEAN{color}
{color:green}+1 RAW_PATCH_ANALYSIS{color}
.    {color:green}+1{color} the patch does not introduce any @author tags
.    {color:green}+1{color} the patch does not introduce any tabs
.    {color:green}+1{color} the patch does not introduce any trailing spaces
.    {color:green}+1{color} the patch does not introduce any line longer than 120
.    {color:green}+1{color} the patch does adds/modifies 1 testcase(s)
{color:red}-1 RAT{color}
.    {color:red}-1{color} the patch seems to introduce 16 new RAT warning(s)
.    {color:red}WARNING: the current HEAD has 7 RAT warning(s), they should be addressed ASAP{color}
{color:green}+1 JAVADOC{color}
.    {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings
.    {color:red}WARNING{color}: the current HEAD has 8 Javadoc warning(s)
{color:green}+1 COMPILE{color}
.    {color:green}+1{color} HEAD compiles
.    {color:green}+1{color} patch compiles
.    {color:green}+1{color} the patch does not seem to introduce new javac warnings
.    {color:red}WARNING{color}: the current HEAD has 9 javac warning(s)
{color:red}-1 FINDBUGS{color}
.    {color:green}+1{color} the patch does not seem to introduce new Findbugs warnings

----------------------------
{color:red}*-1 Overall result, please check the reported -1(s)*{color}

{color:red}.   There is at least one warning, please check{color}

The full output of the test-patch run is available at

.   https://builds.apache.org/job/bookkeeper-trunk-precommit-build/8/
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Commented] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Flavio Junqueira (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13500242#comment-13500242 ] 

Flavio Junqueira commented on BOOKKEEPER-471:
---------------------------------------------

Same here, +1 for getting this going, Ivan!
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: TriggerFindbugsFail.patch
    
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (BOOKKEEPER-471) Add scripts for preCommit testing

Posted by "Ivan Kelly (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/BOOKKEEPER-471?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ivan Kelly updated BOOKKEEPER-471:
----------------------------------

    Attachment: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch

This is the patch with the actual patches. Based heavily on oozie patches. The scripts need to be executable.
                
> Add scripts for preCommit testing
> ---------------------------------
>
>                 Key: BOOKKEEPER-471
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-471
>             Project: Bookkeeper
>          Issue Type: Bug
>            Reporter: Ivan Kelly
>            Assignee: Ivan Kelly
>             Fix For: 4.2.0
>
>         Attachments: 0001-BOOKKEEPER-471-Add-scripts-for-preCommit-testing.patch, failTests.patch, failTests.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch, TriggerFindbugsFail.patch
>
>
> We should have jenkins test patches automatically before submission. This JIRA is to set that up.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira