[GitHub] [cloudstack] shwstppr commented on pull request #4675: Bug fix in displaying public IP address of shared networks

2021-03-11 Thread GitBox
shwstppr commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-796714627 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [cloudstack] blueorangutan commented on pull request #4675: Bug fix in displaying public IP address of shared networks

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-796714927 @shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests [S] -

[GitHub] [cloudstack] edward-stpierre opened a new issue #4797: Snapshot removal from secondary storage with Ceph primary v4.15

2021-03-11 Thread GitBox
edward-stpierre opened a new issue #4797: URL: https://github.com/apache/cloudstack/issues/4797 # ISSUE TYPE * Bug Report # COMPONENT NAME Snapshot cleanup ~~~ ~~~ # CLOUDSTACK VERSION 4.15 ~~~ ~~~ # CONFIGURATION

[GitHub] [cloudstack] GutoVeronezi commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GutoVeronezi commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796719556 @GabrielBrascher I rebased the PR and fixed it. I will ask you to take a look at the commit db1e0f49dc06d16218ce61d9ef6a025075710d8e, which introduced the error yo

[GitHub] [cloudstack] borisstoyanov opened a new issue #4798: UX improvements in the UI

2021-03-11 Thread GitBox
borisstoyanov opened a new issue #4798: URL: https://github.com/apache/cloudstack/issues/4798 # ISSUE TYPE * Improvement Request # COMPONENT NAME ~~~ UI ~~~ # CLOUDSTACK VERSION ~~~ 4.16 ~~~ # OS / ENV

[GitHub] [cloudstack] blueorangutan commented on pull request #4795: api: cpu values with overprovisioning in host metrics response

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4795: URL: https://github.com/apache/cloudstack/pull/4795#issuecomment-796727877 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 91 --

[GitHub] [cloudstack] blueorangutan commented on pull request #4796: db, server: refactor host_view to prevent duplicate entries

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4796: URL: https://github.com/apache/cloudstack/pull/4796#issuecomment-796733287 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 92 --

