Yaniv Dary has posted comments on this change. Change subject: packaging: rework httpd proxy ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/23310/2/packaging/conf/ovirt-engine-reports-proxy.conf.in File packaging/conf/ovirt-engine-reports-proxy.conf.in: Line 6: # Line 7: <IfModule proxy_ajp_module> Line 8: Line 9: <Location /ovirt-engine-reports> Line 10: ProxyPassMatch ajp://127.0.0.1:@JBOSS_AJP_PORT@ > because we have this configurable in the new setup and we need to sync up. Then why did you hard code the value in the setup? Line 11: <IfModule deflate_module> Line 12: AddOutputFilterByType DEFLATE text/javascript text/css text/html text/xml text/json application/xml application/json application/x-yaml Line 13: </IfModule> Line 14: </Location> -- To view, visit http://gerrit.ovirt.org/23310 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4255bdebbe9bf253d62d895a2952bdd18a768626 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
