screwdriver-api 7.0.8 → 7.0.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "screwdriver-api",
-  "version": "7.0.8",
+  "version": "7.0.10",
   "description": "API server for the Screwdriver.cd service",
   "main": "index.js",
   "scripts": {

package/plugins/builds/index.js

@@ -454,7 +454,7 @@ async function updateParentBuilds({ joinParentBuilds, nextBuild, build }) {
     );
 
     nextBuild.parentBuilds = newParentBuilds;
-    nextBuild.parentBuildId = [build.id].concat(nextBuild.parentBuildId || []);
+    nextBuild.parentBuildId = Array.from(new Set([build.id, ...(nextBuild.parentBuildId || [])]));
 
     // FIXME: Is this needed ? Why not update once in handleNewBuild()
     return nextBuild.update();
@@ -683,6 +683,31 @@ async function createJoinObject(nextJobs, current, eventFactory) {
     return joinObj;
 }
 
+/**
+ * Get parentBuildId from parentBuilds object
+ * @param {Object}  parentBuilds    Builds that triggered this build
+ * @param {Array}   joinListNames   Array of join job name
+ * @param {Number}  pipelineId      Pipeline ID
+ * @return {Array}                  Array of parentBuildId
+ */
+function getParentBuildIds({ currentBuildId, parentBuilds, joinListNames, pipelineId }) {
+    const parentBuildIds = [];
+
+    for (let i = 0; i < joinListNames.length; i += 1) {
+        const name = joinListNames[i];
+        const joinInfo = getPipelineAndJob(name, pipelineId);
+
+        if (
+            parentBuilds[joinInfo.externalPipelineId] &&
+            parentBuilds[joinInfo.externalPipelineId].jobs[joinInfo.externalJobName]
+        ) {
+            parentBuildIds.push(parentBuilds[joinInfo.externalPipelineId].jobs[joinInfo.externalJobName]);
+        }
+    }
+
+    return Array.from(new Set([currentBuildId, ...parentBuildIds]));
+}
+
 /**
  * Build API Plugin
  * @method register
@@ -946,7 +971,8 @@ const buildsPlugin = {
 
                             return existingBuild &&
                                 existingBuild.status !== 'CREATED' &&
-                                !existingBuild.parentBuildId.includes(current.build.id)
+                                !existingBuild.parentBuildId.includes(current.build.id) &&
+                                existingBuild.eventId !== current.event.parentEventId
                                 ? existingBuild
                                 : null;
                         })
@@ -1004,6 +1030,9 @@ const buildsPlugin = {
 
                         fillParentBuilds(parentBuilds, current, externalGroupBuilds, externalEvent);
 
+                        const joinList = nextJobs[nextJobName].join;
+                        const joinListNames = joinList.map(j => j.name);
+
                         if (nextBuild) {
                             // update current build info in parentBuilds
                             // nextBuild is not build model, so fetch proper build
@@ -1015,6 +1044,13 @@ const buildsPlugin = {
                         } else {
                             // no existing build, so first time processing this job
                             // in the external pipeline's event
+                            const parentBuildId = getParentBuildIds({
+                                currentBuildId: current.build.id,
+                                parentBuilds,
+                                joinListNames,
+                                pipelineId: externalPipelineId
+                            });
+
                             newBuild = await createInternalBuild({
                                 jobFactory,
                                 buildFactory,
@@ -1026,15 +1062,14 @@ const buildsPlugin = {
                                 event: externalEvent, // this is the parentBuild for the next build
                                 baseBranch: externalEvent.baseBranch || null,
                                 parentBuilds,
-                                parentBuildId: current.build.id,
+                                parentBuildId,
                                 start: false
                             });
                         }
 
-                        const joinList = nextJobs[nextJobName].join;
                         const { hasFailure, done } = await getParentBuildStatus({
                             newBuild,
-                            joinListNames: joinList.map(j => j.name),
+                            joinListNames,
                             pipelineId: externalPipelineId,
                             buildFactory
                         });