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

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

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

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

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

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1945 Trillian env failed to deploy, restarting tests. @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

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

2017-02-17 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 #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1813 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-502 --- 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 #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan test centos7 xenserver-65sp1 --- 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 th

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

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

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

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1813 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 feature enabled

[GitHub] cloudstack pull request #1250: automated CLOUDSTACK 9177 9178 9179

2017-02-17 Thread shwetaag
Github user shwetaag closed the pull request at: https://github.com/apache/cloudstack/pull/1250 --- 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 #1250: automated CLOUDSTACK 9177 9178 9179

2017-02-17 Thread shwetaag
Github user shwetaag commented on the issue: https://github.com/apache/cloudstack/pull/1250 as these cases are also included in PR 1401 so closing this PR --- 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 pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708018 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -781,6 +781,15 @@ "ostype": "CentOS 5.6 (64-bit)" }, +"core

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708093 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -790,6 +799,26 @@ "format": "OVA", "ispublic": "true" },

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708141 --- Diff: test/integration/component/maint/test_escalations_instances.py --- @@ -0,0 +1,270 @@ +# Licensed to the Apache Software Foundation (ASF) u

Re: apidocs build failure

2017-02-17 Thread Rajani Karuturi
since travis is already verifying this, I asked infra to disable this job. Infra ticket https://issues.apache.org/jira/browse/INFRA-13527 Please comment on the ticket if you think otherwise. Thanks, ~ Rajani http://cloudplatform.accelerite.com/ On February 13, 2017 at 12:29 PM, Rohit Yadav (r

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708208 --- Diff: test/integration/component/test_escalations_instances.py --- @@ -2068,6 +2068,94 @@ def test_13_vm_nics(self): "VM NIC is

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

2017-02-17 Thread adwaitpatankar
Github user adwaitpatankar commented on the issue: https://github.com/apache/cloudstack/pull/1857 @ustcweizhou, the issue is that the resource count at domain level are not getting updated implicitly after expunge VM call. If the user tries to add a new instance, and if the primary st

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708342 --- Diff: test/integration/component/test_project_usage.py --- @@ -1819,3 +1844,183 @@ def test_01_vpn_usage(self): "Ch

[GitHub] cloudstack pull request #1401: automated 9277 9276 9275 9274 9273 9179 9178 ...

