Delete vendor from test-infra

[fejta]

Go modules no longer uses this directory, and bazel is more efficient without it.

• Update BUILD.bazel rules to look for @com_github_foo_bar//whatever instead of //vendor/github.com/foo/bar/whatever
• Update hack scripts to use this

/assign @Katharine @stevekuznetsov @michelle192837

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 9106daec5e8afb3dcd7a50f7a67c2a0f8a994c0e

[fejta]

/hold

[fejta]

Now with passing tests!

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 2ea2d6d093e22032ed33bdbaffc4dd360535af2f

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fejta, michelle192837

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 [fejta,michelle192837]

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

[stevekuznetsov]

Does this have any impact on downstream repositories importing Prow types, etc?

[fejta]

Is there a specific scenario you're concerned about?

This should not have a downstream impact as they can use our go.mod to create a correct version of vendor or go.mod for their repo.

[stevekuznetsov]

Generally importing test-infra is an absolute nightmare since any package you pull in has a habit of pulling in ... idk ... the world? Knative? k8s? Everything? So just wanted to make sure this did not make that materially worse

[fejta]

Definitely won't make things worse, may make things better.

[fejta]

/hold cancel