Tal Nisan has submitted this change and it was merged. Change subject: core: Make VmValidator validate single VMs ...................................................................... core: Make VmValidator validate single VMs As all methods in VmValidator are validating a single VM, the support for multiple VMs is overkill and adds unnecessary complexity to every validation that has to be added to this class. This patch make VmValidator test single VMs, for future multiple VM validations there's the MultipleVmsValdator class Change-Id: I84db7fcc79893f8b88d3dd0016adc514e5008869 Signed-off-by: Tal Nisan <tnisan@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/VmValidator.java 1 file changed, 51 insertions(+), 82 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55712 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I84db7fcc79893f8b88d3dd0016adc514e5008869 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>