Skip to content

Commit

Permalink
Merge pull request #163 from openinfradev/byoh_minor_fix
Browse files Browse the repository at this point in the history
add parameter for creating lma
  • Loading branch information
cho4036 authored Oct 20, 2023
2 parents e34183e + 12974fb commit 7f09bf5
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions internal/usecase/app-group.go
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,9 @@ func (u *AppGroupUsecase) Create(ctx context.Context, dto domain.AppGroup) (id d

// check cloudAccount
tksCloudAccountId := ""
tksObjectStore := "minio"
if cluster.CloudService != domain.CloudService_BYOH {
tksObjectStore = "aws"
cloudAccounts, err := u.cloudAccountRepo.Fetch(cluster.OrganizationId, nil)
if err != nil {
return "", httpErrors.NewBadRequestError(fmt.Errorf("Failed to get cloudAccounts"), "", "")
Expand Down Expand Up @@ -126,6 +128,7 @@ func (u *AppGroupUsecase) Create(ctx context.Context, dto domain.AppGroup) (id d
"alert_tks=" + viper.GetString("external-address") + "/system-api/1.0/alerts",
"alert_slack=" + viper.GetString("alert-slack"),
"cloud_account_id=" + tksCloudAccountId,
"object_store=" + tksObjectStore,
}

switch dto.AppGroupType {
Expand Down

0 comments on commit 7f09bf5

Please sign in to comment.