Skip to content

Commit

Permalink
[config vlan]: Remove -t flag from docker exec command (sonic-net#1317
Browse files Browse the repository at this point in the history
)

Remove -t flag from the docker exec command used to restart SWSS after updating the proxy ARP config. Currently with the flag, this command cannot be executed as part of a script (e.g. during a unit test in sonic-mgmt) since Docker is expecting the input device to be a TTY.

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
  • Loading branch information
theasianpianist authored Dec 17, 2020
1 parent 281b157 commit 155f6d5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/vlan.py
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ def del_vlan(db, vid):

def restart_ndppd():
verify_swss_running_cmd = "docker container inspect -f '{{.State.Status}}' swss"
docker_exec_cmd = "docker exec -it swss {}"
docker_exec_cmd = "docker exec -i swss {}"
ndppd_config_gen_cmd = "sonic-cfggen -d -t /usr/share/sonic/templates/ndppd.conf.j2,/etc/ndppd.conf"
ndppd_restart_cmd = "supervisorctl restart ndppd"

Expand Down

0 comments on commit 155f6d5

Please sign in to comment.