-
Notifications
You must be signed in to change notification settings - Fork 998
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Volcano automatically releases the corresponding helm charts package #2823
Volcano automatically releases the corresponding helm charts package #2823
Conversation
Signed-off-by: wangyang <wangyang8126@gmail.com>
/priority important-soon |
Signed-off-by: wangyang <wangyang8126@gmail.com>
6492dd0
to
621f2c1
Compare
CHART_ORIGIN_PATH=${VK_ROOT}/installer/helm/chart | ||
CHART_OUT_PATH=${RELEASE_FOLDER}/chart | ||
cp -r "${CHART_ORIGIN_PATH}" "${CHART_OUT_PATH}" | ||
sed -i "s|image_tag_version: \"latest\"|image_tag_version: \"${VOLCANO_IMAGE_TAG}\"|g" $(find ${CHART_OUT_PATH} -type f | grep values.yaml) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I know, the sed command behaves inconsistently on linux and darwin, is there a way without using sed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done, refer to: #2848
#!/bin/bash | ||
|
||
# | ||
# Copyright 2021 The Volcano Authors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please refresh the date
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done, refer to: #2848
It's cool to generete the helm charts automatically :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: william-wang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description of Requirement:
refer: #2766,
Volcano can be installed through helm at present, but you need to download the source code of the entire warehouse, the operation is relatively troublesome, and it does not conform to the standard usage of helm.
Target:
Build the helm repo corresponding to Volcano, and automatically publish helm charts.
Users can view the latest released packages through
helm repo update
, and install and use them throughhelm install
accomplish:
(It is automatically completed through github action without manual intervention.)
(It is automatically completed through github action without manual intervention.)