feat: refactoring deployment app name usage

api/bean/AppView.go

@@ -144,6 +144,7 @@ type AppEnvironmentContainer struct {
 type DeploymentDetailContainer struct {
 	InstalledAppId                int                    `json:"installedAppId,omitempty"`
 	AppId                         int                    `json:"appId,omitempty"`
+	PcoId                         int                    `json:"pcoId"`
 	CdPipelineId                  int                    `json:"cdPipelineId,omitempty"`
 	TriggerType                   string                 `json:"triggerType,omitempty"`
 	ParentEnvironmentName         string                 `json:"parentEnvironmentName"`
@@ -183,6 +184,8 @@ type DeploymentDetailContainer struct {
 	HelmPackageName               string                 `json:"helmPackageName"`
 	HelmReleaseInstallStatus      string                 `json:"-"`
 	DeploymentConfig              *bean.DeploymentConfig `json:"-"`
+	IsPipelineTriggered           bool                   `json:"isPipelineTriggered"`
+	ReleaseMode                   string                 `json:"releaseMode"`
 }
 
 type AppDetailContainer struct {

api/helm-app/gRPC/applist.proto

@@ -44,6 +44,7 @@ service ApplicationService {
   rpc ValidateOCIRegistry(RegistryCredential) returns(OCIRegistryResponse) {}
   rpc GetResourceTreeForExternalResources(ExternalResourceTreeRequest) returns(ResourceTreeResponse){}
   rpc GetFluxAppDetail(FluxAppDetailRequest)returns(FluxAppDetail){}
+  rpc  GetReleaseDetails(ReleaseIdentifier)returns(DeployedAppDetail){}
 
 }
 

api/helm-app/service/HelmAppService.go

@@ -1136,7 +1136,7 @@ func (impl *HelmAppServiceImpl) appListRespProtoTransformer(deployedApps *gRPC.D
 			// do not add app in the list which are created using cd_pipelines (check combination of clusterId, namespace, releaseName)
 			var toExcludeFromList bool
 			for _, helmCdPipeline := range helmCdPipelines {
-				helmAppReleaseName := util2.BuildDeployedAppName(helmCdPipeline.App.AppName, helmCdPipeline.Environment.Name)
+				helmAppReleaseName := helmCdPipeline.DeploymentAppName
 				if deployedapp.AppName == helmAppReleaseName && int(deployedapp.EnvironmentDetail.ClusterId) == helmCdPipeline.Environment.ClusterId && deployedapp.EnvironmentDetail.Namespace == helmCdPipeline.Environment.Namespace {
 					toExcludeFromList = true
 					break

api/restHandler/app/pipeline/configure/DeploymentPipelineRestHandler.go

@@ -207,6 +207,11 @@ func (handler *PipelineConfigRestHandlerImpl) CreateCdPipeline(w http.ResponseWr
 	handler.Logger.Infow("request payload, CreateCdPipeline", "payload", cdPipeline)
 	userUploaded, err := handler.chartService.CheckIfChartRefUserUploadedByAppId(cdPipeline.AppId)
 	if !userUploaded {
+		for i, p := range cdPipeline.Pipelines {
+			if len(p.ReleaseMode) == 0 {
+				cdPipeline.Pipelines[i].ReleaseMode = util.PIPELINE_RELEASE_MODE_CREATE
+			}
+		}
 		err = handler.validator.Struct(cdPipeline)
 		if err != nil {
 			handler.Logger.Errorw("validation err, CreateCdPipeline", "err", err, "payload", cdPipeline)

internal/sql/repository/AppListingRepository.go

@@ -365,10 +365,10 @@ func (impl AppListingRepositoryImpl) deploymentDetailsByAppIdAndEnvId(ctx contex
 		" p.ci_pipeline_id," +
 		" p.trigger_type" +
 		" FROM pipeline p" +
-		" INNER JOIN pipeline_config_override pco on pco.pipeline_id=p.id" +
+		" LEFT JOIN pipeline_config_override pco on pco.pipeline_id=p.id" +
 		" INNER JOIN environment env ON env.id=p.environment_id" +
 		" INNER JOIN cluster cl on cl.id=env.cluster_id" +
-		" INNER JOIN ci_artifact cia on cia.id = pco.ci_artifact_id" +
+		" LEFT JOIN ci_artifact cia on cia.id = pco.ci_artifact_id" +
 		" INNER JOIN app a ON a.id=p.app_id" +
 		" WHERE a.app_type = 0 AND a.id=? AND env.id=? AND p.deleted = FALSE AND env.active = TRUE" +
 		" ORDER BY pco.created_on DESC LIMIT 1;"
@@ -378,14 +378,15 @@ func (impl AppListingRepositoryImpl) deploymentDetailsByAppIdAndEnvId(ctx contex
 		return deploymentDetail, err
 	}
 	deploymentDetail.EnvironmentId = envId
-	if len(deploymentDetail.DeploymentAppType) == 0 {
-		dc, err := impl.deploymentConfigRepository.GetByAppIdAndEnvId(appId, envId)
-		if err != nil {
-			impl.Logger.Errorw("error in getting deployment config by appId and envId", "appId", appId, "envId", envId, "err", err)
-			return deploymentDetail, err
-		}
-		deploymentDetail.DeploymentAppType = dc.DeploymentAppType
+
+	deploymentDetail.EnvironmentId = envId
+	dc, err := impl.deploymentConfigRepository.GetByAppIdAndEnvId(appId, envId)
+	if err != nil {
+		impl.Logger.Errorw("error in getting deployment config by appId and envId", "appId", appId, "envId", envId, "err", err)
+		return deploymentDetail, err
 	}
+	deploymentDetail.DeploymentAppType = dc.DeploymentAppType
+	deploymentDetail.ReleaseMode = dc.ReleaseMode
 
 	return deploymentDetail, nil
 }
@@ -455,6 +456,9 @@ func (impl AppListingRepositoryImpl) FetchAppDetail(ctx context.Context, appId i
 	if err != nil {
 		impl.Logger.Warn("unable to fetch deployment detail for app")
 	}
+	if deploymentDetail.PcoId > 0 {
+		deploymentDetail.IsPipelineTriggered = true
+	}
 	appWfMapping, _ := impl.appWorkflowRepository.FindWFCDMappingByCDPipelineId(deploymentDetail.CdPipelineId)
 	if appWfMapping.ParentType == appWorkflow2.CDPIPELINE {
 		parentEnvironmentName, _ := impl.getEnvironmentNameFromPipelineId(appWfMapping.ParentId)

internal/sql/repository/chartConfig/PipelineOverrideRepository.go

@@ -58,6 +58,7 @@ type PipelineConfigOverrideMetadata struct {
 
 type PipelineOverrideRepository interface {
 	Save(*PipelineOverride) error
+	Update(pipelineOverride *PipelineOverride) error
 	UpdateStatusByRequestIdentifier(requestId string, newStatus models.ChartStatus) (int, error)
 	GetLatestConfigByRequestIdentifier(requestIdentifier string) (pipelineOverride *PipelineOverride, err error)
 	GetLatestConfigByEnvironmentConfigOverrideId(envConfigOverrideId int) (pipelineOverride *PipelineOverride, err error)
@@ -85,6 +86,10 @@ func (impl PipelineOverrideRepositoryImpl) Save(pipelineOverride *PipelineOverri
 	return impl.dbConnection.Insert(pipelineOverride)
 }
 
+func (impl PipelineOverrideRepositoryImpl) Update(pipelineOverride *PipelineOverride) error {
+	return impl.dbConnection.Update(pipelineOverride)
+}
+
 func (impl PipelineOverrideRepositoryImpl) UpdatePipelineMergedValues(ctx context.Context, tx *pg.Tx, id int, pipelineMergedValues string, userId int32) error {
 	_, span := otel.Tracer("orchestrator").Start(ctx, "PipelineOverrideRepositoryImpl.UpdatePipelineMergedValues")
 	defer span.End()

internal/sql/repository/deploymentConfig/repository.go

@@ -29,6 +29,7 @@ type DeploymentConfig struct {
 	ConfigType        string   `sql:"config_type"`
 	RepoUrl           string   `sql:"repo_url"`
 	RepoName          string   `sql:"repo_name"`
+	ReleaseMode       string   `json:"release_mode"`
 	Active            bool     `sql:"active,notnull"`
 	sql.AuditLog
 }

internal/util/ChartTemplateService.go

@@ -46,6 +46,10 @@ const (
 	CHART_WORKING_DIR_PATH                     = "/tmp/charts/"
 )
 
+const (
+	PIPELINE_RELEASE_MODE_CREATE = "create"
+)
+
 type ChartCreateRequest struct {
 	ChartMetaData       *chart.Metadata
 	ChartPath           string

pkg/bean/app.go

@@ -621,6 +621,7 @@ type CDPipelineConfigObject struct {
 	ChildPipelineId               int                                    `json:"childPipelineId"`
 	IsDigestEnforcedForPipeline   bool                                   `json:"isDigestEnforcedForPipeline"`
 	IsDigestEnforcedForEnv        bool                                   `json:"isDigestEnforcedForEnv"`
+	ReleaseMode                   string                                 `json:"releaseMode" validate:"oneof=create"`
 }
 
 type CDPipelineAddType string

pkg/bulkAction/BulkUpdateService.go

@@ -1080,7 +1080,7 @@ func (impl BulkUpdateServiceImpl) buildHibernateUnHibernateRequestForHelmPipelin
 		return nil, nil, err
 	}
 	var group, kind, version, name string
-	name = fmt.Sprintf("%s-%s", pipeline.App.AppName, pipeline.Environment.Name)
+	name = pipeline.DeploymentAppName
 	if chartInfo.Name == bean3.RolloutChartType && chartInfo.UserUploaded == false {
 		// rollout type chart
 		group = "argoproj.io"

pkg/deployment/common/adapter.go

@@ -14,6 +14,7 @@ func ConvertDeploymentConfigDTOToDbObj(config *bean.DeploymentConfig) *deploymen
 		ConfigType:        config.ConfigType,
 		RepoUrl:           config.RepoURL,
 		Active:            config.Active,
+		ReleaseMode:       config.ReleaseMode,
 	}
 }
 
@@ -26,5 +27,6 @@ func ConvertDeploymentConfigDbObjToDTO(dbObj *deploymentConfig.DeploymentConfig)
 		ConfigType:        dbObj.ConfigType,
 		RepoURL:           dbObj.RepoUrl,
 		Active:            dbObj.Active,
+		ReleaseMode:       dbObj.ReleaseMode,
 	}
 }

pkg/deployment/common/bean/bean.go

@@ -14,6 +14,7 @@ type DeploymentConfig struct {
 	DeploymentAppType string
 	RepoURL           string
 	RepoName          string
+	ReleaseMode       string
 	Active            bool
 }
 

pkg/deployment/common/deploymentConfigService.go

@@ -6,6 +6,7 @@ import (
 	appRepository "github.com/devtron-labs/devtron/internal/sql/repository/app"
 	"github.com/devtron-labs/devtron/internal/sql/repository/deploymentConfig"
 	"github.com/devtron-labs/devtron/internal/sql/repository/pipelineConfig"
+	util2 "github.com/devtron-labs/devtron/internal/util"
 	"github.com/devtron-labs/devtron/pkg/appStore/installedApp/repository"
 	bean3 "github.com/devtron-labs/devtron/pkg/auth/user/bean"
 	chartRepoRepository "github.com/devtron-labs/devtron/pkg/chartRepo/repository"
@@ -101,6 +102,20 @@ func (impl *DeploymentConfigServiceImpl) GetConfigForDevtronApps(appId, envId in
 			impl.logger.Errorw("error in parsing config from charts and pipeline repository", "appId", appId, "envId", envId, "err", err)
 			return nil, err
 		}
+		if envId > 0 {
+			// add columns added after migration (of deployment app type and repo url) here
+			appAndEnvLevelConfig, err := impl.deploymentConfigRepository.GetByAppIdAndEnvId(appId, envId)
+			if err != nil && err != pg.ErrNoRows {
+				impl.logger.Errorw("error in getting deployment config db object by appId and envId", "appId", appId, "envId", envId, "err", err)
+				return nil, err
+			}
+			if err == pg.ErrNoRows {
+				// deployment config is not done
+				configFromOldData.ReleaseMode = util2.PIPELINE_RELEASE_MODE_CREATE
+			} else {
+				configFromOldData.ReleaseMode = appAndEnvLevelConfig.ReleaseMode
+			}
+		}
 		return configFromOldData, nil
 	}
 
@@ -190,6 +205,9 @@ func (impl *DeploymentConfigServiceImpl) GetAndMigrateConfigIfAbsentForDevtronAp
 			impl.logger.Errorw("error in parsing config from charts and pipeline repository", "appId", appId, "envId", envId, "err", err)
 			return nil, err
 		}
+		if envId > 0 {
+			configFromOldData.ReleaseMode = envLevelConfig.ReleaseMode
+		}
 		return configFromOldData, nil
 	}
 
@@ -256,6 +274,7 @@ func (impl *DeploymentConfigServiceImpl) parseEnvLevelConfigForDevtronApps(appLe
 		EnvironmentId: envId,
 		ConfigType:    appLevelConfig.ConfigType,
 		RepoUrl:       appLevelConfig.RepoUrl,
+		ReleaseMode:   util2.PIPELINE_RELEASE_MODE_CREATE, //for migration it is always equal to create as migration is happening for old cd pipelines
 		Active:        true,
 	}
 

pkg/deployment/manifest/ManifestCreationService.go

@@ -265,7 +265,7 @@ func (impl *ManifestCreationServiceImpl) GetValuesOverrideForTrigger(overrideReq
 			appLabelJsonByte = nil
 		}
 		mergedValues, err := impl.mergeOverrideValues(envOverride, releaseOverrideJson, configMapJson.MergedJson, appLabelJsonByte, strategy)
-		appName := fmt.Sprintf("%s-%s", overrideRequest.AppName, envOverride.Environment.Name)
+		appName := pipeline.DeploymentAppName
 		var k8sErr error
 		mergedValues, k8sErr = impl.updatedExternalCmCsHashForTrigger(newCtx, overrideRequest.ClusterId,
 			envOverride.Namespace, mergedValues, configMapJson.ExternalCmList, configMapJson.ExternalCsList)
@@ -275,10 +275,12 @@ func (impl *ManifestCreationServiceImpl) GetValuesOverrideForTrigger(overrideReq
 			// error is not returned as it's not blocking for deployment process
 			// blocking deployments based on this use case can vary for user to user
 		}
-		mergedValues, err = impl.autoscalingCheckBeforeTrigger(newCtx, appName, envOverride.Namespace, mergedValues, overrideRequest)
-		if err != nil {
-			impl.logger.Errorw("error in autoscaling check before trigger", "pipelineId", overrideRequest.PipelineId, "err", err)
-			return valuesOverrideResponse, err
+		if !envOverride.Environment.IsVirtualEnvironment {
+			mergedValues, err = impl.autoscalingCheckBeforeTrigger(newCtx, appName, envOverride.Namespace, mergedValues, overrideRequest)
+			if err != nil {
+				impl.logger.Errorw("error in autoscaling check before trigger", "pipelineId", overrideRequest.PipelineId, "err", err)
+				return valuesOverrideResponse, err
+			}
 		}
 		// handle image pull secret if access given
 		mergedValues, err = impl.dockerRegistryIpsConfigService.HandleImagePullSecretOnApplicationDeployment(newCtx, envOverride.Environment, artifact, pipeline.CiPipelineId, mergedValues)
@@ -806,7 +808,7 @@ func (impl *ManifestCreationServiceImpl) checkAndFixDuplicateReleaseNo(override
 				return err
 			}
 			override.PipelineReleaseCounter = currentReleaseNo + 1
-			err = impl.pipelineOverrideRepository.Save(override)
+			err = impl.pipelineOverrideRepository.Update(override)
 			if err != nil {
 				return err
 			}

pkg/deployment/trigger/devtronApps/TriggerService.go

@@ -956,20 +956,6 @@ func (impl *TriggerServiceImpl) buildManifestPushTemplate(overrideRequest *bean3
 	return manifestPushTemplate, err
 }
 
-// getAcdAppGitOpsRepoName returns the GitOps repository name, configured for the argoCd app
-func (impl *TriggerServiceImpl) getAcdAppGitOpsRepoName(appName string, environmentName string) (string, error) {
-	//this method should only call in case of argo-integration and gitops configured
-	acdToken, err := impl.argoUserService.GetLatestDevtronArgoCdUserToken()
-	if err != nil {
-		impl.logger.Errorw("error in getting acd token", "err", err)
-		return "", err
-	}
-	ctx := context.Background()
-	ctx = context.WithValue(ctx, "token", acdToken)
-	acdAppName := fmt.Sprintf("%s-%s", appName, environmentName)
-	return impl.argoClientWrapperService.GetGitOpsRepoName(ctx, acdAppName)
-}
-
 func (impl *TriggerServiceImpl) getManifestPushService(triggerEvent bean.TriggerEvent) publish.ManifestPushService {
 	var manifestPushService publish.ManifestPushService
 	if triggerEvent.ManifestStorageType == bean2.ManifestStorageGit {

pkg/generateManifest/DeploymentTemplateService.go

@@ -501,7 +501,18 @@ func (impl DeploymentTemplateServiceImpl) GetRestartWorkloadData(ctx context.Con
 		impl.Logger.Errorw("error in fetching environment", "envId", envId, "err", err)
 		return nil, err
 	}
-	installReleaseRequests, err := impl.constructInstallReleaseBulkReq(apps, environment)
+
+	pipelineMap := make(map[string]*pipelineConfig.Pipeline)
+	pipelines, err := impl.pipelineRepository.FindActiveByInFilter(envId, appIds)
+	if err != nil {
+		impl.Logger.Errorw("error in getting pipelines by appIds and envId", "appIds", appIds, "envId", envId, "err", err)
+		return nil, err
+	}
+	for _, p := range pipelines {
+		pipelineMap[fmt.Sprintf("%d-%d", p.AppId, p.EnvironmentId)] = p
+	}
+
+	installReleaseRequests, err := impl.constructInstallReleaseBulkReq(apps, environment, pipelineMap)
 	if err != nil {
 		impl.Logger.Errorw("error in fetching installReleaseRequests", "appIds", appIds, "envId", envId, "err", err)
 		return nil, err

pkg/generateManifest/helper.go

@@ -23,6 +23,7 @@ import (
 	"github.com/devtron-labs/devtron/api/helm-app/bean"
 	"github.com/devtron-labs/devtron/api/helm-app/gRPC"
 	"github.com/devtron-labs/devtron/internal/sql/repository/app"
+	"github.com/devtron-labs/devtron/internal/sql/repository/pipelineConfig"
 	"github.com/devtron-labs/devtron/pkg/cluster/repository"
 	"go.opentelemetry.io/otel"
 	"golang.org/x/exp/maps"
@@ -67,7 +68,7 @@ func (impl DeploymentTemplateServiceImpl) constructRotatePodResponse(templateCha
 	return podResp, nil
 }
 
-func (impl DeploymentTemplateServiceImpl) constructInstallReleaseBulkReq(apps []*app.App, environment *repository.Environment) ([]*gRPC.InstallReleaseRequest, error) {
+func (impl DeploymentTemplateServiceImpl) constructInstallReleaseBulkReq(apps []*app.App, environment *repository.Environment, pipelineMap map[string]*pipelineConfig.Pipeline) ([]*gRPC.InstallReleaseRequest, error) {
 	appIdToInstallReleaseRequest := make(map[int]*gRPC.InstallReleaseRequest)
 	installReleaseRequests := make([]*gRPC.InstallReleaseRequest, 0)
 	var applicationIds []int
@@ -104,9 +105,10 @@ func (impl DeploymentTemplateServiceImpl) constructInstallReleaseBulkReq(apps []
 		impl.Logger.Errorw("error in fetching cluster detail", "clusterId", 1, "err", err)
 		return nil, err
 	}
+
 	for _, app := range apps {
 		if _, ok := appIdToInstallReleaseRequest[app.Id]; ok {
-			appIdToInstallReleaseRequest[app.Id].ReleaseIdentifier = impl.getReleaseIdentifier(config, app, environment)
+			appIdToInstallReleaseRequest[app.Id].ReleaseIdentifier = impl.getReleaseIdentifier(config, app, environment, pipelineMap)
 			appIdToInstallReleaseRequest[app.Id].K8SVersion = k8sServerVersion.String()
 		}
 	}
@@ -140,10 +142,11 @@ func (impl DeploymentTemplateServiceImpl) setChartContent(ctx context.Context, i
 	return err
 }
 
-func (impl DeploymentTemplateServiceImpl) getReleaseIdentifier(config *gRPC.ClusterConfig, app *app.App, env *repository.Environment) *gRPC.ReleaseIdentifier {
+func (impl DeploymentTemplateServiceImpl) getReleaseIdentifier(config *gRPC.ClusterConfig, app *app.App, env *repository.Environment, pipelineMap map[string]*pipelineConfig.Pipeline) *gRPC.ReleaseIdentifier {
+	pipeline := pipelineMap[fmt.Sprintf("%d-%d", app.Id, env.Id)]
 	return &gRPC.ReleaseIdentifier{
 		ClusterConfig:    config,
-		ReleaseName:      fmt.Sprintf("%s-%s", app.AppName, env.Name),
+		ReleaseName:      pipeline.DeploymentAppName,
 		ReleaseNamespace: env.Namespace,
 	}
 }