[GitHub] blueorangutan commented on issue #3120: Rework for destroy VM with volumes UI

2019-01-14 Thread GitBox
blueorangutan commented on issue #3120: Rework for destroy VM with volumes UI URL: https://github.com/apache/cloudstack/pull/3120#issuecomment-454295957 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2536 This is an automate

[GitHub] borisstoyanov commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
borisstoyanov commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454294484 @blueorangutan package This is an automa

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454294551 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #3120: Rework for destroy VM with volumes UI

2019-01-14 Thread GitBox
blueorangutan commented on issue #3120: Rework for destroy VM with volumes UI URL: https://github.com/apache/cloudstack/pull/3120#issuecomment-454286764 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #3120: Rework for destroy VM with volumes UI

2019-01-14 Thread GitBox
borisstoyanov commented on issue #3120: Rework for destroy VM with volumes UI URL: https://github.com/apache/cloudstack/pull/3120#issuecomment-454286581 @blueorangutan package This is an automated message from the Apache Git S

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454281578 Trillian test result (tid-3320) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 22896 s

[cloudstack] Diff for: [GitHub] rafaelweingartner merged pull request #3128: Extend PR#2535 to enable remote debugging for CentOS63

2019-01-14 Thread GitBox
diff --git a/packaging/centos63/cloud-management.rc b/packaging/centos63/cloud-management.rc index 0ef5fc4a171..eb075daee59 100755 --- a/packaging/centos63/cloud-management.rc +++ b/packaging/centos63/cloud-management.rc @@ -88,7 +88,7 @@ start() { echo -n "Starting $PROGNAME" "$SHORTNAME"

[cloudstack] branch master updated: Extend PR#2535 to enable remote debugging for CentOS63 as well (#3128)

2019-01-14 Thread rafael
This is an automated email from the ASF dual-hosted git repository. rafael 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 df6288f Extend PR#2535 to enable remote deb

[GitHub] rafaelweingartner merged pull request #3128: Extend PR#2535 to enable remote debugging for CentOS63

2019-01-14 Thread GitBox
rafaelweingartner merged pull request #3128: Extend PR#2535 to enable remote debugging for CentOS63 URL: https://github.com/apache/cloudstack/pull/3128 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, commits@cloudstack.apache.org -

[GitHub] GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454205801 @blueorangutan test This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454205915 @GabrielBrascher a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] mike-tutkowski commented on a change in pull request #2848: Vmware offline migration

2019-01-14 Thread GitBox
mike-tutkowski commented on a change in pull request #2848: Vmware offline migration URL: https://github.com/apache/cloudstack/pull/2848#discussion_r247647511 ## File path: api/src/main/java/org/apache/cloudstack/api/command/admin/vm/MigrateVMCmd.java ## @@ -119,13 +119,7

[GitHub] mike-tutkowski commented on a change in pull request #2848: Vmware offline migration

2019-01-14 Thread GitBox
mike-tutkowski commented on a change in pull request #2848: Vmware offline migration URL: https://github.com/apache/cloudstack/pull/2848#discussion_r238087352 ## File path: api/src/main/java/org/apache/cloudstack/api/command/admin/vm/MigrateVMCmd.java ## @@ -119,13 +119,7

[GitHub] rafaelweingartner commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool

2019-01-14 Thread GitBox
rafaelweingartner commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool URL: https://github.com/apache/cloudstack/pull/3077#discussion_r247643127 ## File path: server/src/main/java/com/cloud/network/Ipv6AddressManagerImpl.java

[GitHub] rafaelweingartner commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool

2019-01-14 Thread GitBox
rafaelweingartner commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool URL: https://github.com/apache/cloudstack/pull/3077#discussion_r247638106 ## File path: server/src/main/java/com/cloud/network/Ipv6AddressManagerImpl.java

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454133110 Trillian test result (tid-3318) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 21311 s

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454102020 Trillian test result (tid-3319) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] wido commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool

2019-01-14 Thread GitBox
wido commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool URL: https://github.com/apache/cloudstack/pull/3077#discussion_r247535371 ## File path: server/src/main/java/com/cloud/network/Ipv6AddressManagerImpl.java ##

[GitHub] wido commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool

2019-01-14 Thread GitBox
wido commented on a change in pull request #3077: ipv6: Calculate IPv6 address instead of fetching one from a pool URL: https://github.com/apache/cloudstack/pull/3077#discussion_r247534718 ## File path: server/src/main/java/com/cloud/network/Ipv6AddressManagerImpl.java ##

[GitHub] DagSonsteboSB commented on issue #3087: XCP-ng hypervisor version issues

2019-01-14 Thread GitBox
DagSonsteboSB commented on issue #3087: XCP-ng hypervisor version issues URL: https://github.com/apache/cloudstack/issues/3087#issuecomment-454021432 Thanks for the feedback @stormi - we'll take this into account going forward. ---

[GitHub] rafaelweingartner closed pull request #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
rafaelweingartner closed pull request #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130 This is an automated message from the Apac

[GitHub] rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130#issuecomment-454021038 I will close this one in favor of #3131 --

[GitHub] rafaelweingartner commented on issue #3131: Working with ACL List Rules in VPC networks

2019-01-14 Thread GitBox
rafaelweingartner commented on issue #3131: Working with ACL List Rules in VPC networks URL: https://github.com/apache/cloudstack/pull/3131#issuecomment-454020942 Something like this #2475? I guess the Javascripts for these different functions all use the same API method. Therefore,

[GitHub] stormi commented on issue #3087: XCP-ng hypervisor version issues

