Skip to content

Commit 4f9ac00

Browse files
committed
Add test description
1 parent 9280017 commit 4f9ac00

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

tests/suite/reconfig_test.go

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -340,6 +340,9 @@ var _ = Describe("Reconfiguration Performance Testing", Ordered, Label("reconfig
340340
getStartTime := func() time.Time { return startTime }
341341
modifyStartTime := func() { startTime = startTime.Add(500 * time.Millisecond) }
342342

343+
output, err := framework.InstallGatewayAPI(getDefaultSetupCfg().gwAPIVersion)
344+
Expect(err).ToNot(HaveOccurred(), string(output))
345+
343346
if startWithNGFSetup {
344347
setup(getDefaultSetupCfg())
345348

@@ -366,9 +369,6 @@ var _ = Describe("Reconfiguration Performance Testing", Ordered, Label("reconfig
366369
),
367370
).WithTimeout(metricExistTimeout).WithPolling(metricExistPolling).Should(Succeed())
368371
}
369-
} else {
370-
output, err := framework.InstallGatewayAPI(getDefaultSetupCfg().gwAPIVersion)
371-
Expect(err).ToNot(HaveOccurred(), string(output))
372372
}
373373

374374
Expect(test(resourceCount)).To(Succeed())
@@ -493,9 +493,10 @@ var _ = Describe("Reconfiguration Performance Testing", Ordered, Label("reconfig
493493
err = writeReconfigResults(outFile, results)
494494
Expect(err).ToNot(HaveOccurred())
495495

496-
cleanupResources(30)
496+
cleanupResources(resourceCount)
497497
}
498498

499+
// Test 1 - Resources exist before start-up
499500
It("test 1 - 30 resources", func() {
500501
timeToReadyStartingLogSubstring := "Starting NGINX Gateway Fabric"
501502

@@ -518,6 +519,7 @@ var _ = Describe("Reconfiguration Performance Testing", Ordered, Label("reconfig
518519
// )
519520
//})
520521

522+
// Test 2 - Start NGF, deploy Gateway, create many resources attached to GW
521523
It("test 2 - 30 resources", func() {
522524
timeToReadyStartingLogSubstring := "Reconciling the resource\",\"controller\":\"httproute\""
523525

@@ -540,6 +542,7 @@ var _ = Describe("Reconfiguration Performance Testing", Ordered, Label("reconfig
540542
// )
541543
//})
542544

545+
// Test 3: Start NGF, create many resources attached to a Gateway, deploy the Gateway
543546
It("test 3 - 30 resources", func() {
544547
timeToReadyStartingLogSubstring := "Reconciling the resource\",\"controller\":\"gateway\""
545548

0 commit comments

Comments
 (0)