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

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

Merged
merged 3 commits into from
Apr 10, 2018

Conversation

DjangoPeng
Copy link
Member

@DjangoPeng DjangoPeng commented Apr 6, 2018

Provide a solution for Unknown variable: env issue.

Ref: #598

/assign @jlewi
/assign @gaocegege


This change is Reviewable

user_guide.md Outdated
@@ -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).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@k8s-ci-robot
Copy link
Contributor

[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:

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

@DjangoPeng
Copy link
Member Author

comment addressed @jlewi

@DjangoPeng
Copy link
Member Author

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

@jlewi PTAL

@DjangoPeng
Copy link
Member Author

@gaocegege PTAL

@jlewi
Copy link
Contributor

jlewi commented Apr 9, 2018

/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
Copy link

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
Copy link
Contributor

jlewi commented Apr 9, 2018

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

@jlewi
Copy link
Contributor

jlewi commented Apr 10, 2018

@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
Copy link
Member Author

@jlewi LGTM

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

@DjangoPeng
Copy link
Member Author

/retest

@DjangoPeng
Copy link
Member Author

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
Copy link
Contributor

jlewi commented Apr 10, 2018

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
Copy link
Contributor

jlewi commented Apr 10, 2018

/test all

@jlewi
Copy link
Contributor

jlewi commented Apr 10, 2018

/test all

1 similar comment
@jlewi
Copy link
Contributor

jlewi commented Apr 10, 2018

/test all

@jlewi
Copy link
Contributor

jlewi commented Apr 10, 2018

Woo Hoo tests passed.

Merging this manually; because the CLA check is busted.

@jlewi 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
)

* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants