scottk/kubernetes
1
0
Fork 0
mirror of https://github.com/optim-enterprises-bv/kubernetes.git synced 2025-11-04 04:08:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #35812 from rmmh/owner-better

Browse Source 
Automatic merge from submit-queue

Improve update_owners.py username detection and error message.

Fixes the root cause of #35808.
This commit is contained in:
Kubernetes Submit Queue
2016-10-28 14:44:38 -07:00
committed by GitHub
parent cf7178d7c3 20754c0f5c
commit bbe36f9186
2 changed files with 8 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
hack/update_owners.py
View File

@@ -116,12 +116,12 @@ def get_maintainers():
def detect_github_username():
remotes = subprocess.check_output(['git', 'remote', '-v'])
repos = set(re.findall(r'\w+(?=/kubernetes)', remotes))
repos.remove('kubernetes')
if len(repos) == 1:
return repos.pop()
raise ValueError('unable to guess GitHub user from `git remote -v` output, use --user instead')
origin_url = subprocess.check_output(['git', 'config', 'remote.origin.url'])
m = re.search(r'github.com[:/](.*)/', origin_url)
if m and m.group(1) != 'kubernetes':
return m.group(1)
raise ValueError('unable to determine GitHub user from '
'`git config remote.origin.url` output, run with --user instead')
def main():

3
test/list/main.go
View File

@@ -237,7 +237,8 @@ func (t *testList) handlePath(path string, info os.FileInfo, err error) error {
return err
}
if strings.Contains(path, "third_party") ||
strings.Contains(path, "staging") {
strings.Contains(path, "staging") ||
strings.Contains(path, "_output") {
return filepath.SkipDir
}
if strings.HasSuffix(path, ".go") && strings.Contains(path, "e2e") ||