[GitHub] blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline

2017-12-15 Thread GitBox
blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline URL: https://github.com/apache/cloudstack/pull/2318#issuecomment-352054875 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2348: Remove ejb-api 3.0 dependency

2017-12-15 Thread GitBox
blueorangutan commented on issue #2348: Remove ejb-api 3.0 dependency URL: https://github.com/apache/cloudstack/pull/2348#issuecomment-352047436 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] syed commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response

2017-12-15 Thread GitBox
syed commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-352034337 Thanks @rhtyd This is an automated message from the

[GitHub] blueorangutan commented on issue #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job

2017-12-14 Thread GitBox
blueorangutan commented on issue #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job URL: https://github.com/apache/cloudstack/pull/2354#issuecomment-351714932 Packaging result: ?centos6 ?centos7 ?debian. JID-1372

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-12-14 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-351883775 Trillian test result (tid-1786) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 44762 seconds

[GitHub] rhtyd commented on issue #2348: Remove ejb-api 3.0 dependency

2017-12-15 Thread GitBox
rhtyd commented on issue #2348: Remove ejb-api 3.0 dependency URL: https://github.com/apache/cloudstack/pull/2348#issuecomment-352047216 @blueorangutan package This is an automated message from the Apache Git Service. To

[GitHub] rhtyd commented on issue #2184: CLOUDSTACK-10003 automatic configure juniper srx/vsrx nat loopback,

2017-12-15 Thread GitBox
rhtyd commented on issue #2184: CLOUDSTACK-10003 automatic configure juniper srx/vsrx nat loopback, URL: https://github.com/apache/cloudstack/pull/2184#issuecomment-352052969 We don't have srx infra to test this, but we'll run regression tests and accept if they pass. @blueorangutan

[GitHub] blueorangutan commented on issue #2184: CLOUDSTACK-10003 automatic configure juniper srx/vsrx nat loopback,

2017-12-15 Thread GitBox
blueorangutan commented on issue #2184: CLOUDSTACK-10003 automatic configure juniper srx/vsrx nat loopback, URL: https://github.com/apache/cloudstack/pull/2184#issuecomment-352053063 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline

2017-12-15 Thread GitBox
blueorangutan commented on issue #2318: [4.9] Smoketest health checkrun baseline URL: https://github.com/apache/cloudstack/pull/2318#issuecomment-352062033 Packaging result: ?centos6 ?centos7 ?debian. JID-1384 This is an

[GitHub] mike-tutkowski commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-12-15 Thread GitBox
mike-tutkowski commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-352092375 Sounds good, @rhtyd - it should be pretty soon. I needed to mainly attend to other work over the past week or so, so I

[GitHub] blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected

2017-12-15 Thread GitBox
blueorangutan commented on issue #2258: Cloudstack 10064: Secondary storage Usage for uploadedVolume is not collected URL: https://github.com/apache/cloudstack/pull/2258#issuecomment-352116681 Trillian test result (tid-1791) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] blueorangutan commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-12-15 Thread GitBox
blueorangutan commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347#issuecomment-352103578 Trillian test result (tid-1790) Environment: kvm-centos7 (x2), Advanced Networking

[GitHub] WingkaiHo commented on issue #2045: Fix snmptrap alert bug

2017-12-13 Thread GitBox
WingkaiHo commented on issue #2045: Fix snmptrap alert bug URL: https://github.com/apache/cloudstack/pull/2045#issuecomment-351620073 how to add patch to jira This is an automated message from the Apache Git Service. To

[GitHub] marcaurele closed pull request #2359: doc: replace virutal by virtual (typo)

2017-12-13 Thread GitBox
marcaurele closed pull request #2359: doc: replace virutal by virtual (typo) URL: https://github.com/apache/cloudstack/pull/2359 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 provenance: As this is a

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-12-14 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-351661931 Packaging result: ?centos6 ?centos7 ?debian. JID-1371 This is an

[GitHub] rhtyd commented on issue #2045: Fix snmptrap alert bug

2017-12-14 Thread GitBox
rhtyd commented on issue #2045: Fix snmptrap alert bug URL: https://github.com/apache/cloudstack/pull/2045#issuecomment-351660843 @WingkaiHo go to https://issues.apache.org/jira/secure/CreateIssue!default.jspa, create an account if necessary and submit a ticket under CLOUDSTACK project.

[GitHub] niteshsarda commented on issue #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job

2017-12-14 Thread GitBox
niteshsarda commented on issue #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job URL: https://github.com/apache/cloudstack/pull/2354#issuecomment-351664696 @rhtyd : As per your suggestion, I have made required changes in the code. Please check and if everything is

