remove overwriteConfig

fix
cdapio · Jul 19, 2023 · 0682de3 · 0682de3
1 parent 89250f7
commit 0682de3
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 8 deletions.
diff --git a/...onents/PipelineConfigurations/ConfigurationsContent/ConfigModelessActionButtons/index.tsx b/...onents/PipelineConfigurations/ConfigurationsContent/ConfigModelessActionButtons/index.tsx
@@ -89,7 +89,7 @@ export const ConfigModelessActionButtons = ({ ...props }: IConfigModelessActionB
     setSaveLoading(true);
     let observable;
     if (lifecycleManagementEditEnabled) {
-      observable = updatePreferences(lifecycleManagementEditEnabled);
+      observable = updatePreferences();
     } else {
       observable = Observable.forkJoin(updatePipeline(), updatePreferences());
     }

diff --git a/app/cdap/components/PipelineConfigurations/Store/ActionCreator.js b/app/cdap/components/PipelineConfigurations/Store/ActionCreator.js
@@ -132,17 +132,14 @@ const getMacrosResolvedByPrefs = (resolvedPrefs = {}, macrosMap = {}) => {
   return resolvedMacros;
 };
 
-const updatePreferences = (lifecycleManagementEditEnabled = false) => {
+const updatePreferences = () => {
   const { runtimeArgs } = PipelineConfigurationsStore.getState();
   let filteredRuntimeArgs = cloneDeep(runtimeArgs);
   filteredRuntimeArgs.pairs = filteredRuntimeArgs.pairs.filter(
     (runtimeArg) => !runtimeArg.provided
   );
   let appId = PipelineDetailStore.getState().name;
   let prefObj = convertKeyValuePairsObjToMap(runtimeArgs);
-  if (lifecycleManagementEditEnabled) {
-    prefObj = { ...prefObj, 'app.pipeline.overwriteConfig': 'true' };
-  }
 
   return MyPreferenceApi.setAppPreferences(
     {

diff --git a/app/cdap/services/global-constants.js b/app/cdap/services/global-constants.js
@@ -358,7 +358,6 @@ const GENERATED_RUNTIMEARGS = {
   SYSTEM_DRIVER_RESOURCES_CORES: 'task.driver.system.resources.cores',
   SYSTEM_EXECUTOR_RESOURCES_MEMORY: 'task.executor.system.resources.memory',
   SYSTEM_EXECUTOR_RESOURCES_CORES: 'task.executor.system.resources.cores',
-  PIPELINE_CONFIG_OVERWRITE: 'app.pipeline.overwriteConfig',
 }
 
 const SCOPES = {

diff --git a/src/e2e-test/java/io/cdap/cdap/ui/stepsdesign/PipelineEdit.java b/src/e2e-test/java/io/cdap/cdap/ui/stepsdesign/PipelineEdit.java
@@ -204,7 +204,6 @@ public void checkRuntimeArgsEmpty() {
       .map(element -> element.findElement(By.tagName("input")).getAttribute("value"))
       .collect(Collectors.toList());
     Assert.assertTrue(stringKeys.size() == 1);
-    Assert.assertTrue(!stringKeys.contains("app.pipeline.overwriteConfig"));
     Assert.assertTrue(!stringKeys.contains("app.pipeline.instrumentation"));
     ElementHelper.clickOnElement(Helper.locateElementByTestId("pipeline-modeless-close-btn"));
   }
@@ -220,7 +219,6 @@ public void checkRuntimeArgsNotEmpty() {
       .map(element -> element.findElement(By.tagName("input")).getAttribute("value"))
       .collect(Collectors.toList());
     Assert.assertTrue(stringKeys.size() > 1);
-    Assert.assertTrue(stringKeys.contains("app.pipeline.overwriteConfig"));
     Assert.assertTrue(stringKeys.contains("app.pipeline.instrumentation"));
     ElementHelper.clickOnElement(Helper.locateElementByTestId("pipeline-modeless-close-btn"));
   }