Merge pull request #122833 from kannon92/add-density-test-flake
mark density tests as flaky
This commit is contained in:
		| @@ -526,6 +526,21 @@ func withLabel(label string) interface{} { | ||||
| 	return newLabel(label) | ||||
| } | ||||
|  | ||||
| // WithFlaky specifies that a certain test or group of tests are failing randomly. | ||||
| // These tests are usually filtered out and ran separately from other tests. | ||||
| func WithFlaky() interface{} { | ||||
| 	return withFlaky() | ||||
| } | ||||
|  | ||||
| // WithFlaky is a shorthand for the corresponding package function. | ||||
| func (f *Framework) WithFlaky() interface{} { | ||||
| 	return withFlaky() | ||||
| } | ||||
|  | ||||
| func withFlaky() interface{} { | ||||
| 	return newLabel("Flaky") | ||||
| } | ||||
|  | ||||
| type label struct { | ||||
| 	// parts get concatenated with ":" to build the full label. | ||||
| 	parts []string | ||||
|   | ||||
| @@ -76,7 +76,7 @@ var _ = SIGDescribe("Density", framework.WithSerial(), framework.WithSlow(), fun | ||||
| 		rc = NewResourceCollector(containerStatsPollingPeriod) | ||||
| 	}) | ||||
|  | ||||
| 	ginkgo.Context("create a batch of pods", func() { | ||||
| 	f.Context("create a batch of pods", framework.WithFlaky(), func() { | ||||
| 		// TODO(coufon): the values are generous, set more precise limits with benchmark data | ||||
| 		// and add more tests | ||||
| 		dTests := []densityTest{ | ||||
| @@ -225,7 +225,7 @@ var _ = SIGDescribe("Density", framework.WithSerial(), framework.WithSlow(), fun | ||||
| 		} | ||||
| 	}) | ||||
|  | ||||
| 	ginkgo.Context("create a sequence of pods", func() { | ||||
| 	f.Context("create a sequence of pods", framework.WithFlaky(), func() { | ||||
| 		dTests := []densityTest{ | ||||
| 			{ | ||||
| 				podsNr:   10, | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 Kubernetes Prow Robot
					Kubernetes Prow Robot