-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
vendor tests are running in test-infra (?) #5304
Comments
/area bazel |
Err, godep is supposed to drop all |
ah - @spxtr I think you are right - since these days we are adding dependencies manually in test-infra |
hehe, i thought it was by design i saw it a few days ago. |
We shouldn't be running them anyhow. k/k explicitly excludes vendor in bazel test/build for this reason. |
k/k excludes them because of the staging stuff I think. Could be wrong. We can exclude them as in this PR but we should also remove them at the same time. |
https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/pr-logs/pull/test-infra/5284/pull-test-infra-bazel/9538/
The text was updated successfully, but these errors were encountered: