Skip to content

Commit

Permalink
[CELEBORN-971][HELM] Should update Charts appVersion when we update p…
Browse files Browse the repository at this point in the history
…roject version

### What changes were proposed in this pull request?
Should update Charts appVersion when we update project version and we should change this appVersion in future release work.

### Why are the changes needed?
appVersion means:
>This is the version number of the application being deployed. This version number should be incremented each time you make changes to the application. Versions are not expected to follow Semantic Versioning. They should reflect the version the application is using.

### Does this PR introduce _any_ user-facing change?
Yes, user will find appVersion changed when using celeborn chart

### How was this patch tested?
Local

Closes apache#1904 from zwangsheng/CELEBORN-971.

Authored-by: zwangsheng <[email protected]>
Signed-off-by: zky.zhoukeyong <[email protected]>
  • Loading branch information
zwangsheng authored and waitinfuture committed Sep 19, 2023
1 parent 1e49ff7 commit b210698
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion charts/celeborn/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,4 +39,4 @@ version: 0.1.0
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
appVersion: 0.2.1
appVersion: 0.4.0

0 comments on commit b210698

Please sign in to comment.