Change in ovirt-engine[master]: engine: Avoid race condition between gluster peer probe and ...

[kmayilsa at redhat.com]

Kanagaraj M has submitted this change and it was merged.

Change subject: engine: Avoid race condition between gluster peer probe and status
......................................................................


engine: Avoid race condition between gluster peer probe and status

During add of a gluster host, a gluster peer probe command is executed.
If a gluster peer status is also fired at the same time from GlusterSyncJob
this results in the newly added host being removed from the engine database,
as the gluster peer status has not yet recognized the newly added host.

Added lock at cluster level before peer probe to avoid this.

Change-Id: Iaa76cea4c5b5bf7c444680260dd4da5b65665428
Bug-Url: https://bugzilla.redhat.com/972619
Signed-off-by: Sahina Bose <sabose at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
1 file changed, 41 insertions(+), 24 deletions(-)

Approvals:
  Kanagaraj M: Looks good to me, approved
  Sahina Bose: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa76cea4c5b5bf7c444680260dd4da5b65665428
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server