Em, I think this is a trick as the comment states " #distro like sl 6.1 needs 
this folder, or tomcat6 failed to start"
I think we can only remove it after root cause the issue

> -----Original Message-----
> From: Wido den Hollander [mailto:w...@widodh.nl]
> Sent: Monday, May 21, 2012 6:03 AM
> To: cloudstack-dev@incubator.apache.org
> Cc: Wido den Hollander
> Subject: [PATCH] ConfigServer: Don't create /var/log/cloud-management.
> It's not in use
> 
> 
> Signed-off-by: Wido den Hollander <w...@widodh.nl>
> ---
>  python/lib/cloudutils/serviceConfigServer.py |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/python/lib/cloudutils/serviceConfigServer.py
> b/python/lib/cloudutils/serviceConfigServer.py
> index b113390..7c9e6b1 100644
> --- a/python/lib/cloudutils/serviceConfigServer.py
> +++ b/python/lib/cloudutils/serviceConfigServer.py
> @@ -101,7 +101,6 @@ class cloudManagementConfig(serviceCfgBase):
> 
>          #distro like sl 6.1 needs this folder, or tomcat6 failed to start
>          checkHostName()
> -        bash("mkdir /var/log/cloud-management/")
>          #set max process per account is unlimited
>          if os.path.exists("/etc/security/limits.conf"):
>              cfo = configFileOps("/etc/security/limits.conf")
> --
> 1.7.5.4

Reply via email to