[GitHub] [flink-kubernetes-operator] mbalassi commented on a change in pull request #126: Add helm packaging to create_source_release.sh script

[GitBox <gi...@apache.org>]

mbalassi commented on a change in pull request #126:
URL: https://github.com/apache/flink-kubernetes-operator/pull/126#discussion_r837152978



##########
File path: tools/releasing/create_source_release.sh
##########
@@ -77,10 +77,29 @@ rsync -a \
   --exclude ".travis.yml" \
   . flink-kubernetes-operator-${RELEASE_VERSION}
 
+# Package sources
 tar czf ${RELEASE_DIR}/flink-kubernetes-operator-${RELEASE_VERSION}-src.tgz flink-kubernetes-operator-${RELEASE_VERSION}
 gpg --armor --detach-sig ${RELEASE_DIR}/flink-kubernetes-operator-${RELEASE_VERSION}-src.tgz
+
+# Package helm chart
+commit_hash=$(git log -1 --pretty=format:%h)
+
+# TODO: We might want to be more specific here later on what to replace
+perl -pi -e "s#^  repository: .*#  repository: ghcr.io/apache/flink-operator#" helm/flink-operator/values.yaml
+perl -pi -e "s#^  tag: .*#  tag: ${commit_hash}#" helm/flink-operator/values.yaml

Review comment:
       We can certainly improve this once we get some more experience with this, but currently this is the approach we can use for this preview release candidate.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org