Skip to content

Commit d7bc98e

Browse files
author
Shuotian Cheng
authored
[config]: Fix bug of losing data when setting admin status (sonic-net#361)
Shall use mod_entry instead of set_entry when updating partial information in the configuration database. set_entry will delete the rest of the data associated with this entry. Signed-off-by: Shu0T1an ChenG <[email protected]>
1 parent 078b8b8 commit d7bc98e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

config/main.py

+4-4
Original file line numberDiff line numberDiff line change
@@ -749,9 +749,9 @@ def startup(ctx):
749749
interface_name = ctx.obj['interface_name']
750750

751751
if interface_name.startswith("Ethernet"):
752-
config_db.set_entry("PORT", interface_name, {"admin_status": "up"})
752+
config_db.mod_entry("PORT", interface_name, {"admin_status": "up"})
753753
elif interface_name.startswith("PortChannel"):
754-
config_db.set_entry("PORTCHANNEL", interface_name, {"admin_status": "up"})
754+
config_db.mod_entry("PORTCHANNEL", interface_name, {"admin_status": "up"})
755755
#
756756
# 'shutdown' subcommand
757757
#
@@ -764,9 +764,9 @@ def shutdown(ctx):
764764
interface_name = ctx.obj['interface_name']
765765

766766
if interface_name.startswith("Ethernet"):
767-
config_db.set_entry("PORT", interface_name, {"admin_status": "down"})
767+
config_db.mod_entry("PORT", interface_name, {"admin_status": "down"})
768768
elif interface_name.startswith("PortChannel"):
769-
config_db.set_entry("PORTCHANNEL", interface_name, {"admin_status": "down"})
769+
config_db.mod_entry("PORTCHANNEL", interface_name, {"admin_status": "down"})
770770

771771
#
772772
# 'speed' subcommand

0 commit comments

Comments
 (0)