[EPMDEDP-12267]: fix: Set default deploymentScript for codebase to he…

…lm-chart Let's be more agile and do not make this field mandatory, just not to break backward API compatibility. As the next step we should be more restrictive and set this field to enum with validation check Signed-off-by: Sergiy Kulanov <[email protected]> Change-Id: I300043b14db2d20c4389d6944fc19944a5eee30c
epam · Jul 17, 2023 · 87a4711 · 87a4711
1 parent 2972121
commit 87a4711
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 0 deletions.
diff --git a/api/v1/codebase_types.go b/api/v1/codebase_types.go
@@ -90,6 +90,7 @@ type CodebaseSpec struct {
 	JobProvisioning *string `json:"jobProvisioning"`
 
 	// +optional
+	// +kubebuilder:default:=helm-chart
 	DeploymentScript string `json:"deploymentScript,omitempty"`
 
 	Versioning Versioning `json:"versioning"`

diff --git a/config/crd/bases/v2.edp.epam.com_codebases.yaml b/config/crd/bases/v2.edp.epam.com_codebases.yaml
@@ -76,6 +76,7 @@ spec:
                 description: Name of default branch.
                 type: string
               deploymentScript:
+                default: helm-chart
                 type: string
               description:
                 description: A short description of codebase.

diff --git a/deploy-templates/crds/v2.edp.epam.com_codebases.yaml b/deploy-templates/crds/v2.edp.epam.com_codebases.yaml
@@ -76,6 +76,7 @@ spec:
                 description: Name of default branch.
                 type: string
               deploymentScript:
+                default: helm-chart
                 type: string
               description:
                 description: A short description of codebase.

diff --git a/docs/api.md b/docs/api.md
@@ -795,6 +795,8 @@ CodebaseSpec defines the desired state of Codebase.
         <td>string</td>
         <td>
           <br/>
+          <br/>
+            <i>Default</i>: helm-chart<br/>
         </td>
         <td>false</td>
       </tr><tr>