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(a)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(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>