Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-38604

Filter class directories

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • jacoco-plugin
    • None
    • Jenkins 2.7.3 with JaCoCo plugin 2.0.1

      When using the post build step to publish JaCoCo coverage the plugin takes all files in the **/classes directory.
      Having jar files with large resource files leads to xxxGB of wasted space on the Jenkins server, because this is stored for every build job.

      The plugin should only publish *.class files to the jobs\xxx\builds\123\jacoco\classes directory.

          [JENKINS-38604] Filter class directories

          Martin Heinzerling added a comment - - edited

          In addition to that, only *,java should be copied to sources.

           

          centic, is there any scenario where we need jars in the classes path. Do we need to make this configurable?

          Martin Heinzerling added a comment - - edited In addition to that, only *,java should be copied to sources.   centic , is there any scenario where we need jars in the classes path. Do we need to make this configurable?

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Martin Heinzerling
          Path:
          src/main/java/hudson/plugins/jacoco/JacocoPublisher.java
          src/main/java/hudson/plugins/jacoco/JacocoReportDir.java
          src/test/java/hudson/plugins/jacoco/JacocoPublisherTest.java
          http://jenkins-ci.org/commit/jacoco-plugin/d771e0b15933516917208f30dd54bad81b1c290d
          Log:
          JENKINS-38604: limit copying of files to types *.java and *.class, improve logging

          Compare: https://github.com/jenkinsci/jacoco-plugin/compare/370c2774bb71^...d771e0b15933

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Martin Heinzerling Path: src/main/java/hudson/plugins/jacoco/JacocoPublisher.java src/main/java/hudson/plugins/jacoco/JacocoReportDir.java src/test/java/hudson/plugins/jacoco/JacocoPublisherTest.java http://jenkins-ci.org/commit/jacoco-plugin/d771e0b15933516917208f30dd54bad81b1c290d Log: JENKINS-38604 : limit copying of files to types *.java and *.class, improve logging Compare: https://github.com/jenkinsci/jacoco-plugin/compare/370c2774bb71 ^...d771e0b15933

          Martin Heinzerling added a comment -

          centic, please review.

          Martin Heinzerling added a comment - centic , please review.

          centic added a comment -

          Looks good to merge to master!

          centic added a comment - Looks good to merge to master!

          Martin Heinzerling added a comment -

          Integrated into master.

          Martin Heinzerling added a comment - Integrated into master.

          John Slater added a comment -

          This change has prevented .groovy files from being copied over. The coverage report was previously displayed for groovy files before this change.

          John Slater added a comment - This change has prevented .groovy files from being copied over. The coverage report was previously displayed for groovy files before this change.

          centic added a comment -

          See JENKINS-56918 for adjusting the inclusion pattern some more.

          centic added a comment - See JENKINS-56918 for adjusting the inclusion pattern some more.

            mheinzerling Martin Heinzerling
            reitzmichnicht Michael Düsterhus
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: