Change in ovirt-engine[ovirt-engine-4.1]: webadmin: disable dashboard data if DWH isn't available

Code Review gerrit at ovirt.org
Wed Apr 19 13:29:30 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan 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
  Scott Dickerson: Verified



-- 
To view, visit https://gerrit.ovirt.org/75652
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e42b9e32b32ee75e6d1d290c187d35e1e784926
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
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: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Scott Dickerson <sdickers at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan 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