Add a workflow for publishing Helm charts
What you would like to be added?
Add a GitHub action workflow for:
-
Publishing helm charts to GHCR, so user can install the trainer Helm chart by specifying OCI image repository and version:
helm install kubeflow-trainer oci://ghcr.io/kubeflow/helm-charts/kubeflow-trainer --version 2.0.0 \ --namespace kubeflow-system \ --create-namespace -
(Optional) Using GH page to host the Helm charts index file
index.yaml, so users are able to add trainer as a Helm chart repo like:helm repo add kubeflow-trainer https://kubeflow.github.io/trainerIn this way, users only need to add the Helm repo once and then they can:
# Update Helm repo index helm repo update # Search repo for available versions of kubeflow-trainer helm search repo kubeflow-trainer --versions # Install the latest version helm install kubeflow-trainer kubeflow-trainer/kubeflow-trainer \ --namespace kubeflow-system \ --create-namespace # Install a specified version helm install kubeflow-trainer kubeflow-trainer/kubeflow-trainer \ --version 2.0.0 \ --namespace kubeflow-system \ --create-namespace
Why is this needed?
Set a workflow to publish Helm charts automatically.
Love this feature?
Give it a š We prioritize the features with most š
/remove-label lifecycle/needs-triage /good-first-issue /area deployment
@andreyvelich: This request has been marked as suitable for new contributors.
Please ensure the request meets the requirements listed here.
If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.
In response to this:
/remove-label lifecycle/needs-triage /good-first-issue /area deployment
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
Hey I was looking to contribute to this issue. Can I start working on it?
Sure, thank you for your time @RasButAss! While we are finalizing the Helm Chart PR, you can start working on GitHub action to publish it: https://github.com/kubeflow/trainer/pull/2435 /assign @RasButAss
Hello @RasButAss , are you still working on this issue? Would love to work on this issue.
/assign
Hi I am looking to contribute to this project, is this issue still open?
Sure, that would be great @kris-gaudel! @izuku-sds Are you still working on this PR, or @kris-gaudel can take it over: https://github.com/kubeflow/trainer/pull/2571 ?
@andreyvelich if this issue is still actively being worked on, do you have any issues you'd recommend for a beginner like myself?
Hi @andreyvelich , due to other commitments, Iām currently unable to continue working on this issue. Iād be happy for @kris-gaudel to take it over. Some initial work has been done in PR #2571 ā feel free to refer to it,
/unassign @izuku-sds
/assign @kris-gaudel
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
@kris-gaudel Are you still working on this issue ? /remove-lifecycle stale
Unfortunately I'm not anymore
/unassign @kris-gaudel
/good-first-issue /help
We are still looking for contributors to help us with publishing Helm Charts for KF Trainer. /good-first-issue
@andreyvelich: This request has been marked as suitable for new contributors.
Please ensure the request meets the requirements listed here.
If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.
In response to this:
We are still looking for contributors to help us with publishing Helm Charts for KF Trainer. /good-first-issue
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
@adity1raut If you are interested, feel free to take over this issue! @izuku-sds Has pending PR that you should just finalize: https://github.com/kubeflow/trainer/pull/2571
@adity1raut If you are interested, feel free to take over this issue! @izuku-sds Has pending PR that you should just finalize: #2571
Yes @andreyvelich Please Assign me
/assign
Hi @adity1raut, are you still working on publishing Trainer Helm Charts ? It would be great to have charts published, so users can use Helm to install KF Trainer v2.1 cc @kubeflow/kubeflow-trainer-team
Hi @adity1raut, are you still working on publishing Trainer Helm Charts ? It would be great to have charts published, so users can use Helm to install KF Trainer v2.1 cc @kubeflow/kubeflow-trainer-team
Yes @andreyvelich
Sounds great @adity1raut! We are planning to release v2.1 by end of October, so it would be good to include this feature.
/milestone v2.1
Hi @adity1raut, do we have any updates for the Helm publishing PR ?
Fixed by: https://github.com/kubeflow/trainer/pull/2906 /close
@andreyvelich: Closing this issue.
In response to this:
Fixed by: https://github.com/kubeflow/trainer/pull/2906 /close
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.