Removed 'default' row from test_owners.csv. Also updated update_owners.py to no longer handle the 'default' row.

pull/6/head
Cole Wagner 2017-05-05 14:21:23 -07:00
parent 854441643f
commit e9eca4b5ba
2 changed files with 1 additions and 4 deletions

View File

@ -86,8 +86,7 @@ def write_owners(fname, owners):
with open(fname, 'w') as f:
out = csv.writer(f, lineterminator='\n')
out.writerow(['name', 'owner', 'auto-assigned', 'sig'])
sort_key = lambda (k, v): (k != 'DEFAULT', k) # put 'DEFAULT' first.
items = sorted(owners.items(), key=sort_key)
items = sorted(owners.items())
for name, (owner, random_assignment, sig) in items:
out.writerow([name, owner, int(random_assignment), sig])
@ -137,7 +136,6 @@ def main():
test_names = get_test_names_from_test_history()
else:
test_names = get_test_names_from_local_files()
test_names.add('DEFAULT')
test_names = sorted(test_names)
owners = load_owners(OWNERS_PATH)

View File

@ -1,5 +1,4 @@
name,owner,auto-assigned,sig
DEFAULT,rmmh/spxtr/ixdy/apelisse/fejta,0,
Addon update should propagate add-on file changes,eparis,1,
AppArmor should enforce an AppArmor profile,derekwaynecarr,0,node
AppArmor when running with AppArmor should enforce a permissive profile,yujuhong,1,node

1 name owner auto-assigned sig
DEFAULT rmmh/spxtr/ixdy/apelisse/fejta 0
2 Addon update should propagate add-on file changes eparis 1
3 AppArmor should enforce an AppArmor profile derekwaynecarr 0 node
4 AppArmor when running with AppArmor should enforce a permissive profile yujuhong 1 node