Change in ovirt-engine[master]: gluster: Fetch brick statuses only for UP volumes

sanjal at redhat.com sanjal at redhat.com
Mon Jan 21 09:11:15 UTC 2013


Shireesh Anjal has submitted this change and it was merged.

Change subject: gluster: Fetch brick statuses only for UP volumes
......................................................................


gluster: Fetch brick statuses only for UP volumes

Brick statuses are fetched by executing the gluster cli command
'gluster volume status' which works only on UP volumes. Hence
we should not try to fetch and update brick statuses for volumes
that are currently DOWN.

Also made sure that whenever a volume status changes, we update
the status of it's bricks as well.

Introduced more specific audit log types when we detect that a
volume was started/stopped from the gluster cli.

Change-Id: If7f28948036993423d5baad2f4af41875670a18a
Signed-off-by: Shireesh Anjal <sanjal at redhat.com>
---
A backend/manager/dbscripts/upgrade/03_02_0340_gluster_volume_status_events.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterManager.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterManagerTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
6 files changed, 38 insertions(+), 16 deletions(-)

Approvals:
  Shireesh Anjal: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If7f28948036993423d5baad2f4af41875670a18a
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal 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: Shireesh Anjal <sanjal at redhat.com>



More information about the Engine-commits mailing list