Skip to content

Commit 8c8d68d

Browse files
Ci Build config API backward compatibilty (#2598)
* docker build config added as backward compatible changes * dependency injection fix
1 parent cf7b614 commit 8c8d68d

File tree

3 files changed

+27
-6
lines changed

3 files changed

+27
-6
lines changed

api/appbean/AppDetail.go

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,19 @@ type GitMaterial struct {
4242
}
4343

4444
type DockerConfig struct {
45-
DockerRegistry string `json:"dockerRegistry" validate:"required"`
46-
DockerRepository string `json:"dockerRepository" validate:"required"`
47-
CiBuildConfig *bean.CiBuildConfigBean `json:"ciBuildConfig" validate:"required"`
48-
CheckoutPath string `json:"checkoutPath"`
45+
DockerRegistry string `json:"dockerRegistry" validate:"required"`
46+
DockerRepository string `json:"dockerRepository" validate:"required"`
47+
CiBuildConfig *bean.CiBuildConfigBean `json:"ciBuildConfig"`
48+
DockerBuildConfig *DockerBuildConfig `json:"dockerBuildConfig,omitempty"` // Deprecated, should use CiBuildConfig for development
49+
CheckoutPath string `json:"checkoutPath"`
50+
}
51+
52+
type DockerBuildConfig struct {
53+
GitCheckoutPath string `json:"gitCheckoutPath,omitempty" validate:"required"`
54+
DockerfileRelativePath string `json:"dockerfileRelativePath,omitempty" validate:"required"`
55+
Args map[string]string `json:"args,omitempty"`
56+
TargetPlatform string `json:"targetPlatform"`
57+
DockerBuildOptions map[string]string `json:"dockerBuildOptions,omitempty"`
4958
}
5059

5160
type DeploymentTemplate struct {

api/restHandler/CoreAppRestHandler.go

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ import (
3838
chartRepoRepository "github.com/devtron-labs/devtron/pkg/chartRepo/repository"
3939
repository2 "github.com/devtron-labs/devtron/pkg/cluster/repository"
4040
"github.com/devtron-labs/devtron/pkg/pipeline"
41+
bean2 "github.com/devtron-labs/devtron/pkg/pipeline/bean"
4142
"github.com/devtron-labs/devtron/pkg/sql"
4243
"github.com/devtron-labs/devtron/pkg/team"
4344
"github.com/devtron-labs/devtron/pkg/user"
@@ -1244,7 +1245,18 @@ func (handler CoreAppRestHandlerImpl) createGitMaterials(appId int, gitMaterials
12441245
// create docker config
12451246
func (handler CoreAppRestHandlerImpl) createDockerConfig(appId int, dockerConfig *appBean.DockerConfig, userId int32) (error, int) {
12461247
handler.logger.Infow("Create App - creating docker config", "appId", appId, "DockerConfig", dockerConfig)
1247-
1248+
dockerBuildConfig := dockerConfig.DockerBuildConfig
1249+
if dockerBuildConfig != nil {
1250+
dockerConfig.CheckoutPath = dockerBuildConfig.GitCheckoutPath
1251+
dockerConfig.CiBuildConfig = &bean2.CiBuildConfigBean{
1252+
DockerBuildConfig: &bean2.DockerBuildConfig{
1253+
DockerfilePath: dockerBuildConfig.DockerfileRelativePath,
1254+
DockerBuildOptions: dockerBuildConfig.DockerBuildOptions,
1255+
Args: dockerBuildConfig.Args,
1256+
TargetPlatform: dockerBuildConfig.TargetPlatform,
1257+
},
1258+
}
1259+
}
12481260
createDockerConfigRequest := &bean.CiConfigRequest{
12491261
AppId: appId,
12501262
UserId: userId,

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.

0 commit comments

Comments
 (0)