Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Removing go get golint since the test image now has golint #187

Closed

Conversation

nikhiljindal
Copy link
Contributor

@nikhiljindal
Copy link
Contributor Author

Lets see if the test passes :)

@nikhiljindal
Copy link
Contributor Author

/test pull-kubernetes-multicluster-ingress-test
Couldnt find logs for the failure. Retrying

@nikhiljindal
Copy link
Contributor Author

nikhiljindal commented Apr 6, 2018

Cant get details again. Its failing with Unable to load build details.
Looking at https://k8s-gubernator.appspot.com/builds/kubernetes-jenkins/pr-logs/pull/GoogleCloudPlatform_k8s-multicluster-ingress/, I cant get any of the links to work for past runs.

cc @BenTheElder @krzyzacy Any ideas why?

@BenTheElder
Copy link

you should check prow.k8s.io, logs won't upload if your job can't execute or something along those l ines

/retest

@G-Harmon
Copy link
Contributor

G-Harmon commented Apr 6, 2018

/lgtm
but the test still isn't uploading logs...

@G-Harmon
Copy link
Contributor

G-Harmon commented Apr 7, 2018

golint failed- couldn't find golint.
vet failed- W0406 23:53:56.097] app/kubemci/pkg/gcp/forwardingrule/forwardingrulesyncer_test.go:240: github.com/GoogleCloudPlatform/k8s-multicluster-ingress/app/kubemci/pkg/gcp/status.LoadBalancerStatus composite literal uses unkeyed fields

@krzyzacy
Copy link

krzyzacy commented Apr 7, 2018

/retest

@krzyzacy
Copy link

krzyzacy commented Apr 7, 2018

?! no pod logs?
/retest

@k8s-ci-robot
Copy link

@nikhiljindal: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
pull-kubernetes-multicluster-ingress-test bc54927 link /test pull-kubernetes-multicluster-ingress-test

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@krzyzacy
Copy link

krzyzacy commented Apr 7, 2018

NAME                                   READY     STATUS               RESTARTS   AGE
2efee4b9-39f9-11e8-9931-0a580a6c031a   0/1       ContainerCannotRun   0          4m

whoops

@BenTheElder
Copy link

BenTheElder commented Apr 7, 2018 via email

@nikhiljindal
Copy link
Contributor Author

#189 is the better route.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants