You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Ismael Juma (JIRA)" <ji...@apache.org> on 2016/11/29 08:42:59 UTC

[jira] [Created] (KAFKA-4459) rat license check not running in Jenkins

Ismael Juma created KAFKA-4459:
----------------------------------

             Summary: rat license check not running in Jenkins
                 Key: KAFKA-4459
                 URL: https://issues.apache.org/jira/browse/KAFKA-4459
             Project: Kafka
          Issue Type: Bug
            Reporter: Ismael Juma


The following commit introduced files without a license, which causes rat to fail when run locally:

https://github.com/apache/kafka/commit/e035fc039598127e88f31739458f705290b1fdba

However, the build passed in Jenkins:

https://builds.apache.org/job/kafka-trunk-jdk7/1710/

Not sure why, but a potential culprit is the check for `.git` in the build:

{code}
if (new File('.git').exists()) {
  apply from: file('gradle/rat.gradle')
  rat {
    // Exclude everything under the directory that git should be ignoring via .gitignore or that isn't checked in. These
    // restrict us only to files that are checked in or are staged.
    def repo = Grgit.open(project.file('.'))
    excludes = new ArrayList<String>(repo.clean(ignore: false, directories: true, dryRun: true))
    // And some of the files that we have checked in should also be excluded from this check
    excludes.addAll([
        '**/.git/**',
        '**/build/**',
        'CONTRIBUTING.md',
        'gradlew',
        'gradlew.bat',
        '**/README.md'
    ])
  }
}
{code}

If the current directory in Jenkins is not the repo directory, it could fail.





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)