[GitHub] ustcweizhou commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
ustcweizhou commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#discussion_r167781635 ## File path: engine/schema/resources/META-INF/db/schema-41000to41100.sql ## @@ -5

[GitHub] niteshsarda commented on issue #2451: CLOUDSTACK-10284:Creating a snapshot from VM Snapshot generates error if hypervisor is not KVM.

2018-02-12 Thread GitBox
niteshsarda commented on issue #2451: CLOUDSTACK-10284:Creating a snapshot from VM Snapshot generates error if hypervisor is not KVM. URL: https://github.com/apache/cloudstack/pull/2451#issuecomment-365164038 @ustcweizhou : I have address your second issue in latest code, but could not und

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-365082774 Trillian test result (tid-2254) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total ti

[GitHub] nvazquez opened a new pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
nvazquez opened a new pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423 JIRA Ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-10250 This fix allows users to restore a VM from a previously r

[GitHub] DaanHoogland closed pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
DaanHoogland closed pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of prov

[GitHub] DaanHoogland commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's

2018-02-12 Thread GitBox
DaanHoogland commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's URL: https://github.com/apache/cloudstack/pull/2442#issuecomment-365043745 @houthuis did you ask the submitter (of CLOUDSTACK-10147) about the validity of the functionality? --

[GitHub] nvazquez opened a new pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
nvazquez opened a new pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423 JIRA Ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-10250 This fix allows users to restore a VM from a previously r

[GitHub] DaanHoogland closed pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
DaanHoogland closed pull request #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of prov

[GitHub] DaanHoogland commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added

2018-02-12 Thread GitBox
DaanHoogland commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added URL: https://github.com/apache/cloudstack/pull/2446#issuecomment-365042857 @marcaurele are you -1 on this chance as is? Thi

[GitHub] blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's

2018-02-12 Thread GitBox
blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's URL: https://github.com/apache/cloudstack/pull/2442#issuecomment-365032026 Trillian test result (tid-2252) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7

[GitHub] blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423#issuecomment-364999878 Trillian test result (tid-2253) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Tot

[GitHub] blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added

2018-02-12 Thread GitBox
blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added URL: https://github.com/apache/cloudstack/pull/2446#issuecomment-364996703 Trillian test result (tid-2251) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] rhtyd commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2018-02-12 Thread GitBox
rhtyd commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-364967523 @syed great, happy to collaborate you've my support. Perhaps we can also get some help from @ustcweizhou and

[GitHub] syed commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER

2018-02-12 Thread GitBox
syed commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-364958976 @rhtyd Sorry for dropping the ball on this. We will be looking at working on this in the coming months. -

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364954377 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests ---

[GitHub] rhtyd commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
rhtyd commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364954204 @blueorangutan test This is an automated message from the

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364930948 Packaging result: ?centos6 ?centos7 ?debian. JID-1710 -

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364923692 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364901557 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] rhtyd commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
rhtyd commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364923525 @blueorangutan package This is an automated message from t

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364907238 Packaging result: ?centos6 ?centos7 ?debian. JID-1709 -

[GitHub] rhtyd commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
rhtyd commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#discussion_r167555229 ## File path: engine/schema/resources/META-INF/db/schema-41000to41100.sql ## @@ -546,3 +

[GitHub] ustcweizhou commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
ustcweizhou commented on a change in pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#discussion_r167540175 ## File path: engine/schema/resources/META-INF/db/schema-41000to41100.sql ## @@ -5

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364907238 Packaging result: ?centos6 ?centos7 ?debian. JID-1709 -

[GitHub] blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
blueorangutan commented on issue #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452#issuecomment-364901557 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] rhtyd opened a new pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users

2018-02-12 Thread GitBox
rhtyd opened a new pull request #2452: CLOUDSTACK-10285: Fix db upgrade failure for 4.10.0.0 users URL: https://github.com/apache/cloudstack/pull/2452 4.10.0.0 users when upgrade to 4.11.0.0 may face db related discrepancies due to some PRs that got merged without moving their sql cha

[cloudstack] branch 4.11 updated: CLOUDSTACK-10261: Libvirt metadata create only one nuage-extension xml tag (#2441)

2018-02-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new dee5bb5 CLOUDSTACK-10261: Libvirt metadata creat

[GitHub] rhtyd closed pull request #2441: CLOUDSTACK-10261: Libvirt metadata: only create one nuage-extension tag

2018-02-12 Thread GitBox
rhtyd closed pull request #2441: CLOUDSTACK-10261: Libvirt metadata: only create one nuage-extension tag URL: https://github.com/apache/cloudstack/pull/2441 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of prov

[GitHub] rhtyd commented on issue #2441: CLOUDSTACK-10261: Libvirt metadata: only create one nuage-extension tag

2018-02-12 Thread GitBox
rhtyd commented on issue #2441: CLOUDSTACK-10261: Libvirt metadata: only create one nuage-extension tag URL: https://github.com/apache/cloudstack/pull/2441#issuecomment-364891629 LGTM, merging based on review and test results (ignoring intermittent failures). -

[cloudstack-docs-install] 01/01: source: bump version

2018-02-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-install.git commit 3ae5d3e45f0f84d5a65ee9a89932d1d456506b17 Author: Rohit Yadav AuthorDate: Mon Feb 12 12:03:50 2018 +0100

[cloudstack-docs-install] branch master updated (d11db4f -> 3ae5d3e)

2018-02-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-install.git. from d11db4f Updates for 4.11 release add b2eb9cc update preferred versions text (#34) new 3ae

[cloudstack-docs-install] branch 4.11 updated: update preferred versions text (#34)

2018-02-12 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack-docs-install.git The following commit(s) were added to refs/heads/4.11 by this push: new b2eb9cc update preferred versions t

[GitHub] blueorangutan commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
blueorangutan commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364887772 Packaging result: ?centos6 ?centos7 ?debian. JID-1708 --

[GitHub] blueorangutan commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
blueorangutan commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364881789 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] rhtyd commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
rhtyd commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364881765 @blueorangutan package This is an automated mes

[GitHub] DaanHoogland commented on issue #2449: WIP CLOUDSTACK-10278 idempotent column addition

2018-02-12 Thread GitBox
DaanHoogland commented on issue #2449: WIP CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-364879178 @ustcweizhou I agree that something like that would be preferable but someone has to take the heat on implementing that. And nob

[GitHub] borisstoyanov commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-02-12 Thread GitBox
borisstoyanov commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-364874573 @NuxRo we'll be addressing this as well -

[GitHub] NuxRo commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network

2018-02-12 Thread GitBox
NuxRo commented on issue #2448: CLOUDSTACK-10274: L2 network refused to be designed on VXLAN physical network URL: https://github.com/apache/cloudstack/pull/2448#issuecomment-364873501 A bit confused as to whether my previous complaint was noticed, so here goes again: - this is only

[GitHub] marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364857790 Rebased done @rhtyd This bug leaves the systemvm with all ports opened

[GitHub] marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364857790 Rebased done @rhtyd This bug leaves the systemvm with all port opened -

[GitHub] marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab?

2018-02-12 Thread GitBox
marcaurele commented on issue #2450: CLOUDSTACK-10282: firewall rules operation should be done with ip6tab? URL: https://github.com/apache/cloudstack/pull/2450#issuecomment-364857790 Rebased done @rhtyd This bug leaves the systemvm with all port opens --

[GitHub] borisstoyanov commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's

2018-02-12 Thread GitBox
borisstoyanov commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's URL: https://github.com/apache/cloudstack/pull/2442#issuecomment-364855183 @blueorangutan test centos7 xenserver-65sp1

[GitHub] blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's

2018-02-12 Thread GitBox
blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's URL: https://github.com/apache/cloudstack/pull/2442#issuecomment-364855206 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke test

[GitHub] blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423#issuecomment-364855209 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added

2018-02-12 Thread GitBox
blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added URL: https://github.com/apache/cloudstack/pull/2446#issuecomment-364854977 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
borisstoyanov commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423#issuecomment-364855042 @blueorangutan test This is an automated mess

[GitHub] borisstoyanov commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added

2018-02-12 Thread GitBox
borisstoyanov commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added URL: https://github.com/apache/cloudstack/pull/2446#issuecomment-364854961 @blueorangutan test This is an automated messag

[GitHub] blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template

2018-02-12 Thread GitBox
blueorangutan commented on issue #2423: CLOUDSTACK-10250: Reinstall VM from bypassed registered template URL: https://github.com/apache/cloudstack/pull/2423#issuecomment-364854140 Packaging result: ?centos6 ?centos7 ?debian. JID-1707

[GitHub] blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's

2018-02-12 Thread GitBox
blueorangutan commented on issue #2442: CLOUDSTACK-10147 Disabled Xenserver Cluster can still deploy VM's URL: https://github.com/apache/cloudstack/pull/2442#issuecomment-364854142 Packaging result: ?centos6 ?centos7 ?debian. JID-1706 ---

[GitHub] blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added

2018-02-12 Thread GitBox
blueorangutan commented on issue #2446: CLOUDSTACK-10271 maven plugin for owasp dependency check added URL: https://github.com/apache/cloudstack/pull/2446#issuecomment-364854141 Packaging result: ?centos6 ?centos7 ?debian. JID-1705 --

[GitHub] ustcweizhou commented on issue #2451: CLOUDSTACK-10284:Creating a snapshot from VM Snapshot generates error if hypervisor is not KVM.

2018-02-12 Thread GitBox
ustcweizhou commented on issue #2451: CLOUDSTACK-10284:Creating a snapshot from VM Snapshot generates error if hypervisor is not KVM. URL: https://github.com/apache/cloudstack/pull/2451#issuecomment-364853929 @niteshsarda sorry I cannot merge it now. two issues (1) what if the instanc