Skip to content

Commit c1dde5e

Browse files
authored
Increase the timeout limit. (#350)
* Increase the timeout limit. * Increased timeout to 360 seconds. * Addressed build failures: - Increased timeout for kuttl steps - increased CPU requests for the fast preemption tests - reduced the number of app wrappers submitted, simplified code
1 parent e391803 commit c1dde5e

File tree

3 files changed

+9
-19
lines changed

3 files changed

+9
-19
lines changed

hack/run-e2e-kind.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -556,4 +556,4 @@ kuttl-tests
556556
mcad-quota-management-down
557557
mcad-up
558558
echo "==========================>>>>> Running E2E tests... <<<<<=========================="
559-
go test ./test/e2e -v -timeout 75m -count=1
559+
go test ./test/e2e -v -timeout 120m -count=1

kuttl-test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@ apiVersion: kuttl.dev/v1beta1
22
kind: TestSuite
33
testDirs:
44
- ./test/e2e-kuttl/
5-
timeout: 180
5+
timeout: 420

test/e2e/queue.go

Lines changed: 7 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -431,7 +431,7 @@ var _ = Describe("AppWrapper E2E Test", func() {
431431

432432
// This should not fit on any node but should dispatch because there is enough aggregated resources.
433433
aw2 := createGenericDeploymentCustomPodResourcesWithCPUAW(
434-
context, "aw-ff-deployment-1-700-cpu", "700m", "700m", 1, 60)
434+
context, "aw-ff-deployment-1-850-cpu", "850m", "850m", 1, 60)
435435

436436
appwrappers = append(appwrappers, aw2)
437437

@@ -728,36 +728,25 @@ var _ = Describe("AppWrapper E2E Test", func() {
728728

729729
})
730730

731-
It("Create AppWrapper - Generic 100 Deployment Only - 2 pods each", func() {
732-
fmt.Fprintf(os.Stdout, "[e2e] Generic 100 Deployment Only - 2 pods each - Started.\n")
731+
It("Create AppWrapper - Generic 50 Deployment Only - 2 pods each", func() {
732+
fmt.Fprintf(os.Stdout, "[e2e] Generic 50 Deployment Only - 2 pods each - Started.\n")
733733

734734
context := initTestContext()
735735
var aws []*arbv1.AppWrapper
736736
appwrappersPtr := &aws
737737
defer cleanupTestObjectsPtr(context, appwrappersPtr)
738738

739739
const (
740-
awCount = 100
740+
awCount = 50
741741
reportingInterval = 10
742+
cpuDemand = "5m"
742743
)
743744

744745
replicas := 2
745746
modDivisor := int(awCount / reportingInterval)
746747
for i := 0; i < awCount; i++ {
747-
name := fmt.Sprintf("%s%d-", "aw-generic-deployment-", replicas)
748+
name := fmt.Sprintf("aw-generic-deployment-%02d-%03d", replicas, i+1)
748749

749-
// Pad name with '0' when i < 100
750-
if i < (awCount - 1) {
751-
name = fmt.Sprintf("%s%s", name, "0")
752-
}
753-
754-
// Pad name with '0' when i < 10
755-
if i < (reportingInterval - 1) {
756-
name = fmt.Sprintf("%s%s", name, "0")
757-
}
758-
759-
name = fmt.Sprintf("%s%d", name, i+1)
760-
cpuDemand := "5m"
761750
if ((i+1)%modDivisor) == 0 || i == 0 {
762751
fmt.Fprintf(GinkgoWriter, "[e2e] Creating AW %s with %s cpu and %d replica(s).\n", name, cpuDemand, replicas)
763752
}
@@ -766,6 +755,7 @@ var _ = Describe("AppWrapper E2E Test", func() {
766755
}
767756

768757
// Give the deployments time to create pods
758+
// FIXME: do not assume that the pods are in running state in the order of submission.
769759
time.Sleep(2 * time.Minute)
770760
for i := 0; i < len(aws); i++ {
771761
err := waitAWReadyQuiet(context, aws[i])

0 commit comments

Comments
 (0)