Change in ovirt-engine[master]: backend: improve block domains sync on host activation

Code Review gerrit at ovirt.org
Mon Jul 10 08:24:18 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged. ( https://gerrit.ovirt.org/78977 )

Change subject: backend: improve block domains sync on host activation
......................................................................


backend: improve block domains sync on host activation

When activating the first host in the data center, all the storage
domains in the data center are activated.
Today, the engine calls SyncLunsInfoForBlockStorageDomain for each
storage domain in a new thread using the same host.
This patch improves this behavior by synchronizing all the domains
together using SyncStorageDomainsLuns and thus performing one call to
vdsm (instead one per domain) in one thread.

Change-Id: I6539a2e96ac885f828d93276bf939931bdfc372c
Bug-Url: https://bugzilla.redhat.com/1467794
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsEventListener.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/IVdsEventListener.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsProxy.java
3 files changed, 31 insertions(+), 9 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve
  Idan Shaby: Verified



-- 
To view, visit https://gerrit.ovirt.org/78977
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6539a2e96ac885f828d93276bf939931bdfc372c
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list