From 189f598081b4d0eb5b461d7e48682f473184cbbd Mon Sep 17 00:00:00 2001 From: wenyiz2021 <91497961+wenyiz2021@users.noreply.github.com> Date: Mon, 30 Jan 2023 15:35:49 -0800 Subject: [PATCH] remove comments, raise issue instead --- utilities_common/multi_asic.py | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/utilities_common/multi_asic.py b/utilities_common/multi_asic.py index 48401f2053..9e213f67f1 100644 --- a/utilities_common/multi_asic.py +++ b/utilities_common/multi_asic.py @@ -114,17 +114,6 @@ def multi_asic_display_default_option(): ] def multi_asic_namespace_validation_callback(ctx, param, value): - # TODO: this condition has a bug: it should be either "multi_asic.is_multi_asic()" - # or "self.is_multi_asic", but changing it to correct version will lead to the - # following cli not working on single-asic: - # show ip(v6) bgp neighbors/network, show storm_control add/del - # e.g. - # admin@str2-7050cx3-acs-02:/usr$ show ip bgp neighbor -n asic0 - # -n/--namespace is not available for single asic - # Aborted! - # admin@str2-7050cx3-acs-02:/usr$ show ip bgp neighbor - # -n/--namespace is not available for single asic - # Aborted! if not multi_asic.is_multi_asic: click.echo("-n/--namespace is not available for single asic") ctx.abort()