2019-01-14 Thread GitBox
stormi commented on issue #3087: XCP-ng hypervisor version issues URL: https://github.com/apache/cloudstack/issues/3087#issuecomment-454020702 As release manager for XCP-ng I can try to give some insight. Most codebase is shared at the moment and we maintain compatibility with XenServer. Th

[GitHub] raiciks opened a new pull request #3131: Working with ACL List Rules in VPC networks

2019-01-14 Thread GitBox
raiciks opened a new pull request #3131: Working with ACL List Rules in VPC networks URL: https://github.com/apache/cloudstack/pull/3131 ## Description Hello! From our customers we receive questions about some inconvenience working with ACL List of Rules, for example: 1) It is nec

[GitHub] DagSonsteboSB commented on issue #3087: XCP-ng hypervisor version issues

2019-01-14 Thread GitBox
DagSonsteboSB commented on issue #3087: XCP-ng hypervisor version issues URL: https://github.com/apache/cloudstack/issues/3087#issuecomment-454016259 To be discussed: * As per original comment above - design decision needs to be made whether to keep XCP-ng as a version of XenServer or as

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454007691 @dhlaluku a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454007562 @blueorangutan test This is an automated mess

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-454006013 Trillian test result (tid-3317) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454000121 @blueorangutan test This is an automated message from the Apache

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-454000281 @GabrielBrascher a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-453999706 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2535 This is an au

[GitHub] GabrielBrascher commented on a change in pull request #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
GabrielBrascher commented on a change in pull request #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#discussion_r247476431 ## File path: server/src/main/java/com/cloud/network/NetworkServiceImpl.java ## @@ -711,15 +710

[GitHub] rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130#issuecomment-453994659 I guess they all use the same ACL endpoint. Therefore, it is a matter of extending the

[GitHub] blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
blueorangutan commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-453994180 @GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] raiciks commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
raiciks commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130#issuecomment-453994370 Yes! But this feature we need in Guest Networks, ACL and Security Groups. --

[GitHub] GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
GabrielBrascher commented on issue #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#issuecomment-453994057 @blueorangutan package This is an automated message from the Apac

[GitHub] GabrielBrascher commented on a change in pull request #3129: Fix “test_nic_secondaryip_add_remove” test

2019-01-14 Thread GitBox
GabrielBrascher commented on a change in pull request #3129: Fix “test_nic_secondaryip_add_remove” test URL: https://github.com/apache/cloudstack/pull/3129#discussion_r247476431 ## File path: server/src/main/java/com/cloud/network/NetworkServiceImpl.java ## @@ -711,15 +710

[GitHub] rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
rafaelweingartner commented on issue #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130#issuecomment-453992877 Something like this https://github.com/apache/cloudstack/pull/2475? --

[GitHub] raiciks opened a new pull request #3130: Description field option is needed for networking NAT, ACL and Security Group rules.

2019-01-14 Thread GitBox
raiciks opened a new pull request #3130: Description field option is needed for networking NAT, ACL and Security Group rules. URL: https://github.com/apache/cloudstack/pull/3130 ## Description Hello! Many of our customers asking for description field option in networking NAT, ACL rul

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-453941197 @dhlaluku a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --

[GitHub] dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-453941134 @blueorangutan test This is an automated mess

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-453940985 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2534 ---

[GitHub] PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11]

2019-01-14 Thread GitBox
PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11] URL: https://github.com/apache/cloudstack/pull/3127#discussion_r247418803 ## File path: systemvm/debian/opt/cloud/bin/setup/common.sh ## @@ -583,14 +583,14 @@ setup_ntp

[GitHub] PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11]

2019-01-14 Thread GitBox
PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11] URL: https://github.com/apache/cloudstack/pull/3127#discussion_r247417895 ## File path: tools/appliance/systemvmtemplate/scripts/install_systemvm_packages.sh ## @@ -68

[GitHub] PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11]

2019-01-14 Thread GitBox
PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11] URL: https://github.com/apache/cloudstack/pull/3127#discussion_r247417797 ## File path: tools/appliance/systemvmtemplate/scripts/configure_systemvm_services.sh ## @@ -

[GitHub] PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11]

2019-01-14 Thread GitBox
PaulAngus commented on a change in pull request #3127: Improve System VM startup and memory usage [4.11] URL: https://github.com/apache/cloudstack/pull/3127#discussion_r247417249 ## File path: systemvm/debian/etc/vmware-tools/tools.conf ## @@ -0,0 +1,17 @@ +[logging] +# Tu

[GitHub] blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
blueorangutan commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-453936030 @dhlaluku a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
dhlaluku commented on issue #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#issuecomment-453935979 @blueorangutan package This is an automated m

[GitHub] dhlaluku commented on a change in pull request #3125: IP address acquired with associate ip address is marked as source nat

2019-01-14 Thread GitBox
dhlaluku commented on a change in pull request #3125: IP address acquired with associate ip address is marked as source nat URL: https://github.com/apache/cloudstack/pull/3125#discussion_r247415759 ## File path: server/src/main/java/com/cloud/network/IpAddressManagerImpl.java #

[GitHub] marabesi commented on issue #3123: Db exception trying to upload SSL certificate

2019-01-14 Thread GitBox
marabesi commented on issue #3123: Db exception trying to upload SSL certificate URL: https://github.com/apache/cloudstack/issues/3123#issuecomment-453926683 The question now is, should the column be changed in the official cloudstack repo? -

[GitHub] marabesi commented on issue #3123: Db exception trying to upload SSL certificate

2019-01-14 Thread GitBox
marabesi commented on issue #3123: Db exception trying to upload SSL certificate URL: https://github.com/apache/cloudstack/issues/3123#issuecomment-453926589 Hello @rhtyd you are correct, the column fingerprint was varchar(62) and the data being stored was bigger. Executing the same