aboutgitcodebugslistschat
path: root/util.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2022-07-15 15:21:35 +1000
committerStefano Brivio <sbrivio@redhat.com>2022-07-22 19:42:18 +0200
commitd078a1113febf93ec9124521c0d4d7ee8409b178 (patch)
tree2a386c5651b30a01adb04299790cedca19f7ea9c /util.c
parent1aaa2bd189cb42a26ee411fd1652e7479d82f005 (diff)
downloadpasst-d078a1113febf93ec9124521c0d4d7ee8409b178.tar
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.tar.gz
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.tar.bz2
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.tar.lz
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.tar.xz
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.tar.zst
passt-d078a1113febf93ec9124521c0d4d7ee8409b178.zip
tests: Remove unnecessary ^D in passt_in_ns teardown
teardown_passt_in_ns() sends a ^D to the NS pane, which appears to be intended to terminate the nsenter running there, leaving the namespace. However, we've also sent a ^D to the PASST pane which will exit the pasta instance which created the namespace. With the namespace destroyed the nsenter in the NS pane will be killed, so it does not need to be exited explicitly. In fact sending the extra ^D can be harmful, since it will exit the shell in which the nsenter was run, causing the whole pane to be closed. That can then mean that the "pane_wait NS" hangs indefinitely. I believe this will sometimes work, because there's a race between the various options here, but it should be more reliable without the extra ^D. Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'util.c')
0 files changed, 0 insertions, 0 deletions