[GitHub] rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check

2017-12-14 Thread GitBox
rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-351655475 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-12-14 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-35166 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #1637: Command route not available on CentOS 7

2017-12-13 Thread GitBox
rhtyd commented on issue #1637: Command route not available on CentOS 7 URL: https://github.com/apache/cloudstack/pull/1637#issuecomment-351494942 I'll introduce `net-tools` as a dependency for centos7/cloudstack-agent, as there may be other commands/places where router/ifconfig maybe in

[GitHub] rhtyd commented on issue #1637: Command route not available on CentOS 7

2017-12-13 Thread GitBox
rhtyd commented on issue #1637: Command route not available on CentOS 7 URL: https://github.com/apache/cloudstack/pull/1637#issuecomment-351495843 Thanks @lmrv I've ported your patch to #2211 for inclusion crediting the commit to your name/email. I'll close this PR now.

[GitHub] rhtyd closed pull request #1637: Command route not available on CentOS 7

2017-12-13 Thread GitBox
rhtyd closed pull request #1637: Command route not available on CentOS 7 URL: https://github.com/apache/cloudstack/pull/1637 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 provenance: As this is a foreign

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-12-13 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-351480887 Trillian test result (tid-1781) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] mike-tutkowski commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128.

2017-12-13 Thread GitBox
mike-tutkowski commented on issue #2315: A comprehensive solution for CLOUDSTACK-9025 and CLOUDSTACK-10128. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-351507718 @rhtyd I haven't actually examined the code for this PR. I was serving mainly as a source of answers for

[GitHub] blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-12-12 Thread GitBox
blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-351245001 Trillian test result (tid-1776) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-12-12 Thread GitBox
blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-351255572 Trillian test result (tid-1777) Environment: xenserver-72 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-12-12 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-351275916 Trillian test result (tid-1778) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #1709: CLOUDSTACK-7982: KVM live migration with local storage

2017-12-12 Thread GitBox
blueorangutan commented on issue #1709: CLOUDSTACK-7982: KVM live migration with local storage URL: https://github.com/apache/cloudstack/pull/1709#issuecomment-351173963 Trillian test result (tid-1773) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-12-12 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-350976272 Packaging result: ?centos6 ?centos7 ?debian. JID-1365

[GitHub] niteshsarda commented on a change in pull request #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job

2017-12-14 Thread GitBox
niteshsarda commented on a change in pull request #2354: CLOUDSTACK-10176:VM Start Api Job returns success for failed Job URL: https://github.com/apache/cloudstack/pull/2354#discussion_r156902284 ## File path: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157832926 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#issuecomment-352848141 @yvsubhash I've left some additional remarks, please see them. After commit triaging, looks like I made an

[GitHub] rhtyd commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage

2017-12-19 Thread GitBox
rhtyd commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage URL: https://github.com/apache/cloudstack/pull/2352#issuecomment-352851020 @khos2ow the test_deploy_virtio_scsi_vm related failures are new, rest are known issues and ignorable. can you check?

[GitHub] nvazquez commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-19 Thread GitBox
nvazquez commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352899927 @blueorangutan package This is an automated message from the

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-12-19 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-352856628 Packaging result: ?centos6 ?centos7 ?debian. JID-1429

[GitHub] rhtyd closed pull request #2232: CLOUDSTACK-10040 Upload volume fails when management server can not r?

2017-12-19 Thread GitBox
rhtyd closed pull request #2232: CLOUDSTACK-10040 Upload volume fails when management server can not r? URL: https://github.com/apache/cloudstack/pull/2232 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

[GitHub] rhtyd commented on issue #2232: CLOUDSTACK-10040 Upload volume fails when management server can not r?

2017-12-19 Thread GitBox
rhtyd commented on issue #2232: CLOUDSTACK-10040 Upload volume fails when management server can not r? URL: https://github.com/apache/cloudstack/pull/2232#issuecomment-352868878 Thanks for the pr @yvsubhash, closing this as 2074 has been merged.

[GitHub] blueorangutan commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

2017-12-19 Thread GitBox
blueorangutan commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740#issuecomment-352888771 Trillian test result (tid-1829) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] nvazquez commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-19 Thread GitBox
nvazquez commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352917718 @blueorangutan test This is an automated message from the

[GitHub] blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-19 Thread GitBox
blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352917790 @nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157831617 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-12-19 Thread GitBox
rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-352846227 @fmaximus I did testing in my branch/pr #2211 and found that the dev id is not assigned in case of private gate and this

[GitHub] blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters

2017-12-19 Thread GitBox
blueorangutan commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-352870322 Trillian test result (tid-1828) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rhtyd commented on issue #2320: CLOUDSTACK-10178: Hotfixes to make 4.10 work

2017-12-19 Thread GitBox
rhtyd commented on issue #2320: CLOUDSTACK-10178: Hotfixes to make 4.10 work URL: https://github.com/apache/cloudstack/pull/2320#issuecomment-352850794 @bwsw I see most of these commits are already in master, do you want to squash your commits and if there are no new changes/commits for

[GitHub] blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-19 Thread GitBox
blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#issuecomment-352868279 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension

2017-12-19 Thread GitBox
rhtyd commented on issue #1554: CLOUDSTACK-9602: API: improve resource limits comprehension URL: https://github.com/apache/cloudstack/pull/1554#issuecomment-352855948 Test LGTM, additional review requested for merging this.

[GitHub] khos2ow commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage

2017-12-19 Thread GitBox
khos2ow commented on issue #2352: CLOUDSTACK-10175: prevent VPC list leakage URL: https://github.com/apache/cloudstack/pull/2352#issuecomment-352855972 @rhtyd yes I will. This is an automated message from the Apache Git

[GitHub] krissterckx commented on issue #2259: CLOUDSTACK-10024: Network migration support

2017-12-19 Thread GitBox
krissterckx commented on issue #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#issuecomment-352885731 @borisstoyanov can you reassess/approve and re-kick Trillian please. We aim to merge in this week. Thanks.

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157830563 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] rhtyd commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-19 Thread GitBox
rhtyd commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#issuecomment-352868008 @blueorangutan package This is an

[GitHub] blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-19 Thread GitBox
blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#issuecomment-352874854 Packaging result: ?centos6 ?centos7 ?debian. JID-1430

[GitHub] blueorangutan commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support

2017-12-19 Thread GitBox
blueorangutan commented on issue #2350: Cloudstack 10170 - fixes resource tags security bugs and adds account tags support URL: https://github.com/apache/cloudstack/pull/2350#issuecomment-352886023 Trillian test result (tid-1825) Environment: vmware-55u3 (x2), Advanced Networking with

[GitHub] blueorangutan commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca?

2017-12-19 Thread GitBox
blueorangutan commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca? URL: https://github.com/apache/cloudstack/pull/2203#issuecomment-352867209 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to

[GitHub] rhtyd commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca?

2017-12-19 Thread GitBox
rhtyd commented on issue #2203: [CLOUDSTACK-10029] All private, public, and guest interfaceses are marked as untagged ca? URL: https://github.com/apache/cloudstack/pull/2203#issuecomment-352867031 @blueorangutan test This

[GitHub] blueorangutan commented on issue #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM

2017-12-19 Thread GitBox
blueorangutan commented on issue #2042: CLOUDSTACK-9875: Unable to re-apply Explicit dedication to VM URL: https://github.com/apache/cloudstack/pull/2042#issuecomment-352877051 Trillian test result (tid-1830) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157830563 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-19 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157830379 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

2017-12-19 Thread GitBox
rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740#issuecomment-352927548 I have the same doubt as @syed here. > Why is it expected that latest snapshot be present on

[GitHub] rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

2017-12-19 Thread GitBox
rafaelweingartner commented on issue #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740#issuecomment-352927548 I have the same doubt as @syed here. > Why is it expected that latest snapshot be present on

[GitHub] rafaelweingartner commented on a change in pull request #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

2017-12-19 Thread GitBox
rafaelweingartner commented on a change in pull request #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740#discussion_r157915732 ## File path:

[GitHub] blueorangutan commented on issue #2281: CLOUDSTACK-10102: New network type (L2)

2017-12-19 Thread GitBox
blueorangutan commented on issue #2281: CLOUDSTACK-10102: New network type (L2) URL: https://github.com/apache/cloudstack/pull/2281#issuecomment-352937530 Trillian test result (tid-1832) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 28508

[GitHub] DaanHoogland closed pull request #2299: WIP: CLOUDSTACK-10118 Domain level settings

2017-12-20 Thread GitBox
DaanHoogland closed pull request #2299: WIP: CLOUDSTACK-10118 Domain level settings URL: https://github.com/apache/cloudstack/pull/2299 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 provenance: As this is

[GitHub] DaanHoogland commented on issue #2299: WIP: CLOUDSTACK-10118 Domain level settings

2017-12-20 Thread GitBox
DaanHoogland commented on issue #2299: WIP: CLOUDSTACK-10118 Domain level settings URL: https://github.com/apache/cloudstack/pull/2299#issuecomment-352998391 closing in favour of #2369 This is an automated message from the

[GitHub] DaanHoogland closed pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-20 Thread GitBox
DaanHoogland closed pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048 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 provenance: As

[GitHub] DaanHoogland commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data

2017-12-20 Thread GitBox
DaanHoogland commented on issue #2208: CLOUDSTACK-9542 make listNics and ListUserVms return uniform NIC data URL: https://github.com/apache/cloudstack/pull/2208#issuecomment-35326 Yes @rhtyd I will rebase one last time.

[GitHub] blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-20 Thread GitBox
blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-352999798 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-352999756 @blueorangutan package This is an automated

[GitHub] blueorangutan commented on issue #2117: CLOUDSTACK-8735:Browser Based Volume and Template Upload Test Automat?

2017-12-20 Thread GitBox
blueorangutan commented on issue #2117: CLOUDSTACK-8735:Browser Based Volume and Template Upload Test Automat? URL: https://github.com/apache/cloudstack/pull/2117#issuecomment-35329 Packaging result: ?centos6 ?centos7 ?debian. JID-1432

[GitHub] rhtyd commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-20 Thread GitBox
rhtyd commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#issuecomment-353000641 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-20 Thread GitBox
blueorangutan commented on issue #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#issuecomment-353000660 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway

2017-12-20 Thread GitBox
rhtyd commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway URL: https://github.com/apache/cloudstack/pull/2367#issuecomment-353001860 This may cause regressions in other cases with the stated assumption, additional review is requested. I'll run some tests.

[GitHub] krissterckx commented on issue #2259: CLOUDSTACK-10024: Network migration support

2017-12-20 Thread GitBox
krissterckx commented on issue #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#issuecomment-353003052 @borisstoyanov this PR adds MigrateNetworkCmd.class and MigrateVPCCmd.class as new commands, which just causes git not to know in

[GitHub] borisstoyanov commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353003839 @blueorangutan package This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-20 Thread GitBox
blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353004045 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-20 Thread GitBox
blueorangutan commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-353006797 Packaging result: ?centos6 ?centos7 ?debian. JID-1433

[GitHub] blueorangutan commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway

2017-12-20 Thread GitBox
blueorangutan commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway URL: https://github.com/apache/cloudstack/pull/2367#issuecomment-353008201 Packaging result: ?centos6 ?centos7 ?debian. JID-1434 This

[GitHub] rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-12-20 Thread GitBox
rhtyd commented on a change in pull request #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#discussion_r157970919 ## File path: server/src/com/cloud/network/IpAddressManagerImpl.java ## @@

[GitHub] blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-20 Thread GitBox
blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353009956 Packaging result: ?centos6 ?centos7 ?debian. JID-1435 This is an

[GitHub] blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-20 Thread GitBox
blueorangutan commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353011065 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2344: CLOUDSTACK-10163: Component tests sanity

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2344: CLOUDSTACK-10163: Component tests sanity URL: https://github.com/apache/cloudstack/pull/2344#issuecomment-353010895 @blueorangutan test centos7 kvm-centos7 component/test_*.py This is

[GitHub] borisstoyanov commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352997966 I think this looks good now, the failures are not related

[GitHub] DaanHoogland opened a new pull request #2369: CLOUDSTACK-10117 Domain level ldap configuration

2017-12-20 Thread GitBox
DaanHoogland opened a new pull request #2369: CLOUDSTACK-10117 Domain level ldap configuration URL: https://github.com/apache/cloudstack/pull/2369 This contains fixes and implementation of CLOUDSTACK-10117 and CLOUDSTACK-10118, enabling the operator to configure an ldap server per domain.

[GitHub] DaanHoogland closed pull request #2363: 4.8.0.1 rc20160525 t1247

2017-12-20 Thread GitBox
DaanHoogland closed pull request #2363: 4.8.0.1 rc20160525 t1247 URL: https://github.com/apache/cloudstack/pull/2363 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] DaanHoogland commented on issue #2363: 4.8.0.1 rc20160525 t1247

2017-12-20 Thread GitBox
DaanHoogland commented on issue #2363: 4.8.0.1 rc20160525 t1247 URL: https://github.com/apache/cloudstack/pull/2363#issuecomment-352999702 not a vaild pr This is an automated message from the Apache Git Service. To respond to

[GitHub] borisstoyanov commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2368#issuecomment-352999454 @nvazquez it appears this one has conflicts, can you please resolve them and I'll run tests on it.

