tests: Extend SRv6 static SIDs topotest to verify SID structure

The `static_srv6_sids` topotest verifies that staticd correctly programs the SIDs in the zebra RIB. Currently, the topotest only validates the programmed behavior and SID attributes. This commit extends the topotest to also validate the SID structure. Signed-off-by: Carmine Scarpitta <[email protected]>
FRRouting · Feb 8, 2025 · a6d02fe · a6d02fe
1 parent 312f7b3
commit a6d02fe
Show file tree

Hide file tree

Showing 3 changed files with 63 additions and 9 deletions.
diff --git a/tests/topotests/static_srv6_sids/expected_srv6_sids.json b/tests/topotests/static_srv6_sids/expected_srv6_sids.json
@@ -25,7 +25,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End"
+						"action": "End",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 0,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 
@@ -60,7 +66,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT4"
+						"action": "End.DT4",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 10
@@ -95,7 +107,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT6"
+						"action": "End.DT6",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 20
@@ -130,7 +148,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT46"
+						"action": "End.DT46",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 30

diff --git a/tests/topotests/static_srv6_sids/expected_srv6_sids_sid_delete_1.json b/tests/topotests/static_srv6_sids/expected_srv6_sids_sid_delete_1.json
@@ -25,7 +25,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT4"
+						"action": "End.DT4",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 10
@@ -60,7 +66,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT6"
+						"action": "End.DT6",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 20
@@ -95,7 +107,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT46"
+						"action": "End.DT46",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 30

diff --git a/tests/topotests/static_srv6_sids/expected_srv6_sids_sid_delete_2.json b/tests/topotests/static_srv6_sids/expected_srv6_sids_sid_delete_2.json
@@ -25,7 +25,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT4"
+						"action": "End.DT4",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 10
@@ -60,7 +66,13 @@
 					"active": true,
 					"weight": 1,
 					"seg6local": {
-						"action": "End.DT46"
+						"action": "End.DT46",
+						"sidStructure": {
+							"blockLen": 32,
+							"nodeLen": 16,
+							"funcLen": 16,
+							"argLen": 0
+						}
 					},
 					"seg6localContext": {
 						"table": 30