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@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@redhat.com> Gerrit-Reviewer: Alexander Wels <awels@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server