[GitHub] cloudmonger commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places

2017-07-27 Thread git
cloudmonger commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places URL: https://github.com/apache/cloudstack/pull/2123#issuecomment-318562436 ### ACS CI BVT Run **Sumarry:** Build Number 1019 Hypervisor xenserver

[GitHub] cloudmonger commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places

2017-07-27 Thread git
cloudmonger commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places URL: https://github.com/apache/cloudstack/pull/2123#issuecomment-317216823 ### ACS CI BVT Run **Sumarry:** Build Number 998 Hypervisor xenserver

[GitHub] niteshsarda commented on a change in pull request #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users

2017-07-27 Thread git
niteshsarda commented on a change in pull request #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users URL: https://github.com/apache/cloudstack/pull/2144#discussion_r130004621 ## File path:

[GitHub] blueorangutan commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
blueorangutan commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318540323 Trillian test result (tid-1264) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 50493

[GitHub] blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318529057 Trillian test result (tid-1266) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] syed commented on issue #2204: Http handler

2017-07-27 Thread git
syed commented on issue #2204: Http handler URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-318524162 Good job on finally opening the PR Sachin. Can you do a few things first: 1. Create a JIRA ticket at issues.apache.org Under the `Cloudstack` project giving a

[GitHub] syed commented on issue #2204: Http handler

2017-07-27 Thread git
syed commented on issue #2204: Http handler URL: https://github.com/apache/cloudstack/pull/2204#issuecomment-318524162 Good job on finally opening the PR Sachin. Can you do a few things first: 1. Create a JIRA ticket at issues.apache.org Under the `Cloudstack` project giving a

[GitHub] blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318523603 Trillian test result (tid-1265) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] syed commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
syed commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318521510 Thanks @wido and @rhtyd This is an

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318515658 Trillian test result (tid-1259) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 51464 seconds Marvin

[GitHub] cloudmonger commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-07-27 Thread git
cloudmonger commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-318512886 ### ACS CI BVT Run **Sumarry:** Build Number 1018 Hypervisor xenserver NetworkType Advanced

[GitHub] cloudmonger commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing

2017-07-27 Thread git
cloudmonger commented on issue #2109: CLOUDSTACK-9922 Unable to use 8081 port for Load balancing URL: https://github.com/apache/cloudstack/pull/2109#issuecomment-307062116 ### ACS CI BVT Run **Sumarry:** Build Number 810 Hypervisor xenserver NetworkType Advanced

[GitHub] blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318504869 Trillian test result (tid-1263) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318500397 Trillian test result (tid-1257) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 46764 seconds

[cloudstack] branch master updated: CLOUDSTACK-10015: Return storage provider with response to list storage pools (#2200)

2017-07-27 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab 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 623ca0d CLOUDSTACK-10015: Return storage

[GitHub] rhtyd closed pull request #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
rhtyd closed pull request #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200 This is an automated message from the Apache Git Service.

[GitHub] blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318486117 Trillian test result (tid-1262) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] GabrielBrascher commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places

2017-07-27 Thread git
GabrielBrascher commented on issue #2123: CLOUDSTACK-9914: update Quota plugin to support currency values up to 5 decimal places URL: https://github.com/apache/cloudstack/pull/2123#issuecomment-318478745 Thanks for the review @rhtyd and @rafaelweingartner !

[GitHub] syed commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

2017-07-27 Thread git
syed commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318433930 Thanks for the quick review @GabrielBrascher

[GitHub] syed opened a new pull request #2206: [CLOUDSTACK-10020] Changes to make marvin work with projects and VPCs

2017-07-27 Thread git
syed opened a new pull request #2206: [CLOUDSTACK-10020] Changes to make marvin work with projects and VPCs URL: https://github.com/apache/cloudstack/pull/2206 Currently there are a lot of problems when running Marvin with projects and VPCs enabled. Most of them are related to the fact

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

2017-07-27 Thread git
cloudmonger commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318427511 ### ACS CI BVT Run **Sumarry:** Build Number 1017 Hypervisor xenserver NetworkType Advanced

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

2017-07-27 Thread git
cloudmonger commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-286481800 ### ACS CI BVT Run **Sumarry:** Build Number 466 Hypervisor xenserver NetworkType Advanced

[GitHub] GabrielBrascher commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API

2017-07-27 Thread git
GabrielBrascher commented on issue #2205: Add NULL checks for various objects in SolidFire integration test API URL: https://github.com/apache/cloudstack/pull/2205#issuecomment-318420194 Code LGTM. Thanks @syed ! This is an

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318416017 Trillian test result (tid-1258) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26350 seconds Marvin

[GitHub] blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318409007 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
borisstoyanov commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318408918 @blueorangutan test This is an automated

[GitHub] syed opened a new pull request #2205: Add NULL checks for various objects in SolidFire integration test API

2017-07-27 Thread git
syed opened a new pull request #2205: Add NULL checks for various objects in SolidFire integration test API URL: https://github.com/apache/cloudstack/pull/2205 Guard from cases where we pass an invalid `accountID` or `storagePoolID` in the API. CC: @mike-tutkowski

[GitHub] blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318371586 Packaging result: ?centos6 ?centos7 ?debian. JID-872

[GitHub] blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318364933 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
borisstoyanov commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318364638 sure no probs @swill @blueorangutan package

[GitHub] swill commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
swill commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318364112 Sorry for the repetition here. We have changes to marvin in our local CI which we have not had a chance to push

[GitHub] GabrielBrascher commented on a change in pull request #1357: [4.9] CLOUDSTACK-8958: release dedicated ip range in domain removal

2017-07-27 Thread git
GabrielBrascher commented on a change in pull request #1357: [4.9] CLOUDSTACK-8958: release dedicated ip range in domain removal URL: https://github.com/apache/cloudstack/pull/1357#discussion_r129825976 ## File path: engine/schema/src/com/cloud/domain/DomainVO.java ## @@

[GitHub] blueorangutan commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
blueorangutan commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318347837 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318347831 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318347827 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
rhtyd commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318347712 @blueorangutan test This is an

[GitHub] rhtyd commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
rhtyd commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318347790 @blueorangutan test This is an automated message from the

[GitHub] rhtyd commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
rhtyd commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318347645 @blueorangutan test centos7 xenserver-65sp1 This is an automated message from

[GitHub] blueorangutan commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
blueorangutan commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318346748 Packaging result: ?centos6 ?centos7 ?debian. JID-871 This is an

[GitHub] blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318346509 Packaging result: ?centos6 ?centos7 ?debian. JID-870

[GitHub] blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318345664 Packaging result: ?centos6 ?centos7 ?debian. JID-869

[GitHub] GabrielBrascher commented on issue #1558: Remove unused code from "com.cloud.vm.UserVmManagerImpl"

2017-07-27 Thread git
GabrielBrascher commented on issue #1558: Remove unused code from "com.cloud.vm.UserVmManagerImpl" URL: https://github.com/apache/cloudstack/pull/1558#issuecomment-318343893 Thanks for reminding @rhtyd! I will do it. This

[GitHub] GabrielBrascher commented on issue #1443: CLOUDSTACK-9314 - Remove unused code from XenServerStorageProcessor and change methods access level

2017-07-27 Thread git
GabrielBrascher commented on issue #1443: CLOUDSTACK-9314 - Remove unused code from XenServerStorageProcessor and change methods access level URL: https://github.com/apache/cloudstack/pull/1443#issuecomment-318343980 Thanks @rhtyd, I will do it.

[GitHub] blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
blueorangutan commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318340822 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] blueorangutan commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
blueorangutan commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318340826 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain

2017-07-27 Thread git
rhtyd commented on issue #2124: CLOUDSTACK-9432: cluster/host dedicated to a domain is owned by the root domain URL: https://github.com/apache/cloudstack/pull/2124#issuecomment-318340728 @blueorangutan package This is an

[GitHub] rhtyd commented on issue #2152: improved logging, removed usued code

2017-07-27 Thread git
rhtyd commented on issue #2152: improved logging, removed usued code URL: https://github.com/apache/cloudstack/pull/2152#issuecomment-318340647 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] rhtyd commented on issue #2112: Add Datera CloudStack Storage Plugin 4.10.0.0

2017-07-27 Thread git
rhtyd commented on issue #2112: Add Datera CloudStack Storage Plugin 4.10.0.0 URL: https://github.com/apache/cloudstack/pull/2112#issuecomment-318340421 @jcshen007 can you rebase against latest master and fix the version to 4.11.0.0-SNAPSHOT etc?

[GitHub] rhtyd commented on issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm data

2017-07-27 Thread git
rhtyd commented on issue #1919: CLOUDSTACK-9763: set ssh public key when creating vm data URL: https://github.com/apache/cloudstack/pull/1919#issuecomment-318339885 @serbaut can you rebase against latest master This is an

[GitHub] blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
blueorangutan commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318339657 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service

2017-07-27 Thread git
rhtyd commented on issue #1784: CS-505: Marvin test to check VR internal DNS Service URL: https://github.com/apache/cloudstack/pull/1784#issuecomment-318339512 @blueorangutan package This is an automated message from the

[GitHub] rhtyd commented on issue #1558: Remove unused code from "com.cloud.vm.UserVmManagerImpl"

2017-07-27 Thread git
rhtyd commented on issue #1558: Remove unused code from "com.cloud.vm.UserVmManagerImpl" URL: https://github.com/apache/cloudstack/pull/1558#issuecomment-318339330 @GabrielBrascher can you rebase against latest master This

[GitHub] rhtyd commented on issue #1443: CLOUDSTACK-9314 - Remove unused code from XenServerStorageProcessor and change methods access level

2017-07-27 Thread git
rhtyd commented on issue #1443: CLOUDSTACK-9314 - Remove unused code from XenServerStorageProcessor and change methods access level URL: https://github.com/apache/cloudstack/pull/1443#issuecomment-318339268 @GabrielBrascher can you rebase against latest master

[GitHub] rhtyd commented on issue #1448: CLOUDSTACK-9315: Removed unused Classes

2017-07-27 Thread git
rhtyd commented on issue #1448: CLOUDSTACK-9315: Removed unused Classes URL: https://github.com/apache/cloudstack/pull/1448#issuecomment-318339304 @GabrielBrascher can you rebase against latest master This is an automated

[GitHub] rhtyd closed pull request #1655: Fix ajaxviewer.js to solve console on Firefox

2017-07-27 Thread git
rhtyd closed pull request #1655: Fix ajaxviewer.js to solve console on Firefox URL: https://github.com/apache/cloudstack/pull/1655 This is an automated message from the Apache Git Service. To respond to the message, please

