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

Performance plugin lock out Jenkins while running Performance Trend

    Details

    • Similar Issues:

      Description

      When running Performance Trend, Performance plugin lock Jenkins.

      I could see on log :

      
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-100.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-100.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-001.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-001.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-100.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-001.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-001.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-001.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-050.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-010.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-200.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-200.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-200.jtl
      Performance: Parsing JMeter report file jmeterlog-PLF_PERF_03_EcmsBasicRun-200.jtl
      

      CPU is also very high (100%)

      8610 jenkins  20   0 1253m 945m 9.9m S  100  7.9  32:35.24 exe                                                                                   
      

      Plugin is scanning all results files, a very consuming task.
      Why not produce a performance resume for each job, so this resume could be reused, much faster, later in Performance Trend ?

      For now Performance Plugin is no more usable for us since any click could lock up completly Jenkins.

        Attachments

          Issue Links

            Activity

            Hide
            manolo Manuel Carrasco added a comment -

            Actually it is not difficult at all to me, the thing is that I prefer having you as committer of the patch in git history, and it is the preferred way in Jenkins contributions. I don't have any problem doing it in your way.

            Show
            manolo Manuel Carrasco added a comment - Actually it is not difficult at all to me, the thing is that I prefer having you as committer of the patch in git history, and it is the preferred way in Jenkins contributions. I don't have any problem doing it in your way.
            Hide
            mcfly Michel Marti added a comment -

            But a git commit contains both the author and the committer, and it's the author the gets displayed by default when browsing git history. Am I missing something here?

            Show
            mcfly Michel Marti added a comment - But a git commit contains both the author and the committer, and it's the author the gets displayed by default when browsing git history. Am I missing something here?
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Michel Marti
            Path:
            src/main/java/hudson/plugins/performance/HttpSample.java
            src/main/java/hudson/plugins/performance/JMeterParser.java
            src/main/java/hudson/plugins/performance/PerformanceReport.java
            src/main/java/hudson/plugins/performance/UriReport.java
            http://jenkins-ci.org/commit/performance-plugin/be7401b0284466b9f10815d5215e01d33771528f
            Log:
            Cache preprocessed JMeter Reports (JENKINS-9031)

            Instead of parsing all available JTL results upon each request,
            we now save the parsed report as a serialized Java object for later
            reuse. The file will be next to the JTL file with a '.serialized'
            suffix. Additionally the report also gets added to a in-memory cache
            with a maximum capacity of 100 reports. This considerably speeds up the
            performance trend display.

            Signed-off-by: Michel Marti <mma@objectxp.com>

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Michel Marti Path: src/main/java/hudson/plugins/performance/HttpSample.java src/main/java/hudson/plugins/performance/JMeterParser.java src/main/java/hudson/plugins/performance/PerformanceReport.java src/main/java/hudson/plugins/performance/UriReport.java http://jenkins-ci.org/commit/performance-plugin/be7401b0284466b9f10815d5215e01d33771528f Log: Cache preprocessed JMeter Reports ( JENKINS-9031 ) Instead of parsing all available JTL results upon each request, we now save the parsed report as a serialized Java object for later reuse. The file will be next to the JTL file with a '.serialized' suffix. Additionally the report also gets added to a in-memory cache with a maximum capacity of 100 reports. This considerably speeds up the performance trend display. Signed-off-by: Michel Marti <mma@objectxp.com>
            Hide
            manolo Manuel Carrasco added a comment -

            You are right, I didn't realize that your patch was done with git-diff.

            Merged: https://github.com/jenkinsci/performance-plugin/commits/master

            Show
            manolo Manuel Carrasco added a comment - You are right, I didn't realize that your patch was done with git-diff. Merged: https://github.com/jenkinsci/performance-plugin/commits/master
            Hide
            manolo Manuel Carrasco added a comment -

            Fixed in git

            Show
            manolo Manuel Carrasco added a comment - Fixed in git

              People

              • Assignee:
                manolo Manuel Carrasco
                Reporter:
                hgomez Henri Gomez
              • Votes:
                7 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: