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

Tide cannot merge the batch due to error in script #1957

Closed
Ark-kun opened this issue Aug 26, 2019 · 2 comments · Fixed by #1958
Closed

Tide cannot merge the batch due to error in script #1957

Ark-kun opened this issue Aug 26, 2019 · 2 comments · Fixed by #1958
Assignees

Comments

@Ark-kun
Copy link
Contributor

Ark-kun commented Aug 26, 2019

https://prow.k8s.io/tide-history?author=Ark-kun
https://prow.k8s.io/?repo=kubeflow%2Fpipelines&type=batch
https://prow.k8s.io/log?job=kubeflow-pipeline-e2e-test&id=1166128972417732608

 source /home/prow/go/src/github.com/kubeflow/pipelines/test/build-images.sh
++ set -ex
++ IMAGES_BUILDING=false
+++ gcloud container images list --repository=gcr.io/ml-pipeline-test/
ERROR: (gcloud.container.images.list) Image name cannot end with '/'. Remove the trailing '/' and try again.
++ BUILT_IMAGES=
[test-timing] It took 0m0.597s.
{"component":"entrypoint","error":"wrapped process failed: exit status 1","file":"prow/entrypoint/run.go:79","func":"k8s.io/test-infra/prow/entrypoint.Options.Run","level":"error","msg":"Error executing test process","time":"2019-08-26T23:23:23Z"}
@Ark-kun Ark-kun assigned Ark-kun and Bobgy and unassigned Bobgy Aug 27, 2019
Ark-kun added a commit to Ark-kun/pipelines that referenced this issue Aug 27, 2019
Ark-kun added a commit to Ark-kun/pipelines that referenced this issue Aug 27, 2019
Ark-kun added a commit to Ark-kun/pipelines that referenced this issue Aug 27, 2019
@Bobgy
Copy link
Contributor

Bobgy commented Aug 27, 2019

Thanks for the fix. Sorry I didn't know this could be run as batch test too. I'm not super clear what a batch test does.

The only thing I found is: https://github.com/kubernetes/test-infra/tree/master/prow/tide#features
Do you know if there are more detailed explanation of how batch test is run.

@gaoning777
Copy link
Contributor

ajchili pushed a commit to ajchili/pipelines that referenced this issue Aug 28, 2019
Ark-kun added a commit to Ark-kun/pipelines that referenced this issue Aug 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants