[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 Test LGTM, the two failures are intermittent and env related. --- 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 n

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Trillian test result (tid-427) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28223 seconds Marvin logs: https://github.com/blueoranguta

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 Checked centos6 failed due to an intermittent unit test failure, we can ignore that for now. @blueorangutan test --- If your project is set up for it, you can reply to this email and have you

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-24 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-248 --- 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 d

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-23 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-23 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 @blueorangutan package --- 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

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-20 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd @jburwell I believe the last commit should have addressed his concerns. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-203 --- 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 d

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 @nathanejohnson can you reply to @jburwell 's comments --- 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

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-11-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 @blueorangutan package --- 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

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-10-21 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-78 --- 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 do

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-10-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-10-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1635 @blueorangutan package --- 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

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-09-07 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1635 I did find one marvin test that does pass forced flag. Here are the results. https://gist.github.com/nathanejohnson/c4d1ffb8b61b19a8105dcb5326794b15 --- If your project is set

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-09-02 Thread nathanejohnson
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1635 @jburwell sorry I've been silent, I've been on vacation for a bit and I will be juggling plans this weekend as well. I plan to update this PR soon, and as far as a Marvin test I am not a

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-09-01 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1635 @nathanejohnson I want to get this bug fix into 4.8.2.0. In order to make it, we need to complete the following: * [] Resolve the question about the name of the method (``stop`` vs

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-24 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 I'll let him comment on this, but If I recall, it had more to do with implementation of force stopping with KVM rather than the lack of the parameter being passing into the VM Manager. --- I

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-24 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1635 All hypervisors are affected by this, since the parameter is just not passed into the VM Manager. That means its effects don't work at all anywhere. :) I will try to review it to tomorrow.

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-24 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 @ProjectMoon Jeff, could you help review this, as I believe you were the first to point it out on the list. From talking to Nathan, it sounds like we're not sure KVM is actually affec

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-22 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1635 @nathanejohnson please amend your commit message to explain the change(s) made and their motivation? --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-22 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1635 Packaging result: ✔centos6 ✔centos7 ✔debian repo: http://packages.shapeblue.com/cloudstack/pr/1635 Job ID-95 --- If your project is set up for it, you can reply to this email an

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-19 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1635 LGTM --- 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 t

[GitHub] cloudstack issue #1635: CLOUDSTACK-9451

2016-08-11 Thread kiwiflyer
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1635 References: https://github.com/apache/cloudstack/pull/1632 --- 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