Re: Can't create a zone using master

2017-02-16 Thread Kris Sterckx
Thanks Mike Investigating. Kris On 17 February 2017 at 08:18, Tutkowski, Mike wrote: > I added a comment on the code of the relevant PR, but the PR is in the > Merged state, so I don't know if people receive e-mail notifications then. > > > On Feb 17, 2017, at 12:16

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-16 Thread priyankparihar
Github user priyankparihar commented on the issue: https://github.com/apache/cloudstack/pull/1813 Hi @borisstoyanov and @serg38 , Test code is modified. Please take a look. I think now it will not fail. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1857: CLOUDSTACK-8284: On expunge VM, initiate resource co...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1857 @adwaitpatankar this is because _resourceLimitMgr.recalculateResourceCount only recalculate the resource count on account(if accountid is set), or domain (if accountid is not set), not both

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

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1874 harmless change, 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

[GitHub] cloudstack issue #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1916 @ustcweizhou I think it would be good to add support for 16.04/systemd to run mgmt server. But Given 12.04 will EOL in next 2.5 months, let's have support for 16.04 in 4.9 as well, drop support

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1946 @borisstoyanov 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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1946 Trillian build failed: 16:43:38 FAILED - RETRYING: TASK: Wait for default template to be ready before returning (1 retries left). will try again.. @blueorangutan test --- If

[GitHub] cloudstack issue #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

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

[GitHub] cloudstack issue #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1916 @rhtyd I remember Trillian having some issues when using ubuntu images, but cannot find the build failure. Will trigger a new one to investigate. @blueorangutan test ubuntu

[GitHub] cloudstack issue #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1916 @rhtyd then I will close this PR, and create another PR for 4.10/master. Someone who run with 4.9 LTS and want to have 16.04 systemd support, can feel free to merge this PR into their

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
I added a comment on the code of the relevant PR, but the PR is in the Merged state, so I don't know if people receive e-mail notifications then. > On Feb 17, 2017, at 12:16 AM, Wei ZHOU wrote: > > Maybe we need ask Nuage why the lines are added. > > > 2017-02-17 7:54

[GitHub] cloudstack issue #1856: [4.9] CLOUDSTACK-9569: propagate global configuratio...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1856 @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

Re: Can't create a zone using master

2017-02-16 Thread Wei ZHOU
Maybe we need ask Nuage why the lines are added. 2017-02-17 7:54 GMT+01:00 Tutkowski, Mike : > Hopefully we can now find someone with experience in CloudStack's > networking code to tell us if that fix is OK or if that was just a Band-Aid > and we should do something

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
Hopefully we can now find someone with experience in CloudStack's networking code to tell us if that fix is OK or if that was just a Band-Aid and we should do something more thorough to correct the issue. > On Feb 16, 2017, at 11:48 PM, Rajani Karuturi wrote: > > Thanks for

[GitHub] cloudstack issue #1856: [4.9] CLOUDSTACK-9569: propagate global configuratio...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1856 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-501 --- 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

[GitHub] cloudstack issue #1903: [4.9] CLOUDSTACK-9356: FIX Cannot add users in VPC V...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1903 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-500 --- 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

Re: Can't create a zone using master

2017-02-16 Thread Rajani Karuturi
Thanks for digging it Mike. ~ Rajani http://cloudplatform.accelerite.com/ On February 17, 2017 at 12:18 AM, Tutkowski, Mike (mike.tutkow...@netapp.com) wrote: Hi, I opened a blocker for 4.10 regarding this issue: https://issues.apache.org/jira/browse/CLOUDSTACK-9790 Thanks, Mike On

[GitHub] cloudstack issue #1770: CLOUDSTACK-9628: Fix Template Size in Swift as Secon...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1770 @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

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

2017-02-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1847 oops.. will wait for travis --- 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

[GitHub] cloudstack pull request #1913: CLOUDSTACK-9752: [Vmware] Optimization of vol...

2017-02-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1913 --- 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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1847 merging --- 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 #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1727 merging --- 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 #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-16 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1913 merging --- 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 #1944: CLOUDSTACK-9783: Improve metrics view performance

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1944 I've created a Trillian env out of the PR and run the test and it has passed: ``` test_list_vms_metrics (tests.smoke.test_metrics_api.TestMetrics) ... === TestName:

[GitHub] cloudstack pull request #1727: CLOUDSTACK-9539: Support changing Service off...

2017-02-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1727 --- 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 #1770: CLOUDSTACK-9628: Fix Template Size in Swift as Secon...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1770 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-499 --- 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

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

2017-02-16 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 @karuturi @nvazquez , Added the marvin test from #1735. --- 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

[GitHub] cloudstack issue #1856: [4.9] CLOUDSTACK-9569: propagate global configuratio...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1856 @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

[GitHub] cloudstack issue #1856: [4.9] CLOUDSTACK-9569: propagate global configuratio...

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

[GitHub] cloudstack issue #1901: [4.9] CLOUDSTACK-9405: add details parameter in list...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1901 LGTM, @ustcweizhou can you add a marvin test to validate api changes? --- 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

[GitHub] cloudstack issue #1903: [4.9] CLOUDSTACK-9356: FIX Cannot add users in VPC V...

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

[GitHub] cloudstack issue #1903: [4.9] CLOUDSTACK-9356: FIX Cannot add users in VPC V...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1903 @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

[GitHub] cloudstack issue #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1916 @borisstoyanov can you check Trillian failures? @ustcweizhou can you check Jenkins/Travis failures? I'm in favour of removing 12.04 support (for mgmt server) in 4.9+, since the support for

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1947 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 #1770: CLOUDSTACK-9628: Fix Template Size in Swift as Secon...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1770 @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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1946 This might affect other/all APIs therefore would require regression testing, but LGTM (did not test this). --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1770: CLOUDSTACK-9628: Fix Template Size in Swift as Secon...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1770 LGTM, did not test it. @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

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed I'll work on the upgrade path today and include your fix, thanks. --- 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

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

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

[GitHub] cloudstack issue #1857: CLOUDSTACK-8284: On expunge VM, initiate resource co...

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

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

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/847 LGTM, @kishankavala please squash your changes and amend git summary with jira id. /cc @karuturi --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1913 LGTM. /cc @karuturi --- 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 #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1829 LGTM. /cc @karuturi --- 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 #1886: CLOUDSTACK-9728: Fixed traffic sentinel HTTP 414 err...

2017-02-16 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1886 @rafaelweingartner Traffic sentinel supports the post request. --- 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

[GitHub] cloudstack issue #1838: CLOUDSTACK-9682: Block VM migration to a storage whi...

2017-02-16 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1838 Code changes LGTM @karuturi This can be merged --- 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

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

2017-02-16 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1847 @anshul1886 great, thanks! --- 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

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

2017-02-16 Thread anshul1886
Github user anshul1886 commented on the issue: https://github.com/apache/cloudstack/pull/1847 @nvazquez, Yeah I am fine with it. I will add the test from your PR to mine PR --- 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 #1847: CLOUDSTACK-9691: Fixed unhandeled excetion in list s...

2017-02-16 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1847 Hi @anshul1886, I've deployed and tested your PR by replicating issue we had in our environment with Vmware and passed successfully! I think your solution is much cleaner and simpler than

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 Trillian test result (tid-842) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33489 seconds Marvin logs:

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 Trillian test result (tid-840) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32519 seconds Marvin logs:

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-16 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1918 That is a good idea @nvazquez, thanks for the help here ;) What do you think guys (@nvazquez, @borisstoyanov, @jayakarteek)? if we use the "summary.config.cpuReservatio" as

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-16 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1813 @priyankparihar Resize test still fails on vmware. It is hard to see the reason in blueorangutain output but you might want to make sure the test provisioning uses full clones via setting

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 Trillian test result (tid-835) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 43251 seconds Marvin logs:

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-16 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1913 @karuturi This one is ready for merging. LGTM and test results are positive. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack issue #1913: CLOUDSTACK-9752: [Vmware] Optimization of volume att...

2017-02-16 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1913 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 pull request #1771: CLOUDSTACK-9611: Dedicating a Guest VLAN rang...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1771#discussion_r101618476 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -3085,9 +3085,10 @@ public GuestVlan

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 Trillian test result (tid-836) Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 38546 seconds Marvin logs:

[GitHub] cloudstack pull request #1579: CLOUDSTACK-9403 : Support for shared networks...

2017-02-16 Thread mike-tutkowski
Github user mike-tutkowski commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1579#discussion_r101596817 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -2909,51 +2911,21 @@ public Vlan doInTransaction(final

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
Hi, I opened a blocker for 4.10 regarding this issue: https://issues.apache.org/jira/browse/CLOUDSTACK-9790 Thanks, Mike On 2/16/17, 8:35 AM, "Tutkowski, Mike" wrote: I believe the issue is here: https://github.com/apache/cloudstack/pull/1579

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

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/847 Trillian test result (tid-837) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28112 seconds Marvin logs:

[GitHub] cloudstack issue #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-02-16 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1918 @rafaelweingartner I checked 'objProps' object and it doesn't contain the property. However, I think it can be easily retrieved by adding "summary.config.cpuReservation" on array at line 5089

[GitHub] cloudstack issue #1944: CLOUDSTACK-9783: Improve metrics view performance

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1944 Just triggered Trillian env to check the test failure. --- 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

[GitHub] cloudstack issue #1944: CLOUDSTACK-9783: Improve metrics view performance

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1944 Trillian test result (tid-833) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28094 seconds Marvin logs:

[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1940 @jayantpatil1234 can you please have a look on the volumes and snapshots failures please. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1944: CLOUDSTACK-9783: Improve metrics view performance

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1944 Thanks for testing/reviewing @borisstoyanov --- 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1947 @blueorangutan test --- 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

[GitHub] cloudstack issue #1940: CLOUDSTACK-9781:ACS records ID in events tables inst...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1940 Trillian test result (tid-832) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28868 seconds Marvin logs:

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 @borisstoyanov 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1829 @borisstoyanov I've done my testing on a linux based HVM VM and it works as expected after the fix. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 thanks @syed --- 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 #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-498 --- 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1829 @borisstoyanov 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 @blueorangutan test centos7 vmware-60u2 --- 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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 @blueorangutan test centos7 kvm-centos7 --- 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 @borisstoyanov 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1947 @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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1946 @borisstoyanov 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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1946 @blueorangutan test --- 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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1946 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-497 --- 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-496 --- 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

Re: Can't create a zone using master

2017-02-16 Thread Tutkowski, Mike
I believe the issue is here: https://github.com/apache/cloudstack/pull/1579 In ConfigurationManagerImpl.java, it appears the lines that led to NPEs being thrown were added on lines 2924 – 2926: https://github.com/apache/cloudstack/pull/1579/files On 2/16/17, 2:42 AM, "Wido den Hollander"

[GitHub] cloudstack issue #1945: CLOUDSTACK-9787: Fix wrong return value in NetUtils....

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1945 @borisstoyanov 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

[GitHub] cloudstack issue #1945: CLOUDSTACK-9787: Fix wrong return value in NetUtils....

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1945 @blueorangutan test --- 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

[GitHub] cloudstack issue #1945: CLOUDSTACK-9787: Fix wrong return value in NetUtils....

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1945 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-495 --- 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

[GitHub] cloudstack issue #1943: CLOUDSTACK-9339: Send correct network type to router...

2017-02-16 Thread remibergsma
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1943 This issue was also handled in this PR: https://github.com/apache/cloudstack/pull/1821/files although that was done on the Python side. This seems cleaner. Should we keep both fixes? Won't

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1946 @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

[GitHub] cloudstack issue #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1946 @borisstoyanov 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1947 @borisstoyanov 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

[GitHub] cloudstack issue #1947: CLOUDSTACK-9789: Fix releasing secondary guest IP fa...

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1947 @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

[GitHub] cloudstack issue #1945: CLOUDSTACK-9787: Fix wrong return value in NetUtils....

2017-02-16 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1945 @borisstoyanov 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

[GitHub] cloudstack issue #1945: CLOUDSTACK-9787: Fix wrong return value in NetUtils....

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1945 @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

[GitHub] cloudstack issue #1829: CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer

2017-02-16 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1829 I've verified this with Windows HVM enabled template. Having more than 4 disks (6 data disks in my case), removing the disk on deviceId=4. After a reboot the VM boots as expected, user is

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-16 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1727 @karuturi sure, done! Thanks! --- 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

[GitHub] cloudstack issue #1927: ipv6: Set IPv6 CIDR and Gateway in 'nic' profile

2017-02-16 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1927 @karuturi @rhtyd: This only applies when you have IPv6 in Basic Networking enabled and are also using it. I can't think of another use-case where this goes bad. --- If your project is

[GitHub] cloudstack issue #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut I agree with reducing the size. However, I suggest to (1) remove the delaycompress, then the files like syslog.1 or rsyslog.1 will be compressed. (2) keep rotate ,

[GitHub] cloudstack pull request #1937: CLOUDSTACK-9779 : Releasing secondary guest I...

2017-02-16 Thread ustcweizhou
Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1937#discussion_r101515754 --- Diff: server/src/com/cloud/network/NetworkServiceImpl.java --- @@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long ipAddressId) {

[GitHub] cloudstack pull request #1947: CLOUDSTACK-9789: Fix releasing secondary gues...

2017-02-16 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1947 CLOUDSTACK-9789: Fix releasing secondary guest IP fails with associated static nat which is actually not used You can merge this pull request into a Git repository by running: $ git

[GitHub] cloudstack pull request #1946: CLOUDSTACK-9788: Fix exception listNetworks w...

2017-02-16 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1946 CLOUDSTACK-9788: Fix exception listNetworks with pagesize=0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/ustcweizhou/cloudstack

[GitHub] cloudstack pull request #1945: CLOUDSTACK-9787: Fix wrong return value in Ne...

2017-02-16 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1945 CLOUDSTACK-9787: Fix wrong return value in NetUtils.isNetworkAWithinNetworkB You can merge this pull request into a Git repository by running: $ git pull

  1   2   >