Generate performance report even if previous builds are still running

Fixes JENKINS-26515
jenkinsci · Jan 31, 2015 · e02e148 · e02e148
1 parent 76e2d5f
commit e02e148
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/main/java/hudson/plugins/performance/PerformancePublisher.java b/src/main/java/hudson/plugins/performance/PerformancePublisher.java
@@ -186,7 +186,7 @@ public Action getProjectAction(AbstractProject<?, ?> project) {
   }
 
   public BuildStepMonitor getRequiredMonitorService() {
-    return BuildStepMonitor.BUILD;
+    return BuildStepMonitor.NONE;
   }
 
   public List<PerformanceReportParser> getParsers() {

diff --git a/src/main/java/hudson/plugins/performance/PerformanceReportMap.java b/src/main/java/hudson/plugins/performance/PerformanceReportMap.java
@@ -195,7 +195,7 @@ public void addAll(Collection<PerformanceReport> parse) {
               }
             }
           }, parameter);
-      previousBuild = previousBuild.getPreviousBuild();
+      previousBuild = previousBuild.getPreviousCompletedBuild();
     }
     // Now we should have the data necessary to generate the graphs!
     DataSetBuilder<String, NumberOnlyBuildLabel> dataSetBuilderAverage = new DataSetBuilder<String, NumberOnlyBuildLabel>();
@@ -231,7 +231,7 @@ public void addAll(Collection<PerformanceReport> parse) {
               }
             }
           }, parameter);
-      previousBuild = previousBuild.getPreviousBuild();
+      previousBuild = previousBuild.getPreviousCompletedBuild();
     }
     DataSetBuilder<NumberOnlyBuildLabel, String> dataSetBuilderSummarizer = new DataSetBuilder<NumberOnlyBuildLabel, String>();
     for (AbstractBuild<?, ?> currentBuild : buildReports.keySet()) {
@@ -317,7 +317,7 @@ private void addPreviousBuildReports() {
       }
     }
 
-    AbstractBuild<?, ?> previousBuild = getBuild().getPreviousBuild();
+    AbstractBuild<?, ?> previousBuild = getBuild().getPreviousCompletedBuild();
     if (previousBuild == null) {
       return;
     }