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

Pipeline API Server Swagger Client (Go) for Pipeline Upload returns incomplete output #122

Closed
vicaire opened this issue Nov 7, 2018 · 5 comments

Comments

@vicaire
Copy link
Contributor

vicaire commented Nov 7, 2018

The Pipeline API Server Swagger Client (Go) returns incomplete output for the pipeline.upload method.

For instance, using the CLI in the directory:

kubeflow/pipelines/backend/src/cmd/ml

With the command:

go run main.go pipeline --namespace kubeflow upload ./samples/hello-world.yaml --name pipleline87 -o json

We get the output:

{
"created_at": "0001-01-01T00:00:00.000Z",
"parameters": null
}

@vicaire
Copy link
Contributor Author

vicaire commented Nov 9, 2018

Raw request/response for the PIPELINE GET call:

GET /api/v1/namespaces/kubeflow/services/ml-pipeline:8888/proxy/apis/v1beta1/pipelines/1b02b12c-21ff-4216-a610-22c6a5b9881d HTTP/1.1
Host: 35.224.113.48
User-Agent: Go-http-client/1.1
Accept: application/json
Accept-Encoding: gzip

HTTP/2.0 200 OK
Content-Length: 98
Audit-Id: d1759187-0d5f-4351-b816-1072c5a30982
Content-Type: application/json
Date: Fri, 09 Nov 2018 06:21:00 GMT
Grpc-Metadata-Content-Type: application/grpc

{"id":"1b02b12c-21ff-4216-a610-22c6a5b9881d","created_at":"2018-11-09T06:20:22Z","name":"toto106"}
created_at: "2018-11-09T06:20:22.000Z"
id: 1b02b12c-21ff-4216-a610-22c6a5b9881d
name: toto106
parameters: null

@vicaire
Copy link
Contributor Author

vicaire commented Nov 9, 2018

Raw request/response for the PIPELINE UPLOAD call:

POST /api/v1/namespaces/kubeflow/services/ml-pipeline:8888/proxy/apis/v1beta1/pipelines/upload?name=toto106 HTTP/1.1
Host: 35.224.113.48
User-Agent: Go-http-client/1.1
Transfer-Encoding: chunked
Accept: application/json
Content-Type: multipart/form-data; boundary=0cf8a8d5b4b3e59d424a44a79e2ccc39362b82f6acff7d4fb621ee43f769
Accept-Encoding: gzip
Content-Disposition: form-data; name="uploadfile"; filename="hello-world.yaml"
Content-Type: application/octet-stream

HTTP/2.0 200 OK
Content-Length: 98
Audit-Id: 6a849149-4d3e-4cde-b8bd-ce26a5c2179e
Content-Type: text/plain; charset=utf-8
Date: Fri, 09 Nov 2018 06:20:23 GMT

{"id":"1b02b12c-21ff-4216-a610-22c6a5b9881d","name":"toto106","created_at":"2018-11-09T06:20:22Z"}

@vicaire
Copy link
Contributor Author

vicaire commented Nov 9, 2018

For PIPELINE GET, the swagger config specifies that the output is of content-type application/json

https://github.com/kubeflow/pipelines/blob/master/backend/api/swagger/pipeline.swagger.json#L15

This is correct.

For PIPELINE UPLOAD, the swagger config specifies that the output is of content-type application/json

https://github.com/kubeflow/pipelines/blob/master/backend/api/swagger/pipeline.upload.swagger.json#L15

but the output is in fact of type text/plain

@IronPan
Copy link
Member

IronPan commented Nov 9, 2018

fix #112

@vicaire
Copy link
Contributor Author

vicaire commented Mar 26, 2019

Looks like the fix has been checked in

@vicaire vicaire closed this as completed Mar 26, 2019
Linchin pushed a commit to Linchin/pipelines that referenced this issue Apr 11, 2023
* Added optional param replicas

function wait_for_deployment may now check specified number of replicas running

* Fix typo

* Start comment with capital letter
HumairAK pushed a commit to red-hat-data-services/data-science-pipelines that referenced this issue Mar 11, 2024
* add feature list

* Update FEATURES.md

* b headers

* b headers
HumairAK added a commit to red-hat-data-services/data-science-pipelines that referenced this issue Mar 11, 2024
…objstore-creds

feat(backend): Source ObjStore Creds from Env in Tekton Template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants