Skip to content

Commit

Permalink
properly set new chart version
Browse files Browse the repository at this point in the history
  • Loading branch information
fharper committed Jul 10, 2023
1 parent 46757aa commit 52e2676
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions .argo/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,7 @@ spec:
- name: chartDir
value: 'charts/{{workflow.parameters.appName}}'
- name: chartVersion
value: '{{steps.get-initial-chart-version.outputs.result}}'
- name: shortSha
value: '{{workflow.parameters.shortSha}}'
value: '{{steps.get-initial-chart-version.outputs.result}}-rc.{{workflow.parameters.shortSha}}'
- - name: publish-helm-chart
templateRef:
name: cwft-helm
Expand Down Expand Up @@ -78,7 +76,7 @@ spec:
- name: fullChartPath
value: 'registry/{{workflow.parameters.clusterName}}/components/{{workflow.parameters.environment}}/{{workflow.parameters.appName}}/Chart.yaml'
- name: chartVersion
value: '{{steps.get-initial-chart-version.outputs.result}}-rc.{{workflow.parameters.shortSha}}'
value: '{{steps.get-initial-chart-version.outputs.result}}'
- name: environment
value: '{{workflow.parameters.environment}}'
- - name: commit-production
Expand All @@ -96,7 +94,7 @@ spec:
- name: gitUrlNoProtocol
value: '{{workflow.parameters.gitUrlNoProtocol}}'
- name: commitMessage
value: 'setting {{workflow.parameters.appName}} {{workflow.parameters.environment}} to chart version {{steps.get-initial-chart-version.outputs.result}}'
value: 'setting {{workflow.parameters.appName}} {{workflow.parameters.environment}} to chart version {{steps.set-chart-versions.parameters.chartVersion}}'
- - name: increment-chart-minor
templateRef:
name: cwft-helm
Expand All @@ -112,7 +110,7 @@ spec:
- name: chartDir
value: 'charts/{{workflow.parameters.appName}}'
- name: chartVersion
value: '{{steps.get-initial-chart-version.outputs.result}}'
value: '{{steps.set-chart-versions.parameters.chartVersion}}'
- - name: get-published-chart-version
templateRef:
name: cwft-helm
Expand Down

0 comments on commit 52e2676

Please sign in to comment.