Skip to content

Commit 5b5d7b6

Browse files
committed
Updated overlay test plans.
- Removed one dirrection lookup test based on PR #238. - Added connection tracking tests for the custom ageing timeout. - Fixed spellcheck errors. Signed-off-by: Anton Putria <[email protected]>
1 parent 5a648a9 commit 5b5d7b6

File tree

4 files changed

+35
-30
lines changed

4 files changed

+35
-30
lines changed

.wordlist.txt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ adaptor
1313
adaptors
1414
ADDR
1515
addr
16+
ageing
1617
agnostically
1718
amd
1819
apache
@@ -293,6 +294,7 @@ lookups
293294
loopback
294295
LPM
295296
lts
297+
MACs
296298
Macsec
297299
makefile
298300
Makefile
@@ -310,6 +312,7 @@ MaxTcpFlow
310312
MaxTotalFlowEntry
311313
MaxUdpFlow
312314
md
315+
mgmt
313316
microsoft
314317
misorder
315318
MSFT
@@ -598,6 +601,7 @@ Vnic
598601
vnic
599602
vnics
600603
VNID
604+
VNIs
601605
VPC
602606
vpc
603607
vport

test/docs/testplans/conntrack.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,10 +56,12 @@ Test cases are automated:
5656

5757
| # | Test case | Test Class.Method
5858
| --- | --- | ---
59-
| 1 | Standard ageing (fully correct TCP session). | -
60-
| 2 | Standard ageing (fully correct UDP session). | -
61-
| 3 | Open TCP session but no data and no FIN | -
62-
| 4 | TCP session started from the middle (no SYN packet) | -
59+
| 1 | Standard ageing (fully correct TCP session, session dropped immediately). | -
60+
| 2 | Standard ageing (fully correct UDP session, session dropped after timeout). | -
61+
| 3 | Custom ageing (fully correct TCP session). | -
62+
| 4 | Custom ageing (fully correct UDP session). | -
63+
| 5 | Open TCP session but no data and no FIN. | -
64+
| 6 | TCP session started from the middle (no SYN packet). | -
6365

6466
## Integration
6567

test/docs/testplans/eni.md

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -37,28 +37,28 @@ Verifies create operations, an association with VNI, MAC.
3737

3838
| # | Test case | Test Class.Method
3939
| --- | --- | ---
40-
| 1 | create inbound/outbound DASH ACL groups | CreateDeleteEniTest.createInOutAclGroupsTest
41-
| 2 | create VNET | CreateDeleteEniTest.createVnetTest
42-
| 3 | create ENI | CreateDeleteEniTest.createEniTest
43-
| 4 | create ENI Ether address map entry | CreateDeleteEniTest.createEniEtherAddressMapTest
44-
| 5 | create PA validation entry | CreateDeleteEniTest.createPaValidationTest
45-
| 6 | create Outbound routing entry | CreateDeleteEniTest.createOutboundRoutingEntryTest
46-
| 7 | create Inbound routing entry | CreateDeleteEniTest.createInboundRoutingEntryTest |
47-
| 8 | verify Inbound routing entry attributes getting/setting | CreateDeleteEniTest.inboundRoutingEntryAttributesTest |
48-
| 9 | verify ENI attributes getting/setting | CreateDeleteEniTest.eniAttributesTest
49-
| 10 | verify ENI Ether address map entry attributes getting/setting | CreateDeleteEniTest.eniEtherAddressMapAttributesTest
50-
| 11 | verify PA validation entry attributes getting/setting | CreateDeleteEniTest.paValidationEntryAttributesTest
51-
| 12 | verify Outbound routing entry attributes getting/setting | CreateDeleteEniTest.outboundRoutingEntryAttributesTest
40+
| 1 | create inbound/outbound DASH ACL groups | `CreateDeleteEniTest.createInOutAclGroupsTest`
41+
| 2 | create VNET | `CreateDeleteEniTest.createVnetTest`
42+
| 3 | create ENI | `CreateDeleteEniTest.createEniTest`
43+
| 4 | create ENI Ether address map entry | `CreateDeleteEniTest.createEniEtherAddressMapTest`
44+
| 5 | create PA validation entry | `CreateDeleteEniTest.createPaValidationTest`
45+
| 6 | create Outbound routing entry | `CreateDeleteEniTest.createOutboundRoutingEntryTest`
46+
| 7 | create Inbound routing entry | `CreateDeleteEniTest.createInboundRoutingEntryTest`
47+
| 8 | verify Inbound routing entry attributes getting/setting | `CreateDeleteEniTest.inboundRoutingEntryAttributesTest`
48+
| 9 | verify ENI attributes getting/setting | `CreateDeleteEniTest.eniAttributesTest`
49+
| 10 | verify ENI Ether address map entry attributes getting/setting | `CreateDeleteEniTest.eniEtherAddressMapAttributesTest`
50+
| 11 | verify PA validation entry attributes getting/setting | `CreateDeleteEniTest.paValidationEntryAttributesTest`
51+
| 12 | verify Outbound routing entry attributes getting/setting | `CreateDeleteEniTest.outboundRoutingEntryAttributesTest`
5252

5353
## ENI removal
5454

5555
Verifies remove operations.
5656

5757
| # | Test case | Test Class.Method
5858
| --- | --- | ---
59-
| 1 | normal delete:<br>verify deletion of: inbound/outbound DASH ACL groups, VNET, ENI, ENI Ether address map entry, PA validation entry, inbound/outbound routing entry | Is a part of CreateDeleteEniTest.destroy_teardown_obj
60-
| 2 | error id mapped rules exist:<br>verify ENI cannot be deleted when map exist | CreateDeleteEniTest.deleteEniWhenMapExistTest
61-
| 3 | duplicated deletion<br>no errors | CreateDeleteEniTest.duplicatedEniDeletionTest
59+
| 1 | normal delete:<br>verify deletion of: inbound/outbound DASH ACL groups, VNET, ENI, ENI Ether address map entry, PA validation entry, inbound/outbound routing entry | Is a part of `CreateDeleteEniTest.destroy_teardown_obj`
60+
| 2 | error id mapped rules exist:<br>verify ENI cannot be deleted when map exist | `CreateDeleteEniTest.deleteEniWhenMapExistTest`
61+
| 3 | duplicated deletion<br>no errors | `CreateDeleteEniTest.duplicatedEniDeletionTest`
6262
| 4 | normal bulk delete | -
6363
| 5 | bulk delete does not remove any if there is a mapping for some ENI | -
6464

@@ -68,5 +68,5 @@ Verifies basic ENI scale, create/remove/recreate maximum number of ENIs.
6868

6969
| # | Test case | Test Class.Method
7070
| --- | --- | ---
71-
| 1 | Create/remove a max number of ENI entries | EniScaleTest.eniScaleTest
72-
| 2 | Recreate (repeated creation/removal a max number of ENI entries) | EniScaleTest.eniScaleTest
71+
| 1 | Create/remove a max number of ENI entries | `EniScaleTest.eniScaleTest`
72+
| 2 | Recreate (repeated creation/removal a max number of ENI entries) | `EniScaleTest.eniScaleTest`

