Skip to content

Commit 653cac0

Browse files
committed
Merge remote-tracking branch 'origin/release-candidate-v0.28.0' into fix-build-infra-cm-cs
2 parents 87e110d + 44adbfb commit 653cac0

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

pkg/pipeline/WorkflowService.go

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -175,18 +175,18 @@ func (impl *WorkflowServiceImpl) createWorkflowTemplate(workflowRequest *types.W
175175
}
176176
} else {
177177
if shouldAddExistingCmCsInWorkflow {
178-
workflowConfigMaps, workflowSecrets, err = impl.addExistingCmCsInWorkflowForCDStage(workflowRequest, workflowConfigMaps, workflowSecrets)
178+
cdStageConfigMaps, cdStageSecrets, err := impl.addExistingCmCsInWorkflowForCDStage(workflowRequest)
179179
if err != nil {
180180
impl.Logger.Errorw("error occurred while adding existing cm/ cs", "err", err)
181181
return bean3.WorkflowTemplate{}, err
182182
}
183-
cdStageConfigMaps, cdStageSecrets, err := impl.prepareCmCsForWorkflowTemplate(workflowRequest, workflowConfigMaps, workflowSecrets)
183+
cdStageModifiedConfigMaps, cdStageModifiedSecrets, err := impl.prepareCmCsForWorkflowTemplate(workflowRequest, cdStageConfigMaps, cdStageSecrets)
184184
if err != nil {
185185
impl.Logger.Errorw("error occurred while preparing cd stage cm/ cs for workflow template", "err", err)
186186
return bean3.WorkflowTemplate{}, err
187187
}
188-
workflowConfigMaps = append(workflowConfigMaps, cdStageConfigMaps...)
189-
workflowSecrets = append(workflowSecrets, cdStageSecrets...)
188+
workflowConfigMaps = append(workflowConfigMaps, cdStageModifiedConfigMaps...)
189+
workflowSecrets = append(workflowSecrets, cdStageModifiedSecrets...)
190190
}
191191
}
192192
// internally inducing BlobStorageCmName and BlobStorageSecretName for getting logs, caches and artifacts from
@@ -273,7 +273,9 @@ func (impl *WorkflowServiceImpl) appendGlobalCMCS(workflowRequest *types.Workflo
273273
return workflowConfigMaps, workflowSecrets, nil
274274
}
275275

276-
func (impl *WorkflowServiceImpl) addExistingCmCsInWorkflowForCDStage(workflowRequest *types.WorkflowRequest, workflowConfigMaps []bean.ConfigSecretMap, workflowSecrets []bean.ConfigSecretMap) ([]bean.ConfigSecretMap, []bean.ConfigSecretMap, error) {
276+
func (impl *WorkflowServiceImpl) addExistingCmCsInWorkflowForCDStage(workflowRequest *types.WorkflowRequest) ([]bean.ConfigSecretMap, []bean.ConfigSecretMap, error) {
277+
workflowConfigMaps := make([]bean.ConfigSecretMap, 0)
278+
workflowSecrets := make([]bean.ConfigSecretMap, 0)
277279
existingConfigMap, existingSecrets, err := impl.configMapService.GetCmCsForPrePostStageTrigger(workflowRequest.Scope, workflowRequest.AppId, workflowRequest.EnvironmentId, false)
278280
if err != nil {
279281
impl.Logger.Errorw("failed to get configmap data", "err", err)

scripts/sql/31602800_build_infra_cm_cs.down.sql

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,8 @@ DROP TABLE IF EXISTS "public"."infra_config_trigger_history";
66
-- Drop Sequence for infra_config_trigger_history
77
DROP SEQUENCE IF EXISTS "public"."id_seq_infra_config_trigger_history";
88

9+
--hard deleting the entries
10+
DELETE FROM "public"."infra_profile_configuration"
11+
WHERE key = 8 or key =9;
12+
913
END;

0 commit comments

Comments
 (0)