#11: Race in NetTestController.__prepare_device() ------------------------+----------------------- Reporter: rpazdera | Owner: rpazdera Type: defect | Status: new Priority: critical | Milestone: Component: component1 | Version: Keywords: | Blocked By: Blocking: | ------------------------+----------------------- There's a race in this method of NetTestController class. Sometimes multiple devices with a same MAC address can be accepted by LNST.
It can happen if you try to create a device with MAC that already exist on the target domain. If the interface addition is somehow delayed (because of scheduling for instance), the wait_for() function will pass correctly. It will find the already existing device, while the freshly created doesn't exist yet.
The correct behavior would be to recognize this and fail because of a hwaddr collision between two interfaces.
I will look at this and send a patch asap.
#11: Race in NetTestController.__prepare_device() -------------------------+---------------------- Reporter: rpazdera | Owner: rpazdera Type: defect | Status: assigned Priority: critical | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: -------------------------+---------------------- Changes (by rpazdera):
* status: new => assigned
#11: Race in NetTestController.__prepare_device() -------------------------+---------------------- Reporter: rpazdera | Owner: rpazdera Type: defect | Status: assigned Priority: critical | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: -------------------------+----------------------
Comment (by rpazdera):
Patch sent: https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000395.html
#11: Race in NetTestController.__prepare_device() -------------------------+---------------------- Reporter: rpazdera | Owner: rpazdera Type: defect | Status: closed Priority: critical | Milestone: Component: component1 | Version: Resolution: fixed | Keywords: Blocked By: | Blocking: -------------------------+---------------------- Changes (by rpazdera):
* resolution: => fixed * status: assigned => closed
Comment:
Fixed by: https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000405.html
lnst-developers@lists.fedorahosted.org