Change in ovirt-engine[master]: webadmin: disable dashboard data if DWH isn't available
Code Review
gerrit at ovirt.org
Tue Apr 18 14:10:08 UTC 2017
>From Vojtech Szocs <vszocs at redhat.com>:
Vojtech Szocs has submitted this change and it was merged.
Change subject: webadmin: disable dashboard data if DWH isn't available
......................................................................
webadmin: disable dashboard data if DWH isn't available
Two checks are made to verify DWH configuration. First, the engine DWH
config table is verified. Second, the ability of the JNDI managed DWH
DataSource to create a DB connection is verified. If either test fails,
the query caching and direct queries are disabled.
The change will eliminate error logging for developers who don't need
to work with DWH components.
Bug-Url: https://bugzilla.redhat.com/1441857
Change-Id: I7e42b9e32b32ee75e6d1d290c187d35e1e784926
Signed-off-by: Scott J Dickerson <sdickers at redhat.com>
---
M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/dashboard/DashboardDataServlet.java
1 file changed, 76 insertions(+), 4 deletions(-)
Approvals:
Jenkins CI: Passed CI tests
Greg Sheremeta: Looks good to me, approved
Vojtech Szocs: Looks good to me, but someone else must approve
Scott Dickerson: Verified
--
To view, visit https://gerrit.ovirt.org/68664
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7e42b9e32b32ee75e6d1d290c187d35e1e784926
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Scott Dickerson <sdickers at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Scott Dickerson <sdickers at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list