[GitHub] [cloudstack] weizhouapache commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
weizhouapache commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796771636 > @GabrielBrascher I rebased the PR and fixed it. > > I will ask you to take a look at the commit [db1e0f4](https://github.com/apache/cloudstack/commit/db1e0f49dc

[GitHub] [cloudstack] nvazquez opened a new issue #4799: GSoC 2021 idea: New marvin tests service

2021-03-11 Thread GitBox
nvazquez opened a new issue #4799: URL: https://github.com/apache/cloudstack/issues/4799 # ISSUE TYPE * Improvement Request # COMPONENT NAME ~~~ UI, marvin tests ~~~ # CLOUDSTACK VERSION ~~~ 4.16 ~~~ # CONFIGURATION N/A

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GabrielBrascher commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796780782 @GutoVeronezi thanks for rebasing and fixing the compilation issue. Answering your question: You had correctly addressed most of the callers by adding `Li

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GabrielBrascher commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796784579 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack] blueorangutan commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796785118 @GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

[GitHub] [cloudstack] blueorangutan commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796792136 Packaging result: :heavy_multiplication_x: centos7 :heavy_multiplication_x: centos8 :heavy_multiplication_x: debian. SL-JID 93

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4212: Migrate/Stop VMs with local storage when preparing host for maintenance

2021-03-11 Thread GitBox
GabrielBrascher commented on pull request #4212: URL: https://github.com/apache/cloudstack/pull/4212#issuecomment-796798108 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack] blueorangutan commented on pull request #4212: Migrate/Stop VMs with local storage when preparing host for maintenance

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4212: URL: https://github.com/apache/cloudstack/pull/4212#issuecomment-796798819 @GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

[GitHub] [cloudstack] ustcweizhou opened a new pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
ustcweizhou opened a new pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800 ### Description We do not need to rebase the qcow2 image of DATADISK and ROOT disk which does not have a backing file. ### T

[GitHub] [cloudstack] weizhouapache commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
weizhouapache commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796805586 @GutoVeronezi tested ok. There are ignored NPE when migrate vm with datadisk. It is caused by commit db1e0f4 Fixed by #4800 --

[GitHub] [cloudstack] blueorangutan commented on pull request #4212: Migrate/Stop VMs with local storage when preparing host for maintenance

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4212: URL: https://github.com/apache/cloudstack/pull/4212#issuecomment-796815120 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 94 --

[GitHub] [cloudstack] rhtyd commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
rhtyd commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-796827098 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [cloudstack] blueorangutan commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-796827718 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S] T

[GitHub] [cloudstack] blueorangutan commented on pull request #4727: Marvin3

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-796839874 [S] Trillian test result (tid-85) Environment: kvm-centos8 (x2), Advanced Networking with Mgmt server 8 Total time taken: 19423 seconds Marvin logs: https://git

[GitHub] [cloudstack] blueorangutan commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-796844514 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 95 --

[cloudstack-www] branch DaanHoogland-patch-1 updated (604bd98 -> b53f8ff)

2021-03-11 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch DaanHoogland-patch-1 in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git. from 604bd98 update issues link add b53f8ff checksum procedures No new revisions were added by

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GabrielBrascher commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796875959 @GutoVeronezi Travis now points to `ScaleIOStorageAdaptor.java` (lines 271 & 365). Can you please re-check it? ``` [INFO] --

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4790: Externalize secondary storage capacity threshold

2021-03-11 Thread GitBox
DaanHoogland commented on a change in pull request #4790: URL: https://github.com/apache/cloudstack/pull/4790#discussion_r592529180 ## File path: server/src/main/java/com/cloud/server/StatsCollector.java ## @@ -1599,10 +1621,21 @@ public String getConfigComponentName() {

[GitHub] [cloudstack] DaanHoogland commented on pull request #4727: Marvin3

2021-03-11 Thread GitBox
DaanHoogland commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-796883992 up to 50 failing suites now, :facepalm: not sure how to address this anymore. I think these are all due to the router scripts, but diagnostics is very bad, so I can't

[GitHub] [cloudstack] GutoVeronezi commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GutoVeronezi commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796916228 @GabrielBrascher done. I looked for more places that needed to handle it, but found none. This is

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4567: Qemu 2.10 requires `-U` flag to read volume metadata

2021-03-11 Thread GitBox
GabrielBrascher commented on pull request #4567: URL: https://github.com/apache/cloudstack/pull/4567#issuecomment-796944608 Thanks, @GutoVeronezi! This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [cloudstack] blueorangutan commented on pull request #4793: systemvmtemplate: new template for 4.15.1

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4793: URL: https://github.com/apache/cloudstack/pull/4793#issuecomment-796963600 [S] Trillian test result (tid-80) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34834 seconds Marvin logs: https://git

[GitHub] [cloudstack] blueorangutan commented on pull request #4779: Fix NPE while cloudstack agent failed to connect to mgt server

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4779: URL: https://github.com/apache/cloudstack/pull/4779#issuecomment-796993428 [S] Trillian test result (tid-83) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36092 seconds Marvin logs: https://git

[GitHub] [cloudstack] weizhouapache commented on issue #4757: issues with life migration on centos (was: Trillian test failures)

2021-03-11 Thread GitBox
weizhouapache commented on issue #4757: URL: https://github.com/apache/cloudstack/issues/4757#issuecomment-797026320 > @rhtyd Live migration on centos8 seems to work fine. However, there is an issue noticed during live migration of VMs with data disks (test_02_migrate_VM_with_two_data_disk

[GitHub] [cloudstack] blueorangutan commented on pull request #4577: [HEALTH] 4.16/master Health Check please don't merge this

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4577: URL: https://github.com/apache/cloudstack/pull/4577#issuecomment-797036175 [S] Trillian test result (tid-87) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35335 seconds Marvin logs: https://git

[GitHub] [cloudstack] shwstppr commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
shwstppr commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797054398 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [cloudstack] blueorangutan commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797054860 @shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests [S] -

[GitHub] [cloudstack] blueorangutan commented on pull request #4738: Fix VMware OVF properties copy from template

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4738: URL: https://github.com/apache/cloudstack/pull/4738#issuecomment-797058004 [S] Trillian test result (tid-78) Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 47517 seconds Marvin logs: https://git

[GitHub] [cloudstack] blueorangutan commented on pull request #4577: [HEALTH] 4.16/master Health Check please don't merge this

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4577: URL: https://github.com/apache/cloudstack/pull/4577#issuecomment-797080568 [S] Trillian test result (tid-88) Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40272 seconds Marvin logs: https://git

[GitHub] [cloudstack] blueorangutan commented on pull request #4141: [New feature] Load balancer customization (haproxy-based)

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4141: URL: https://github.com/apache/cloudstack/pull/4141#issuecomment-797104620 [S] Trillian test result (tid-84) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45581 seconds Marvin logs: https://git

[GitHub] [cloudstack] blueorangutan commented on pull request #4675: Bug fix in displaying public IP address of shared networks

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797126584 [S] Trillian test result (tid-90) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 37102 seconds Marvin logs: https://git

[GitHub] [cloudstack] davidjumani commented on pull request #4571: uservmjoindaoimpl: Set free memory to zero if greater than total memory

2021-03-11 Thread GitBox
davidjumani commented on pull request #4571: URL: https://github.com/apache/cloudstack/pull/4571#issuecomment-797233467 @blueorangutan package This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [cloudstack] blueorangutan commented on pull request #4571: uservmjoindaoimpl: Set free memory to zero if greater than total memory

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4571: URL: https://github.com/apache/cloudstack/pull/4571#issuecomment-797234399 @davidjumani a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

[GitHub] [cloudstack] davidjumani commented on issue #4778: Listhostsmetric cpuallocated >100%

2021-03-11 Thread GitBox
davidjumani commented on issue #4778: URL: https://github.com/apache/cloudstack/issues/4778#issuecomment-797234454 +1 for getting listHosts as a subset of listHostsMetrics This is an automated message from the Apache Git Serv

[GitHub] [cloudstack] blueorangutan commented on pull request #4571: uservmjoindaoimpl: Set free memory to zero if greater than total memory

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4571: URL: https://github.com/apache/cloudstack/pull/4571#issuecomment-797241955 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 96 --

[GitHub] [cloudstack] rhtyd commented on pull request #4795: api: cpu values with overprovisioning in host metrics response

2021-03-11 Thread GitBox
rhtyd commented on pull request #4795: URL: https://github.com/apache/cloudstack/pull/4795#issuecomment-797263842 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [cloudstack] blueorangutan commented on pull request #4795: api: cpu values with overprovisioning in host metrics response

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4795: URL: https://github.com/apache/cloudstack/pull/4795#issuecomment-797264084 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests [S]

[GitHub] [cloudstack] rhtyd commented on pull request #4794: server: filter null details during volume to template creation

2021-03-11 Thread GitBox
rhtyd commented on pull request #4794: URL: https://github.com/apache/cloudstack/pull/4794#issuecomment-797264246 @blueorangutan test This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [cloudstack] blueorangutan commented on pull request #4794: server: filter null details during volume to template creation

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4794: URL: https://github.com/apache/cloudstack/pull/4794#issuecomment-797264487 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests [S]

[GitHub] [cloudstack] rhtyd commented on issue #4798: GSoC Idea - UX improvements in the UI

2021-03-11 Thread GitBox
rhtyd commented on issue #4798: URL: https://github.com/apache/cloudstack/issues/4798#issuecomment-797285312 @borisstoyanov can you use the structure/format as in https://github.com/apache/cloudstack/issues/4696 This is an a

[GitHub] [cloudstack] rhtyd commented on issue #4799: GSoC 2021 idea: New marvin tests service

2021-03-11 Thread GitBox
rhtyd commented on issue #4799: URL: https://github.com/apache/cloudstack/issues/4799#issuecomment-797285259 @nvazquez can you use the structure/format as in https://github.com/apache/cloudstack/issues/4696 This is an automa

[GitHub] [cloudstack] blueorangutan commented on pull request #4800: kvm: Do not rebase DATADISK in vm start/migration

2021-03-11 Thread GitBox
blueorangutan commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797292083 [S] Trillian test result (tid-91) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34323 seconds Marvin logs: https://git

[GitHub] [cloudstack] shwstppr commented on a change in pull request #4727: Marvin3

2021-03-11 Thread GitBox
shwstppr commented on a change in pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#discussion_r592961107 ## File path: test/integration/smoke/test_vm_life_cycle.py ## @@ -1443,11 +1433,13 @@ def setUp(self): if len(self.hosts) < 2: s

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4727: Marvin3

2021-03-11 Thread GitBox
DaanHoogland commented on a change in pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#discussion_r592964447 ## File path: test/integration/smoke/test_vm_life_cycle.py ## @@ -1443,11 +1433,13 @@ def setUp(self): if len(self.hosts) < 2:

[GitHub] [cloudstack] shwstppr commented on pull request #4789: api/server: fix hahost value in listHosts

2021-03-11 Thread GitBox
shwstppr commented on pull request #4789: URL: https://github.com/apache/cloudstack/pull/4789#issuecomment-797298881 @weizhouapache yes #4796 will conflict in the current form. I'll rebase my changes there if this is accepted. --

[cloudstack] branch marvin3 updated (004cd05 -> bc09184)

2021-03-11 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch marvin3 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 004cd05 ip to net address conversions add bc09184 clean up VMs No new revisions were added by this update.

[GitHub] [cloudstack] DaanHoogland opened a new pull request #4801: skip lifemigration for centos

2021-03-11 Thread GitBox
DaanHoogland opened a new pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801 ### Description This PR skips live migration on centos as the default installations for qemu/libvirt don't support it. ### Type

[GitHub] [cloudstack] DaanHoogland commented on pull request #4801: skip lifemigration for centos

2021-03-11 Thread GitBox
DaanHoogland commented on pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801#issuecomment-797301870 @weizhouapache @shwstppr @rhtyd please let me know if this needs to go on 4.15. as it is only an intergration test skipping, I didn't deeem it important. --

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4727: Marvin3

2021-03-11 Thread GitBox
DaanHoogland commented on a change in pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#discussion_r592968380 ## File path: test/integration/smoke/test_vm_life_cycle.py ## @@ -1443,11 +1433,13 @@ def setUp(self): if len(self.hosts) < 2:

<    1   2