Skip to content

Commit 90f36f6

Browse files
prkhrkatprkhrkat
authored
Merge branch 'develop' into mandatory-tag-v2
2 parents 67c1204 + 45e9738 commit 90f36f6

File tree

14 files changed

+60
-28
lines changed

14 files changed

+60
-28
lines changed

‎client/argocdServer/connection/Connection.go

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -155,17 +155,11 @@ func (impl *ArgoCDConnectionManagerImpl) getConnectionWithToken(connectionConfig
155155
}
156156

157157
func (impl *ArgoCDConnectionManagerImpl) getLatestDevtronArgoCdUserToken(grpcConfig *bean.ArgoGRPCConfig) (string, error) {
158-
gitOpsConfigurationStatus, err := impl.gitOpsConfigReadService.IsGitOpsConfigured()
159-
if err != nil {
160-
impl.logger.Errorw("error while checking if gitOps is configured", "err", err)
161-
return "", err
162-
}
163-
if !gitOpsConfigurationStatus.IsGitOpsConfigured {
164-
//here acd token only required in context for argo cd calls
165-
return "", nil
166-
}
158+
var (
159+
k8sClient *v1.CoreV1Client
160+
err error
161+
)
167162
authConfig := grpcConfig.AuthConfig
168-
var k8sClient *v1.CoreV1Client
169163
if authConfig.ClusterId == bean2.DefaultClusterId {
170164
k8sClient, err = impl.k8sUtil.GetCoreV1ClientInCluster()
171165
if err != nil {

‎cmd/external-app/wire_gen.go

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎pkg/infraConfig/service/infraConfigService.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ func (impl *InfraConfigServiceImpl) loadDefaultProfile() error {
228228
impl.logger.Errorw("error in fetching platforms from db", "error", err)
229229
return err
230230
}
231-
231+
232232
runnerPlatFormMapping := &repository.ProfilePlatformMapping{}
233233
//one platform is expected
234234
if len(platformsFromDb) > 0 {

‎pkg/policyGovernance/security/imageScanning/ImageScanService.go

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ package imageScanning
1919
import (
2020
"context"
2121
bean4 "github.com/devtron-labs/devtron/api/bean"
22+
"github.com/devtron-labs/devtron/internal/util"
2223
"github.com/devtron-labs/devtron/pkg/cluster/environment"
2324
"github.com/devtron-labs/devtron/pkg/cluster/environment/bean"
2425
bean2 "github.com/devtron-labs/devtron/pkg/deployment/trigger/devtronApps/bean"
@@ -326,9 +327,13 @@ func (impl ImageScanServiceImpl) FetchExecutionDetailResult(request *bean3.Image
326327
}
327328
imageScanResponse.Image = ciArtifact.Image
328329
scanExecution, err := impl.scanHistoryRepository.FindByImageAndDigestWithHistoryMapping(ciArtifact.ImageDigest, ciArtifact.Image)
329-
if err != nil {
330+
if err != nil && !util.IsErrNoRows(err) {
330331
impl.Logger.Errorw("error while fetching scan execution result", "err", err)
331332
return nil, err
333+
} else if util.IsErrNoRows(err) {
334+
// image not scanned
335+
imageScanResponse.Scanned = false
336+
return imageScanResponse, nil
332337
}
333338
ciPipeline, err := impl.ciPipelineRepository.FindByIdIncludingInActive(ciArtifact.PipelineId)
334339
if err != nil {
@@ -779,14 +784,16 @@ func (impl ImageScanServiceImpl) fetchLatestArtifactMetadataDeployedOnAllEnvsAcr
779784
}
780785
appEnvToCiArtifactMap[bean3.NewAppEnvMetadata(item.AppId, item.EnvId)] = ciArtifactId
781786
}
782-
parentCiArtifacts, err := impl.ciArtifactRepository.GetByIds(parentCiArtifactIds)
783-
if err != nil {
784-
impl.Logger.Errorw("error in getting artifacts by ids", "ids", parentCiArtifactIds, "err", err)
785-
return nil, nil, err
786-
}
787-
for _, parentCiArtifact := range parentCiArtifacts {
788-
// for linked ci case
789-
ciArtifactIdToScannedMap[parentCiArtifact.Id] = parentCiArtifact.Scanned
787+
if len(parentCiArtifactIds) > 0 {
788+
parentCiArtifacts, err := impl.ciArtifactRepository.GetByIds(parentCiArtifactIds)
789+
if err != nil {
790+
impl.Logger.Errorw("error in getting artifacts by ids", "ids", parentCiArtifactIds, "err", err)
791+
return nil, nil, err
792+
}
793+
for _, parentCiArtifact := range parentCiArtifacts {
794+
// for linked ci case
795+
ciArtifactIdToScannedMap[parentCiArtifact.Id] = parentCiArtifact.Scanned
796+
}
790797
}
791798
return appEnvToCiArtifactMap, ciArtifactIdToScannedMap, nil
792799
}

‎scripts/devtron-reference-helm-charts/deployment-chart_4-19-0/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ spec:
186186
protocol: TCP
187187
name: envoy-admin
188188
{{- range $index, $element := .Values.ContainerPort }}
189-
- name: {{ $element.name}}
189+
- name: envoy-{{ $element.name}}
190190
containerPort: {{ $element.envoyPort | default (add 8790 $index) }}
191191
protocol: TCP
192192
{{- end }}

‎scripts/devtron-reference-helm-charts/deployment-chart_4-19-0/templates/service.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ spec:
3333
{{- end }}
3434
{{- if .targetPort }}
3535
targetPort: {{ .targetPort }}
36+
{{- else if $.Values.appMetrics }}
37+
targetPort: envoy-{{ .name }}
3638
{{- else }}
3739
targetPort: {{ .name }}
3840
{{- end }}
@@ -69,7 +71,13 @@ spec:
6971
{{- else }}
7072
- port: {{ .port }}
7173
{{- end }}
74+
{{- if .targetPort }}
75+
targetPort: {{ .targetPort }}
76+
{{- else if $.Values.appMetrics }}
77+
targetPort: envoy-{{ .name }}
78+
{{- else }}
7279
targetPort: {{ .name }}
80+
{{- end }}
7381
protocol: TCP
7482
name: {{ .name }}
7583
{{- end }}

‎scripts/devtron-reference-helm-charts/deployment-chart_4-20-0/templates/deployment.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ spec:
513513
protocol: TCP
514514
name: envoy-admin
515515
{{- range $index, $element := .Values.ContainerPort }}
516-
- name: {{ $element.name}}
516+
- name: envoy-{{ $element.name}}
517517
containerPort: {{ $element.envoyPort | default (add 8790 $index) }}
518518
protocol: TCP
519519
{{- end }}
@@ -1123,7 +1123,7 @@ spec:
11231123
protocol: TCP
11241124
name: envoy-admin
11251125
{{- range $index, $element := .Values.ContainerPort }}
1126-
- name: {{ $element.name}}
1126+
- name: envoy-{{ $element.name}}
11271127
containerPort: {{ $element.envoyPort | default (add 8790 $index) }}
11281128
protocol: TCP
11291129
{{- end }}

‎scripts/devtron-reference-helm-charts/deployment-chart_4-20-0/templates/service.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ spec:
3333
{{- end }}
3434
{{- if .targetPort }}
3535
targetPort: {{ .targetPort }}
36+
{{- else if $.Values.appMetrics }}
37+
targetPort: envoy-{{ .name }}
3638
{{- else }}
3739
targetPort: {{ .name }}
3840
{{- end }}
@@ -80,7 +82,13 @@ spec:
8082
{{- else }}
8183
- port: {{ .port }}
8284
{{- end }}
85+
{{- if .targetPort }}
86+
targetPort: {{ .targetPort }}
87+
{{- else if $.Values.appMetrics }}
88+
targetPort: envoy-{{ .name }}
89+
{{- else }}
8390
targetPort: {{ .name }}
91+
{{- end }}
8492
protocol: TCP
8593
name: {{ .name }}
8694
{{- end }}

‎scripts/devtron-reference-helm-charts/reference-chart_4-19-0/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ spec:
186186
protocol: TCP
187187
name: envoy-admin
188188
{{- range $index, $element := .Values.ContainerPort }}
189-
- name: {{ $element.name}}
189+
- name: envoy-{{ $element.name}}
190190
containerPort: {{ $element.envoyPort | default (add 8790 $index) }}
191191
protocol: TCP
192192
{{- end }}

‎scripts/devtron-reference-helm-charts/reference-chart_4-19-0/templates/service.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ spec:
3333
{{- end }}
3434
{{- if .targetPort }}
3535
targetPort: {{ .targetPort }}
36+
{{- else if $.Values.appMetrics }}
37+
targetPort: envoy-{{ .name }}
3638
{{- else }}
3739
targetPort: {{ .name }}
3840
{{- end }}
@@ -69,7 +71,13 @@ spec:
6971
{{- else }}
7072
- port: {{ .port }}
7173
{{- end }}
74+
{{- if .targetPort }}
75+
targetPort: {{ .targetPort }}
76+
{{- else if $.Values.appMetrics }}
77+
targetPort: envoy-{{ .name }}
78+
{{- else }}
7279
targetPort: {{ .name }}
80+
{{- end }}
7381
protocol: TCP
7482
name: {{ .name }}
7583
{{- end }}

0 commit comments

Comments
 (0)