[GitHub] rhtyd commented on a change in pull request #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name

2017-12-20 Thread GitBox
rhtyd commented on a change in pull request #2366: CLOUDSTACK-10168 - VR duplicate entries in /etc/hosts when reusing VM name URL: https://github.com/apache/cloudstack/pull/2366#discussion_r157964380 ## File path: systemvm/patches/debian/config/opt/cloud/bin/cs_dhcp.py ##

[GitHub] blueorangutan commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway

2017-12-20 Thread GitBox
blueorangutan commented on issue #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway URL: https://github.com/apache/cloudstack/pull/2367#issuecomment-353001946 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] krissterckx commented on issue #2097: [4.11] CLOUDSTACK-9813: Extending Config Drive support

2017-12-20 Thread GitBox
krissterckx commented on issue #2097: [4.11] CLOUDSTACK-9813: Extending Config Drive support URL: https://github.com/apache/cloudstack/pull/2097#issuecomment-353004576 Hi @rhtyd i see you removed 4.11 label from the PR. See discussion thread with @wido. At Nuage we see lots of

[GitHub] borisstoyanov commented on issue #2259: CLOUDSTACK-10024: Network migration support

2017-12-20 Thread GitBox
borisstoyanov commented on issue #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#issuecomment-353005252 @krissterckx sure I can kick the marvin tests, have you managed to add the default network provider tests?

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-12-19 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-352941800 Trillian test result (tid-1833) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] nvazquez commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM

2017-12-19 Thread GitBox
nvazquez commented on issue #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2368#issuecomment-352944809 @rhtyd @DaanHoogland @borisstoyanov @nitin-maharana please review

[GitHub] blueorangutan commented on issue #2104: CLOUDSTACK-9908 : Primary Storage allocated capacity goes very high after VM snapshot

2017-12-19 Thread GitBox
blueorangutan commented on issue #2104: CLOUDSTACK-9908 : Primary Storage allocated capacity goes very high after VM snapshot URL: https://github.com/apache/cloudstack/pull/2104#issuecomment-352949554 Trillian test result (tid-1831) Environment: vmware-55u3 (x2), Advanced Networking

[GitHub] nvazquez commented on issue #2305: [PLACEHOLDER] CLOUDSTACK-10126: Separate Subnet for CPVM and SSVM

2017-12-19 Thread GitBox
nvazquez commented on issue #2305: [PLACEHOLDER] CLOUDSTACK-10126: Separate Subnet for CPVM and SSVM URL: https://github.com/apache/cloudstack/pull/2305#issuecomment-352949270 Closing this PR as work is continued on #2368

[GitHub] nvazquez closed pull request #2305: [PLACEHOLDER] CLOUDSTACK-10126: Separate Subnet for CPVM and SSVM

2017-12-19 Thread GitBox
nvazquez closed pull request #2305: [PLACEHOLDER] CLOUDSTACK-10126: Separate Subnet for CPVM and SSVM URL: https://github.com/apache/cloudstack/pull/2305 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

[GitHub] voloshanenko opened a new pull request #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway

2017-12-19 Thread GitBox
voloshanenko opened a new pull request #2367: Fix ACL_INBOUND/OUTBOUND rules for PrivateGateway URL: https://github.com/apache/cloudstack/pull/2367 We found bug in ACL rules for PrivateGateway for VPC At a glance - rules not applied - switching Allow All or Deny All (default ACL) -

[GitHub] nvazquez opened a new pull request #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM

2017-12-19 Thread GitBox
nvazquez opened a new pull request #2368: CLOUDSTACK-10126: Separate Subnet for SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2368 JIRA Ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-10126 ### Background This extends work presented on #2048 on which the

[GitHub] nvazquez commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics

2017-12-19 Thread GitBox
nvazquez commented on issue #2035: CLOUDSTACK-9867:VM snapshot on primary storage usage metrics URL: https://github.com/apache/cloudstack/pull/2035#issuecomment-352959690 @rhtyd sure, conflicts fixed This is an automated

[GitHub] blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-19 Thread GitBox
blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352900152 @nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

2017-12-19 Thread GitBox
blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-352905755 Packaging result: ?centos6 ?centos7 ?debian. JID-1431 This

[GitHub] nvazquez commented on issue #2146: Multidisk ova import

2017-12-19 Thread GitBox
nvazquez commented on issue #2146: Multidisk ova import URL: https://github.com/apache/cloudstack/pull/2146#issuecomment-352969262 Sure @rhtyd, conflicts fixed This is an automated message from the Apache Git Service. To

<    5   6   7   8   9   10   11   12   13   14   >