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
support PMML server #1141
support PMML server #1141
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
Hi @AnyISalIn. Thanks for your PR. I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the 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. |
@googlebot I signed it! |
/ok-to-test |
@animeshsingh Hi, I see that the build fails, The log in Prow only shows
Where can I see more detailed logs? |
I found the log of argo, It seems to be stuck in
|
cc @yuzisun |
@yuzisun Maybe we can try this solution.
|
/retest |
It seems that the main problem is the expansion caused by PodSpec, We might be able to learn from Tekton Like this
|
b8a97e2
to
3df4fe7
Compare
/retest |
1 similar comment
/retest |
fbdf0bc
to
0019151
Compare
0854851
to
8ca26d7
Compare
/retest |
@@ -5,7 +5,7 @@ XGBoostSpec defines arguments for configuring XGBoost model serving. | |||
Name | Type | Description | Notes | |||
------------ | ------------- | ------------- | ------------- | |||
**nthread** | **int** | Number of thread to be used by XGBoost | [optional] | |||
**resources** | [**V1ResourceRequirements**](https://github.com/kubernetes-client/python/blob/master/kubernetes/docs/V1ResourceRequirements.md) | Defaults to requests and limits of 1CPU, 2Gb MEM. | [optional] | |||
**resources** | [**V1ResourceRequirements**](V1ResourceRequirements.md) | Defaults to requests and limits of 1CPU, 2Gb MEM. | [optional] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jinchihe How are we generating these reference links?
83d967f
to
468645e
Compare
@yuzisun Hi, I already rebase to the latest, and the e2e test passed, when can we merge this PR? of course, I have another question, Do I still need to write the definition of v1alpha API? Thanks. |
468645e
to
82ede9f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AnyISalIn Great job, left a final comment otherwise good to merge.
82ede9f
to
5b6538e
Compare
@yuzisun How do we upload PMML samples to KFServing public GCS and release docker image to DockerHub? |
@AnyISalIn We will take care of the docker image publish when releasing 0.5, please also add a OWNER file under pmmlserver directory, I am assuming you are going to own and maintain this server going forward? |
@yuzisun I hope to be able to push pmmlserver image to KFServing public gcr.io, maintenance refers to maintaining code or docker image? |
5b6538e
to
d4c3b2f
Compare
@AnyISalIn I meant for maintaining the pmmlserver related code, OWNER file is for that purpose. Also I created the github action to push pmmlserver to KFS docker registry here #1198. |
Thanks @AnyISalIn for this great contribution! 🚀 /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AnyISalIn, yuzisun 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 |
What this PR does / why we need it:
This request will provide KFServing with PMML model support. It completes the PMML Server package based on pypmml and InferenceService PMML API definition, e2e test cases and some documents
Which issue(s) this PR fixes *
Fixes # #462
Special notes for your reviewer:
The CustomResourceDefinition "inferenceservices.serving.kubeflow.org" is invalid: metadata.annotations: Too long: must have at most 262144 bytes
, Exceeds the limit of kubernetes, so thekubectl apply
inmake deploy-dev
cannot take effect normally, need to usekubectl replace
orcreate
to replaceRelease note: