Change in ovirt-engine[master]: engine: Avoid PM/storage alerts for GlusterOnly clusters

ofrenkel at redhat.com ofrenkel at redhat.com
Wed Jun 20 10:53:35 UTC 2012


Omer Frenkel has submitted this change and it was merged.

Change subject: engine: Avoid PM/storage alerts for GlusterOnly clusters
......................................................................


engine: Avoid PM/storage alerts for GlusterOnly clusters

https://bugzilla.redhat.com/829842
https://bugzilla.redhat.com/830820

Bugs:
 - Activation of server triggers "Failed to connect host to storage servers" event
 - Power Management related events/alerts should be removed

This was happening because various class level boolean variables indicating errors related to
storage pool and fencing are set to false by default, and trigger corresponding audit logs/
events when getAuditLogTypeValue() is called. This should not happen if the host belongs to
a cluster that does not support virtualization e.g. gluster cluster.

Also modified VdsCommand to generate the power management alert only when virt service
is enabled on the cluster.

Change-Id: I22d93df0cf6b2594d7eabf972c038baf893eccdf
Signed-off-by: Shireesh Anjal <sanjal at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java
2 files changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Kanagaraj M: Verified
  Omer Frenkel: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I22d93df0cf6b2594d7eabf972c038baf893eccdf
Gerrit-PatchSet: 7
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: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>



More information about the Engine-commits mailing list