> On Feb. 28, 2013, 12:10 p.m., Abhinandan Prateek wrote:
> > Ship It!
>
> Prasanna Santhanam wrote:
> Wait - doesn't this break the developer environment? It will fix the
> binary however.
yes this does break:
web app root system property: 'webapp.root' =
[/Users/agneya2001/citrix/asf/client/target/cloud-client-ui-4.2.0-SNAPSHOT]
2013-02-28 17:47:59.018:INFO:/client:Initializing log4j from
[file:/etc/cloudstack/management/log4j-cloud.xml]
log4j:ERROR Could not parse url
[file:/etc/cloudstack/management/log4j-cloud.xml].
java.io.FileNotFoundException: /etc/cloudstack/management/log4j-cloud.xml (No
such file or directory)
at java.io.FileInputStream.open(Native Method)
at java.io.FileInputStream.<init>(FileInputStream.java:138)
at java.io.FileInputStream.<init>(FileInputStream.java:97)
at
sun.net.www.protocol.file.FileURLConnection.connect(FileURLConnection.java:90)
- Abhinandan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9676/#review17188
-----------------------------------------------------------
On Feb. 28, 2013, 9:46 a.m., Noa Resare wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9676/
> -----------------------------------------------------------
>
> (Updated Feb. 28, 2013, 9:46 a.m.)
>
>
> Review request for cloudstack.
>
>
> Description
> -------
>
> The log4j initialization filter introduced to catch logging from
> spring (before the code that previously initialized log4j was run)
> a new filter was introduced to the client web.xml to set up log4j
> even earlier. Unfortunatly that filter was configured to look for
> log4j-cloud.xml in a deprecated location. This patch refers the
> log4j init filter to the canonical location in /etc
>
> CLOUDSTACK-1436: 4.1 management server fails to start
> CLOUDSTACK-1423: Unable to launch UI
>
>
> This addresses bugs CLOUDSTACK-1423 and CLOUDSTACK-1436.
>
>
> Diffs
> -----
>
> client/WEB-INF/web.xml 626305b
>
> Diff: https://reviews.apache.org/r/9676/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Noa Resare
>
>
