Skip to content

Commit 07d452d

Browse files
authored
Merge pull request #6696 from devtron-labs/fix-panic
fix: app clone panic
2 parents 555b987 + 75fd84e commit 07d452d

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

pkg/appClone/AppCloneService.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ import (
4141
bean3 "github.com/devtron-labs/devtron/pkg/pipeline/bean"
4242
globalUtil "github.com/devtron-labs/devtron/util"
4343
"github.com/go-pg/pg"
44+
"github.com/pkg/errors"
4445
"go.uber.org/zap"
4546
"net/http"
4647
"strings"
@@ -767,6 +768,9 @@ func (impl *AppCloneServiceImpl) createWfInstances(refWfMappings []bean4.AppWork
767768
impl.logger.Errorw("error in creating ci pipeline, app clone", "err", err)
768769
return createWorkflowMappingDto, err
769770
}
771+
if ci == nil || ci.CiPipelines == nil || len(ci.CiPipelines) == 0 {
772+
return createWorkflowMappingDto, errors.New("ci pipeline not created")
773+
}
770774
impl.logger.Debugw("ci created", "ci", ci)
771775
}
772776

@@ -812,6 +816,7 @@ type cloneCiPipelineRequest struct {
812816
func (impl *AppCloneServiceImpl) CreateCiPipeline(req *cloneCiPipelineRequest) (*bean.CiConfigRequest, error) {
813817
refCiConfig, err := impl.pipelineBuilder.GetCiPipeline(req.refAppId)
814818
if err != nil {
819+
impl.logger.Errorw("error in fetching ci config", "err", err)
815820
return nil, err
816821
}
817822

@@ -825,6 +830,7 @@ func (impl *AppCloneServiceImpl) CreateCiPipeline(req *cloneCiPipelineRequest) (
825830
}
826831
}
827832
if refCiPipeline == nil {
833+
impl.logger.Errorw("error in fetching ci pipeline", "err", err)
828834
return nil, nil
829835
}
830836
pipelineName := refCiPipeline.Name

0 commit comments

Comments
 (0)