Skip to content

Commit

Permalink
remove comments, raise issue instead
Browse files Browse the repository at this point in the history
  • Loading branch information
wenyiz2021 authored Jan 30, 2023
1 parent 400fd00 commit 189f598
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions utilities_common/multi_asic.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down

0 comments on commit 189f598

Please sign in to comment.