[GitHub] asfgit closed pull request #2081: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot

2017-10-18 Thread GitBox
asfgit closed pull request #2081: CLOUDSTACK-9894 Separate creation and backup operations for a volume snapshot URL: https://github.com/apache/cloudstack/pull/2081 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] DaanHoogland closed pull request #2297: CLOUDSTACK-9957 tyopos in test

2017-10-19 Thread GitBox
DaanHoogland closed pull request #2297: CLOUDSTACK-9957 tyopos in test URL: https://github.com/apache/cloudstack/pull/2297 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 pull

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

2017-10-19 Thread GitBox
borisstoyanov commented on issue #2281: CLOUDSTACK-10102: New network type (L2) URL: https://github.com/apache/cloudstack/pull/2281#issuecomment-337858174 @DaanHoogland I see this error here when listing hosts: ``` 2017-10-19 09:45:50,498 ERROR [c.c.a.ApiServer]

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

2017-10-19 Thread GitBox
ustcweizhou 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_r145624172 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java

[GitHub] fmaximus commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-10-19 Thread GitBox
fmaximus commented on a change in pull request #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#discussion_r145625247 ## File path: test/integration/component/test_host.py ## @@ -197,8 +194,8

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

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

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

2017-10-19 Thread GitBox
blueorangutan commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337876824 Packaging result: ?centos6 ?centos7 ?debian. JID-1172

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

2017-10-19 Thread GitBox
borisstoyanov commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337877873 @syed could this be a related failure? ``` Results : Failed tests:

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

2017-10-19 Thread GitBox
blueorangutan commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337878062 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

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

2017-10-19 Thread GitBox
blueorangutan commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337882976 Packaging result: ?centos6 ?centos7 ?debian. JID-1173

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

2017-10-19 Thread GitBox
borisstoyanov commented on issue #2281: CLOUDSTACK-10102: New network type (L2) URL: https://github.com/apache/cloudstack/pull/2281#issuecomment-337883771 @nvazquez can you please rebase and push This is an automated message

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

2017-10-19 Thread GitBox
borisstoyanov commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337883292 @blueorangutan test This is an automated

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

2017-10-19 Thread GitBox
blueorangutan commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-337883390 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

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

2017-10-18 Thread GitBox
nvazquez commented on issue #2295: CLOUDSTACK-10109: Enable dedication of public IPs to SSVM and CPVM URL: https://github.com/apache/cloudstack/pull/2295#issuecomment-337573323 @borisstoyanov done. The only main concern for this community PR is fix for `domain_vlan_map`. @ustcweizhou

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

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

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

2017-10-23 Thread GitBox
mike-tutkowski commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-338831054 @borisroman Are we good to go with this PR? Should I merge it? Thanks!

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146415652 ## File path: engine/schema/src/com/cloud/tags/dao/ResourceTagDao.java ## @@ -34,5 +34,22 @@

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146319286 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateVPCCmd.java ##

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146328309 ## File path: core/src/com/cloud/agent/api/ReplugNicAnswer.java ## @@ -0,0 +1,26 @@ +/// +//

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r141434102 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateNetworkCmd.java

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146418796 ## File path:

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146401454 ## File path: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ## @@

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146315585 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateNetworkCmd.java

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146415351 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,28 @@ IpAddress

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r141427671 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,26 @@ IpAddress

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146328468 ## File path: core/src/com/cloud/agent/api/ReplugNicCommand.java ## @@ -0,0 +1,67 @@ +/// +//

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146415530 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,28 @@ IpAddress

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146421388 ## File path: server/src/com/cloud/network/NetworkMigrationManager.java ## @@ -0,0 +1,84 @@

[GitHub] GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-23 Thread GitBox
GabrielBrascher commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146415623 ## File path: engine/schema/src/com/cloud/tags/dao/ResourceTagDao.java ## @@ -34,5 +34,22 @@

[GitHub] nitin-maharana commented on a change in pull request #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain

2017-10-23 Thread GitBox
nitin-maharana commented on a change in pull request #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain URL: https://github.com/apache/cloudstack/pull/2102#discussion_r146226118 ## File path: api/src/com/cloud/network/GuestVlanDomain.java ## @@ -0,0 +1,33

[GitHub] nitin-maharana commented on a change in pull request #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain

2017-10-23 Thread GitBox
nitin-maharana commented on a change in pull request #2102: CLOUDSTACK-9889 Dedication of guest vlan range to a domain URL: https://github.com/apache/cloudstack/pull/2102#discussion_r146232407 ## File path: engine/schema/src/com/cloud/network/dao/DomainGuestVlanMapDao.java

[GitHub] svenvogel commented on a change in pull request #2271: CLOUDSTACK-10087 Template registration errors out when template URL i?

2017-10-23 Thread GitBox
svenvogel commented on a change in pull request #2271: CLOUDSTACK-10087 Template registration errors out when template URL i? URL: https://github.com/apache/cloudstack/pull/2271#discussion_r146237581 ## File path: systemvm/patches/debian/config/etc/init.d/cloud-early-config

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

2017-10-19 Thread GitBox
blueorangutan commented on issue #2294: [CLOUDSTACK-10039] Adding allocated IOPS to storage pool response URL: https://github.com/apache/cloudstack/pull/2294#issuecomment-338021583 Trillian test result (tid-1597) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

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

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

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

2017-10-23 Thread GitBox
nitin-maharana commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-338866778 If there is no more review, shall we consider merging this PR? Ping @karuturi @borisstoyanov

[GitHub] karuturi commented on a change in pull request #2271: CLOUDSTACK-10087 Template registration errors out when template URL i?

2017-10-23 Thread GitBox
karuturi commented on a change in pull request #2271: CLOUDSTACK-10087 Template registration errors out when template URL i? URL: https://github.com/apache/cloudstack/pull/2271#discussion_r146451794 ## File path: systemvm/patches/debian/config/etc/init.d/cloud-early-config

[GitHub] sgoeminn commented on issue #2238: [CLOUDSTACK-10053] Performance improvement: caching of NuageVsp ID

2017-10-24 Thread GitBox
sgoeminn commented on issue #2238: [CLOUDSTACK-10053] Performance improvement: caching of NuageVsp ID URL: https://github.com/apache/cloudstack/pull/2238#issuecomment-338892248 CI failure seems not to be related to this PR (known issue).

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

2017-10-24 Thread GitBox
borisstoyanov commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-338934337 No, I'm not the right person to do that This is an

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

2017-10-24 Thread GitBox
nitin-maharana commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-338936352 @borisstoyanov, Sorry!! @jayapalu, Can you please merge this one.

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492979 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateVPCCmd.java ## @@ -0,0

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492740 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,28 @@ IpAddress

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492848 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateNetworkCmd.java ## @@

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492729 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,28 @@ IpAddress

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492712 ## File path: api/src/com/cloud/network/NetworkService.java ## @@ -82,6 +83,26 @@ IpAddress

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146492964 ## File path: api/src/org/apache/cloudstack/api/command/admin/network/MigrateNetworkCmd.java ## @@

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493133 ## File path: core/src/com/cloud/agent/api/ReplugNicCommand.java ## @@ -0,0 +1,67 @@ +/// +//

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493117 ## File path: core/src/com/cloud/agent/api/ReplugNicAnswer.java ## @@ -0,0 +1,26 @@ +/// +//

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493419 ## File path: engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java ## @@ -3636,6

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493509 ## File path: engine/schema/src/com/cloud/tags/dao/ResourceTagDao.java ## @@ -34,5 +34,22 @@

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493531 ## File path: server/src/com/cloud/network/NetworkMigrationManager.java ## @@ -0,0 +1,84 @@ +//

[GitHub] sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support

2017-10-24 Thread GitBox
sgoeminn commented on a change in pull request #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#discussion_r146493484 ## File path: engine/schema/src/com/cloud/tags/dao/ResourceTagDao.java ## @@ -34,5 +34,22 @@

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

2017-10-24 Thread GitBox
sgoeminn commented on issue #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#issuecomment-338921135 @GabrielBrascher Thanks for reviewing, really appreciate it! We tried to address all your comments.

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

2017-10-24 Thread GitBox
nitin-maharana commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-338925612 @borisstoyanov, Can you please merge this. Thanks. This

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150464505 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -990,4 +1091,53 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150464567 ## File path: agent/conf/agent.properties ## @@ -30,6 +30,19 @@ workers=5 #host=

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150464671 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -699,16 +719,97 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150464767 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -699,16 +719,97 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150465994 ## File path: server/src/org/apache/cloudstack/agent/mslb/AgentMSLBServiceImpl.java

[GitHub] bwsw opened a new pull request #2322: CLOUDSTACK-10140: Fix for when template is created from snapshot template.properties are corrupted

2017-11-12 Thread GitBox
bwsw opened a new pull request #2322: CLOUDSTACK-10140: Fix for when template is created from snapshot template.properties are corrupted URL: https://github.com/apache/cloudstack/pull/2322 This is an automated message from

[GitHub] bwsw commented on issue #2231: [CLOUDSTACK-10039] Adding IOPS/GB offering

2017-11-12 Thread GitBox
bwsw commented on issue #2231: [CLOUDSTACK-10039] Adding IOPS/GB offering URL: https://github.com/apache/cloudstack/pull/2231#issuecomment-343830206 Cool feature. Recently have thought about it. This is an automated message

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150462987 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -699,16 +719,97 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150463117 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -699,16 +719,97 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150463227 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -990,4 +1091,53 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150465571 ## File path: server/src/com/cloud/resource/ResourceManagerImpl.java ## @@ -800,6

[GitHub] mrunalinikankariya commented on issue #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua?

2017-11-13 Thread GitBox
mrunalinikankariya commented on issue #2292: CLOUDSTACK-10108:ConfigKey based approach for reading 'ping' configua? URL: https://github.com/apache/cloudstack/pull/2292#issuecomment-343848027 @rhtyd Could this be merged? This

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150463668 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -990,4 +1091,53 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150465147 ## File path: agent/src/com/cloud/agent/AgentShell.java ## @@ -107,18 +111,48 @@

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150465256 ## File path: agent/src/com/cloud/agent/IAgentShell.java ## @@ -22,33 +22,64 @@

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150465190 ## File path: agent/src/com/cloud/agent/AgentShell.java ## @@ -291,6 +325,11 @@

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150463839 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -990,4 +1091,53 @@ public

[GitHub] rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-12 Thread GitBox
rhtyd commented on a change in pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#discussion_r150464946 ## File path: agent/src/com/cloud/agent/Agent.java ## @@ -990,4 +1091,53 @@ public

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343414715 Packaging result: ?centos6 ?centos7 ?debian. JID-1240

[GitHub] borisstoyanov commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-10 Thread GitBox
borisstoyanov commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-343443823 @blueorangutan test This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-10 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-343443874 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150219895 ## File path: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/XenServerGuru.java

[GitHub] rafaelweingartner commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343462921 @DaanHoogland I did not understand what you mean with more is needed? I did not add code here, I only re-worked the

[GitHub] blueorangutan commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
blueorangutan commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343417258 Packaging result: ?centos6 ?centos7 ?debian. JID-1241 This is

[GitHub] blueorangutan commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present

2017-11-10 Thread GitBox
blueorangutan commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present URL: https://github.com/apache/cloudstack/pull/2316#issuecomment-343424968 Packaging result: ?centos6 ?centos7 ?debian. JID-1242

[GitHub] rhtyd commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
rhtyd commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343407424 @blueorangutan package This is an automated message

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343406682 Packaging result: ?centos6 ?centos7 ?debian. JID-1238

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-10 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-343443118 Packaging result: ?centos6 ?centos7 ?debian. JID-1243 This is an automated message

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150218941 ## File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java ## @@ -792,11 +791,8

[GitHub] blueorangutan commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present

2017-11-10 Thread GitBox
blueorangutan commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present URL: https://github.com/apache/cloudstack/pull/2316#issuecomment-343419216 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present

2017-11-10 Thread GitBox
rhtyd commented on issue #2316: CLOUDSTACK-10137: Only link log4j if not already present URL: https://github.com/apache/cloudstack/pull/2316#issuecomment-343419067 @marcaurele thanks, I went ahead and fixed similar issues for centos6 and centos7 as well. Will fire some tests.

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343419700 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
rhtyd commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343419461 @blueorangutan test This is an automated message

[GitHub] anshul1886 commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
anshul1886 commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343452822 Also see the fix in https://github.com/apache/cloudstack/pull/2317 which I am working on.

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150220323 ## File path: server/src/com/cloud/hypervisor/HypervisorGuruBase.java ## @@ -194,6

[GitHub] anshul1886 opened a new pull request #2317: CLOUDSTACK-9025: Fixed Unable to deploy VM instance from template if ?

2017-11-10 Thread GitBox
anshul1886 opened a new pull request #2317: CLOUDSTACK-9025: Fixed Unable to deploy VM instance from template if ? URL: https://github.com/apache/cloudstack/pull/2317 ?template spin from linked clone snapshot Re-introduce the code to send CopyCommand to hypervisor host instead of

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150217798 ## File path: engine/schema/src/com/cloud/host/dao/HostDao.java ## @@ -72,14 +72,6 @@

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150217867 ## File path: engine/schema/src/com/cloud/host/dao/HostDao.java ## @@ -103,4 +95,13 @@

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

2017-11-10 Thread GitBox
blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-343407670 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
borisstoyanov commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343411629 Thanks for this PR @rafaelweingartner, let me run some testing on it. @blueorangutan package

[GitHub] rhtyd commented on a change in pull request #2316: CLOUDSTACK-10137: Only link log4j if not already present

2017-11-10 Thread GitBox
rhtyd commented on a change in pull request #2316: CLOUDSTACK-10137: Only link log4j if not already present URL: https://github.com/apache/cloudstack/pull/2316#discussion_r150183528 ## File path: debian/cloudstack-management.postinst ## @@ -57,7 +57,9 @@ if [ "$1" =

[GitHub] blueorangutan commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
blueorangutan commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343417716 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
borisstoyanov commented on issue #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#issuecomment-343417614 @blueorangutan test centos7 xenserver-65sp1 This is an

[GitHub] borisstoyanov commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-10 Thread GitBox
borisstoyanov commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-343435386 @blueorangutan package This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-10 Thread GitBox
blueorangutan commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-343435458 @borisstoyanov 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-11-10 Thread GitBox
blueorangutan commented on issue #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#issuecomment-343443425 Packaging result: ?centos6 ?centos7 ?debian. JID-1244 This

[GitHub] rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025.

2017-11-10 Thread GitBox
rafaelweingartner commented on a change in pull request #2315: A comprehensive solution for #CLOUDSTACK-9025. URL: https://github.com/apache/cloudstack/pull/2315#discussion_r150218941 ## File path: engine/schema/src/com/cloud/host/dao/HostDaoImpl.java ## @@ -792,11 +791,8

  1   2   3   4   5   6   7   8   9   10   >