diff --git a/core/src/main/java/hudson/model/AbstractProject.java b/core/src/main/java/hudson/model/AbstractProject.java index 19b973bbb9..d12d1d3859 100644 --- a/core/src/main/java/hudson/model/AbstractProject.java +++ b/core/src/main/java/hudson/model/AbstractProject.java @@ -174,6 +174,12 @@ public abstract class AbstractProject

,R extends A */ protected volatile boolean disabled; + /** + * True to keep builds of this project in queue when downstream projects are + * building. False by default to keep from breaking existing behavior. + */ + protected volatile boolean blockBuildWhenDownstreamBuilding = false; + /** * True to keep builds of this project in queue when upstream projects are * building. False by default to keep from breaking existing behavior. @@ -500,6 +506,15 @@ public abstract class AbstractProject

,R extends A return true; } + public boolean blockBuildWhenDownstreamBuilding() { + return blockBuildWhenDownstreamBuilding; + } + + public void setBlockBuildWhenDownstreamBuilding(boolean b) throws IOException { + blockBuildWhenDownstreamBuilding = b; + save(); + } + public boolean blockBuildWhenUpstreamBuilding() { return blockBuildWhenUpstreamBuilding; } @@ -998,6 +1013,22 @@ public abstract class AbstractProject

,R extends A return Messages.AbstractProject_BuildInProgress(lbn, eta); } } + + /** + * Because the downstream build is in progress, and we are configured to wait for that. + */ + public static class BecauseOfDownstreamBuildInProgress extends CauseOfBlockage { + public final AbstractProject up; + + public BecauseOfDownstreamBuildInProgress(AbstractProject up) { + this.up = up; + } + + @Override + public String getShortDescription() { + return Messages.AbstractProject_DownstreamBuildInProgress(up.getName()); + } + } /** * Because the upstream build is in progress, and we are configured to wait for that. @@ -1018,7 +1049,11 @@ public abstract class AbstractProject

,R extends A public CauseOfBlockage getCauseOfBlockage() { if (isBuilding() && !isConcurrentBuild()) return new BecauseOfBuildInProgress(getLastBuild()); - if (blockBuildWhenUpstreamBuilding()) { + if (blockBuildWhenDownstreamBuilding()) { + AbstractProject bup = getBuildingDownstream(); + if (bup!=null) + return new BecauseOfDownstreamBuildInProgress(bup); + } else if (blockBuildWhenUpstreamBuilding()) { AbstractProject bup = getBuildingUpstream(); if (bup!=null) return new BecauseOfUpstreamBuildInProgress(bup); @@ -1026,6 +1061,24 @@ public abstract class AbstractProject

,R extends A return null; } + /** + * Returns the project if any of the downstream project (or itself) is either + * building or is in the queue. + *

+ * This means eventually there will be an automatic triggering of + * the given project (provided that all builds went smoothly.) + */ + protected AbstractProject getBuildingDownstream() { + DependencyGraph graph = Hudson.getInstance().getDependencyGraph(); + Set tups = graph.getTransitiveDownstream(this); + tups.add(this); + for (AbstractProject tup : tups) { + if(tup!=this && (tup.isBuilding() || tup.isInQueue())) + return tup; + } + return null; + } + /** * Returns the project if any of the upstream project (or itself) is either * building or is in the queue. @@ -1570,6 +1623,7 @@ public abstract class AbstractProject

,R extends A } else { scmCheckoutRetryCount = null; } + blockBuildWhenDownstreamBuilding = req.getParameter("blockBuildWhenDownstreamBuilding")!=null; blockBuildWhenUpstreamBuilding = req.getParameter("blockBuildWhenUpstreamBuilding")!=null; if(req.getParameter("hasSlaveAffinity")!=null) { diff --git a/core/src/main/resources/hudson/model/AbstractItem/configure-common.jelly b/core/src/main/resources/hudson/model/AbstractItem/configure-common.jelly index fb40c91665..cb33be6aa6 100644 --- a/core/src/main/resources/hudson/model/AbstractItem/configure-common.jelly +++ b/core/src/main/resources/hudson/model/AbstractItem/configure-common.jelly @@ -61,6 +61,7 @@ THE SOFTWARE. + diff --git a/core/src/main/resources/hudson/model/Messages.properties b/core/src/main/resources/hudson/model/Messages.properties index a8b058517e..471a956882 100644 --- a/core/src/main/resources/hudson/model/Messages.properties +++ b/core/src/main/resources/hudson/model/Messages.properties @@ -33,6 +33,7 @@ AbstractProject.Pronoun=Project AbstractProject.Aborted=Aborted AbstractProject.BuildInProgress=Build #{0} is already in progress{1} AbstractProject.UpstreamBuildInProgress=Upstream project {0} is already building. +AbstractProject.DownstreamBuildInProgress=Downstream project {0} is already building. AbstractProject.Disabled=Build disabled AbstractProject.ETA=\ (ETA:{0}) AbstractProject.NoBuilds=No existing build. Scheduling a new one. @@ -289,4 +290,4 @@ CLI.quiet-down.shortDescription=Quiet down Hudson, in preparation for a restart. CLI.cancel-quiet-down.shortDescription=Cancel the effect of the "quiet-down" command. CLI.reload-configuration.shortDescription=Discard all the loaded data in memory and reload everything from file system. Useful when you modified config files directly on disk. -BuildAuthorizationToken.InvalidTokenProvided=Invalid token provided. \ No newline at end of file +BuildAuthorizationToken.InvalidTokenProvided=Invalid token provided. diff --git a/core/src/main/resources/lib/hudson/project/config-blockWhenDownstreamBuilding.jelly b/core/src/main/resources/lib/hudson/project/config-blockWhenDownstreamBuilding.jelly new file mode 100644 index 0000000000..04400e5cd9 --- /dev/null +++ b/core/src/main/resources/lib/hudson/project/config-blockWhenDownstreamBuilding.jelly @@ -0,0 +1,31 @@ + + + + + + diff --git a/war/src/main/webapp/help/project-config/block-downstream-building.html b/war/src/main/webapp/help/project-config/block-downstream-building.html new file mode 100644 index 0000000000..c76433a373 --- /dev/null +++ b/war/src/main/webapp/help/project-config/block-downstream-building.html @@ -0,0 +1,3 @@ +

+ When this option is checked, Hudson will prevent the project from building when a child of this project is in the queue, or building. The children include the direct as well as the transitive children. +