Change in ovirt-engine[master]: webadmin: Added sorting of hosts by SPM priority upon activa...

tnisan at redhat.com tnisan at redhat.com
Tue Sep 10 14:58:49 UTC 2013


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Added sorting of hosts by SPM priority upon activating
......................................................................


webadmin: Added sorting of hosts by SPM priority upon activating

When sending a multiple activate host command the list of the hosts is not
oredered by SPM priority and that can cause a scenario in which no SPM is
elected, multiple hosts are activated one after another and the first one
is selected as an SPM regardless of the SPM priority of the rest.
In order to avoid this scenario, the list is sorted by SPM priority prior
to running the multiple actions

Change-Id: Ie1b3f564491372c8934451c774127ae29e1ac879
Bug-Url: https://bugzilla.redhat.com/998997
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java
2 files changed, 17 insertions(+), 2 deletions(-)

Approvals:
  Tal Nisan: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1b3f564491372c8934451c774127ae29e1ac879
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list