Change in ovirt-engine[master]: services: reports: serve reports ui xml as servlet

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Jan 15 09:31:49 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: services: reports: serve reports ui xml as servlet
......................................................................


services: reports: serve reports ui xml as servlet

instead of using direct static content which does not support logic, use
a servlet that can acquire information from whatever source.

in this patch the source is local config, to allow easy integration with
reports without reports package overwrite the engine data directory.

currently we put the reports.xml at package state, so it can be added
and removed without restarting engine, this may change in future.

Change-Id: I78f7822354ac6850a49698b0262209489a5ab5b3
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/ReportInit.java
M packaging/services/ovirt-engine/ovirt-engine.conf.in
3 files changed, 28 insertions(+), 1 deletion(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I78f7822354ac6850a49698b0262209489a5ab5b3
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list