Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wenyiz2021 opened this issue Jul 18, 2023 · 1 comment
Open

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

wenyiz2021 opened this issue Jul 18, 2023 · 1 comment

Comments

@wenyiz2021
Copy link
Contributor

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

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
Copy link
Contributor

rlhui commented Jul 26, 2023

Without this, what's the current impact?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

2 participants