Sync Upstream/master with ODH/master #471
Open
Conversation
This file contains 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
Signed-off-by: Jin Dong <[email protected]>
* squash commits Signed-off-by: Jin Dong <[email protected]> * localmodel agent -> localmodelnode agent Signed-off-by: Jin Dong <[email protected]> * Revert change Signed-off-by: Jin Dong <[email protected]> --------- Signed-off-by: Jin Dong <[email protected]>
* Fix model server fails to gracefully shutdown Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Update unit tests Signed-off-by: Sivanantham Chinnaiyan <[email protected]> --------- Signed-off-by: Sivanantham Chinnaiyan <[email protected]> Co-authored-by: Dan Sun <[email protected]>
kserve#4152) Ensure root model directory exists and add protection for # of jobs created Signed-off-by: Dan Sun <[email protected]>
Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
* updated package versions Signed-off-by: ayush <[email protected]> * updated go.mod and pip Signed-off-by: ayush <[email protected]> * updated poetry lock Signed-off-by: ayush <[email protected]> * refactored handling of task argument Signed-off-by: ayush <[email protected]> * removed go.mod changes Signed-off-by: ayush <[email protected]> --------- Signed-off-by: ayush <[email protected]>
* Add workflow for verifying go mod Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Merge with go lint workflow Signed-off-by: Sivanantham Chinnaiyan <[email protected]> --------- Signed-off-by: Sivanantham Chinnaiyan <[email protected]> Co-authored-by: Dan Sun <[email protected]>
upgrade aiohttp Signed-off-by: Andrews Arokiam <[email protected]> Co-authored-by: Dan Sun <[email protected]>
* Allow other engine builds other than docker chore: This change will allow user to use other engine builds. In addition, for Podman/Buildah engine builder, users can set the target architecture, useful when using ARM and there is a need to build x86 Container images. Can be done with `ENGINE=podman ARCH="--arch x86_64" make docker...` Signed-off-by: Spolti <[email protected]> * review changes Signed-off-by: Spolti <[email protected]> --------- Signed-off-by: Spolti <[email protected]>
Signed-off-by: Jin Dong <[email protected]>
* Fixes Non-linear parsing of case-insensitive content chore: Fixes CVE-2024-45338 Non-linear parsing of case-insensitive content in golang.org/x/net/html https://github.com/kserve/kserve/security/dependabot/1871 Signed-off-by: Spolti <[email protected]> * update go.sum Signed-off-by: Spolti <[email protected]> --------- Signed-off-by: Spolti <[email protected]>
* Helm chart - option to run daemonset as root Signed-off-by: Jin Dong <[email protected]> * make generate Signed-off-by: Jin Dong <[email protected]> * Revert default values Signed-off-by: Jin Dong <[email protected]> * Fix job count test Signed-off-by: Dan Sun <[email protected]> --------- Signed-off-by: Jin Dong <[email protected]> Signed-off-by: Dan Sun <[email protected]> Co-authored-by: Dan Sun <[email protected]>
Replace nodeGroup with nodeGroups Signed-off-by: Rituraj Singh <[email protected]> Co-authored-by: Rituraj Singh <[email protected]>
* Add affinity and tolerations to localmodel daemonset Signed-off-by: Jin Dong <[email protected]> * make generate Signed-off-by: Jin Dong <[email protected]> --------- Signed-off-by: Jin Dong <[email protected]>
…#4157) * sets virtual addressing style for legacy s3 endpoint Signed-off-by: bentohset <[email protected]> * add unit test Signed-off-by: bentohset <[email protected]> * fix formatting Signed-off-by: bentohset <[email protected]> * fix unit tests Signed-off-by: bentohset <[email protected]> --------- Signed-off-by: bentohset <[email protected]> Co-authored-by: Lize Cai <[email protected]>
* Make label and annotation propagation configurable chore: Make the DisallaowedAnnotations and Labels configurable through ConfigMap so users can configured it quickly. fixes kserve#3710 Signed-off-by: Spolti <[email protected]> * generate boilerplate code Signed-off-by: Spolti <[email protected]> * Edgar's review changes Signed-off-by: Spolti <[email protected]> --------- Signed-off-by: Spolti <[email protected]>
* Add client sdk for localmodelcache, localmodelnodegroup Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Add e2e test for modelcache Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Use docker driver and minikube tunnel Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Merge "Allow multiple node groups in the model cache CR (kserve#4134)" Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Try mounting image dir Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Add local model agent to image scan Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Debug Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Create model root directory beforehand Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Restart kserve controller after patch Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Enablepvc direct mount in e2e test Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Reduce pv storage to 1GB Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Update modelcache test Signed-off-by: Sivanantham Chinnaiyan <[email protected]> * Update status-check to include modelcache logs Signed-off-by: Sivanantham Chinnaiyan <[email protected]> --------- Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
Signed-off-by: Rajat Vig <[email protected]> Co-authored-by: Dan Sun <[email protected]>
* [bugfix] fix s3 storage download filename bug - ensure correct path and file name preservation during s3 downloads in storage-initializer Signed-off-by: Jaeyeon Kim <[email protected]> * update lint - fix format Signed-off-by: Jaeyeon Kim <[email protected]> * fix format Signed-off-by: Jaeyeon Kim <[email protected]> --------- Signed-off-by: Jaeyeon Kim <[email protected]>
Signed-off-by: tjandy98 <[email protected]>
* Add boilerplate setup for embeddings api Signed-off-by: Fabian Scheidt <[email protected]> * Integrate call to model with embedding request Signed-off-by: Fabian Scheidt <[email protected]> * Adjust error responses to be consistent with openai Signed-off-by: Fabian Scheidt <[email protected]> * Adjust doc strings to match implementation Signed-off-by: Fabian Scheidt <[email protected]> * Update openai input validation to use pydantic Signed-off-by: Fabian Scheidt <[email protected]> * Refactor embedding object type Signed-off-by: Fabian Scheidt <[email protected]> * Add e2e test for OpenAI embedding endpoint Signed-off-by: Fabian Scheidt <[email protected]> * Add unit test for OpenAI embeddings Signed-off-by: Fabian Scheidt <[email protected]> --------- Signed-off-by: Fabian Scheidt <[email protected]>
Signed-off-by: Matt Gottlieb <[email protected]> Co-authored-by: Sivanantham <[email protected]>
Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
Signed-off-by: Andrews Arokiam <[email protected]>
Signed-off-by: jooho lee <[email protected]> Co-authored-by: Sivanantham <[email protected]>
Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
Signed-off-by: jooho lee <[email protected]>
…4186) Signed-off-by: Rituraj Singh <[email protected]> Co-authored-by: Rituraj Singh <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: spolti 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 |
|
@spolti: The following tests failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
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.
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #
Type of changes
Please delete options that are not relevant.
Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.
Test A
Test B
Logs
Special notes for your reviewer:
Checklist:
Release note:
Re-running failed tests
/rerun-all- rerun all failed workflows./rerun-workflow <workflow name>- rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.