feat(docs): Changelog for Kubeflow Trainer v2.2.0#3369
feat(docs): Changelog for Kubeflow Trainer v2.2.0#3369google-oss-prow[bot] merged 1 commit intokubeflow:masterfrom
Conversation
Signed-off-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>
|
@andreyvelich: GitHub didn't allow me to request PR reviews from the following users: vsoch, andreyvelich, robert-bell, Krishna-kg732, Goku2099, XploY04, sameerdattav, Sanskarzz, kapil27. Note that only kubeflow members and repo collaborators can review this PR, and authors cannot review their own PRs. DetailsIn response to this:
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. |
There was a problem hiding this comment.
Pull request overview
This PR finalizes the Kubeflow Trainer v2.2.0 release documentation by moving version references from the release candidate to the stable tag and adding the v2.2.0 changelog entry.
Changes:
- Bump release/version markers from
2.2.0-rc.0to2.2.0across the Helm chart, Python API package, and rootVERSIONfile. - Add the v2.2.0 release notes section to
CHANGELOG.md. - Update the Helm chart README version badge to reflect the stable release.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| charts/kubeflow-trainer/README.md | Updates chart version badge for the stable v2.2.0 release. |
| charts/kubeflow-trainer/Chart.yaml | Bumps Helm chart version to 2.2.0. |
| api/python_api/kubeflow_trainer_api/init.py | Updates Python package __version__ to 2.2.0. |
| VERSION | Updates repository release version marker to v2.2.0. |
| CHANGELOG.md | Adds v2.2.0 release notes and links. |
| # kubeflow-trainer | ||
|
|
||
|   | ||
|   |
There was a problem hiding this comment.
The Helm install examples in this README still reference version 2.1.0 even though the chart badge (and Chart.yaml) were bumped to 2.2.0; update the example version numbers to 2.2.0 to avoid users installing the wrong release.
| # kubeflow-trainer | ||
|
|
||
|   | ||
|   |
There was a problem hiding this comment.
The torch-distributed-with-cache example text says to enable dataCache.runtimes.torchDistributed.enabled, but the values key is dataCache.runtimes.torchDistributedWithCache.enabled; fix the key name in the comment to match values.yaml so users can enable the runtime correctly.
|
|
||
| ## Breaking Changes | ||
|
|
||
| - feat(api): BREAKING CHANGE: Replace PodTemplateOverrides with RuntimePatches API ([#3309](https://github.com/kubeflow/trainer/pull/3309) by @astefanutti) |
There was a problem hiding this comment.
There are two spaces before the PR link in this bullet ("API ([#3309]"); remove the extra space to keep changelog formatting consistent.
| - feat(api): BREAKING CHANGE: Replace PodTemplateOverrides with RuntimePatches API ([#3309](https://github.com/kubeflow/trainer/pull/3309) by @astefanutti) | |
| - feat(api): BREAKING CHANGE: Replace PodTemplateOverrides with RuntimePatches API ([#3309](https://github.com/kubeflow/trainer/pull/3309) by @astefanutti) |
astefanutti
left a comment
There was a problem hiding this comment.
/lgtm
Thanks all, amazing release!
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andreyvelich The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes: #3116
Kubeflow Trainer v2.2 has been released. Thanks everyone for your contributions 🎉
/cc @andreyvelich @astefanutti @kaisoz @robert-bell @akshaychitneni @vsoch @Krishna-kg732 @XploY04 @sameerdattav @rudeigerc @Sanskarzz @Fiona-Waters @Goku2099 @jaiakash @kannon92 @kapil27
@milinddethe15 @khushiiagrawal @adity1raut @Snehadas2005 @Ishtiyaque-Alam @ryanHwH20 @sksingh2005 @ChughShilpa @aniketpati1121 @yosri-brh @puwun @ghazariann @Amir380-A @krishdef7