[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-10-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/799 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-10-23 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-150552760 In the mean while work has been done on CentOS 7 (systemd) so that is not relevant any more. There also is another open issue that needs to address CentOS 7

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-28 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-143724455 @remibergsma rebased against current master and also squashed 4 commits to one. --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-27 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-143590376 @sudhansu7 Any update on this? Please rebase against current master so we can run the tests. Also, squash teh commits (4 of them have the same title)

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-21 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-141945369 @bhaisaab For CentOS 7 we have we cloud-management.service and cloud-management.config files. I don't see any other .rc file for cloud-management service.

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-21 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-142060873 @sudhansu7 I think in the mean while this problem fixed itself: @wido contributed systemd profiles so it's not needed any more to fix the CentOS 7 sysv scripts.

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-12 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-139783672 @sudhansu7 I will merge it once the other files are edited too. CentOS 7, but also the KVM agent files for the different distributions. Let me know if you need

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-11 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-139538952 LGTM. With @bhaisaab though. Please fix Centos 7 as well. Btw, we should really use systemd :) --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-10 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-139273531 Hi @sudhansu7, thanks for the fix! Could you do me a favour and look through the packaging folder and see if more scripts have this? It's also in the agent, and

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-10 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/799 CLOUDSTACK-8815 : Issues with cloudstack-management init script When a management server is halted or rebooted, the cloudstack-management init script does not successfully kill the

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-10 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-139333599 LGTM @sudhansu7 please also fix the CentOS7 one as well --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as