incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Noa Resare" <...@resare.com>
Subject Re: Review Request: Make log4j-cloud.xml path in client web.xml reference /etc
Date Thu, 28 Feb 2013 13:25:31 GMT


> 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.
> 
> Abhinandan Prateek wrote:
>     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)

Ok. For now it seems like a good idea to solve this with a symlink in the .rpm package. I'm
building a patch to test that as I'm writing this.


- Noa


-----------------------------------------------------------
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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message