Update OWNERS

[aaronlevy]

We haven't updated this in a very long time, but moving to the upstream k8s approvals process which uses this. Updating to more recent direct maintainers.

cc @dghubble -- any interest in being added to this list?

[spiffxp]

/lgtm
@rphillips you should be able to /approve and then this will merge.

Note that the tku-bookube-e2e-* tests aren't listed as required for merge so they're not going to be paid attention to. If they need to be, you can open a PR into k/test-infra

This is what is currently listed as required https://github.com/kubernetes/test-infra/blob/master/prow/config.yaml#L255-L258

[dghubble]

@aaronlevy sure, sgtm

[dghubble]

/lgtm

[k8s-ci-robot]

@dghubble: changing LGTM is restricted to assignees, and only kubernetes-incubator/bootkube repo collaborators may be assigned issues.

In response to this:

/lgtm

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.

[spiffxp]

@dghubble file an issue on kubernetes/org and we'll get you added as a member so that works

[aaronlevy]

@dghubble updated this to include you. I also sent an invite to be a collaborator on this repo -- that might be enough to get the bot to like you (and/or can join the incubator org as well).

@rphillips mind lgtm'ing?

[rphillips]

/lgtm

[rphillips]

/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rphillips

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 [rphillips]

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

[spiffxp]

tide Pending — Not mergeable. Jobs tku-bootkube-e2e-calico, tku-bootkube-e2e-flannel have not succeeded.

It's not clear to me why tide is paying attention to the failing job contexts; they aren't listed as Required for merge? I'm going to manually merge this since there are other examples of PR's where this is happening, and I'd like to get your updated OWNERS file in.