[cloudstack] branch master updated: Fix ajaxviewer.js to solve console on Firefox (#1655)

2017-07-27 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab 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 6d5c61b Fix ajaxviewer.js to solve

[GitHub] rhtyd commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring

2017-07-27 Thread git
rhtyd commented on issue #1593: CLOUDSTACK-9417: Usage module refactoring URL: https://github.com/apache/cloudstack/pull/1593#issuecomment-318338865 @nvazquez rebase against latest master please This is an automated message

[GitHub] rhtyd commented on issue #1407: 4.8.0 fix i18n es

2017-07-27 Thread git
rhtyd commented on issue #1407: 4.8.0 fix i18n es URL: https://github.com/apache/cloudstack/pull/1407#issuecomment-318338696 @ntavares rebase against latest master, change base branch please. This may not be applicable anymore.

[GitHub] rhtyd commented on issue #1355: Refactor test_vpc_vpn, add test for multiple VPNs

2017-07-27 Thread git
rhtyd commented on issue #1355: Refactor test_vpc_vpn, add test for multiple VPNs URL: https://github.com/apache/cloudstack/pull/1355#issuecomment-318338649 @bheuvel rebase against latest master, change base branch please. This may not be applicable anymore.

[GitHub] rhtyd closed pull request #1104: bugfix ui/scripts/accounts.js L1339

2017-07-27 Thread git
rhtyd closed pull request #1104: bugfix ui/scripts/accounts.js L1339 URL: https://github.com/apache/cloudstack/pull/1104 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] rhtyd commented on issue #1104: bugfix ui/scripts/accounts.js L1339

2017-07-27 Thread git
rhtyd commented on issue #1104: bugfix ui/scripts/accounts.js L1339 URL: https://github.com/apache/cloudstack/pull/1104#issuecomment-318338561 Compared diff, issue merged fixed by some other PR/person. Closing this. This is

[GitHub] rhtyd closed pull request #997: Add all tests in /test/integration/smoke to Travis run and nove nicir?

2017-07-27 Thread git
rhtyd closed pull request #997: Add all tests in /test/integration/smoke to Travis run and nove nicir? URL: https://github.com/apache/cloudstack/pull/997 This is an automated message from the Apache Git Service. To respond

[GitHub] cloudmonger commented on issue #1437: removed unused HypervDummyResourceBase class

2017-07-27 Thread git
cloudmonger commented on issue #1437: removed unused HypervDummyResourceBase class URL: https://github.com/apache/cloudstack/pull/1437#issuecomment-318337669 ### ACS CI BVT Run **Sumarry:** Build Number 1016 Hypervisor xenserver NetworkType Advanced Passed=104

[GitHub] kishankavala commented on a change in pull request #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users

2017-07-27 Thread git
kishankavala commented on a change in pull request #2144: CLOUDSTACK-9955 : Featured Templates/Iso's created by Root/admin user are not visible to Domain Admin users URL: https://github.com/apache/cloudstack/pull/2144#discussion_r129809062 ## File path:

[GitHub] blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318318296 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-07-27 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318318289 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
rhtyd commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318318097 @blueorangutan test This is an

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

2017-07-27 Thread git
rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318318024 @marcaurele I checked there exists a 4.10.0.0 -> 4.11.0.0 upgrade path on master branch that you can use.

[GitHub] rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-07-27 Thread git
rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318318126 @blueorangutan test This is an automated

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

2017-07-27 Thread git
rhtyd commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318317849 @marcaurele we already have 4.10.0.0 release and branch cut, we won't be able to accept PRs with schema changes

[GitHub] blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318313865 Packaging result: ?centos6 ?centos7 ?debian. JID-868

[cloudstack] 01/01: Merge pull request #1959 from Accelerite/CLOUDSTACK-9786

2017-07-27 Thread kishan
This is an automated email from the ASF dual-hosted git repository. kishan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 91f7012b015249d9d74dace8874ee8f38b54bfc2 Merge: e1c4b1b 030d34d Author: kishankavala

[cloudstack] branch master updated (e1c4b1b -> 91f7012)

2017-07-27 Thread kishan
This is an automated email from the ASF dual-hosted git repository. kishan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from e1c4b1b Enhance SSL protocol used by Console Proxy add 030d34d CLOUDSTACK-9786:API reference guide entry

[GitHub] kishankavala closed pull request #1959: CLOUDSTACK-9786:API reference guide entry for associateIpAddress needs additional information

2017-07-27 Thread git
kishankavala closed pull request #1959: CLOUDSTACK-9786:API reference guide entry for associateIpAddress needs additional information URL: https://github.com/apache/cloudstack/pull/1959 This is an automated message from

[GitHub] blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-07-27 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318312865 Packaging result: ?centos6 ?centos7 ?debian. JID-867

[GitHub] blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
blueorangutan commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318307533 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] rhtyd commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools

2017-07-27 Thread git
rhtyd commented on issue #2200: [CLOUDSTACK-10015] Return storage provider with response to list storage pools URL: https://github.com/apache/cloudstack/pull/2200#issuecomment-318307478 LGTM. @blueorangutan package This

[GitHub] rhtyd closed pull request #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect

2017-07-27 Thread git
rhtyd closed pull request #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect URL: https://github.com/apache/cloudstack/pull/1640 This is an automated message from the Apache Git Service. To respond

[GitHub] rhtyd commented on issue #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect

2017-07-27 Thread git
rhtyd commented on issue #1640: CLOUDSTACK-9458: Fix HA bug when VMs are stopped on agent disconnect URL: https://github.com/apache/cloudstack/pull/1640#issuecomment-318307352 Closing on the comment that the original bug may not be applicable for current codebase, please re-open to resend

[GitHub] blueorangutan commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

2017-07-27 Thread git
blueorangutan commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account URL: https://github.com/apache/cloudstack/pull/2171#issuecomment-318307298 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been

[GitHub] blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections

2017-07-27 Thread git
blueorangutan commented on issue #2190: CLOUDSTACK-10010: Fixed the negotiation of S2S VPN connections URL: https://github.com/apache/cloudstack/pull/2190#issuecomment-318307019 Trillian test result (tid-1256) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rhtyd closed pull request #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent?

2017-07-27 Thread git
rhtyd closed pull request #1933: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent? URL: https://github.com/apache/cloudstack/pull/1933 This is an automated message from the Apache Git Service. To

[GitHub] rhtyd commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account

2017-07-27 Thread git
rhtyd commented on issue #2171: CLOUDSTACK-9990 : Account name is giving null in event tab after successful creation of account URL: https://github.com/apache/cloudstack/pull/2171#issuecomment-318307086 @blueorangutan test

[cloudstack] branch 4.9 updated: CLOUDSTACK-9569: add router.aggregation.command.each.timeout to agent.properties (#1933)

2017-07-27 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch 4.9 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.9 by this push: new 7ed3179 CLOUDSTACK-9569: add

[GitHub] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

2017-07-27 Thread git
rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318306243 @wido Travis has some failures, can you check?

[GitHub] rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets

2017-07-27 Thread git
rhtyd commented on issue #2028: CLOUDSTACK-9853: Add support for Secondary IPv6 Addresses and Subnets URL: https://github.com/apache/cloudstack/pull/2028#issuecomment-318306164 @blueorangutan package This is an automated

[GitHub] rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-07-27 Thread git
rhtyd commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318306023 @blueorangutan package This is an

[GitHub] blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls

2017-07-27 Thread git
blueorangutan commented on issue #2046: CLOUDSTACK-7958: Add configuration for limit to CIDRs for Admin API calls URL: https://github.com/apache/cloudstack/pull/2046#issuecomment-318306094 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

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

2017-07-27 Thread git
marcaurele commented on issue #1760: CLOUDSTACK-9593: userdata: enforce data is a multiple of 4 characters URL: https://github.com/apache/cloudstack/pull/1760#issuecomment-318305754 @rhtyd I still don't get it why on new release there isn't the creation of the SQL migration files done by

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318304874 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
rhtyd commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318304705 @blueorangutan test matrix This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318297864 Packaging result: ?centos6 ?centos7 ?debian. JID-866 This is an automated message

[GitHub] rhtyd commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
rhtyd commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318292983 @blueorangutan package This is an automated message from the Apache Git Service. To respond

[GitHub] blueorangutan commented on issue #2115: [4.9] Smoketests Health

2017-07-27 Thread git
blueorangutan commented on issue #2115: [4.9] Smoketests Health URL: https://github.com/apache/cloudstack/pull/2115#issuecomment-318293088 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on a change in pull request #1357: [4.9] CLOUDSTACK-8958: release dedicated ip range in domain removal

2017-07-27 Thread git
rhtyd commented on a change in pull request #1357: [4.9] CLOUDSTACK-8958: release dedicated ip range in domain removal URL: https://github.com/apache/cloudstack/pull/1357#discussion_r129777825 ## File path: server/src/com/cloud/user/DomainManagerImpl.java ## @@ -326,6

[GitHub] rhtyd commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients

2017-07-27 Thread git
rhtyd commented on issue #2201: CLOUDSTACK-10016: VPC VR doesn't respond to DNS requests from remote access vpn clients URL: https://github.com/apache/cloudstack/pull/2201#issuecomment-318290578 @Slair1 can you use the JIRA id in the git commit message, @swill can you review as well?

[GitHub] rhtyd closed pull request #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

2017-07-27 Thread git
rhtyd closed pull request #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy URL: https://github.com/apache/cloudstack/pull/2177 This is an automated message from the Apache Git Service. To respond to the

[GitHub] rhtyd commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy

2017-07-27 Thread git
rhtyd commented on issue #2177: CLOUDSTACK-9977: Enhance SSL protocol used by Console Proxy URL: https://github.com/apache/cloudstack/pull/2177#issuecomment-318289872 Test LGTM, the failed tests are intermittent (for r/vpc) and some env induced errors for vmware/xenserver.

[cloudstack] branch master updated: Enhance SSL protocol used by Console Proxy

2017-07-27 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab 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 e1c4b1b Enhance SSL protocol used by