Details

    • Type: Bug
    • Status: Resolved
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.0.0, 1.1.0
    • Component/s: build
    • Labels:

      Description

      The following files are missing ASL header

        bigtop-deploy/puppet/modules/hue/templates/hue.ini
        bigtop-deploy/puppet/modules/tachyon/templates/log4j.properties
        bigtop-deploy/puppet/modules/tachyon/templates/tachyon-env.sh
        bigtop-deploy/vm/utils/smoke-tests.sh
        bigtop-deploy/vm/vagrant-puppet-docker/boot2docker/Vagrantfile
        bigtop-deploy/vm/vagrant-puppet-docker/vagrantconfig.yaml
        bigtop-deploy/vm/vagrant-puppet-docker/vagrantconfig_debian.yaml
        bigtop-deploy/vm/vagrant-puppet-docker/docker-hadoop.sh
        bigtop-deploy/vm/vagrant-puppet-vm/vagrantconfig.yaml
        bigtop-deploy/vm/vagrant-puppet-openstack/para-provision.sh
        bigtop-deploy/vm/vagrant-puppet-openstack/vagrantconfig.yaml
        bigtop-ci/jenkins/jobsCreator.groovy
        MAINTAINERS.txt
        docker/bigtop-puppet/centos-6/Dockerfile
        docker/bigtop-puppet/centos-6/build.sh
        docker/bigtop-puppet/centos-7/Dockerfile
        docker/bigtop-puppet/centos-7/build.sh
        docker/bigtop-puppet/debian-8/Dockerfile
        docker/bigtop-puppet/debian-8/build.sh
        docker/bigtop-puppet/fedora-20/Dockerfile
        docker/bigtop-puppet/fedora-20/build.sh
        docker/bigtop-puppet/opensuse-13.2/Dockerfile
        docker/bigtop-puppet/opensuse-13.2/build.sh
        docker/bigtop-puppet/ubuntu-14.04/Dockerfile
        docker/bigtop-puppet/ubuntu-14.04/build.sh
        bigtop-bigpetstore/README.md
        bigtop-bigpetstore/bigpetstore-data-generator/README.md
        bigtop-bigpetstore/bigpetstore-spark/README.md
        bigtop-bigpetstore/bigpetstore-spark/src/test/scala/org/apache/bigpetstore/spark/TestFullPipeline.scala
        bigtop-bigpetstore/bigpetstore-transaction-queue/Dockerfile
        bigtop-bigpetstore/bigpetstore-transaction-queue/README.md
        bigtop-bigpetstore/bigpetstore-transaction-queue/build.gradle
      
      1. BIGTOP-1951.v1.1.patch
        38 kB
        Konstantin Boudnik
      2. BIGTOP-1951.v1.0.patch
        22 kB
        Konstantin Boudnik

        Issue Links

          Activity

          Hide
          rnowling RJ Nowling added a comment - - edited

          Hi Konstantin Boudnik,

          Do we need ASL headers in README files / docs (if they are read directly, not compiled)?

          Other projects don't seem to require ASL headers in docs.

          RJ

          Show
          rnowling RJ Nowling added a comment - - edited Hi Konstantin Boudnik , Do we need ASL headers in README files / docs (if they are read directly, not compiled)? Other projects don't seem to require ASL headers in docs. RJ
          Hide
          cos Konstantin Boudnik added a comment -

          Yeah, README.txt are excluded from the RAT check automatically. *.md files aren't. And as they are interpreted - let's have the license there as well.

          Show
          cos Konstantin Boudnik added a comment - Yeah, README.txt are excluded from the RAT check automatically. *.md files aren't. And as they are interpreted - let's have the license there as well.
          Hide
          cos Konstantin Boudnik added a comment -

          Patches for branch 1.0 and branch 1.1
          Please review as I want to add 1.0 fix for the license so we don't have to deal with this issue anymore.

          Thank you!

          Show
          cos Konstantin Boudnik added a comment - Patches for branch 1.0 and branch 1.1 Please review as I want to add 1.0 fix for the license so we don't have to deal with this issue anymore. Thank you!
          Hide
          rnowling RJ Nowling added a comment -

          Reviewed +1.

          Show
          rnowling RJ Nowling added a comment - Reviewed +1.
          Hide
          cos Konstantin Boudnik added a comment -

          Fixed and pushed to 1.0 and master branches

          Show
          cos Konstantin Boudnik added a comment - Fixed and pushed to 1.0 and master branches

            People

            • Assignee:
              cos Konstantin Boudnik
              Reporter:
              cos Konstantin Boudnik
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development