Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

optimise dockerfile #6226

Open
wants to merge 81 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
81 commits
Select commit Hold shift + click to select a range
3b7b718
optimise dockerfile
nishant-d Dec 27, 2024
10f39a3
build optimise
nishant-d Dec 27, 2024
bc34abf
handle the case when a cve is not found in our database to enforce po…
prakash100198 Dec 30, 2024
df4beab
removed casbin database creation call (#6238)
kartik-579 Dec 31, 2024
b64dca1
fix: graceful shutdown handling and logs flush (#6239)
Shivam-nagar23 Dec 31, 2024
33f0ad6
isProd in cluster
prkhrkat Dec 31, 2024
385432f
Merge pull request #6241 from devtron-labs/cluster-fix
prkhrkat Dec 31, 2024
2c3ea7d
common lib hash update and add pg failure queries flag
ayu-devtron Jan 2, 2025
69097b9
chore: merging user service with connection manager (#6225)
iamayushm Jan 2, 2025
de8652b
chore: Argocd repository secret creation service refactoring (#6229)
iamayushm Jan 2, 2025
2ddc994
Merge branch 'develop' into img-scan-bug-fixes
prakash100198 Jan 3, 2025
c168da4
fix app level cve add and it wasn't enforcing it at app level only
prakash100198 Jan 3, 2025
54e776a
soft delete previous policies so that consistent data is created in s…
prakash100198 Jan 5, 2025
703bf36
use ilike with regex matching in app name filter at cve search
prakash100198 Jan 5, 2025
15ccde9
bypass vulnerability check for rollback trigger
prakash100198 Jan 6, 2025
11b47d5
update common lib hash to use updated query processor function
ayu-devtron Jan 6, 2025
d60c186
main sync develop
vikramdevtron Jan 6, 2025
e93cadb
Merge pull request #6256 from devtron-labs/main-sync-develop-6jan
vikramdevtron Jan 6, 2025
0c2479c
update common lib hash
ayu-devtron Jan 6, 2025
0733fad
SeverityStringToEnum
prakash100198 Jan 6, 2025
ef09af6
FilterDeployInfoByScannedArtifactsDeployedInEnv
prakash100198 Jan 6, 2025
5e2dbeb
Merge branch 'develop' into img-scan-bug-fixes
prakash100198 Jan 6, 2025
8cffe77
wire
prakash100198 Jan 6, 2025
c8dfc37
some changes after self review
prakash100198 Jan 7, 2025
cb7f9ac
added schme json for cronjobs (#6260)
badal773 Jan 7, 2025
36d9331
code review comments + self review comments incorporation
prakash100198 Jan 7, 2025
dcb7a39
Merge branch 'develop' into img-scan-bug-fixes
prakash100198 Jan 7, 2025
390dea2
wip
kartik-579 Jan 8, 2025
12f01f2
wip
kartik-579 Jan 8, 2025
96a9a3b
update common-lib hash
ayu-devtron Jan 8, 2025
0baf4e7
created resourceTree service
kartik-579 Jan 8, 2025
fe30a55
resrouce tree service ent
kartik-579 Jan 8, 2025
44b3a0f
Added licenses
kartik-579 Jan 8, 2025
c2ba630
removed resource tree
kartik-579 Jan 8, 2025
24701e6
wip
kartik-579 Jan 8, 2025
de84a08
removed unused dependencies
kartik-579 Jan 8, 2025
233ec55
oss/ent
kartik-579 Jan 8, 2025
6ecdabf
wip
kartik-579 Jan 8, 2025
4226677
wip
kartik-579 Jan 8, 2025
1328df4
oss/ent
kartik-579 Jan 8, 2025
1f916e4
minor changes
kartik-579 Jan 8, 2025
eadaccf
updated log
kartik-579 Jan 8, 2025
a1ba8d7
added ent func
kartik-579 Jan 8, 2025
c4389f6
len check in soft delete policies
prakash100198 Jan 8, 2025
ffef7f9
added pointer in receiver
kartik-579 Jan 8, 2025
d414602
return statement
kartik-579 Jan 8, 2025
d7713a4
wip
kartik-579 Jan 8, 2025
1e32846
erview comments
kartik-579 Jan 10, 2025
940bc2f
review comments
kartik-579 Jan 10, 2025
c6f8cca
wip
kartik-579 Jan 10, 2025
c472b6e
wip
kartik-579 Jan 10, 2025
384a769
query update
kartik-579 Jan 10, 2025
b758948
Merge pull request #6262 from devtron-labs/helm-type-pipeline-app-status
kartik-579 Jan 10, 2025
9672e0f
Merge branch 'develop' of github.com:devtron-labs/devtron into pg-ena…
ayu-devtron Jan 10, 2025
e17ca74
put deleted check in FindLatestCdWorkflowRunnerArtifactMetadataForApp…
prakash100198 Jan 10, 2025
3008f93
fix
prakash100198 Jan 10, 2025
17284db
update oss hash
ayu-devtron Jan 10, 2025
36fb5ba
hash update
ayu-devtron Jan 10, 2025
366bf84
fix: dependabot dt25 (#6237)
RajeevRanjan27 Jan 11, 2025
aff3919
use common query monitoring bean
ayu-devtron Jan 11, 2025
922aecb
env_gen update
ayu-devtron Jan 11, 2025
800291f
update oss hash
ayu-devtron Jan 12, 2025
3005129
Merge branch 'develop' of github.com:devtron-labs/devtron into pg-ena…
ayu-devtron Jan 12, 2025
4c5c318
udpate oss hash
ayu-devtron Jan 12, 2025
a9cd284
update oss hash
ayu-devtron Jan 12, 2025
abd7531
Merge pull request #6243 from devtron-labs/pg-enable-metrics
ayu-devtron Jan 13, 2025
0bf67af
Merge branch 'develop' into img-scan-bug-fixes
prakash100198 Jan 13, 2025
a0ef645
Merge pull request #6259 from devtron-labs/img-scan-bug-fixes
prakash100198 Jan 13, 2025
1f3d001
main sync develop
vikramdevtron Jan 13, 2025
205b6ba
authenticator and common lib updated
vikramdevtron Jan 13, 2025
0f5cf58
Merge pull request #6273 from devtron-labs/main-sync-develop-13jan
vikramdevtron Jan 13, 2025
64229e2
added labels on app-manual-sync job (#6269)
pawan-59 Jan 13, 2025
be8504d
Merge branch 'main' into sync-main-develop
RajeevRanjan27 Jan 13, 2025
ba6b6fb
Merge pull request #6276 from devtron-labs/sync-main-develop
RajeevRanjan27 Jan 13, 2025
ab1d13e
remove unused variable causing double register in prometheus
ayu-devtron Jan 14, 2025
376206d
Merge pull request #6279 from devtron-labs/pg-descriptor-fix
ayu-devtron Jan 14, 2025
ded2c48
chore: Argo common client (#6266)
iamayushm Jan 15, 2025
6321361
updated Dockerfile to remove extra layers
pawan-59 Jan 16, 2025
820aa60
updated DockerfileEA to remove extra layers
pawan-59 Jan 16, 2025
18c494a
updated dockerfile to add vendor step
pawan-59 Jan 16, 2025
2974559
Merge branch 'develop' into dockerfile-optimise
pawan-59 Jan 16, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
3 changes: 2 additions & 1 deletion App.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ package main
import (
"context"
"crypto/tls"
"errors"
"fmt"
"github.com/devtron-labs/common-lib/middlewares"
pubsub "github.com/devtron-labs/common-lib/pubsub-lib"
Expand Down Expand Up @@ -137,7 +138,7 @@ func (app *App) Start() {
err = server.ListenAndServe()
}
//err := http.ListenAndServe(fmt.Sprintf(":%d", port), auth.Authorizer(app.Enforcer, app.sessionManager)(app.MuxRouter.Router))
if err != nil {
if err != nil && !errors.Is(err, http.ErrServerClosed) {
app.Logger.Errorw("error in startup", "err", err)
os.Exit(2)
}
Expand Down
66 changes: 30 additions & 36 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,52 +1,46 @@
FROM golang:1.21 AS build-env

RUN echo $GOPATH
RUN apt update
RUN apt install git gcc musl-dev make -y
RUN go install github.com/google/wire/cmd/wire@latest
RUN echo $GOPATH && \
apt update && \
apt install git gcc musl-dev make -y && \
go install github.com/google/wire/cmd/wire@latest

WORKDIR /go/src/github.com/devtron-labs/devtron

ADD . /go/src/github.com/devtron-labs/devtron/

ADD ./vendor/github.com/Microsoft/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/microsoft/
RUN GOOS=linux make build-all

RUN GOOS=linux make build

# uncomment this post build arg
FROM ubuntu:22.04@sha256:1b8d8ff4777f36f19bfe73ee4df61e3a0b789caeff29caa019539ec7c9a57f95 as devtron-all

RUN apt update
RUN apt install ca-certificates git curl -y
RUN apt clean autoclean
RUN apt autoremove -y && rm -rf /var/lib/apt/lists/*
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/devtron .
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/auth_model.conf .
#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/argocd-assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/devtron-reference-helm-charts scripts/devtron-reference-helm-charts
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/sql scripts/sql
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/casbin scripts/casbin
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/argo-assets/APPLICATION_TEMPLATE.tmpl scripts/argo-assets/APPLICATION_TEMPLATE.tmpl

COPY ./git-ask-pass.sh /git-ask-pass.sh
RUN chmod +x /git-ask-pass.sh

RUN useradd -ms /bin/bash devtron
RUN chown -R devtron:devtron ./devtron
RUN chown -R devtron:devtron ./git-ask-pass.sh
RUN chown -R devtron:devtron ./auth_model.conf
RUN chown -R devtron:devtron ./scripts
RUN apt update && \
apt install ca-certificates git curl -y && \
apt clean autoclean && \
apt autoremove -y && \
rm -rf /var/lib/apt/lists/* && \
useradd -ms /bin/bash devtron

USER devtron
COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/devtron .

COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/auth_model.conf .

CMD ["./devtron"]
COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/argocd-assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets

COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/devtron-reference-helm-charts scripts/devtron-reference-helm-charts

#FROM alpine:3.15.0 as devtron-ea
COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/sql scripts/sql

#RUN apk add --no-cache ca-certificates
#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/auth_model.conf .
#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/cmd/external-app/devtron-ea .
COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/casbin scripts/casbin

#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets
#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/devtron-reference-helm-charts scripts/devtron-reference-helm-charts
#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/argo-assets/APPLICATION_TEMPLATE.JSON scripts/argo-assets/APPLICATION_TEMPLATE.JSON
COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/argo-assets/APPLICATION_TEMPLATE.tmpl scripts/argo-assets/APPLICATION_TEMPLATE.tmpl

COPY --chown=devtron:devtron ./git-ask-pass.sh /git-ask-pass.sh

RUN chmod +x /git-ask-pass.sh

USER devtron

#CMD ["./devtron-ea"]
CMD ["./devtron"]
35 changes: 21 additions & 14 deletions DockerfileEA
Original file line number Diff line number Diff line change
@@ -1,33 +1,40 @@
FROM golang:1.21 AS build-env

RUN echo $GOPATH
RUN apt update
RUN apt install git gcc musl-dev make -y
RUN go install github.com/google/wire/cmd/wire@latest
RUN echo $GOPATH && \
apt update && \
apt install git gcc musl-dev make -y && \
go install github.com/google/wire/cmd/wire@latest

WORKDIR /go/src/github.com/devtron-labs/devtron

ADD . /go/src/github.com/devtron-labs/devtron/

ADD ./vendor/github.com/Microsoft/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/microsoft/

RUN GOOS=linux make build-all

FROM ubuntu:22.04@sha256:1b8d8ff4777f36f19bfe73ee4df61e3a0b789caeff29caa019539ec7c9a57f95 as devtron-ea

RUN apt update
RUN apt install ca-certificates curl -y
RUN apt clean autoclean
RUN apt autoremove -y && rm -rf /var/lib/apt/lists/*
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/auth_model.conf .
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/cmd/external-app/devtron-ea .
RUN apt update && \
apt install ca-certificates curl -y && \
apt clean autoclean && \
apt autoremove -y && rm -rf /var/lib/apt/lists/* && \
useradd -ms /bin/bash devtron

COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/auth_model.conf .

COPY --chown=devtron:devtron --from=build-env /go/src/github.com/devtron-labs/devtron/cmd/external-app/devtron-ea .

#COPY --from=build-env /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets
COPY --from=build-env /go/src/github.com/devtron-labs/devtron/argocd-assets/ /go/src/github.com/devtron-labs/devtron/vendor/github.com/argoproj/argo-cd/assets

COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/devtron-reference-helm-charts scripts/devtron-reference-helm-charts

COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/sql scripts/sql

COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/casbin scripts/casbin

COPY --from=build-env /go/src/github.com/devtron-labs/devtron/scripts/argo-assets/APPLICATION_TEMPLATE.tmpl scripts/argo-assets/APPLICATION_TEMPLATE.tmpl

RUN useradd -ms /bin/bash devtron
RUN chown -R devtron:devtron ./devtron-ea
RUN chown -R devtron:devtron ./auth_model.conf
RUN chown -R devtron:devtron ./scripts

USER devtron
Expand Down
28 changes: 17 additions & 11 deletions Wire.go
Original file line number Diff line number Diff line change
Expand Up @@ -79,12 +79,16 @@ import (
"github.com/devtron-labs/devtron/cel"
"github.com/devtron-labs/devtron/client/argocdServer"
"github.com/devtron-labs/devtron/client/argocdServer/application"
"github.com/devtron-labs/devtron/client/argocdServer/bean"
"github.com/devtron-labs/devtron/client/argocdServer/certificate"
cluster2 "github.com/devtron-labs/devtron/client/argocdServer/cluster"
"github.com/devtron-labs/devtron/client/argocdServer/config"
"github.com/devtron-labs/devtron/client/argocdServer/connection"
"github.com/devtron-labs/devtron/client/argocdServer/repoCredsK8sClient"
repocreds "github.com/devtron-labs/devtron/client/argocdServer/repocreds"
repository2 "github.com/devtron-labs/devtron/client/argocdServer/repository"
session2 "github.com/devtron-labs/devtron/client/argocdServer/session"
"github.com/devtron-labs/devtron/client/argocdServer/version"
"github.com/devtron-labs/devtron/client/cron"
"github.com/devtron-labs/devtron/client/dashboard"
eClient "github.com/devtron-labs/devtron/client/events"
Expand Down Expand Up @@ -117,7 +121,6 @@ import (
repository9 "github.com/devtron-labs/devtron/pkg/appStore/installedApp/repository"
deployment3 "github.com/devtron-labs/devtron/pkg/appStore/installedApp/service/FullMode/deployment"
"github.com/devtron-labs/devtron/pkg/appWorkflow"
"github.com/devtron-labs/devtron/pkg/argoRepositoryCreds"
"github.com/devtron-labs/devtron/pkg/asyncProvider"
"github.com/devtron-labs/devtron/pkg/attributes"
"github.com/devtron-labs/devtron/pkg/build"
Expand Down Expand Up @@ -167,7 +170,6 @@ import (
workflow3 "github.com/devtron-labs/devtron/pkg/workflow"
"github.com/devtron-labs/devtron/pkg/workflow/dag"
util2 "github.com/devtron-labs/devtron/util"
"github.com/devtron-labs/devtron/util/argo"
cron2 "github.com/devtron-labs/devtron/util/cron"
"github.com/devtron-labs/devtron/util/rbac"
"github.com/google/wire"
Expand Down Expand Up @@ -229,7 +231,7 @@ func InitializeApp() (*App, error) {
connection.SettingsManager,
// auth.GetConfigForDevtronApps,

connection.GetConfig,
bean.GetConfig,
wire.Bind(new(session2.ServiceClient), new(*middleware.LoginService)),

sse.NewSSE,
Expand Down Expand Up @@ -449,8 +451,8 @@ func InitializeApp() (*App, error) {
wire.Bind(new(repository8.ImageTaggingRepository), new(*repository8.ImageTaggingRepositoryImpl)),
imageTagging.NewImageTaggingServiceImpl,
wire.Bind(new(imageTagging.ImageTaggingService), new(*imageTagging.ImageTaggingServiceImpl)),
argocdServer.NewVersionServiceImpl,
wire.Bind(new(argocdServer.VersionService), new(*argocdServer.VersionServiceImpl)),
version.NewVersionServiceImpl,
wire.Bind(new(version.VersionService), new(*version.VersionServiceImpl)),

router.NewGitProviderRouterImpl,
wire.Bind(new(router.GitProviderRouter), new(*router.GitProviderRouterImpl)),
Expand Down Expand Up @@ -801,9 +803,9 @@ func InitializeApp() (*App, error) {

connection.NewArgoCDConnectionManagerImpl,
wire.Bind(new(connection.ArgoCDConnectionManager), new(*connection.ArgoCDConnectionManagerImpl)),
argo.NewArgoUserServiceImpl,
wire.Bind(new(argo.ArgoUserService), new(*argo.ArgoUserServiceImpl)),
//util2.GetEnvironmentVariables,
//argo.NewArgoUserServiceImpl,
//wire.Bind(new(argo.ArgoUserService), new(*argo.ArgoUserServiceImpl)),
////util2.GetEnvironmentVariables,
// AuthWireSet,

cron.NewCdApplicationStatusUpdateHandlerImpl,
Expand Down Expand Up @@ -926,6 +928,7 @@ func InitializeApp() (*App, error) {
wire.Bind(new(resourceQualifiers.QualifierMappingService), new(*resourceQualifiers.QualifierMappingServiceImpl)),

argocdServer.NewArgoClientWrapperServiceImpl,
argocdServer.NewArgoClientWrapperServiceEAImpl,
wire.Bind(new(argocdServer.ArgoClientWrapperService), new(*argocdServer.ArgoClientWrapperServiceImpl)),

pipeline.NewPluginInputVariableParserImpl,
Expand All @@ -937,7 +940,7 @@ func InitializeApp() (*App, error) {
wire.Bind(new(imageDigestPolicy.ImageDigestPolicyService), new(*imageDigestPolicy.ImageDigestPolicyServiceImpl)),

certificate.NewServiceClientImpl,
wire.Bind(new(certificate.Client), new(*certificate.ServiceClientImpl)),
wire.Bind(new(certificate.ServiceClient), new(*certificate.ServiceClientImpl)),

appStoreRestHandler.FullModeWireSet,

Expand All @@ -950,14 +953,17 @@ func InitializeApp() (*App, error) {
common.NewDeploymentConfigServiceImpl,
wire.Bind(new(common.DeploymentConfigService), new(*common.DeploymentConfigServiceImpl)),

argoRepositoryCreds.NewRepositorySecret,
wire.Bind(new(argoRepositoryCreds.RepositorySecret), new(*argoRepositoryCreds.RepositorySecretImpl)),
repoCredsK8sClient.NewRepositoryCredsK8sClientImpl,
wire.Bind(new(repoCredsK8sClient.RepositoryCredsK8sClient), new(*repoCredsK8sClient.RepositoryCredsK8sClientImpl)),

repocreds.NewServiceClientImpl,
wire.Bind(new(repocreds.ServiceClient), new(*repocreds.ServiceClientImpl)),

dbMigration.NewDbMigrationServiceImpl,
wire.Bind(new(dbMigration.DbMigration), new(*dbMigration.DbMigrationServiceImpl)),

config.NewArgoCDConfigGetter,
wire.Bind(new(config.ArgoCDConfigGetter), new(*config.ArgoCDConfigGetterImpl)),
)
return &App{}, nil
}
30 changes: 11 additions & 19 deletions api/appStore/InstalledAppRestHandler.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,22 +21,22 @@ import (
"encoding/json"
"errors"
"fmt"
bean2 "github.com/devtron-labs/devtron/api/bean/AppView"
client "github.com/devtron-labs/devtron/api/helm-app/gRPC"
"github.com/devtron-labs/devtron/pkg/appStore/installedApp/service/FullMode"
"github.com/devtron-labs/devtron/pkg/appStore/installedApp/service/FullMode/deploymentTypeChange"
"github.com/devtron-labs/devtron/pkg/appStore/installedApp/service/FullMode/resource"
util3 "github.com/devtron-labs/devtron/pkg/appStore/util"
"github.com/devtron-labs/devtron/pkg/bean"
"github.com/devtron-labs/devtron/pkg/cluster/read"
"gopkg.in/go-playground/validator.v9"
"net/http"
"reflect"
"strconv"
"strings"
"time"

bean2 "github.com/devtron-labs/devtron/api/bean"
"github.com/devtron-labs/devtron/api/restHandler/common"
"github.com/devtron-labs/devtron/client/argocdServer/application"
"github.com/devtron-labs/devtron/client/cron"
"github.com/devtron-labs/devtron/internal/constants"
"github.com/devtron-labs/devtron/internal/middleware"
Expand All @@ -50,7 +50,6 @@ import (
"github.com/devtron-labs/devtron/pkg/auth/user"
"github.com/devtron-labs/devtron/pkg/cluster"
"github.com/devtron-labs/devtron/util"
"github.com/devtron-labs/devtron/util/argo"
"github.com/devtron-labs/devtron/util/rbac"
"github.com/devtron-labs/devtron/util/response"
"github.com/go-pg/pg"
Expand Down Expand Up @@ -84,29 +83,30 @@ type InstalledAppRestHandlerImpl struct {
chartGroupService chartGroup.ChartGroupService
validator *validator.Validate
clusterService cluster.ClusterService
acdServiceClient application.ServiceClient
appStoreDeploymentService service.AppStoreDeploymentService
appStoreDeploymentDBService service.AppStoreDeploymentDBService
helmAppClient client.HelmAppClient
argoUserService argo.ArgoUserService
cdApplicationStatusUpdateHandler cron.CdApplicationStatusUpdateHandler
installedAppRepository repository.InstalledAppRepository
appCrudOperationService app2.AppCrudOperationService
installedAppDeploymentTypeChangeService deploymentTypeChange.InstalledAppDeploymentTypeChangeService
clusterReadService read.ClusterReadService
}

func NewInstalledAppRestHandlerImpl(Logger *zap.SugaredLogger, userAuthService user.UserService,
enforcer casbin.Enforcer, enforcerUtil rbac.EnforcerUtil, enforcerUtilHelm rbac.EnforcerUtilHelm,
installedAppService FullMode.InstalledAppDBExtendedService,
installedAppResourceService resource.InstalledAppResourceService,
chartGroupService chartGroup.ChartGroupService, validator *validator.Validate, clusterService cluster.ClusterService,
acdServiceClient application.ServiceClient, appStoreDeploymentService service.AppStoreDeploymentService,
appStoreDeploymentService service.AppStoreDeploymentService,
appStoreDeploymentDBService service.AppStoreDeploymentDBService,
helmAppClient client.HelmAppClient, argoUserService argo.ArgoUserService,
helmAppClient client.HelmAppClient,

cdApplicationStatusUpdateHandler cron.CdApplicationStatusUpdateHandler,
installedAppRepository repository.InstalledAppRepository,
appCrudOperationService app2.AppCrudOperationService,
installedAppDeploymentTypeChangeService deploymentTypeChange.InstalledAppDeploymentTypeChangeService) *InstalledAppRestHandlerImpl {
installedAppDeploymentTypeChangeService deploymentTypeChange.InstalledAppDeploymentTypeChangeService,
clusterReadService read.ClusterReadService) *InstalledAppRestHandlerImpl {
return &InstalledAppRestHandlerImpl{
Logger: Logger,
userAuthService: userAuthService,
Expand All @@ -118,15 +118,14 @@ func NewInstalledAppRestHandlerImpl(Logger *zap.SugaredLogger, userAuthService u
chartGroupService: chartGroupService,
validator: validator,
clusterService: clusterService,
acdServiceClient: acdServiceClient,
appStoreDeploymentService: appStoreDeploymentService,
appStoreDeploymentDBService: appStoreDeploymentDBService,
helmAppClient: helmAppClient,
argoUserService: argoUserService,
cdApplicationStatusUpdateHandler: cdApplicationStatusUpdateHandler,
installedAppRepository: installedAppRepository,
appCrudOperationService: appCrudOperationService,
installedAppDeploymentTypeChangeService: installedAppDeploymentTypeChangeService,
clusterReadService: clusterReadService,
}
}
func (handler *InstalledAppRestHandlerImpl) FetchAppOverview(w http.ResponseWriter, r *http.Request) {
Expand Down Expand Up @@ -550,7 +549,7 @@ func (impl *InstalledAppRestHandlerImpl) DefaultComponentInstallation(w http.Res
return
}
impl.Logger.Errorw("request payload, DefaultComponentInstallation", "clusterId", clusterId)
cluster, err := impl.clusterService.FindById(clusterId)
cluster, err := impl.clusterReadService.FindById(clusterId)
if err != nil {
impl.Logger.Errorw("service err, DefaultComponentInstallation", "error", err, "clusterId", clusterId)
common.WriteJsonResp(w, err, nil, http.StatusInternalServerError)
Expand Down Expand Up @@ -648,13 +647,6 @@ func (handler *InstalledAppRestHandlerImpl) DeleteArgoInstalledAppWithNonCascade
return
}
//rback block ends here
acdToken, err := handler.argoUserService.GetLatestDevtronArgoCdUserToken()
if err != nil {
handler.Logger.Errorw("error in getting acd token", "err", err)
common.WriteJsonResp(w, err, nil, http.StatusInternalServerError)
return
}
ctx := context.WithValue(r.Context(), "token", acdToken)
request := &appStoreBean.InstallAppVersionDTO{}
request.InstalledAppId = installedAppId
request.AppName = installedApp.AppName
Expand All @@ -668,7 +660,7 @@ func (handler *InstalledAppRestHandlerImpl) DeleteArgoInstalledAppWithNonCascade
request.Namespace = installedApp.Namespace
request.AcdPartialDelete = true

request, err = handler.appStoreDeploymentService.DeleteInstalledApp(ctx, request)
request, err = handler.appStoreDeploymentService.DeleteInstalledApp(r.Context(), request)
if err != nil {
handler.Logger.Errorw("service err, DeleteInstalledApp", "err", err, "installAppId", installedAppId)
common.WriteJsonResp(w, err, nil, http.StatusInternalServerError)
Expand Down
Loading
Loading