[GitHub] [cloudstack] blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2019-11-30 Thread GitBox
blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559985413 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-414 This is

[GitHub] [cloudstack] blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2019-11-30 Thread GitBox
blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559985414 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-413

[GitHub] [cloudstack] DaanHoogland commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559996124 @blueorangutan test This is an

[GitHub] [cloudstack] DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is updated URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559996050 @blueorangutan test This is an automated message from the

[GitHub] [cloudstack] rhtyd commented on issue #3729: config: add isdynamic flag in configuration response

2019-11-30 Thread GitBox
rhtyd commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559956321 This will largely work for most global settings, however, there are configkeys that are defined to not be dynamic in nature and

[GitHub] [cloudstack] DaanHoogland commented on issue #3694: Ldap fixes

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3694: Ldap fixes URL: https://github.com/apache/cloudstack/pull/3694#issuecomment-559979127 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the

[GitHub] [cloudstack] blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2019-11-30 Thread GitBox
blueorangutan commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559979305 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] [cloudstack] blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2019-11-30 Thread GitBox
blueorangutan commented on issue #3726: vrouter: reload haproxy when cfg file is updated URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559979461 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] [cloudstack] DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is updated

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3726: vrouter: reload haproxy when cfg file is updated URL: https://github.com/apache/cloudstack/pull/3726#issuecomment-559979380 @blueorangutan package This is an automated message from the

[GitHub] [cloudstack] blueorangutan commented on issue #3694: Ldap fixes

2019-11-30 Thread GitBox
blueorangutan commented on issue #3694: Ldap fixes URL: https://github.com/apache/cloudstack/pull/3694#issuecomment-559979155 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] [cloudstack] DaanHoogland commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3727: server: Capacity check should take vms in Migrating state into calculation URL: https://github.com/apache/cloudstack/pull/3727#issuecomment-559979278 @blueorangutan package This is an

[GitHub] [cloudstack] DaanHoogland commented on issue #3729: config: add isdynamic flag in configuration response

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559978904 good catch @rhtyd , however, will this be harmful? Or can we merge and plan the small fixes that will follow as they come

[GitHub] [cloudstack] weizhouapache edited a comment on issue #3729: config: add isdynamic flag in configuration response

2019-11-30 Thread GitBox
weizhouapache edited a comment on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559960939 @rhtyd this change is based on the assumption that the "dynamic" is set correctly for the configurations.

[GitHub] [cloudstack] DaanHoogland commented on issue #3730: [WIP] update VR's scripts from python2 to python3

2019-11-30 Thread GitBox
DaanHoogland commented on issue #3730: [WIP] update VR's scripts from python2 to python3 URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-559980217 some pylint errors in travis job 1, mostly requiring extra blank lines at places @GabrielBrascher .

[GitHub] [cloudstack] weizhouapache commented on issue #3729: config: add isdynamic flag in configuration response

2019-11-30 Thread GitBox
weizhouapache commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559960939 @rhtyd this change is based on the assumption that the "dynamic" is set correctly for the configurations. We need

[GitHub] [cloudstack] blueorangutan commented on issue #3694: Ldap fixes

2019-11-30 Thread GitBox
blueorangutan commented on issue #3694: Ldap fixes URL: https://github.com/apache/cloudstack/pull/3694#issuecomment-560035025 Trillian test result (tid-570) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34022 seconds Marvin logs: