chore(chart-deps): update kserve to version v0.20.0 - #3497
Draft
svcAPLBot wants to merge 4 commits into
Draft
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR bumps the vendored/managed KServe Helm artifacts to align with kserve-resources v0.20.0, updating version references across the chart metadata, dependency pinning, and associated documentation/config defaults.
Changes:
- Update KServe chart/dependency version references from v0.19.0 → v0.20.0.
- Update README badges/examples and apps catalog metadata to reflect 0.20.0.
- Extend the KServe controller ConfigMap defaults with new storage initializer OCI settings (
enableOciModelSupport,ociModelMode).
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| charts/kserve/values.yaml | Bumps default kserve.version to v0.20.0. |
| charts/kserve/README.md | Updates version badges and install example to v0.20.0. |
| charts/kserve/files/common/configmap.yaml | Adds v0.20.0 storage initializer OCI-related settings. |
| charts/kserve/Chart.yaml | Updates chart version and appVersion to v0.20.0. |
| charts/dependencies.yaml | Pins kserve-crd and kserve-resources to v0.20.0. |
| apps.yaml | Updates KServe app catalog entry version to 0.20.0. |
Suppressed comments (1)
charts/kserve/files/common/configmap.yaml:161
data.storageInitializeris defined twice in this ConfigMap (the second definition starting at line 161 overrides the earlier one). Duplicate mapping keys are invalid YAML and can break tooling; it also makes it unclear which initializer config is intended to be used.
"ociModelMode": "modelcar",
"cpuModelcar": "10m",
"memoryModelcar": "15Mi"
}
storageInitializer: |-
Comment on lines
155
to
+157
| "enableModelcar": false, | ||
| "enableOciModelSupport": false, | ||
| "ociModelMode": "modelcar", |
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.
Suppressed comments (1)
charts/dependencies.yaml:67
- PR description says it updates only the kserve-resources dependency, but this change also bumps kserve-crd to v0.20.0. Please update the PR description (or split the change) so reviewers/users understand both dependencies are being upgraded together.
- name: kserve-crd
version: v0.20.0
repository: oci://ghcr.io/kserve/charts/kserve-crd
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR updates the dependency kserve-resources to version v0.20.0.