Skip to content

Commit 43ca007

Browse files
author
Kate Osborn
committed
Add attached routes notes
1 parent 035ca66 commit 43ca007

File tree

2 files changed

+23
-13
lines changed

2 files changed

+23
-13
lines changed

tests/reconfig/results/1.1.0/1.1.0.md

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,13 @@ NGF deployment:
6565
6666
## NumResources to Total Resources
6767

68-
| NumResources | Gateways | Secrets | ReferenceGrants | Namespaces | application Pods | application Services | HTTPRoutes | Total Resources |
69-
|--------------|----------|---------|-----------------|------------|------------------|----------------------|------------|-----------------|
70-
| x | 1 | 1 | 1 | x+1 | 2x | 2x | 3x | <total> |
71-
| 30 | 1 | 1 | 1 | 31 | 60 | 60 | 90 | 244 |
72-
| 150 | 1 | 1 | 1 | 151 | 300 | 300 | 450 | 1204 |
68+
| NumResources | Gateways | Secrets | ReferenceGrants | Namespaces | application Pods | application Services | HTTPRoutes | Attached HTTPRoutes | Total Resources |
69+
|--------------|----------|---------|-----------------|------------|------------------|----------------------|------------|---------------------|-----------------|
70+
| x | 1 | 1 | 1 | x+1 | 2x | 2x | 3x | 2x | <total> |
71+
| 30 | 1 | 1 | 1 | 31 | 60 | 60 | 90 | 60 | 244 |
72+
| 150 | 1 | 1 | 1 | 151 | 300 | 300 | 450 | 300 | 1204 |
73+
74+
> Note: Only 2x HTTPRoutes attach to the Gateway because the parentRef name in the `cafe-tls-redirect` HTTPRoute is incorrect. This will be fixed in the next release.
7375
7476
## Observations
7577

@@ -83,3 +85,7 @@ NGF deployment:
8385
3. No errors (NGF or NGINX) were observed in any test run.
8486

8587
4. The majority of the event batches were processed in 500ms or less except the 3rd test. In the 3rd test, we create the Gateway resource after all the apps and routes. The batch that contains the Gateway is the only one that takes longer than 500ms. It takes ~13s.
88+
89+
## Future Improvements
90+
91+
1. Fix the parentRef name in the `cafe-tls-redirect` [HTTPRoute](/tests/reconfig/scripts/cafe-routes.yaml), so it matches the deployed Gateway.

tests/reconfig/setup.md

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,17 @@
5858
- Note: Clean up after each test run for isolated results. There's a script provided for removing all the test
5959
fixtures `scripts/delete-multiple.sh` which takes a number (needs to be the same number as what was used in the
6060
create script.)
61-
5. After each individual test run, grab logs of both NGF containers and grab metrics.
62-
Note: You can expose metrics by running the below snippet and then navigating to `127.0.0.1:9113/metrics`:
63-
64-
```console
65-
GW_POD=$(k get pods -n nginx-gateway | sed -n '2s/^\([^[:space:]]*\).*$/\1/p')
66-
kubectl port-forward $GW_POD -n nginx-gateway 9113:9113 &
67-
```
61+
5. After each individual test:
62+
- Describe the Gateway resource and make sure the status is correct.
63+
- Check the logs of both NGF containers for errors.
64+
- Parse the logs for TimeToReady numbers (see steps 6-7 below).
65+
- Grab metrics.
66+
Note: You can expose metrics by running the below snippet and then navigating to `127.0.0.1:9113/metrics`:
67+
68+
```console
69+
GW_POD=$(k get pods -n nginx-gateway | sed -n '2s/^\([^[:space:]]*\).*$/\1/p')
70+
kubectl port-forward $GW_POD -n nginx-gateway 9113:9113 &
71+
```
6872

6973
6. Measure NGINX Reloads and Time to Ready Results
7074
1. TimeToReadyTotal as described in each test - NGF logs.
@@ -90,7 +94,7 @@
9094
e.g. `cd scripts && bash create-resources-gw-last.sh 30`. The script will deploy backend apps and services, wait
9195
60 seconds for them to be ready, and deploy 1 Gateway, 1 RefGrant, 1 Secret, and HTTPRoutes.
9296
2. Deploy NGF
93-
3. Measure TimeToReadyTotal as the time it takes from start-up -> config written and
97+
3. Measure TimeToReadyTotal as the time it takes from start-up -> final config written and
9498
NGINX reloaded. Measure the other results as described in steps 6-7 of the [Setup](#setup) section.
9599

96100
### Test 2: Start NGF, deploy Gateway, create many resources attached to GW

0 commit comments

Comments
 (0)