[GitHub] cloudstack issue #847: Bug-ID: CLOUDSTACK-8880: calculate free memory on hos...

2017-02-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/847 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 the

[GitHub] cloudstack issue #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

2017-02-19 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1917 @jayapalu thanks, yes it seems an intermittent issue with `ipmisim`, `pyghmi` library that is used by the test to simulate a live ipmi server/board. I'll try to fix it soon, meanwhile you can

[GitHub] cloudstack issue #847: Bug-ID: CLOUDSTACK-8880: calculate free memory on hos...

2017-02-19 Thread kishankavala
Github user kishankavala commented on the issue: https://github.com/apache/cloudstack/pull/847 @rhtyd Done --- 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,

Re: [DISCUSS][FS] Host HA for CloudStack

2017-02-19 Thread Koushik Das
Rohit, Thanks for the effort you have put in writing the FS. I have some questions based on my initial reading of the FS. 1. “Host HA” – In the FS you are talking about a generic HA framework but it is not clear what is meaning of “host HA”. Is it something like all or some VMs running on a

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-19 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 @karuturi, Done --- 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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-19 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 Ok, Doing that. --- 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 #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

2017-02-19 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1379 ### ACS CI BVT Run **Sumarry:** Build Number 362 Hypervisor xenserver NetworkType Advanced Passed=105 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-19 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1847 Hi @anshul1886, I just observed that the related PR #1735 was for 4.9. Since this bug also exists in 4.9, can you change the base branch of this PR to 4.9 and rebase? --- If your project is

[GitHub] cloudstack issue #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-19 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 @karuturi Test failing is unrelated to this PR. === TestName: test_add_user_to_project | Status : EXCEPTION === === TestName: test_add_user_to_project | Status : EXCEPTION

[GitHub] cloudstack issue #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

2017-02-19 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1917 Travis CI test_outofbandmanagement is failing which is not related to this PR changes. It seems there is problem with the test case. @rhtyd Do you have any idea on this ? Running

[GitHub] cloudstack issue #1379: CLOUDSTACK-8324: config drive data set/get scripts f...

2017-02-19 Thread bvbharatk
Github user bvbharatk commented on the issue: https://github.com/apache/cloudstack/pull/1379 Code changes look ok to me 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

[GitHub] cloudstack issue #1874: CLOUDSTACK-9711: Fixed error reporting while adding ...

2017-02-19 Thread bvbharatk
Github user bvbharatk commented on the issue: https://github.com/apache/cloudstack/pull/1874 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

[GitHub] cloudstack issue #1922: CLOUDSTACK-9757: Fixed issue in traffic from additio...

2017-02-19 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1922 ### ACS CI BVT Run **Sumarry:** Build Number 361 Hypervisor xenserver NetworkType Advanced Passed=105 Failed=0 Skipped=7 _Link to logs Folder

[GitHub] cloudstack issue #1953: CLOUDSTACK-9794: Unable to attach more than 14 devic...

2017-02-19 Thread remibergsma
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1953 Thanks for the fix! Although I think that after id=25 stuff breaks on KVM, as the code generates a deviceName that is invalid. Id 25 results in vdz, after that it continues with

[GitHub] cloudstack issue #1917: CLOUDSTACK-9756: Configure to ignore the ipassoc fai...

2017-02-19 Thread cloudmonger
Github user cloudmonger commented on the issue: https://github.com/apache/cloudstack/pull/1917 ### ACS CI BVT Run **Sumarry:** Build Number 360 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=1 Skipped=7 _Link to logs Folder

[GitHub] cloudstack pull request #1952: CLOUDSTACK-9790: fix NPE in case of Basic zon...

2017-02-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1952 --- 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

[GitHub] cloudstack issue #1952: CLOUDSTACK-9790: fix NPE in case of Basic zone.

2017-02-19 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1952 we have review from @ustcweizhou and @mike-tutkowski tested the fix. merging this now. Thanks you all. --- If your project is set up for it, you can reply to this email and have your

Re: Can't create a zone using master

2017-02-19 Thread Rajani Karuturi
Thanks Sigert. ~ Rajani http://cloudplatform.accelerite.com/ On February 17, 2017 at 5:53 PM, Sigert GOEMINNE (sigert.goemi...@nuagenetworks.net) wrote: Hi all, We have a new PR that fixes this issue: https://github.com/apache/cloudstack/pull/1952 Kind regards, *Sigert Goeminne* Software

Re: PRs for 4.10

2017-02-19 Thread Rajani Karuturi
noted. ~ Rajani http://cloudplatform.accelerite.com/ On February 17, 2017 at 6:03 PM, Frank Maximus (frank.maxi...@nuagenetworks.net) wrote: I have a couple of bugfixes on previous version outstanding, which I would like to have merged to 4.10, both still requiring review: on 4.8: PR#1912