test/docs/testplans/vnet.md

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,8 @@ Scale and performance tests automation approach - to be defined.
8181
| # | Test case | Test Class.Method
8282
| --- | --- | ---
8383
| 1 | Route action ROUTE_VNET | -
84-
| 2 | Route action ROUTE_VNET_DIRECT | Vnet2VnetInboundTest.<br>Vnet2VnetOutboundRouteVnetDirectTest
85-
| 3 | Route action ROUTE_DIRECT | Vnet2VnetOutboundTest.<br>Vnet2VnetOutboundRouteDirectTest
84+
| 2 | Route action ROUTE_VNET_DIRECT | `Vnet2VnetInboundTest.`<br>`Vnet2VnetOutboundRouteVnetDirectTest`
85+
| 3 | Route action ROUTE_DIRECT | `Vnet2VnetOutboundTest.`<br>`Vnet2VnetOutboundRouteDirectTest`
8686
| 4 | dst_vnet_id True/False in OUTBOUND_CA_TO_PA_ENTRY | -
8787
| 5 | Use same CA prefixes in different outbound routing tables (different ENIs) | -
8888
| 6 | Use multiple overlapping routing prefixes in the same outbound routing table. | -
@@ -94,17 +94,16 @@ Original table [link](https://github.com/Azure/DASH/blob/main/documentation/gene
9494

9595
| # | Test case | Test Class.Method
9696
| --- | --- | ---
97-
| 1 | VNET2VNET routing with PA validation entry PERMIT.<br>SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP_PA_VALIDATE<br>SAI_PA_VALIDATION_ENTRY_ACTION_PERMIT| Vnet2VnetInboundTest.<br>vnet2VnetInboundPaValidatePermitTest
98-
| 2 | Direction lookup DENY action | Vnet2VnetInboundTest.<br/>vnet2VnetInboundDenyVniTest
99-
| 3 | VNET2VNET routing without PA validation entry<br>SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP | -
97+
| 1 | VNET2VNET routing with PA validation entry PERMIT.<br>SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP_PA_VALIDATE<br>SAI_PA_VALIDATION_ENTRY_ACTION_PERMIT| `Vnet2VnetInboundTest`.<br>`vnet2VnetInboundPaValidatePermitTest`
98+
| 2 | VNET2VNET routing without PA validation entry<br>SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP | -
10099

101100

102101
### **Integration**
103102

104103
| # | Test case | Test Class.Method
105104
| --- | --- | ---
106105
| 1 |Multiple inbound and outbound configurations at the same time. Send multiple allowed and forbidden traffic types. | -
107-
| 2 |Send non VXLAN traffic. (**to clarify** underlay routing?) | VnetRouteTest
106+
| 2 |Send non VXLAN traffic. (**to clarify** underlay routing?) | `VnetRouteTest`
108107
| 3 |Use multiple VIPs | -
109108
| 4 | Use same prefixes in CA and PA networks for outbound and inbound VNET at the same time | -
110109

@@ -115,8 +114,8 @@ Original table [link](https://github.com/Azure/DASH/blob/main/documentation/gene
115114
| 1 | Traffic with invalid VIP (Inbound and Outbound) | -
116115
| 2 | Traffic with valid VNI but no match to any ENI MAC | -
117116
| 3 | Invalid configurations:<br>- Multiple MACs for same ENI<br>- All different VNIs in ENI, direction lookup, vnet configuration.<br>- Add same VNI for different direction lookup entries. | -
118-
| 4 | Drop if CA DMAC does not match | Vnet2VnetInboundTest.<br/>vnet2VnetInboundInvalidEniMacTest
119-
| 5 | Drop if PA SIP does not match on PA validation | Vnet2VnetInboundTest.<br/>vnet2VnetInboundInvalidPaSrcIpTest
117+
| 4 | Drop if CA DMAC does not match | `Vnet2VnetInboundTest.`<br/>`vnet2VnetInboundInvalidEniMacTest`
118+
| 5 | Drop if PA SIP does not match on PA validation | `Vnet2VnetInboundTest.`<br/>`vnet2VnetInboundInvalidPaSrcIpTest`
120119

121120
### **Scaling & Performance**
122121

@@ -132,7 +131,7 @@ To be defined.
132131
- static rule
133132
- based on mapping lookup
134133
- inner packet SRC/DEST IP calculated based on part of outer packet SRC/DEST IP<br>
135-
Question: What is static rule nad calculated values?
134+
Question: What is static rule and calculated values?
136135
1. How to test - Inbound (priority) route rules processing:
137136
- Most Outer Source IP Prefix
138137
- Most Outer Destination IP Prefix

0 commit comments

Comments
 (0)