Skip to content
Permalink
Browse files
[FIXED JENKINS-28585] Skips aggregation processing if no publisher su…
…pports aggregation.
  • Loading branch information
ikedam committed Jun 3, 2015
1 parent 0dc77cf commit ab0a50453b112f04a0c3b5a0ce68e77aae0ffdf9
Showing 1 changed file with 13 additions and 13 deletions.
@@ -207,26 +207,26 @@ public boolean perform(final AbstractBuild<?, ?> build, final Launcher launcher,
}

public ConditionalMatrixAggregator createAggregator(MatrixBuild build, Launcher launcher, BuildListener listener) {
if (isConfiguredAggregation()) {
// alerts if all publishers doesn't support aggregation
// even if configured for aggregation
boolean supportAggregation = false;

for (BuildStep publisher: getPublisherList()) {
if (publisher instanceof MatrixAggregatable) {
supportAggregation = true;
break;
}
boolean supportAggregation = false;

for (BuildStep publisher: getPublisherList()) {
if (publisher instanceof MatrixAggregatable) {
supportAggregation = true;
break;
}

if (!supportAggregation) {
}

if (!supportAggregation) {
if (isConfiguredAggregation()) {
// alerts if all publishers doesn't support aggregation
// even if configured for aggregation
listener.getLogger().println(String.format(
"[%s] WARNING: Condition for Matrix Aggregation is configured for %s which does not support aggregation",
getDescriptor().getDisplayName(),
FlexiblePublisher.getBuildStepShortName(getPublisherList())
));
return null;
}
return null;
}
// First, decide whether the condition is satisfied
// in the parent scope.

0 comments on commit ab0a504

Please sign in to comment.