From Allon Mureinik <amureini@redhat.com>:

Allon Mureinik has submitted this change and it was merged. ( https://gerrit.ovirt.org/79060 ) Change subject: core: DiskValidator#isDiskAttachedToAnyVm streams ...................................................................... core: DiskValidator#isDiskAttachedToAnyVm streams This patches changes the implementation of the isDiskAttachedToAnyVm method to first generate the list of offending VM names (which should be a non-op if there are none), and then perform the check on the generated string instead of the original list. While this patch doesn't have too much meaning on its own, it sets up the subsequent patch, which will add another filter clause to the aforementioned stream. Change-Id: I7c89929c040c9e6cb7913b24ba2484db34562752 Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/DiskValidator.java 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/79060 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c89929c040c9e6cb7913b24ba2484db34562752 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>