Change in vdsm[master]: BZ#788640 - Check move image conditions from SD data.

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#788640 - Check move image conditions from SD data.
......................................................................


BZ#788640 - Check move image conditions from SD data.

getAllChildrenList() should be removed since it is intrinsically
race prone.
Removing uses of getAllChilfrenList and validating conditions
near the input instead of waste process.

Change-Id: I2422217717fe0f3cb668b4dc5f775e4a971c8d9b
Signed-off-by: Eduardo <ewarszaw(a)redhat.com&gt;
---
M vdsm/storage/hsm.py
M vdsm/storage/image.py
M vdsm/storage/sp.py
3 files changed, 38 insertions(+), 46 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Dan Kenigsberg: Verified


--
To view, visit http://gerrit.ovirt.org/3466
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2422217717fe0f3cb668b4dc5f775e4a971c8d9b
Gerrit-PatchSet: 14
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw(a)redhat.com&gt;
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Eduardo <ewarszaw(a)redhat.com&gt;
Gerrit-Reviewer: Haim Ateya <hateya(a)redhat.com&gt;
Gerrit-Reviewer: Igor Lvovsky <ilvovsky(a)redhat.com&gt;