You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Wido den Hollander <wi...@widodh.nl> on 2012/05/21 15:03:18 UTC
[PATCH] ConfigServer: Don't create /var/log/cloud-management. It's not in use
Signed-off-by: Wido den Hollander <wi...@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
Re: [PATCH] ConfigServer: Don't create /var/log/cloud-management.
It's not in use
Posted by David Nalley <da...@gnsa.us>.
On Mon, May 21, 2012 at 2:21 PM, Frank Zhang <Fr...@citrix.com> wrote:
> 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
>
We commented? No bug for this?
I just searched and found two related bugs, one was closed for
needinfo not being responded to, the other was closed saying that we
corrected the need for having /var/log/cloud-management.
http://bugs.cloudstack.org/browse/CS-10181 <-- says that we removed
at least some of the stuff that was looking for
/var/log/cloud-management. I also just grepped the source code and see
only
Bigger question is, does it actually work with that removed?
--David
RE: [PATCH] ConfigServer: Don't create /var/log/cloud-management.
It's not in use
Posted by Frank Zhang <Fr...@citrix.com>.
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:wido@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 <wi...@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