From 812677d45dc62b578a3139a7c3cdb01527d6a272 Mon Sep 17 00:00:00 2001 From: Yavor16 Date: Fri, 14 Aug 2026 09:18:49 +0300 Subject: [PATCH 1/3] Replace inheritVariables with explicit in-mappings for undeployAppSubProcess All 5 call sites of undeployAppSubProcess across 4 BPMN files previously used flowable:inheritVariables="true", which implicitly passes the entire parent process variable scope into the subprocess. This creates hidden coupling and can cause variables to leak unintentionally between process instances in multi-instance scenarios. Replace with flowable:inheritVariables="false" and an explicit list of 27 flowable:in variable mappings on each call site, consistent with the same pattern applied to the processBatches subprocess in #1893. Affected files: - rollback-mta.bpmn (1 call site) - xs2-bg-deploy.bpmn (2 call sites) - xs2-deploy.bpmn (1 call site) - xs2-undeploy.bpmn (1 call site) JIRA:LMCROSSITXSADEPLOY-3028 --- .../controller/process/rollback-mta.bpmn | 31 +++++++++- .../controller/process/xs2-bg-deploy.bpmn | 62 ++++++++++++++++++- .../controller/process/xs2-deploy.bpmn | 31 +++++++++- .../controller/process/xs2-undeploy.bpmn | 31 +++++++++- 4 files changed, 150 insertions(+), 5 deletions(-) diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn index 40d93d6806..0faaa19b6f 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn @@ -56,7 +56,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn index cb1efe93d7..235bc40692 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn @@ -125,7 +125,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + @@ -136,7 +165,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn index a0b6d112e9..0797b5be0f 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn @@ -105,7 +105,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn index 5f2e7175a0..726b5d2366 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn @@ -63,7 +63,36 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + From 23694cbca6a3598ad0757c762278063d48862ba5 Mon Sep 17 00:00:00 2001 From: Yavor16 Date: Fri, 14 Aug 2026 11:08:22 +0300 Subject: [PATCH 2/3] Add missing appToProcess to undeployAppSubProcess flowable:in mappings The appToProcess variable is set as the multi-instance elementVariable by the parent call activity (flowable:elementVariable="appToProcess"), but with inheritVariables="false" it must be listed explicitly in flowable:in to be available inside the subprocess. Without it, UndeployAppStep.executeStepInternal() reads a null APP_TO_PROCESS, causing a NullPointerException in getStepErrorMessage() when DeleteApplicationRoutesStep runs. JIRA:LMCROSSITXSADEPLOY-3028 --- .../cloudfoundry/multiapps/controller/process/rollback-mta.bpmn | 1 + .../multiapps/controller/process/xs2-bg-deploy.bpmn | 2 ++ .../cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn | 1 + .../cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn | 1 + 4 files changed, 5 insertions(+) diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn index 0faaa19b6f..594adc978e 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/rollback-mta.bpmn @@ -85,6 +85,7 @@ + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn index 235bc40692..554b005ce6 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-bg-deploy.bpmn @@ -154,6 +154,7 @@ + @@ -194,6 +195,7 @@ + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn index 0797b5be0f..768800e546 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-deploy.bpmn @@ -134,6 +134,7 @@ + diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn index 726b5d2366..591335fca0 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn @@ -92,6 +92,7 @@ + From 76ee27b2c97ba9da51faffda72f09debc07078d5 Mon Sep 17 00:00:00 2001 From: Yavor16 Date: Tue, 18 Aug 2026 09:44:05 +0300 Subject: [PATCH 3/3] fix comment from Stoyan --- .../multiapps/controller/process/xs2-undeploy.bpmn | 3 --- 1 file changed, 3 deletions(-) diff --git a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn index 591335fca0..f7a09120d7 100644 --- a/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn +++ b/multiapps-controller-process/src/main/resources/org/cloudfoundry/multiapps/controller/process/xs2-undeploy.bpmn @@ -89,9 +89,6 @@ - - -