Feat(cv_deploy): Add CV Pathfinder AVT hop count to metadata studio

ansible_collections/arista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/cv-pathfinder-custom-control-plane-policy-pathfinder-1.yml

@@ -531,6 +531,7 @@ metadata:
       avts:
       - constraints:
           jitter: 5
+          hop_count: lowest
         id: 254
         name: CUSTOM-CP-POLICY
         pathgroups:

ansible_collections/arista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/cv-pathfinder-pathfinder.yml

@@ -645,6 +645,7 @@ metadata:
       avts:
       - constraints:
           jitter: 42
+          hop_count: lowest
         id: 2
         name: PROD-AVT-POLICY-VOICE
         pathgroups:

ansible_collections/arista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/cv-pathfinder-pathfinder1.yml

@@ -640,6 +640,7 @@ metadata:
       avts:
       - constraints:
           jitter: 42
+          hop_count: lowest
         id: 2
         name: PROD-AVT-POLICY-VOICE
         pathgroups:

ansible_collections/arista/avd/molecule/eos_designs_unit_tests/intended/structured_configs/cv-pathfinder-pathfinder2.yml

@@ -668,6 +668,7 @@ metadata:
       avts:
       - constraints:
           jitter: 42
+          hop_count: lowest
         id: 2
         name: PROD-AVT-POLICY-VOICE
         pathgroups:

ansible_collections/arista/avd/plugins/plugin_utils/cv_client/workflows/deploy_cv_pathfinder_metadata_to_cv.py

@@ -34,6 +34,11 @@ def is_pathfinder_location_supported(studio_schema: InputSchema) -> bool:
     return attributes.issubset(pathfinder_group_fields)
 
 
+def is_avt_hop_count_supported(studio_schema: InputSchema) -> bool:
+    """Detect if AVT hop count is supported by the metadata studio"""
+    return bool(get_v2(studio_schema, "fields.values.avtHopCount"))
+
+
 def is_internet_exit_zscaler_supported(studio_schema: InputSchema) -> bool:
     """Detect if zscaler internet exit is supported by the metadata studio"""
     return bool(get_v2(studio_schema, "fields.values.zscaler"))
@@ -58,17 +63,26 @@ async def get_metadata_studio_schema(result: DeployToCvResult, cv_client: CVClie
     return studio_schema
 
 
-def update_general_metadata(metadata: dict, studio_inputs: dict) -> None:
+def update_general_metadata(metadata: dict, studio_inputs: dict, studio_schema: InputSchema) -> list[str]:
     """
     In-place update general metadata in studio_inputs.
     """
+    warnings = []
+
     # Temporary fix for default values in metadata studio
     for vrf in get(metadata, "vrfs", default=[]):
         for avt in get(vrf, "avts", default=[]):
             constraints: dict = avt.setdefault("constraints", {})
             constraints.setdefault("latency", 4294967295)
             constraints.setdefault("jitter", 4294967295)
             constraints.setdefault("lossrate", 99.0)
+            if is_avt_hop_count_supported(studio_schema):
+                constraints["hopCount"] = constraints.pop("hop_count", "")
+            elif constraints.pop("hop_count", ""):
+                # hop count is set but not supported by metadata studio.
+                warning = "deploy_cv_pathfinder_metadata_to_cv: Ignoring AVT hop-count information since it is not supported by metadata studio."
+                LOGGER.info(warning)
+                warnings.append(warning)
 
     studio_inputs.update(
         {
@@ -85,6 +99,8 @@ def update_general_metadata(metadata: dict, studio_inputs: dict) -> None:
         }
     )
 
+    return warnings
+
 
 def upsert_pathfinder(metadata: dict, device: CVDevice, studio_inputs: dict, studio_schema: InputSchema) -> list[str]:
     """
@@ -329,7 +345,7 @@ async def deploy_cv_pathfinder_metadata_to_cv(cv_pathfinder_metadata: list[CVPat
 
     if pathfinders:
         # All pathfinders must have the same be general metadata, so we just set it in the studio based on the first one.
-        update_general_metadata(metadata=pathfinders[0].metadata, studio_inputs=studio_inputs)
+        result.warnings.extend(update_general_metadata(metadata=pathfinders[0].metadata, studio_inputs=studio_inputs, studio_schema=studio_schema))
 
     for pathfinder in pathfinders:
         result.warnings.extend(

ansible_collections/arista/avd/roles/eos_cli_config_gen/docs/tables/metadata.md

@@ -60,6 +60,7 @@
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;jitter</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].constraints.jitter") | Integer |  |  |  |  |
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;latency</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].constraints.latency") | Integer |  |  |  |  |
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;lossrate</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].constraints.lossrate") | String |  |  |  |  |
+    | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;hop_count</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].constraints.hop_count") | String |  |  |  |  |
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;description</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].description") | String |  |  |  |  |
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;id</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].id") | Integer |  |  |  |  |
     | [<samp>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;name</samp>](## "metadata.cv_pathfinder.vrfs.[].avts.[].name") | String |  |  |  |  |
@@ -144,6 +145,7 @@
                   jitter: <int>
                   latency: <int>
                   lossrate: <str>
+                  hop_count: <str>
                 description: <str>
                 id: <int>
                 name: <str>

ansible_collections/arista/avd/roles/eos_cli_config_gen/schemas/schema_fragments/metadata.schema.yml

@@ -170,6 +170,8 @@ keys:
                             type: str
                             convert_types:
                               - float
+                          hop_count:
+                            type: str
                       description:
                         type: str
                       id:

ansible_collections/arista/avd/roles/eos_designs/python_modules/metadata/cv_pathfinder.py

@@ -166,6 +166,7 @@ def _metadata_vrfs(self: AvdStructuredConfigMetadata) -> list:
                             "jitter": lb_policy.get("jitter"),
                             "latency": lb_policy.get("latency"),
                             "lossrate": float(lb_policy["loss_rate"]) if "loss_rate" in lb_policy else None,
+                            "hop_count": "lowest" if lb_policy.get("lowest_hop_count") else None,
                         },
                         "description": "",  # TODO: Not sure we have this field anywhere
                         "id": profile["id"],