[GitHub] blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336588451 Trillian test result (tid-1590) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] resmo commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP

2017-10-13 Thread git
resmo commented on issue #2267: CLOUDSTACK-10077: allow to have different VPN customer gateway configs for same gateway IP URL: https://github.com/apache/cloudstack/pull/2267#issuecomment-336580653 issue is unrelated, ready_to_merge

[GitHub] blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336580200 Trillian test result (tid-1591) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 38463 seconds

[GitHub] blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#issuecomment-336538399 Trillian test result (tid-1589) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] mike-tutkowski commented on issue #2081: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot

2017-10-13 Thread git
mike-tutkowski commented on issue #2081: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot URL: https://github.com/apache/cloudstack/pull/2081#issuecomment-336525261 The problem that has prevented me from testing this PR is being addressed in the following PR:

[GitHub] mike-tutkowski commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
mike-tutkowski commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336525000 I have tested this code in a Basic Zone with XenServer and it worked. Also, the code LGTM. Thanks, @DaanHoogland!

[GitHub] nvazquez commented on a change in pull request #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
nvazquez commented on a change in pull request #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#discussion_r144577489 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java

[GitHub] nvazquez commented on a change in pull request #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
nvazquez commented on a change in pull request #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#discussion_r144577489 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java

[GitHub] sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements

2017-10-13 Thread git
sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements URL: https://github.com/apache/cloudstack/pull/2293#discussion_r144572626 ## File path: engine/schema/src/com/cloud/dc/dao/DataCenterVnetDaoImpl.java ## @@ -100,29

[GitHub] sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements

2017-10-13 Thread git
sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements URL: https://github.com/apache/cloudstack/pull/2293#discussion_r144576505 ## File path: vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java ## @@

[GitHub] sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements

2017-10-13 Thread git
sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements URL: https://github.com/apache/cloudstack/pull/2293#discussion_r144575754 ## File path: vmware-base/src/com/cloud/hypervisor/vmware/mo/HypervisorHostHelper.java ## @@

[GitHub] sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements

2017-10-13 Thread git
sgoeminn commented on a change in pull request #2293: CLOUDSTACK-10047: DVSwitch fixes and improvements URL: https://github.com/apache/cloudstack/pull/2293#discussion_r144570747 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/CreateNetworkCmdByAdmin.java

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

2017-10-13 Thread git
nitin-maharana commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-336463203 @karuturi @sateesh-chodapuneedi @kishankavala, Can you please review this.

[GitHub] blueorangutan commented on issue #2290: WIP: add track id to async api calls

2017-10-13 Thread git
blueorangutan commented on issue #2290: WIP: add track id to async api calls URL: https://github.com/apache/cloudstack/pull/2290#issuecomment-335492525 Packaging result: ?centos6 ?centos7 ?debian. JID-1147 This is an

[GitHub] blueorangutan commented on issue #2290: WIP: add track id to async api calls

2017-10-13 Thread git
blueorangutan commented on issue #2290: WIP: add track id to async api calls URL: https://github.com/apache/cloudstack/pull/2290#issuecomment-335484490 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #2290: WIP: add track id to async api calls

2017-10-13 Thread git
DaanHoogland commented on issue #2290: WIP: add track id to async api calls URL: https://github.com/apache/cloudstack/pull/2290#issuecomment-335484114 @blueorangutan package This is an automated message from the Apache Git

[GitHub] nitin-maharana commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
nitin-maharana commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336432514 LGTM. Thanks @DaanHoogland This is an automated message from the Apache Git

[GitHub] DaanHoogland commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
DaanHoogland commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336426786 @blueorangutan test This is an automated message from the Apache Git Service.

[GitHub] blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336426878 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336425927 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336425777 @blueorangutan test centos7 xenserver-65sp1 This is

[GitHub] DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336424997 @blueorangutan help This is an automated message

[GitHub] blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336425033 @DaanHoogland I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test

[GitHub] blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336420218 Packaging result: ?centos6 ?centos7 ?debian. JID-1160 This is an automated

[GitHub] blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
blueorangutan commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336415802 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
DaanHoogland commented on issue #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297#issuecomment-336415583 @blueorangutan package This is an automated message from the Apache Git

[GitHub] DaanHoogland opened a new pull request #2297: CLOUDSTACK-9957 tyopos in test

2017-10-13 Thread git
DaanHoogland opened a new pull request #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336409559 Packaging result: ?centos6 ?centos7 ?debian. JID-1159

[cloudstack] branch master updated: CLOUDSTACK-9957 Annotations (#2181)

2017-10-13 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new a379230 CLOUDSTACK-9957 Annotations (#2181)

[GitHub] DaanHoogland closed pull request #2181: CLOUDSTACK-9957 Annotations

2017-10-13 Thread git
DaanHoogland closed pull request #2181: CLOUDSTACK-9957 Annotations URL: https://github.com/apache/cloudstack/pull/2181 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
blueorangutan commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336403168 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
DaanHoogland commented on issue #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296#issuecomment-336403101 @blueorangutan package This is an automated message

[GitHub] blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#issuecomment-336395083 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland opened a new pull request #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone

2017-10-13 Thread git
DaanHoogland opened a new pull request #2296: CLOUDSTACK-10007 no isolationmethod check for basic zone URL: https://github.com/apache/cloudstack/pull/2296 in case of DirectPodBasedNetworkGuru This is an automated message

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

2017-10-13 Thread git
blueorangutan commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-336376718 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

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

2017-10-13 Thread git
borisstoyanov commented on issue #2204: [CLOUDSTACK-10025] Adding Support for NoVNC Console for KVM and XENSERVER URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-336376548 @blueorangutan test This is an

[GitHub] blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
blueorangutan commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#issuecomment-336376508 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM

2017-10-13 Thread git
borisstoyanov commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#issuecomment-336376353 looks like the deployment failed waiting for default template to install, I'll restart @blueorangutan