Skip to content

Commit f525222

Browse files
authored
Merge pull request #17 from cloudogu/feature/update_readme_helm_git_notes
update helm deployment with notes for git repotype
2 parents b15be9f + 4ca6696 commit f525222

1 file changed

Lines changed: 8 additions & 1 deletion

File tree

README.md

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -527,7 +527,7 @@ def gitopsConfig = [
527527
deployments: [
528528
helm: [
529529
repoType: 'GIT',
530-
repoUrl: "https://git-repo/namespace/name",
530+
repoUrl: "https://git-repo/namespace/name", // git-repo must have a 'main' branch due to limitations of the git-step within jenkins
531531
credentialsId: 'creds',
532532
version: '1.2.3', // tag, commit or branch
533533
chartPath: 'chart', // the path relative to root in the git repo. If the chart is at root level you can ommit this property
@@ -537,6 +537,13 @@ def gitopsConfig = [
537537
]
538538
```
539539

540+
**Note:**
541+
542+
Due to limitations in the git-step of Jenkins, we have to clone from a specific branch rather then having the
543+
git client checkout the default branch given within the HEADs meta information. This specific branch is the
544+
`main` branch. Make sure the git-repository has a main-branch, else the deployment step will fail. After a successful clone
545+
it checks out the given version as expected.
546+
540547
#### Conventions for helm deployment
541548
- Application name is used as the release-name in Flux (not for argo, argo creates plain resources using `helm template`)
542549
> Helm Release name = `${application}`

0 commit comments

Comments
 (0)