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

Filter class directories

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Reopened (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Component/s: jacoco-plugin
    • Labels:
      None
    • Environment:
      Jenkins 2.7.3 with JaCoCo plugin 2.0.1
    • Similar Issues:

      Description

      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.

        Attachments

          Activity

          Hide
          mheinzerling 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?

          Show
          mheinzerling 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?
          Hide
          scm_issue_link 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

          Show
          scm_issue_link 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
          Hide
          mheinzerling Martin Heinzerling added a comment -

          centic, please review.

          Show
          mheinzerling Martin Heinzerling added a comment - centic , please review.
          Hide
          centic centic added a comment -

          Looks good to merge to master!

          Show
          centic centic added a comment - Looks good to merge to master!
          Hide
          mheinzerling Martin Heinzerling added a comment -

          Integrated into master.

          Show
          mheinzerling Martin Heinzerling added a comment - Integrated into master.
          Hide
          john_slater 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.

          Show
          john_slater 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.

            People

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

              Dates

              • Created:
                Updated: