[GitHub] [cloudstack] blueorangutan commented on issue #3649: Garbage snapshots are left on Primary and Secondary

2019-11-29 Thread GitBox
blueorangutan commented on issue #3649: Garbage snapshots are left on Primary and Secondary URL: https://github.com/apache/cloudstack/pull/3649#issuecomment-559909219 Trillian test result (tid-567) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] [cloudstack] blueorangutan commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values

2019-11-29 Thread GitBox
blueorangutan commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values URL: https://github.com/apache/cloudstack/pull/3681#issuecomment-559904497 Trillian test result (tid-566) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Tota

[GitHub] [cloudstack] GabrielBrascher opened a new pull request #3730: [WIP] update VR's scripts from python2 to python3

2019-11-29 Thread GitBox
GabrielBrascher opened a new pull request #3730: [WIP] update VR's scripts from python2 to python3 URL: https://github.com/apache/cloudstack/pull/3730 ## Description - Considering that Python 2.7 will not be maintained in 2020 (will be retired on 1st January 2020) next VR templa

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

2019-11-29 Thread GitBox
andrijapanicsb commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559852498 LGTM Tested manually on 10-15 different settings.

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

2019-11-29 Thread GitBox
blueorangutan commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559845479 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-412 This

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

2019-11-29 Thread GitBox
blueorangutan commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559841164 @andrijapanicsb a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

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

2019-11-29 Thread GitBox
andrijapanicsb commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-55984 @blueorangutan package This is an automated message from

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

2019-11-29 Thread GitBox
blueorangutan commented on issue #3694: Ldap fixes URL: https://github.com/apache/cloudstack/pull/3694#issuecomment-559828934 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-411 This is an automated message from the Apache Gi

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

2019-11-29 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-559824817 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-409 ---

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

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

[GitHub] [cloudstack] blueorangutan commented on issue #3728: server: double check host capacity when start/migrate a vm

2019-11-29 Thread GitBox
blueorangutan commented on issue #3728: server: double check host capacity when start/migrate a vm URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-559824366 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-408 ---

[GitHub] [cloudstack] blueorangutan commented on issue #3725: convert protocol names to be found as labels

2019-11-29 Thread GitBox
blueorangutan commented on issue #3725: convert protocol names to be found as labels URL: https://github.com/apache/cloudstack/pull/3725#issuecomment-559823936 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-407 This is an a

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

2019-11-29 Thread GitBox
blueorangutan commented on issue #3694: Ldap fixes URL: https://github.com/apache/cloudstack/pull/3694#issuecomment-559811595 @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 #3694: Ldap fixes

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

[cloudstack] branch master updated (2427114 -> f30b5ce)

2019-11-29 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 2427114 a conditional to prevent creation of a field (#3723) add f30b5ce Fix resource count of expunged volu

[GitHub] [cloudstack] DaanHoogland merged pull request #3669: server: Fix resource count of primary storage/volume because of Expunged volumes

2019-11-29 Thread GitBox
DaanHoogland merged pull request #3669: server: Fix resource count of primary storage/volume because of Expunged volumes URL: https://github.com/apache/cloudstack/pull/3669 This is an automated message from the Apache Git Se

[GitHub] [cloudstack] blueorangutan commented on issue #3649: Garbage snapshots are left on Primary and Secondary

2019-11-29 Thread GitBox
blueorangutan commented on issue #3649: Garbage snapshots are left on Primary and Secondary URL: https://github.com/apache/cloudstack/pull/3649#issuecomment-559810747 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] [cloudstack] DaanHoogland commented on issue #3649: Garbage snapshots are left on Primary and Secondary

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3649: Garbage snapshots are left on Primary and Secondary URL: https://github.com/apache/cloudstack/pull/3649#issuecomment-559810661 @blueorangutan test This is an automated message from the A

[GitHub] [cloudstack] DaanHoogland merged pull request #3723: a conditional to prevent creation of a field

2019-11-29 Thread GitBox
DaanHoogland merged pull request #3723: a conditional to prevent creation of a field URL: https://github.com/apache/cloudstack/pull/3723 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [cloudstack] DaanHoogland closed issue #3716: creating template from snapshot does not work from snapshot list

2019-11-29 Thread GitBox
DaanHoogland closed issue #3716: creating template from snapshot does not work from snapshot list URL: https://github.com/apache/cloudstack/issues/3716 This is an automated message from the Apache Git Service. To respond to

[cloudstack] branch master updated (b8676ae -> 2427114)

2019-11-29 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from b8676ae Merge release branch 4.13 to master add 2427114 a conditional to prevent creation of a field (#3723)

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

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

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

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

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

2019-11-29 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-559806817 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progre

[GitHub] [cloudstack] weizhouapache commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt

2019-11-29 Thread GitBox
weizhouapache commented on issue #3371: Fix virtual template size for managed storage for KVM / refactor cloud-install-sys-tmplt URL: https://github.com/apache/cloudstack/pull/3371#issuecomment-559806786 @svenvogel I totally agree with what @andrijapanicsb said in https://github.com/apache

[GitHub] [cloudstack] blueorangutan commented on issue #3728: server: double check host capacity when start/migrate a vm

2019-11-29 Thread GitBox
blueorangutan commented on issue #3728: server: double check host capacity when start/migrate a vm URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-559806557 @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 #3728: server: double check host capacity when start/migrate a vm

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3728: server: double check host capacity when start/migrate a vm URL: https://github.com/apache/cloudstack/pull/3728#issuecomment-559806536 @blueorangutan package This is an automated message

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

2019-11-29 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-559806567 @blueorangutan package This is an au

[GitHub] [cloudstack] DaanHoogland commented on issue #3725: convert protocol names to be found as labels

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3725: convert protocol names to be found as labels URL: https://github.com/apache/cloudstack/pull/3725#issuecomment-559806205 @blueorangutan package This is an automated message from the Apach

[GitHub] [cloudstack] blueorangutan commented on issue #3725: convert protocol names to be found as labels

2019-11-29 Thread GitBox
blueorangutan commented on issue #3725: convert protocol names to be found as labels URL: https://github.com/apache/cloudstack/pull/3725#issuecomment-559806295 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] [cloudstack] blueorangutan commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values

2019-11-29 Thread GitBox
blueorangutan commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values URL: https://github.com/apache/cloudstack/pull/3681#issuecomment-559806002 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests ---

[GitHub] [cloudstack] DaanHoogland commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values URL: https://github.com/apache/cloudstack/pull/3681#issuecomment-559805817 @blueorangutan test This is an automated messa

[GitHub] [cloudstack] DaanHoogland commented on issue #3724: Storage-based Snapshots for KVM VMs

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3724: Storage-based Snapshots for KVM VMs URL: https://github.com/apache/cloudstack/pull/3724#issuecomment-559805527 @blueorangutan package This is an automated message from the Apache Git Serv

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

2019-11-29 Thread GitBox
andrijapanicsb commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559800334 thx Wei. This is an automated message from the Apache G

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

2019-11-29 Thread GitBox
DaanHoogland commented on issue #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729#issuecomment-559799015 @blueorangutan package This is an automated message from t

[GitHub] [cloudstack] GabrielBrascher commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values

2019-11-29 Thread GitBox
GabrielBrascher commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values URL: https://github.com/apache/cloudstack/pull/3681#issuecomment-559795153 @andrijapanicsb Qemu supports IOPS bursts since version 2.6: - https://blogs.igalia.com/berto/2016/05/24/i

[GitHub] [cloudstack] weizhouapache commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it

2019-11-29 Thread GitBox
weizhouapache commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it URL: https://github.com/apache/cloudstack/issues/3683#issuecomment-559794134 @andrijapanicsb I have created the pull request #3729 could you please test it when

[GitHub] [cloudstack] ustcweizhou opened a new pull request #3729: config: add isdynamic flag in configuration response

2019-11-29 Thread GitBox
ustcweizhou opened a new pull request #3729: config: add isdynamic flag in configuration response URL: https://github.com/apache/cloudstack/pull/3729 ## Description Some configurations are dynamic so it does not require restart of mgt server when we change it. Add isdynamic i

[GitHub] [cloudstack] andrijapanicsb commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values

2019-11-29 Thread GitBox
andrijapanicsb commented on issue #3681: Validate disk offering IOPS normal and maximum read/write values URL: https://github.com/apache/cloudstack/pull/3681#issuecomment-559787869 @GabrielBrascher - just FMI - when/where was this burst length added? I mean in which Qemu version? That's

[GitHub] [cloudstack] andrijapanicsb commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it

2019-11-29 Thread GitBox
andrijapanicsb commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it URL: https://github.com/apache/cloudstack/issues/3683#issuecomment-559736973 thanks (as usual) @weizhouapache ! --

[GitHub] [cloudstack] weizhouapache commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it

2019-11-29 Thread GitBox
weizhouapache commented on issue #3683: Stop "mgmt server needs restart" message on global setting that doesn't require it URL: https://github.com/apache/cloudstack/issues/3683#issuecomment-559720597 @andrijapanicsb we have a fix for this issue I will port it to 4.13 and create a pull re

[GitHub] [cloudstack] ustcweizhou opened a new pull request #3728: server: double check host capacity when start/migrate a vm

2019-11-29 Thread GitBox
ustcweizhou opened a new pull request #3728: server: double check host capacity when start/migrate a vm URL: https://github.com/apache/cloudstack/pull/3728 ## Description When start a vm or migrate a vm (away from a host in host maintenance), cloudstack will check capacity of al

[GitHub] [cloudstack] ustcweizhou opened a new pull request #3727: server: Capacity check should take vms in Migrating state into calculation

2019-11-29 Thread GitBox
ustcweizhou opened a new pull request #3727: server: Capacity check should take vms in Migrating state into calculation URL: https://github.com/apache/cloudstack/pull/3727 ## Description When we calculate a resource consumption of a host, we need to take the vms in following sta

[GitHub] [cloudstack] ustcweizhou opened a new pull request #3726: vrouter: reload haproxy when cfg file is updated

2019-11-29 Thread GitBox
ustcweizhou opened a new pull request #3726: vrouter: reload haproxy when cfg file is updated URL: https://github.com/apache/cloudstack/pull/3726 since 4.11.3, haproxy is always restarted when add/delete a lb rule. When haproxy is started, the processes are ``` root@r-854-VM:~# ps