Change:
/assign /cc Please ensure that every sub directory of / contains an owners file. The following sub directories do not define them: The result of this is that any PR which changes anything in api, Godeps, staging, vendor, etc will always get assigned to one of you for both the review and approval.
I recall someone also mentioning that certain files or dirs may not need approvers files. If that would help, we can add that functionality (i've already started the work)
staging and api should be easy to find owners for. may have ideas for translations. The big one is Godeps, third_party, vendor. Those are just grunt work. And figuring out criteria for approving is hard as they are a lot of code and it is unclear that things are tested well or automatically. (I'm thinking things like libraries for accessing cloud APIs). Having a dedicated team/process there might help.
I'll open some proposals for staging and API.
Godep in particular needs to be watched for things like licenses, which I have mostly been doing.
Thanks Clayton for the staging and api additions! How about if we pick two of you at random and shove them in as owners for each of the remaining directories?
What's left: translations: the kubectl specific translations is being handled by sig-cli, but the rest of the files like README and test have been handled and approved mostly by thirdparty: this is a tricky mix of stuff like forked code.. mostly handled by the likes of or for approvals. Godep/vendor: These could probably be solved by having a process to follow, and then delegating to a team in OWNERSALIASES for both folders.
Issues go stale after 90d of inactivity. Mark the issue as fresh with . Stale issues rot after an additional 30d of inactivity and eventually close. Prevent issues from auto-closing with an comment. If this issue is safe to close now please do so with . Send feedback to sig-testing, kubernetes/test-infra and/or . /lifecycle stale
Still relevant. /remove-lifecycle stale

Discussion:
- piosz - jsafrane - jbeda <ins> vendor-reviewers: - thockin vendor-approvers: - thockin </ins>

ret=1 fi <del> if ! _out="$(diff -Naupr -x "BUILD" -x "AUTHORS*" -x "CONTRIBUTORS*" vendor ${_kubetmp}/vendor)"; then </del> <ins> if ! _out="$(diff -Naupr -x "BUILD" -x "OWNERS" -x "AUTHORS*" -x "CONTRIBUTORS*" vendor ${_kubetmp}/vendor)"; then </ins> echo "Your vendored results are different:" echo "${_out}" echo "Godeps Verify failed."

// Code generated by go-bindata. // sources: <ins> // translations/OWNERS </ins> // translations/extract.py // translations/kubectl/OWNERS // translations/kubectl/de_DE/LC_MESSAGES/k8s.mo

return nil } <ins> var _translationsOwners = []byte(`reviewers: - brendandburns approvers: - brendandburns `) func translationsOwnersBytes() ([]byte, error) { return _translationsOwners, nil } func translationsOwners() (*asset, error) { bytes, err := translationsOwnersBytes() if err != nil { return nil, err } info := bindataFileInfo{name: "translations/OWNERS", size: 0, mode: os.FileMode(0), modTime: time.Unix(0, 0)} a := &asset{bytes: bytes, info: info} return a, nil } </ins> var _translationsExtractPy = []byte(`#!/usr/bin/env python # Copyright 2017 The Kubernetes Authors.

// _bindata is a table, holding each asset generator, mapped to its name. var _bindata = map[string]func() (*asset, error){ <ins> "translations/OWNERS":                             translationsOwners, </ins> "translations/extract.py":                         translationsExtractPy, "translations/kubectl/OWNERS":                     translationsKubectlOwners, "translations/kubectl/de_DE/LC_MESSAGES/k8s.mo":   translationsKubectlDe_deLc_messagesK8sMo,

var _bintree = &bintree{nil, map[string]*bintree{ "translations": {nil, map[string]*bintree{ <ins> "OWNERS":     {translationsOwners, map[string]*bintree{}}, </ins> "extract.py": {translationsExtractPy, map[string]*bintree{}}, "kubectl": {nil, map[string]*bintree{ "OWNERS": {translationsKubectlOwners, map[string]*bintree{}},

<ins> reviewers: - lavalamp - smarterclayton - thockin approvers: - lavalamp - smarterclayton - thockin </ins>

<ins> reviewers: - brendandburns approvers: - brendandburns </ins>

<ins> reviewers: - vendor-reviewers approvers: - vendor-approvers </ins>