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

Use kubeflow-ci as the test cluster #411

Merged
merged 12 commits into from Mar 15, 2018
Merged

Conversation

lluunn
Copy link
Contributor

@lluunn lluunn commented Mar 12, 2018

Related: kubeflow/testing#18


This change is Reviewable

@lluunn
Copy link
Contributor Author

lluunn commented Mar 12, 2018

/retest

@lluunn
Copy link
Contributor Author

lluunn commented Mar 12, 2018

/test kubeflow-presubmit

2 similar comments
@lluunn
Copy link
Contributor Author

lluunn commented Mar 12, 2018

/test kubeflow-presubmit

@lluunn
Copy link
Contributor Author

lluunn commented Mar 12, 2018

/test kubeflow-presubmit

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

/test kubeflow-presubmit

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

The test timeout waiting for checkout. But it actually succeeded later.
argo get -n=kubeflow-test-infra kubeflow-presubmit-kubeflow-e2e-411-d3b1875-609-112b

Guessing github-token not being used.

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

kubeflow/testing workflow still stuck at checkout.
TF serving workflow already passed.

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

I kubectl exec into the container.
git clone https://github.com/kubeflow/tf-operator.git tmp-aa is fast
git clone https://github.com/kubeflow/tf-operator.git /mnt/test-data-volume/aaa is extremely slow

So the problem should be the mnt

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

I ssh into the NFS server, verified the files are there, so connection should be set correctly..

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

/test kubeflow-presubmit

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

The NFS server was in a different zone from the k8s cluster. I deployed a new one in the same zone

@lluunn lluunn changed the title [WIP] Use kubeflow-ci as the test cluster Use kubeflow-ci as the test cluster Mar 14, 2018
@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

/uncc wbuchwalter
/uncc zjj2wry
/cc ankushagarwal

@k8s-ci-robot k8s-ci-robot requested review from ankushagarwal and removed request for wbuchwalter and zjj2wry March 14, 2018 22:51
@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

Ankush, with this change the e2e tests will be run in kubeflow-ci cluster. Presubmit passed.

@lluunn
Copy link
Contributor Author

lluunn commented Mar 14, 2018

/cc jlewi

@ankushagarwal
Copy link
Contributor

/lgtm
/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ankushagarwal

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit 08512aa into kubeflow:master Mar 15, 2018
elenzio9 pushed a commit to arrikto/kubeflow that referenced this pull request Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants