[config]: Use mod_entry when editing VLAN_INTERFACE (#1602)

theasianpianist · web-flow · commit 331c5a5df0a2 · 2021-05-10T11:29:29.000-07:00
Replace set_entry with mod_entry when setting the proxy_arp value for a VLAN.

Using set_entry will delete any other fields set for the key used, which is not desirable.

Signed-off-by: Lawrence Lee &lt;lawlee@microsoft.com&gt;
diff --git a/config/vlan.py b/config/vlan.py
@@ -92,7 +92,7 @@ def config_proxy_arp(db, vid, mode):
     if not clicommon.is_valid_vlan_interface(db.cfgdb, vlan):
         ctx.fail("Interface {} does not exist".format(vlan))
 
-    db.cfgdb.set_entry('VLAN_INTERFACE', vlan, {"proxy_arp": mode})
+    db.cfgdb.mod_entry('VLAN_INTERFACE', vlan, {"proxy_arp": mode})
     click.echo('Proxy ARP setting saved to ConfigDB')
     restart_ndppd()
 #

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,7 @@ def config_proxy_arp(db, vid, mode):`
`92`	`92`	`if not clicommon.is_valid_vlan_interface(db.cfgdb, vlan):`
`93`	`93`	`ctx.fail("Interface {} does not exist".format(vlan))`
`94`	`94`
`95`		`- db.cfgdb.set_entry('VLAN_INTERFACE', vlan, {"proxy_arp": mode})`
	`95`	`+ db.cfgdb.mod_entry('VLAN_INTERFACE', vlan, {"proxy_arp": mode})`
`96`	`96`	`click.echo('Proxy ARP setting saved to ConfigDB')`
`97`	`97`	`restart_ndppd()`
`98`	`98`	`#`