Uploaded image for project: 'Flume'
  1. Flume
  2. FLUME-1787

Implement Pre-commit testing with Jenkins

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 1.3.0
        • Component/s: None
        • Labels:
          None

          Description

          Hadoop does pre-commit testing. It'd be nice if we could do the same. This was discussed here: http://markmail.org/message/ie4zrpv4kviu6n4y

            Issue Links

            is related to

            Task - A task that needs to be done. HIVE-4862 Create automatic Precommit testing

            • Major - Major loss of function.
            • Resolved
            links to

            Web Link Review Board

              Activity

              Descending order - Click to sort in ascending order
              Hide
              Permalink
              hudson Hudson added a comment -

              Integrated in flume-trunk #384 (See https://builds.apache.org/job/flume-trunk/384/)
              FLUME-1787. Implement Pre-commit testing with Jenkins. (Revision c0c4947b295013d051dad98c07b87deaa9427af2)

              Result = FAILURE
              hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=c0c4947b295013d051dad98c07b87deaa9427af2
              Files :

              • pom.xml
              • dev-support/test-patch.py
              • .gitignore
              • atlassian-ide-plugin.xml
              Show
              hudson Hudson added a comment - Integrated in flume-trunk #384 (See https://builds.apache.org/job/flume-trunk/384/ ) FLUME-1787 . Implement Pre-commit testing with Jenkins. (Revision c0c4947b295013d051dad98c07b87deaa9427af2) Result = FAILURE hshreedharan : http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=c0c4947b295013d051dad98c07b87deaa9427af2 Files : pom.xml dev-support/test-patch.py .gitignore atlassian-ide-plugin.xml
              Hide
              Permalink
              brocknoland Brock Noland added a comment -

              Thanks Hari! I did notice that one file, atlassian-ide-plugin got committed and wasn't in the patch AFAIK.

              Show
              brocknoland Brock Noland added a comment - Thanks Hari! I did notice that one file, atlassian-ide-plugin got committed and wasn't in the patch AFAIK.
              Hide
              Permalink
              hshreedharan Hari Shreedharan added a comment -

              Committed to trunk and flume-1.4, rev: c0c4947. Thanks Brock!

              Show
              hshreedharan Hari Shreedharan added a comment - Committed to trunk and flume-1.4, rev: c0c4947. Thanks Brock!
              Hide
              Permalink
              hshreedharan Hari Shreedharan added a comment -

              Looks like it is and does not seem to affect any existing code. +1.

              Show
              hshreedharan Hari Shreedharan added a comment - Looks like it is and does not seem to affect any existing code. +1.
              Hide
              Permalink
              hshreedharan Hari Shreedharan added a comment -

              Hey Brock,

              Is this ready for commit?

              Show
              hshreedharan Hari Shreedharan added a comment - Hey Brock, Is this ready for commit?
              Hide
              Permalink
              brocknoland Brock Noland added a comment -

              FWIW, here is Hadoop's script https://github.com/apache/hadoop-common/blob/trunk/dev-support/test-patch.sh

              Show
              brocknoland Brock Noland added a comment - FWIW, here is Hadoop's script https://github.com/apache/hadoop-common/blob/trunk/dev-support/test-patch.sh
              Hide
              Permalink
              brocknoland Brock Noland added a comment -

              I have a build setup to use the attached script. I have manually tested it but we need to commit it to trunk to do the final test and before enabling our pre-commit build which is located here: https://builds.apache.org/job/PreCommit-FLUME-Build/

              Show
              brocknoland Brock Noland added a comment - I have a build setup to use the attached script. I have manually tested it but we need to commit it to trunk to do the final test and before enabling our pre-commit build which is located here: https://builds.apache.org/job/PreCommit-FLUME-Build/

                People

                • Assignee:
                  brocknoland Brock Noland
                  Reporter:
                  brocknoland Brock Noland
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development