New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removes additional columns in test_owners.csv #41660
Removes additional columns in test_owners.csv #41660
Conversation
Sorry for breaking the bot! **cc: @ethernetdan, @pwittrock, @kubernetes/test-infra-reviewers ** |
/lgtm |
looks like it also adds about 200 lines... that intentional? |
test/test_owners.csv
Outdated
HA-master survive addition/removal replicas different zones,derekwaynecarr,0,api-machinery | ||
HA-master survive addition/removal replicas multizone workers,rkouj,0,api-machinery | ||
HA-master survive addition/removal replicas same zone,derekwaynecarr,0,api-machinery | ||
AppArmor should enforce an AppArmor profile,derekwaynecarr,0,node |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@calebamiles Looks like the file got repeated here. Could it be an issue with the hack/update_owners.py
script?
- sorts all e2e tests (with sort (GNU coreutils) 8.25) - moves all k8s.io/* tests to the end - removes duplicated tests (with uniq (GNU coreutils) 8.25)
|
Related to kubernetes/test-infra#1935 |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED The following people have approved this PR: calebamiles, spxtr Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/lgtm |
Automatic merge from submit-queue |
What this PR does / why we need it:
fixes a huge collection of typos with the number of columns in the CSV file that probably has broken the auto assign bot
Special notes for your reviewer:
None
Release note:
NONE