Skip to content
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

[pull] master from kserve:master #67

Closed
wants to merge 23 commits into from
Closed

Conversation

pull[bot]
Copy link

@pull pull bot commented Aug 24, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

sivanantha321 and others added 21 commits July 13, 2023 20:15
* Loosen ray range

Signed-off-by: ddelange <[email protected]>

* Fix import of RayServeHandle

ref ray-project/ray#34714

Signed-off-by: ddelange <[email protected]>

* Allow pandas 2.0

Signed-off-by: ddelange <[email protected]>

* Remove exclusion markers

Signed-off-by: ddelange <[email protected]>

* Run poetry lock --no-update

Signed-off-by: ddelange <[email protected]>

---------

Signed-off-by: ddelange <[email protected]>
Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
…d` (#3049)

* Create a separate action to free-up disk space on GH action runner
* Move 'free-up-disk-space' step outside 'minikube-setup' action
* Free-up disk space before running 'predictor-runtime-build'

Signed-off-by: Christian Kadner <[email protected]>
* Bump scikit-learn version to 1.3.0

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Update test models

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Downgrade ray serve to 2.4

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

---------

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
* template out kube-rbac-proxy image

Signed-off-by: Tessa Pham <[email protected]>

* add kserve-addressable-resolver to clusterroles

Signed-off-by: Tessa Pham <[email protected]>

---------

Signed-off-by: Tessa Pham <[email protected]>
Except for Paddle, which does not support arm64/aarch64

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
* Add annotation for secret name overrides

Signed-off-by: Dan Sun <[email protected]>

* Remove the field

Signed-off-by: Dan Sun <[email protected]>

* Fix secret name annotation

Signed-off-by: Dan Sun <[email protected]>

* Update kserve url storage example docs

Signed-off-by: Dan Sun <[email protected]>

* Address review comments

Signed-off-by: Dan Sun <[email protected]>

* Update config docs for storage secret

Signed-off-by: Dan Sun <[email protected]>

* Fix storage secret name

Signed-off-by: Dan Sun <[email protected]>

* Add test for secret name annotation override

Signed-off-by: Dan Sun <[email protected]>

* Fix flake8 lint

Signed-off-by: Dan Sun <[email protected]>

---------

Signed-off-by: Dan Sun <[email protected]>
Signed-off-by: Andrews Arokiam <[email protected]>
…3069)

* document new status conditions

Signed-off-by: Tessa Pham <[email protected]>

* make generate

Signed-off-by: Tessa Pham <[email protected]>

* fix linting

Signed-off-by: Tessa Pham <[email protected]>

* remove requirements.txt in python

Signed-off-by: Tessa Pham <[email protected]>

* comment out unnecessary git checkout lines

Signed-off-by: Tessa Pham <[email protected]>

* indicate conditions for serverless mode only

Signed-off-by: Tessa Pham <[email protected]>

* go mod tidy

Signed-off-by: Tessa Pham <[email protected]>

---------

Signed-off-by: Tessa Pham <[email protected]>
The update was done by running the following commands in both the
project root and also inside `qpext/`:

```
go get -u golang.org/x/net/html
go mod tidy
```

This is to move away from CVE-2023-3978.

Signed-off-by: Edgar Hernández <[email protected]>
* Fix kserve kubeflow crd

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Generate kubeflow release artifact

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

---------

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
* Use knative operator for installing knative in e2e tests

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Get logs for debugging

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Configure cni for minikube

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Upgrade minikube and knative operator

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Fix triton test fails due to insufficient resources

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Fix kourier namespace

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Use yaml configuration instead of knative cli

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

---------

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
* Storage container CRD

Signed-off-by: Jin Dong <[email protected]>

* Add CRD to helm

Signed-off-by: Jin Dong <[email protected]>

* Add default CR

Signed-off-by: Jin Dong <[email protected]>

* Fix default storage container cr

Signed-off-by: Jin Dong <[email protected]>

* Update kserve-manager-role in helm charts

Signed-off-by: Jin Dong <[email protected]>

* Update crd format, modify controller code and tests

Signed-off-by: Jin Dong <[email protected]>

* Update helm charts

Signed-off-by: Jin Dong <[email protected]>

* fix cluster role

Signed-off-by: Jin Dong <[email protected]>

* Remove name from CRD

Signed-off-by: Jin Dong <[email protected]>

* Update CRD in helm

Signed-off-by: Jin Dong <[email protected]>

* Error handling

Signed-off-by: Jin Dong <[email protected]>

* Fix kubebuilder rbac annotation

Signed-off-by: Jin Dong <[email protected]>

* Add comments

Signed-off-by: Jin Dong <[email protected]>

* Fix test

Signed-off-by: Jin Dong <[email protected]>

* Update comments

Signed-off-by: Jin Dong <[email protected]>

---------

Signed-off-by: Jin Dong <[email protected]>
@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-ci
Copy link

openshift-ci bot commented Aug 24, 2023

Hi @pull[bot]. Thanks for your PR.

I'm waiting for a opendatahub-io member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@openshift-ci
Copy link

openshift-ci bot commented Aug 24, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pull[bot]
Once this PR has been reviewed and has the lgtm label, please assign vaibhavjainwiz for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@pull pull bot removed the needs-rebase label Aug 28, 2023
* List Models v2 REST API

Signed-off-by: Jagan Vujjini <[email protected]>
Signed-off-by: jvujjini <[email protected]>

* Fixed lint error

Signed-off-by: jvujjini <[email protected]>

---------

Signed-off-by: Jagan Vujjini <[email protected]>
Signed-off-by: jvujjini <[email protected]>
Co-authored-by: Jagan Vujjini <[email protected]>
@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

* Introduce priority field

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Implement validator webhook

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Add v2 support and priority to servingruntime yaml

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Fix protocol version in serving runtime sorting and reverse sort the serving runtime

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Add v2 protocol support in servingruntime charts

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Add webhooks to helm chart

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Add tests

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* make generate

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* fix supported modelformat sort

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* fix helm chart

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* rebase master

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Make model name comparision case insensitive

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

* Resolve comments

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>

---------

Signed-off-by: Sivanantham Chinnaiyan <[email protected]>
@openshift-merge-robot
Copy link

PR needs rebase.

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.

@openshift-merge-robot
Copy link

PR needs rebase.

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.

@pull pull bot removed the needs-rebase label Sep 5, 2023
@israel-hdez
Copy link

Superseeded by #70.

@israel-hdez israel-hdez closed this Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⤵️ pull merge-conflict Resolve conflicts manually
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.