Skip to content

Commit

Permalink
server_test: add wait after terminate
Browse files Browse the repository at this point in the history
In server_test started_imageio fixture
creates servers with built ovirt-imageio,
and terminates the process at the end,
between runs.

However, if runs do happen too fast
it may try to create the next server socket
before the previous is closed, resulting in
an "address already in use" error.

Add a wait() after terminate() to ensure we do
not run into such issue.

Fixes: e6ed821
Signed-off-by: Albert Esteve <[email protected]>
  • Loading branch information
aesteve-rh authored and nirs committed Jul 13, 2022
1 parent 1f7585f commit e189066
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions test/server_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -231,6 +231,7 @@ def started_imageio(tmpdir, drop_privileges="true"):
yield proc
finally:
proc.terminate()
proc.wait()


def user_groups(user):
Expand Down

0 comments on commit e189066

Please sign in to comment.