Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1235#issuecomment-164254871
Before PR this was seen after log rotate:
```
root@r-996-VM:/etc# lsof | grep cloud.log.1
_plutorun 767 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
logger 768 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
_plutorun 772 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
_plutoloa 773 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
xl2tpd 843 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
python 854 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 860 root 1w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 860 root 2w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 860 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
python 863 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 869 root 1w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 869 root 2w REG 202,10 26054919 71 /var/log/cloud.log.1
passwd_se 869 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
python 871 root 3w REG 202,10 26054919 71 /var/log/cloud.log.1
```
After change:
```
root@r-996-VM:/etc# lsof | grep cloud.log.1
(no results)
```
They all still have `cloud.log` open.
Logrotate works:
```
root@r-996-VM:/var/log# ls -lah cloud*
-rw-r--r-- 1 root root 3.6M Dec 13 10:49 cloud_demo
-rw-r--r-- 1 root root 0 Dec 13 10:50 cloud.log
-rw-r--r-- 1 root root 18M Dec 13 10:50 cloud.log.1
-rw-r----- 1 root adm 0 Dec 13 10:16 cloud-service.log
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