2017-02-17 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1401#discussion_r101708312 --- Diff: test/integration/component/test_project_usage.py --- @@ -221,10 +219,35 @@ def test_01_vm_usage(self): #VM.Destroy and volume

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

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1857 @adwaitpatankar oh, I totally understand the issue and the change you made. You are right as I remember. However, I think the recalculation on other accounts in the domain is not necessary (

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

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1916 @rhtyd As 4.9 is LTS version, it seems not good to drop ubuntu 12.04 support in next minor release (4.9.3 ?), some customers may still use ubuntu 12.04 as management server. I will

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

2017-02-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1916 @ustcweizhou makes sense, in that case please close this PR and open a new PR for 4.10/master that adds support for cloudstack mgmt server on ubuntu 16.04, dropping support for 12.04 etc. --- If

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

2017-02-17 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1916 @ustcweizhou @rhtyd Seems good to me. Let's drop 12.04 support in 4.10 Java 8 anyway which is not supported on 12.04, so we will need to drop it. If we just can get rid of 14.04 the

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

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1916 we got this failure when installing cloudstack-management ``` root@pr1916-t848-kvm-ubuntu-mgmt1:~# apt-get install cloudstack-management Reading package lists... Done Bu

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

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1916 @borisstoyanov It is weird I did not specify the minimal version of init-system-helpers in debian/control. Did you build the packages on ubuntu 16.04 and install it on ubuntu 14.04 ?

[GitHub] cloudstack pull request #1948: Faster IP in subnet check

2017-02-17 Thread ProjectMoon
GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1948 Faster IP in subnet check This change removes the conversion from IPNetwork to list in one of the router scripts. This makes the router faster at processing static NAT rules, which can pre

Re: apidocs build failure

2017-02-17 Thread Rohit Yadav
Thanks Rajani, I've no objections. Regards. From: Rajani Karuturi Sent: 17 February 2017 14:07:34 To: dev@cloudstack.apache.org Subject: Re: apidocs build failure since travis is already verifying this, I asked infra to disable this job. Infra ticket https://i

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

2017-02-17 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1943 I didn't actually find the other PR, otherwise I probably wouldn't have submitted this one. But I do think this is cleaner, and more centralized. There's no real harm in having both, and hav

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

2017-02-17 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 f

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

2017-02-17 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 #1916: [4.9] CLOUDSTACK-9462: Build packages on Ubuntu 12.0...

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1916 used dpkg-buildpackage the docker container just for the build is 16.04 but @rhtyd reports he's able to install this package on his local 14.04 --- If your project is set up for i

[GitHub] cloudstack pull request #1949: Cruds2

2017-02-17 Thread shwetaag
GitHub user shwetaag opened a pull request: https://github.com/apache/cloudstack/pull/1949 Cruds2 Addressed all review comments. Changed all URL to pubic available url where ever possible . For few tests needing rhel7 template or windows server templates added a comment in test ca

[GitHub] cloudstack pull request #1950: CLOUDSTACK-9462: Build packages on Ubuntu 14....

2017-02-17 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1950 CLOUDSTACK-9462: Build packages on Ubuntu 14.04/16.04 and support tomcat6/tomcat7 Changes (1) add systemd support in debian/control and debian/rules for 16.04 (2) add python-setupto

[GitHub] cloudstack issue #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

2017-02-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1950 @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 #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1950 @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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

[GitHub] cloudstack issue #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1950 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-503 --- 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 #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

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

[GitHub] cloudstack issue #1950: [4.10] CLOUDSTACK-9462: Build packages on Ubuntu 14....

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

[GitHub] cloudstack issue #1949: Cruds2

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1949 @shwetaag Hi, I'm sorry but I don't get what is this PR for? Are these some internal issues that you're referring to? Each PR should start with an ACS JIRA ticket number and should be foll

[GitHub] cloudstack pull request #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread rhtyd
GitHub user rhtyd opened a pull request: https://github.com/apache/cloudstack/pull/1951 CLOUDSTACK-9792: Add upgrade path for 4.9.3.0 This adds an upgrade path from 4.9.2.0 to 4.9.3.0, this also includes changes from PR https://github.com/apache/cloudstack/pull/1928 that adds missi

[GitHub] cloudstack pull request #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread rhtyd
Github user rhtyd commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1951#discussion_r101735670 --- Diff: setup/db/db/schema-4920to4930.sql --- @@ -0,0 +1,22 @@ +-- Licensed to the Apache Software Foundation (ASF) under one +-- or more contrib

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

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

[GitHub] cloudstack pull request #1949: Cruds2

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1949#discussion_r101735942 --- Diff: tools/marvin/marvin/config/test_data.py --- @@ -121,7 +121,7 @@ "name": "LargeInstance", "displaytext":

[GitHub] cloudstack pull request #1908: CLOUDSTACK-9317: Fixed disable static nat on ...

2017-02-17 Thread ProjectMoon
Github user ProjectMoon commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1908#discussion_r101736186 --- Diff: server/src/com/cloud/network/router/CommandSetupHelper.java --- @@ -175,6 +176,8 @@ @Inject private IPAddressDao _ipAddre

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

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

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

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

[GitHub] cloudstack issue #1949: Cruds2

2017-02-17 Thread shwetaag
Github user shwetaag commented on the issue: https://github.com/apache/cloudstack/pull/1949 @borisstoyanov This PR is about automating these 8 cloudstack bugs. automated these 8 test bugs 9277 9276 9275 9274 9273 9179 9178 9177 - Test to verify Scale CentOS7 VM does

[GitHub] cloudstack issue #1949: Cruds2

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1949 Thank you @shwetaag can you please edit the PR description --- 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 #1948: Faster IP in subnet check

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1948 @ProjectMoon thank you for this PR, is there an ACS JIRA issue for this change, if so it'll be good to add it in the title same as the other PRs. --- If your project is set up for it, yo

[GitHub] cloudstack issue #1948: Faster IP in subnet check

2017-02-17 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1948 We have not entered one into JIRA, no. But we can. --- 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 ha

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1951 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-504 --- 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 #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread priyankparihar
Github user priyankparihar commented on the issue: https://github.com/apache/cloudstack/pull/1773 Hi @jburwell , Thanks for teaching me something new. Test is also added. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

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

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1951 @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 #1948: Faster IP in subnet check

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

[GitHub] cloudstack issue #1948: Faster IP in subnet check

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

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1951 @borisstoyanov an explicit integration test is not necessary if Travis passes, since all changes are around db upgrade/version. --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack issue #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1951 Just want to confirm it fresh installs and upgrades the DB to 4.9.3. will cancel the run after as I'm planning to do an upgrade scenario as well. Thanks @rhtyd --- If your project i

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1773 @priyankparihar 1. the file change from 10644 to 10755 is not necessary 2. the default value of forced is false, might cause issue on backwards compatibility 3. In the vm install

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread priyankparihar
Github user priyankparihar commented on the issue: https://github.com/apache/cloudstack/pull/1773 Hi @jburwell , Would like to add something on @ustcweizhou comment ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

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

2017-02-17 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1838 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 t

[GitHub] cloudstack issue #1948: Faster IP in subnet check

2017-02-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1948 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-505 --- 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 pull request #1952: CLOUDSTACK-9790: fix NPE in case of Basic zon...

2017-02-17 Thread sgoeminn
GitHub user sgoeminn opened a pull request: https://github.com/apache/cloudstack/pull/1952 CLOUDSTACK-9790: fix NPE in case of Basic zone. This PR fixes the creation of a basic zone. https://issues.apache.org/jira/browse/CLOUDSTACK-9790 You can merge this pull request into a Git

[GitHub] cloudstack pull request #1838: CLOUDSTACK-9682: Block VM migration to a stor...

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

Re: Can't create a zone using master

2017-02-17 Thread Sigert GOEMINNE
Hi all, We have a new PR that fixes this issue: https://github.com/apache/cloudstack/pull/1952 Kind regards, *Sigert Goeminne* Software Development Engineer *nuage*networks.net Copernicuslaan 50 2018 Antwerp Belgium On Fri, Feb 17, 2017 at 8:55 AM, Kris Sterckx < kris.ster...@nuagenetworks.

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

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

Re: PRs for 4.10

2017-02-17 Thread Frank Maximus
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 : which fixes password service running on internal lb vms, making it impossible to do loadbalancing on

[GitHub] cloudstack issue #1948: [CLOUDSTACK-9793] Faster IP in subnet check

2017-02-17 Thread ProjectMoon
Github user ProjectMoon commented on the issue: https://github.com/apache/cloudstack/pull/1948 Updated with the JIRA ticket. --- 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 enabl

[GitHub] cloudstack issue #1948: [CLOUDSTACK-9793] Faster IP in subnet check

2017-02-17 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1948 Thanks, @ProjectMoon I cannot run tests against it since we don't have an automated way to build the systemVM. --- If your project is set up for it, you can reply to this email and have

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

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

[GitHub] cloudstack pull request #1735: CLOUDSTACK-9570: Bug in listSnapshots for sna...

2017-02-17 Thread nvazquez
Github user nvazquez closed the pull request at: https://github.com/apache/cloudstack/pull/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 not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack issue #1735: CLOUDSTACK-9570: Bug in listSnapshots for snapshots ...

2017-02-17 Thread nvazquez
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1735 Closing this PR as #1847 includes this changes in a simpler way --- 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 #1946: CLOUDSTACK-9788: Fix exception listNetworks with pag...

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

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

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

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

2017-02-17 Thread leprechau
Github user leprechau commented on the issue: https://github.com/apache/cloudstack/pull/1915 We are seeing issues under heavy user interaction cycles with redundant VRs. In testing you can reliably completely fill the `/var/log/` partition with data primarily from `cloud.log` when yo

Modify the system vm build scripts

2017-02-17 Thread David Mabry
Hello everyone, I’m looking at making some changes to the system vm, but I have found that there looks like there are 2 different places in the code that “build” the systemvm. There is there is https://github.com/apache/cloudstack/tree/13bfdd71e6f52d2f613a802b3d16c9b40af7/systemvm/patches/

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

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 Agreed with @leprechau . I don't think just changing logrotate to compress and rotate at smaller size intervals will not completely solve our issues here. We really need to adjust logrotate to

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1773 @priyankparihar I agree with @ustcweizhou . Default behavior should remain forced. And in this case Web UI we should give a warning with "yes and no" if template has deployed VM so users can canc

Re: Modify the system vm build scripts

2017-02-17 Thread Will Stevens
So the System VM is "built" from two sources. 1) https://github.com/apache/cloudstack/tree/master/tools/appliance/definitions/systemvmtemplate This defines what is actually built and is distributed as the SystemVM Template. You MUST use it if you change the packages included in the SystemVM templ

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

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

Re: Modify the system vm build scripts

2017-02-17 Thread David Mabry
Awesome. Thanks for the quick answer. That totally makes sense. Package changes (installation, etc…) are done in the “appliance” section of code. Any “config” changes required beyond package installation are done in the “patches” section of code. Thanks, David Mabry On 2/17/17, 10:52 AM, "

Re: Modify the system vm build scripts

2017-02-17 Thread Will Stevens
Yes. Exactly. The "patches" don't require a new system vm template to be distributed, but changes to the "appliance" does require a new system vm template. On Feb 17, 2017 12:04 PM, "David Mabry" wrote: > Awesome. Thanks for the quick answer. That totally makes sense. Package > changes (insta

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

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

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1773 @priyankparihar @ustcweizhou There seems to be just one case where it will affect resetVM operations. If template is deleted and root disk is migrated to another PS where there is no template cop

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

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 agree with @dmabry logrotate need to be moved (or copied) to cron.hourly ``` diff --git a/systemvm/patches/debian/config/etc/cron.hourly/logrotate b/systemvm/patches/debian/confi

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

2017-02-17 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue: https://github.com/apache/cloudstack/pull/1952 LGTM I have also incorporated this code and created a Basic Zone with it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

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

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

[GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1773 @serg38 yes, your approach is the best solution for now. By the way, I've implemented the copy of template between storage pools on kvm/nfs in our branch, not implemented for other s

Re: [GitHub] cloudstack issue #1773: CLOUDSTACK-9607: Preventing template deletion when t...

2017-02-17 Thread Sergey Levitskiy
@ustcweizhou that's great . If you can open PR here @nvazquez can try implementing in-between PS copies for VMware . Also for VMware storage, migration doesn't require base template at destination PS even for linked clones. VMware will always convert link clone to a full one during migration Se

[GitHub] cloudstack-www issue #35: Update "who" page to reflect the new additions to ...

2017-02-17 Thread swill
Github user swill commented on the issue: https://github.com/apache/cloudstack-www/pull/35 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 #1915: CLOUDSTACK-9746 system-vm: logrotate config causes c...

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @ustcweizhou - Thanks for the +1 on cron.hourly move. From looking at the code, it looks like non-redundant VRs are already moving to cron.hourly, but vpcrouter isn't. In our case, we are using

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

2017-02-17 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1915 @dmabry I will test it and update you. --- 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 fea

[GitHub] cloudstack pull request #1951: CLOUDSTACK-9792: Add upgrade path for 4.9.3.0

2017-02-17 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1951#discussion_r101851324 --- Diff: setup/db/db/schema-4920to4930.sql --- @@ -0,0 +1,22 @@ +-- Licensed to the Apache Software Foundation (ASF) under one +-- or more contribu

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

2017-02-17 Thread David Mabry
On 2/16/17, 5:18 AM, "Rohit Yadav" wrote: All, I would like to start discussion on a new feature - Host HA for CloudStack. CloudStack lacks a way to reliably fence a host, the idea of the host-ha feature is to provide a general purpose HA framework and HA provider im

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

2017-02-17 Thread dmabry
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1915 @serbaut Just curious, are you using VPC routers? --- 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 thi

[GitHub] cloudstack issue #1883: CLOUDSTACK-9723: Enable unique mac address across th...

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

Re: Modify the system vm build scripts

2017-02-17 Thread Daan Hoogland
nice explanation Will, c&p to the wiki and docs, i'd say On Fri, Feb 17, 2017 at 6:06 PM, Will Stevens wrote: > Yes. Exactly. The "patches" don't require a new system vm template to be > distributed, but changes to the "appliance" does require a new system vm > template. > > On Feb 17, 2017 12:04