docs(troubleshooting): add a new item into troubleshooting #602

DjangoPeng · 2018-04-06T16:27:57Z

Provide a solution for Unknown variable: env issue.

This change is

jlewi · 2018-04-06T17:35:34Z

user_guide.md

@@ -473,6 +473,18 @@ To fix this issue first create Github API token using this [guide](https://help.
 export GITHUB_TOKEN=<< token >>
 ```

+### Unknown variable: env
+
+It always happens to an old version of ksonnet. Actually there is a requirement of ksonnet version [here](https://github.com/kubeflow/kubeflow/blob/master/user_guide.md#requirements).


Can you include the error message that you saw in the bug?

k8s-ci-robot · 2018-04-07T06:34:22Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gaocegege

Assign the PR to them by writing /assign @gaocegege in a comment when ready.

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:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

DjangoPeng · 2018-04-07T06:34:26Z

comment addressed @jlewi

DjangoPeng · 2018-04-07T13:29:12Z

No idea about the kubeflow-presubmit. It seems like all 7 tests succeeded, but e2e minikube failed.

@jlewi PTAL

DjangoPeng · 2018-04-08T04:20:26Z

@gaocegege PTAL

jlewi · 2018-04-09T14:50:06Z

/test all

I made some suggested changes to the wording of the error message. It seemed easier to do this as a commit then as a comment. Feel free to reject the changes.

googlebot · 2018-04-09T14:59:20Z

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

jlewi · 2018-04-09T15:00:08Z

@DjangoPeng I made some suggested changes to the wording of the error message PTAL.

jlewi · 2018-04-10T13:48:21Z

@DjangoPeng Do my changes look good to you? It would be good to get this merged and unblock your other PR #604 adding the TOC?

DjangoPeng · 2018-04-10T15:33:12Z

@jlewi LGTM

Sorry for the late reply. All day outside, just come back now.

DjangoPeng · 2018-04-10T15:33:41Z

/retest

DjangoPeng · 2018-04-10T16:15:22Z

From the testing error log, I guess there are some trouble in our CI env?

Command: go run hack/e2e.go -v --test --test_args='--ginkgo.focus=simple\-tfjob\-gke$'

Timeout waiting for simple-tfjob-gke in namespace kubeflow-presubmit-kubeflow-e2e-gke-602-28b03b9-1115-c7f3 to finish.

jlewi · 2018-04-10T17:05:12Z

Doesn't look like a quota issue

gcloud --project=kubeflow-ci compute regions list
NAME                     CPUS    DISKS_GB    ADDRESSES  RESERVED_ADDRESSES  STATUS  TURNDOWN_DATE
asia-east1               0/24    0/4096      0/8        0/8                 UP
asia-northeast1          0/24    0/4096      0/8        0/8                 UP
asia-south1              0/24    0/4096      0/8        0/8                 UP
asia-southeast1          0/24    0/4096      0/8        0/8                 UP
australia-southeast1     0/24    0/4096      0/8        0/8                 UP
europe-west1             0/24    0/4096      0/8        0/8                 UP
europe-west2             0/24    0/4096      0/8        0/8                 UP
europe-west3             0/24    0/4096      0/8        0/8                 UP
europe-west4             0/24    0/4096      0/8        0/8                 UP
northamerica-northeast1  0/24    0/4096      0/8        0/8                 UP
southamerica-east1       0/24    0/4096      0/8        0/8                 UP
us-central1              0/320   0/4096      0/32       0/8                 UP
us-east1                 72/320  6200/16384  9/32       0/8                 UP
us-east4                 0/24    0/4096      0/8        0/8                 UP
us-west1                 0/24    0/4096      0/8        0/8                 UP

jlewi · 2018-04-10T17:13:58Z

/test all

jlewi · 2018-04-10T21:09:21Z

/test all

jlewi · 2018-04-10T22:53:17Z

/test all

jlewi · 2018-04-10T23:19:47Z

Woo Hoo tests passed.

Merging this manually; because the CLA check is busted.

) * add a new item into troubleshooting * add error message * Improve wording of the error message

add a new item into troubleshooting

a897a2d

k8s-ci-robot assigned gaocegege and jlewi Apr 6, 2018

k8s-ci-robot requested review from jimexist and zjj2wry April 6, 2018 16:27

k8s-ci-robot added the size/S label Apr 6, 2018

jlewi reviewed Apr 6, 2018

View reviewed changes

add error message

ec3c479

DjangoPeng mentioned this pull request Apr 7, 2018

docs(TOC): add auto-generated TOC for all docs #604

Merged

Improve wording of the error message

28b03b9

I made some suggested changes to the wording of the error message. It seemed easier to do this as a commit then as a comment. Feel free to reject the changes.

This was referenced Apr 10, 2018

TfJob controllers are not namespace scoped so tests aren't isolated #134

Closed

Timeout waiting for simple-tfjob-gke #636

Closed

jlewi merged commit 3cf930a into kubeflow:master Apr 10, 2018

saffaalvi pushed a commit to StatCan/kubeflow that referenced this pull request Feb 11, 2021

docs(troubleshooting): add a new item into troubleshooting (kubeflow#602

917a212

) * add a new item into troubleshooting * add error message * Improve wording of the error message

yanniszark pushed a commit to arrikto/kubeflow that referenced this pull request Feb 15, 2021

Moving folders (kubeflow#602)

2bc89ed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs(troubleshooting): add a new item into troubleshooting #602

docs(troubleshooting): add a new item into troubleshooting #602

DjangoPeng commented Apr 6, 2018 •

edited by jlewi

Loading

jlewi Apr 6, 2018

k8s-ci-robot commented Apr 7, 2018

DjangoPeng commented Apr 7, 2018

DjangoPeng commented Apr 7, 2018

DjangoPeng commented Apr 8, 2018

jlewi commented Apr 9, 2018

googlebot commented Apr 9, 2018

jlewi commented Apr 9, 2018

jlewi commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018 •

edited

Loading

docs(troubleshooting): add a new item into troubleshooting #602

docs(troubleshooting): add a new item into troubleshooting #602

Conversation

DjangoPeng commented Apr 6, 2018 • edited by jlewi Loading

jlewi Apr 6, 2018

Choose a reason for hiding this comment

k8s-ci-robot commented Apr 7, 2018

DjangoPeng commented Apr 7, 2018

DjangoPeng commented Apr 7, 2018

DjangoPeng commented Apr 8, 2018

jlewi commented Apr 9, 2018

googlebot commented Apr 9, 2018

jlewi commented Apr 9, 2018

jlewi commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

DjangoPeng commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018

jlewi commented Apr 10, 2018 • edited Loading

DjangoPeng commented Apr 6, 2018 •

edited by jlewi

Loading

jlewi commented Apr 10, 2018 •

edited

Loading