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

Allow custom ServingRuntimes #92

Merged

Conversation

hansinikarunarathne
Copy link
Contributor

fix issue with #47 PR

Signed-off-by: hansinikarunarathne <hansini.20@cse.mrt.ac.lk>
@hansinikarunarathne
Copy link
Contributor Author

@juliusvonkohout can you please review this? Then I can fix the other two PRs which give the same issue.

@juliusvonkohout
Copy link
Contributor

I see

 make: *** No rule to make target 'prettier-check'.  Stop.
Error: Process completed with exit code 2.

Signed-off-by: Hansini Karunarathne <107214435+hansinikarunarathne@users.noreply.github.com>
@hansinikarunarathne
Copy link
Contributor Author

I see

 make: *** No rule to make target 'prettier-check'.  Stop.
Error: Process completed with exit code 2.

can you run it again @juliusvonkohout

@juliusvonkohout
Copy link
Contributor

@hansinikarunarathne can you test this image once with our kserve workflow and check whether it works in the UI? https://github.com/kubeflow/manifests/blob/master/.github/workflows/kserve_m2m_test.yaml
please upload a screenshot

@juliusvonkohout
Copy link
Contributor

/ok-to-test

Signed-off-by: hansinikarunarathne <hansini.20@cse.mrt.ac.lk>
@hansinikarunarathne
Copy link
Contributor Author

hansinikarunarathne commented Sep 3, 2024

@juliusvonkohout can you please approve to run the test

@hansinikarunarathne
Copy link
Contributor Author

@hansinikarunarathne can you test this image once with our kserve workflow and check whether it works in the UI? https://github.com/kubeflow/manifests/blob/master/.github/workflows/kserve_m2m_test.yaml please upload a screenshot

can you explain this bit more ?

@hansinikarunarathne
Copy link
Contributor Author

If there is no any concerns regarding this PR, please approve this

@juliusvonkohout
Copy link
Contributor

/lgtm
/approve

@juliusvonkohout juliusvonkohout merged commit 4675bb9 into kserve:master Sep 3, 2024
8 checks passed
@juliusvonkohout
Copy link
Contributor

@hansinikarunarathne did you not test this locally as discussed? otherwise i have to revert it.

@juliusvonkohout
Copy link
Contributor

Please create a PR to revert the change (keep the test fixes) if it does not pass the tests https://github.com/kubeflow/manifests/blob/master/.github/workflows/kserve_m2m_test.yaml and https://github.com/kubeflow/manifests/blob/master/.github/workflows/kserve_test.yaml

It should be in the workflows above and here is the explicit file
kserve_test.yaml

They reported issues here with the PR #47 (comment) and you need to make sure that there is no regression.

juliusvonkohout pushed a commit that referenced this pull request Sep 4, 2024
* Revert "Allow custom ServingRuntimes (#92)"

This reverts commit 4675bb9.

Signed-off-by: hansinikarunarathne <hansini.20@cse.mrt.ac.lk>

* Add prettier to makefile

Signed-off-by: hansinikarunarathne <hansini.20@cse.mrt.ac.lk>

---------

Signed-off-by: hansinikarunarathne <hansini.20@cse.mrt.ac.lk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants