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

validate test/workload names in validateTestCases

Browse Source
This commit is contained in:
Nicolas Mitchell
2021-04-04 14:18:39 -04:00
parent 5ad79eae2d
commit 338b06fb69
1 changed files with 19 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
test/integration/scheduler_perf/scheduler_perf_test.go
View File

@@ -111,6 +111,17 @@ func (tc *testCase) collectsMetrics() bool {
return false return false
} }
func (tc *testCase) workloadNamesUnique() bool {
workloadUniqueNames := map[string]bool{}
for _, w := range tc.Workloads {
if workloadUniqueNames[w.Name] {
return false
}
workloadUniqueNames[w.Name] = true
}
return true
}
// workload is a subtest under a testCase that tests the scheduler performance // workload is a subtest under a testCase that tests the scheduler performance
// for a certain ordering of ops. The set of nodes created and pods scheduled // for a certain ordering of ops. The set of nodes created and pods scheduled
// in a workload may be heterogeneous. // in a workload may be heterogeneous.
@@ -766,10 +777,18 @@ func validateTestCases(testCases []*testCase) error {
if len(testCases) == 0 { if len(testCases) == 0 {
return fmt.Errorf("no test cases defined") return fmt.Errorf("no test cases defined")
} }
testCaseUniqueNames := map[string]bool{}
for _, tc := range testCases { for _, tc := range testCases {
if testCaseUniqueNames[tc.Name] {
return fmt.Errorf("%s: name is not unique", tc.Name)
}
testCaseUniqueNames[tc.Name] = true
if len(tc.Workloads) == 0 { if len(tc.Workloads) == 0 {
return fmt.Errorf("%s: no workloads defined", tc.Name) return fmt.Errorf("%s: no workloads defined", tc.Name)
} }
if !tc.workloadNamesUnique() {
return fmt.Errorf("%s: workload names are not unique", tc.Name)
}
if len(tc.WorkloadTemplate) == 0 { if len(tc.WorkloadTemplate) == 0 {
return fmt.Errorf("%s: no ops defined", tc.Name) return fmt.Errorf("%s: no ops defined", tc.Name)
} }