[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-05-28 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-222304350 ### ACS CI BVT Run **Sumarry:** Build Number 72 Hypervisor xenserver NetworkType Advanced Passed=73 Failed=0 Skipped=3 _

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-216210063 @sureshanaparti please rebase against latest master, how do we test your change; additional tests would be great --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-05-02 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-216209760 @ --- 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 a

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-04-27 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-215327572 @sureshanaparti There are some comments, please address them. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-04-13 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-209286654 ### ACS CI BVT Run **Sumarry:** Build Number 172 Hypervisor xenserver NetworkType Advanced Passed=18 Failed=63 Skipped=3

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-03-06 Thread alexandrelimassantana
Github user alexandrelimassantana commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1252#discussion_r55169762 --- Diff: server/src/com/cloud/vm/UserVmManagerImpl.java --- @@ -4468,6 +4472,16 @@ private boolean checkIfHostIsDedicated(HostVO host) {

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-02-21 Thread pedro-martins
Github user pedro-martins commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1252#discussion_r53579947 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java --- @@ -1472,6 +1473,12 @@ private void advanceStop(final VMInstance

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-01-10 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-170381257 @DaanHoogland I see... With that option in settings, things could be more manageable. That is a good suggestion. --- If your project is set up for it, you can re

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-01-10 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-170357234 @rodrigo93 I agree but it seems that this behavioral change might be a compatibility problem to some. I would like this to be tweekable with a setting. People

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2016-01-08 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-170130323 Did someone tested it? This can be very handy. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

2015-12-16 Thread sureshanaparti
GitHub user sureshanaparti opened a pull request: https://github.com/apache/cloudstack/pull/1252 CLOUDSTACK-9182: Some running VMs turned off on manual migration when auto migration failed while host preparing for maintenance. CLOUDSTACK-9182: Some running VMs turned off on manual m