search

sonic-net / sonic-utilities

Command line utilities for the SONiC project
Other
152 stars 643 forks source link

[chassis] [MA] configlet CLI not supporting multi-asic #2914

Open wenyiz2021 opened 1 year ago

wenyiz2021 commented 1 year ago

Description

configlet right now is not supporting multi-asic, it does not get config db from namespace. result in configlet fail to delete configuration on either config_db.json/config_db0.json etc for multi-asic platform.

we should do sth like: config_db = multi_asic.connect_config_db_for_ns(namespace)

right now: https://github.com/sonic-net/sonic-utilities/blob/c301f3bb5c62d14a764706164488ccf679a2ac8d/scripts/configlet#L90C1-L90C4

Steps to reproduce the issue

1. 2. 3.

Describe the results you received

Describe the results you expected

Additional information you deem important (e.g. issue happens only occasionally)

Output of show version

(paste your output here)
rlhui commented 1 year ago

Without this, what's the current impact?