feat: [KSBP-101776] - Add flag for parallelTestApp

Author: kindredjcharters

k8s-deployer/src/pitfile/schema-v1.ts

@@ -67,6 +67,12 @@ export class DeployableComponent {
 export class Graph {
   testApp: DeployableComponent
   components: Array<DeployableComponent>
+  /**
+   * When true, the test app is deployed concurrently with the component chain.
+   * Only enable this if the test app can start independently of the components.
+   * Defaults to false (sequential: test app starts after all components are ready).
+   */
+  parallelTestApp?: boolean
 }
 
 export class Deployment {

k8s-deployer/src/test-suite-handler.ts

@@ -103,13 +103,23 @@ export const deployGraph = async (config: Config, workspace: string, testSuiteId
     }
   })()
 
-  // Deploy test app concurrently with the component levels; deployGraph does not return until both are done.
   logger.info("%s Deploying test app \"%s\" for suite \"%s\" %s", LOG_SEPARATOR_LINE, graph.testApp.name, testSuiteId, LOG_SEPARATOR_LINE)
   const params = [ testSuiteId ]
-  const deployTestAppPromise = Deployer.deployComponent(config, workspace, graph.testApp, namespace, params)
-    .then(commitSha => new DeployedComponent(commitSha, graph.testApp))
 
-  const [ , testAppDeployedComponent] = await Promise.all([deployComponentsPromise, deployTestAppPromise])
+  let testAppDeployedComponent: DeployedComponent
+  if (graph.parallelTestApp === true) {
+    // Deploy test app concurrently with the component levels (opt-in).
+    // Only use this when the test app can start independently of the components.
+    const deployTestAppPromise = Deployer.deployComponent(config, workspace, graph.testApp, namespace, params)
+      .then(commitSha => new DeployedComponent(commitSha, graph.testApp))
+    const [ , resolved] = await Promise.all([deployComponentsPromise, deployTestAppPromise])
+    testAppDeployedComponent = resolved
+  } else {
+    // Default: wait for all components to be ready before deploying the test app.
+    await deployComponentsPromise
+    const commitSha = await Deployer.deployComponent(config, workspace, graph.testApp, namespace, params)
+    testAppDeployedComponent = new DeployedComponent(commitSha, graph.testApp)
+  }
 
   logger.info("")
   return new GraphDeploymentResult(deployments, testAppDeployedComponent)

k8s-deployer/test/test-suite-handler.spec.ts

@@ -57,6 +57,7 @@ describe("Deployment happy path", async () => {
     location: { type: LocationType.Local },
     deployment: {
       graph: {
+        parallelTestApp: true,
         testApp: {
           name: "comp-1-test-app-name",
           id: "comp-1-test-app",
@@ -498,6 +499,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     const deployStub = sinon.stub().callsFake(async (_cfg, _ws, spec) => `sha-${spec.id}`)
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
+      parallelTestApp: true,
       testApp: makeSpec("test-app"),
       components: [makeSpec("comp-a"), makeSpec("comp-b")]
     }
@@ -520,6 +522,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
+      parallelTestApp: true,
       testApp: makeSpec("testApp"),
       components: [makeSpec("B", { parallel: true }), makeSpec("C", { parallel: true })]
     }
@@ -549,6 +552,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     const SuiteHandler = await loadWithStub(deployStub)
     // B is parallel:true, C has no parallel flag — same dependency level, so neither waits on the other
     const graph = {
+      parallelTestApp: true,
       testApp: makeSpec("testApp"),
       components: [makeSpec("B", { parallel: true }), makeSpec("C")]
     }
@@ -579,6 +583,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
+      parallelTestApp: true,
       testApp: makeSpec("testApp"),
       components: [
         makeSpec("A"),
@@ -624,6 +629,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
+      parallelTestApp: true,
       testApp: makeSpec("testApp"),
       components: [makeSpec("A")]
     }