network: Do not show evpn_vni if not returned by API

cubeek · cubeek · commit c572b008aa49 · 2026-06-05T08:49:30.000-04:00
The Router object in the SDK defines the evpn_vni in the Depends-On
patch. The attribute is returned in the body only if the EVPN extension
is enabled in Neutron but the Router object in SDK attempts to show it
always.

This patch makes the column visible only if it's really present in
the API response.

Assisted-By: Claude Opus 4.6
Related-Bug: #2144617

Change-Id: I8747f3cb753a115e7955b0a0ea136431fdade0dd
Signed-off-by: Jakub Libosvar &lt;libosvar@redhat.com&gt;
diff --git a/openstackclient/network/v2/router.py b/openstackclient/network/v2/router.py
@@ -88,6 +88,8 @@ def _get_columns(
     if item.is_distributed is None:
         hidden_columns.append('is_distributed')
         column_map.pop('is_distributed')
+    if not hasattr(item, 'evpn_vni'):
+        hidden_columns.append('evpn_vni')
     return utils.get_osc_show_columns_for_sdk_resource(
         item, column_map, hidden_columns
     )

Original file line number	Diff line number	Diff line change
`@@ -88,6 +88,8 @@ def _get_columns(`
`88`	`88`	`if item.is_distributed is None:`
`89`	`89`	`hidden_columns.append('is_distributed')`
`90`	`90`	`column_map.pop('is_distributed')`
	`91`	`+ if not hasattr(item, 'evpn_vni'):`
	`92`	`+ hidden_columns.append('evpn_vni')`
`91`	`93`	`return utils.get_osc_show_columns_for_sdk_resource(`
`92`	`94`	`item, column_map, hidden_columns`
`93`	`95`	`)`