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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-797872671 [S] Trillian test result (tid-119) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 39187 seconds Marvin logs: https://gi

[GitHub] [cloudstack] aleskxyz commented on issue #4810: Cant run instance

2021-03-12 Thread GitBox
aleskxyz commented on issue #4810: URL: https://github.com/apache/cloudstack/issues/4810#issuecomment-797868141 Here is not the right place for asking help or questions. You can use mailing lists for this purpose. Please subscribe to users mailing list and ask you questions there: htt

[GitHub] [cloudstack] abdelouahabb edited a comment on issue #4734: Cant upload ISO Using TrueNas with NFS

2021-03-12 Thread GitBox
abdelouahabb edited a comment on issue #4734: URL: https://github.com/apache/cloudstack/issues/4734#issuecomment-797843380 @rhtyd Both system VM are on, but cant accept uploading the iso to create a new machine ``` 2021-03-13 03:36:46,546 DEBUG [c.c.n.r.VirtualNetworkApplianceMana

[GitHub] [cloudstack] abdelouahabb opened a new issue #4810: Cant run instance

2021-03-12 Thread GitBox
abdelouahabb opened a new issue #4810: URL: https://github.com/apache/cloudstack/issues/4810 ``` - Cloudstack 4.15 - Ubuntu 18.04.5 - Virt Manager 3.2 (nested on Ubuntu 20.04) ``` Everything was installed, secondary storage works (same machine), uploaded the template as i

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-797845090 [S] Trillian test result (tid-106) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40451 seconds Marvin logs: https://gi

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

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

[GitHub] [cloudstack] abdelouahabb commented on issue #4734: Cant upload ISO Using TrueNas with NFS

2021-03-12 Thread GitBox
abdelouahabb commented on issue #4734: URL: https://github.com/apache/cloudstack/issues/4734#issuecomment-797843380 @rhtyd Both system VM are on, but cant accept uploading the iso to create a new machine ``` "connections": [] }","wait":"0"}}] } 2021-03-11 04:29:23,388 DEBU

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-797840913 [S] Trillian test result (tid-107) Environment: xenserver-72 (x2), Advanced Networking with Mgmt server 7 Total time taken: 38032 seconds Marvin logs: https://g

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4577: URL: https://github.com/apache/cloudstack/pull/4577#issuecomment-797834252 [S] Trillian test result (tid-108) Environment: kvm-ubuntu20 (x2), Advanced Networking with Mgmt server u20 Total time taken: 35428 seconds Marvin logs: https:/

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

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

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4793: URL: https://github.com/apache/cloudstack/pull/4793#issuecomment-797735541 [S] Trillian test result (tid-96) Environment: xcpng81 (x2), Advanced Networking with Mgmt server 7 Total time taken: 45763 seconds Marvin logs: https://github.

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4796: URL: https://github.com/apache/cloudstack/pull/4796#issuecomment-797723409 [S] Trillian test result (tid-101) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 38889 seconds Marvin logs: https://gi

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4212: URL: https://github.com/apache/cloudstack/pull/4212#issuecomment-797693398 [S] Trillian test result (tid-102) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35571 seconds Marvin logs: https://gi

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

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

[GitHub] [cloudstack] aleskxyz opened a new issue #4809: Missing password.txt in ConfigDrive in multihomed VM

2021-03-12 Thread GitBox
aleskxyz opened a new issue #4809: URL: https://github.com/apache/cloudstack/issues/4809 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ ConfigDrive ~~~ # CLOUDSTACK VERSION ~~~ 4.14.1 ~~~ # CONFIGURATION

[GitHub] [cloudstack] blueorangutan commented on pull request #4773: Fix deploy VM from ISOs with UEFI

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4773: URL: https://github.com/apache/cloudstack/pull/4773#issuecomment-797664139 [S] Trillian test result (tid-100) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33524 seconds Marvin logs: https://gi

[GitHub] [cloudstack] blueorangutan commented on pull request #4807: Migrate VR codebase from Python2 to Python3

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4807: URL: https://github.com/apache/cloudstack/pull/4807#issuecomment-797648739 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 100 -

[GitHub] [cloudstack] aleskxyz opened a new issue #4808: CloudStack uses 1st cdrom of VM to mount ConfigDrive ISO

2021-03-12 Thread GitBox
aleskxyz opened a new issue #4808: URL: https://github.com/apache/cloudstack/issues/4808 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ ConfigDrive ~~~ # CLOUDSTACK VERSION ~~~ 4.14.1 ~~~ # CONFIGURATION

[GitHub] [cloudstack] GabrielBrascher opened a new pull request #4807: Migrate VR codebase from Python2 to Python3

2021-03-12 Thread GitBox
GabrielBrascher opened a new pull request #4807: URL: https://github.com/apache/cloudstack/pull/4807 ### Description Considering the Python2 end of life we must migrate the VR's codebase to Python3. This PR covers partially issue #3195. Additionally, this PR is a joint of the

[cloudstack] 09/12: read files in vr as list(byte[])

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit e7ba52ab4b5d1641de53cd5e5faa0fa2564cc032 Author: Daan Hoogland AuthorDate: Tue Mar 9 17:51:01 2021 +0100 r

[cloudstack] 02/12: Fix pycodestyle 'errors'

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 1a9402d568a80afbd96cff53344e96f634d824a3 Author: Gabriel Brascher AuthorDate: Mon Aug 10 11:30:49 2020 -0300

[cloudstack] 11/12: byte as separator on byte[]

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit a9043dccaa0cf98d28a882f36a8bceb7a211f548 Author: Daan Hoogland AuthorDate: Wed Mar 10 16:12:26 2021 +0100

[cloudstack] 12/12: ip to net address conversions

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 12b2a2f4f35afdf60def6b4418975d96685ab464 Author: Daan Hoogland AuthorDate: Wed Mar 10 17:01:35 2021 +0100

[cloudstack] 06/12: replace mock with unittest.mock

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit de248a629bcea8ff6bab4b5fb8186c2cc51d2244 Author: Daan Hoogland AuthorDate: Thu Feb 25 10:20:54 2021 +0100

[cloudstack] 01/12: Make Virtual Router '.py' scripts Python3 compatible

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit ab0118064494a57d58845a3289ec326778372c53 Author: Gabriel Brascher AuthorDate: Fri Jul 24 09:02:58 2020 -0300

[cloudstack] branch python3-vr created (now 12b2a2f)

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a change to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git. at 12b2a2f ip to net address conversions This branch includes the following new commits: new ab01180 M

[cloudstack] 10/12: write as byte[]

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit f151e7b543b6cb959f34c7ff75a330d80deeb524 Author: Daan Hoogland AuthorDate: Tue Mar 9 17:54:05 2021 +0100 w

[cloudstack] 04/12: Change Pylint to Pylint3

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 3a7a4f672d69dda6d8646597817a29974a197e7c Author: Gabriel Brascher AuthorDate: Sat Sep 26 01:23:50 2020 -0300

[cloudstack] 07/12: pylint issues

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit e3c8b34954fa52ef878e3001aa0a6eb9cec3737a Author: Daan Hoogland AuthorDate: Thu Feb 25 15:04:15 2021 +0100

[cloudstack] 05/12: Fix indentation issues and use python3 when running pycodestyle checker

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 4604d472ceaa4e10961f76bd3b917fb73a9223c0 Author: Gabriel Brascher AuthorDate: Thu Feb 25 03:49:27 2021 -0300

[cloudstack] 03/12: Fix conflict with master

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 6b2330bbda0c61478c66b3c08c07d62f9ffdd831 Author: Gabriel Brascher AuthorDate: Thu Sep 24 06:09:53 2020 -0300

[cloudstack] 08/12: str f'ormat'

2021-03-12 Thread gabriel
This is an automated email from the ASF dual-hosted git repository. gabriel pushed a commit to branch python3-vr in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 20288b2e3a77ed0346bfb35066dac3090f31969b Author: Daan Hoogland AuthorDate: Thu Feb 25 15:50:17 2021 +0100

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

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

[GitHub] [cloudstack] ThyLAW closed issue #4700: Uefi.properties doesn't Exist / connection w/ libvirtd broken

2021-03-12 Thread GitBox
ThyLAW closed issue #4700: URL: https://github.com/apache/cloudstack/issues/4700 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [cloudstack] ThyLAW commented on issue #4700: Uefi.properties doesn't Exist / connection w/ libvirtd broken

2021-03-12 Thread GitBox
ThyLAW commented on issue #4700: URL: https://github.com/apache/cloudstack/issues/4700#issuecomment-797616585 (First off sorry for posting on these forums w/ this issue, I didn't know the mail client existed) Update in case someone eventually sees this: I got it working, however I

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

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

[GitHub] [cloudstack] ustcweizhou opened a new pull request #4806: vpc: dnsmasq is not started if use.external.dns is true

2021-03-12 Thread GitBox
ustcweizhou opened a new pull request #4806: URL: https://github.com/apache/cloudstack/pull/4806 ### Description This PR fixes the issue that dnsmasq cannot be started in VPC VR is use.external.dns is true Steps to reproduce the issue (1) change global setting use.external.

[GitHub] [cloudstack] ustcweizhou opened a new pull request #4805: server: create DB entry for storage pool capacity when create storage pool

2021-03-12 Thread GitBox
ustcweizhou opened a new pull request #4805: URL: https://github.com/apache/cloudstack/pull/4805 ### Description This PR fixes the issue that new storage pool can not be used in the first few minute after adding to cloudstack. steps to reproduce the issue (1) change global sett

[cloudstack] branch marvin3 updated (adb8b6b -> d99b685)

2021-03-12 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 adb8b6b clean up VMs add d99b685 disable pylint3 running on py2 code (needs addressing and reverting) No

[GitHub] [cloudstack] ustcweizhou opened a new pull request #4804: server: allow copy cross-zone templates to other zone

2021-03-12 Thread GitBox
ustcweizhou opened a new pull request #4804: URL: https://github.com/apache/cloudstack/pull/4804 ### Description This PR fixes the issue that cross-zone templates cannot be copied to new zone. steps to reproduce the issue (1) register a cross-zone template (2) when it is

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
weizhouapache commented on a change in pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#discussion_r593171611 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java ## @@ -4266,7 +4266,8 @@ pub

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4727: URL: https://github.com/apache/cloudstack/pull/4727#issuecomment-797490189 Packaging result: :heavy_multiplication_x: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 99

[cloudstack] branch marvin3 updated (25fb029 -> adb8b6b)

2021-03-12 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. omit 25fb029 remove superfluent arg omit bc09184 clean up VMs omit 004cd05 ip to net address conversions

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
weizhouapache commented on a change in pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#discussion_r593155348 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java ## @@ -4266,7 +4266,8 @@ pub

[GitHub] [cloudstack] weizhouapache opened a new issue #4803: GSoC 2021 Idea: Add SPICE console for vms on KVM/XenServer

2021-03-12 Thread GitBox
weizhouapache opened a new issue #4803: URL: https://github.com/apache/cloudstack/issues/4803 ## Background CloudStack provides ajax vnc view and novnc console to access vm. It would be good to have spice console for vms on kvm/xenserver. ## Proposed Tasks - Get started

[GitHub] [cloudstack] rhtyd commented on a change in pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
rhtyd commented on a change in pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#discussion_r593146838 ## File path: plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java ## @@ -4266,7 +4266,8 @@ public void

[GitHub] [cloudstack] rhtyd commented on pull request #4768: display nics deviceid and order nics by deviceid on Nics tab of insta…

2021-03-12 Thread GitBox
rhtyd commented on pull request #4768: URL: https://github.com/apache/cloudstack/pull/4768#issuecomment-797464622 +1 I didn't look at code changes, if it involved db changes. On Fri, 12 Mar, 2021, 4:27 pm Wei Zhou, ***@***.***> wrote: > @rhtyd (cc @so

[cloudstack] branch python3-benchmark created (now 25fb029)

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch python3-benchmark in repository https://gitbox.apache.org/repos/asf/cloudstack.git. at 25fb029 remove superfluent arg No new revisions were added by this update.

[GitHub] [cloudstack] weizhouapache commented on pull request #4768: display nics deviceid and order nics by deviceid on Nics tab of insta…

2021-03-12 Thread GitBox
weizhouapache commented on pull request #4768: URL: https://github.com/apache/cloudstack/pull/4768#issuecomment-797413805 > @rhtyd (cc @soreana ) though it is a minor change it does include AI and DB changes, so I'd say leave it for 4.16 (that is not a definite 👎 at all, just a expression

[GitHub] [cloudstack] slavkap commented on pull request #4773: Fix deploy VM from ISOs with UEFI

2021-03-12 Thread GitBox
slavkap commented on pull request #4773: URL: https://github.com/apache/cloudstack/pull/4773#issuecomment-797413493 @DaanHoogland, I could do the python tests, but the only problem (for me) is the automating of the installation. I should research how and is it possible to automate the inst

[GitHub] [cloudstack] sebbASF commented on issue #4752: Bug: incorrect link to issue tracker

2021-03-12 Thread GitBox
sebbASF commented on issue #4752: URL: https://github.com/apache/cloudstack/issues/4752#issuecomment-797408876 AFAICT the web page has not been updated This is an automated message from the Apache Git Service. To respond to t

[GitHub] [cloudstack] DaanHoogland commented on pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797406054 > > @weizhouapache I'm confused by your description in context of the code change. If we do have a disk, but no backing format, we go ahead and rebase it as qcow2, am I

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

2021-03-12 Thread GitBox
DaanHoogland commented on issue #4799: URL: https://github.com/apache/cloudstack/issues/4799#issuecomment-797404730 Nota Bene: a lot of tests leave garbage around, I've done a lot of cleanup during python3 migration of marvin tests, but it is far from clean now. --

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

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

[GitHub] [cloudstack] weizhouapache commented on issue #4696: GSoC 2021 Idea: Add virt-v2v support in CloudStack for cold migration of VM from VMware/XenServer to KVM

2021-03-12 Thread GitBox
weizhouapache commented on issue #4696: URL: https://github.com/apache/cloudstack/issues/4696#issuecomment-797401605 @rhtyd fyi, we used libguestfs/guestfish and rsync to convert openvz containers to kvm vm in the past. https://www.youtube.com/watch?v=mmWs9H7dnNA --

[GitHub] [cloudstack] DaanHoogland commented on pull request #4773: Fix deploy VM from ISOs with UEFI

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4773: URL: https://github.com/apache/cloudstack/pull/4773#issuecomment-797397490 > Hi @DaanHoogland, what should be the expectations of Marvin's tests? I mean, do they have to do an automated install (probably with a kickstart for CentOS) and check i

[GitHub] [cloudstack] DaanHoogland commented on pull request #4768: display nics deviceid and order nics by deviceid on Nics tab of insta…

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4768: URL: https://github.com/apache/cloudstack/pull/4768#issuecomment-797393607 @rhtyd (cc @soreana ) though it is a minor change it does include AI and DB changes, so I'd say leave it for 4.16 (that is not a definite :-1: at all, just a expression

[GitHub] [cloudstack] weizhouapache edited a comment on pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
weizhouapache edited a comment on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797392700 > @weizhouapache I'm confused by your description in context of the code change. If we do have a disk, but no backing format, we go ahead and rebase it as qcow2,

[GitHub] [cloudstack] weizhouapache commented on pull request #4800: kvm: Do not set backing file format of DATADISK in vm start/migration

2021-03-12 Thread GitBox
weizhouapache commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797392700 > @weizhouapache I'm confused by your description in context of the code change. If we do have a disk, but no backing format, we go ahead and rebase it as qcow2, am I r

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

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4779: URL: https://github.com/apache/cloudstack/pull/4779#issuecomment-797392702 Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 98 --

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

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4800: URL: https://github.com/apache/cloudstack/pull/4800#issuecomment-797389521 @weizhouapache I'm confused by your description in context of the code change. If we do have a disk, but no backing format, we go ahead and rebase it as qcow2, am I righ

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

2021-03-12 Thread GitBox
weizhouapache commented on issue #4799: URL: https://github.com/apache/cloudstack/issues/4799#issuecomment-797389443 @nvazquez good. I do not know if administrators are interested in it, but it will help cloudstack developers a lot !

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

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4794: URL: https://github.com/apache/cloudstack/pull/4794#issuecomment-797386465 clgtm, but; while this solves the problem that null values are allowed on volume details and not on template values, this problem is not the root of the issue. Why this

[GitHub] [cloudstack] weizhouapache commented on a change in pull request #4801: skip lifemigration for centos

2021-03-12 Thread GitBox
weizhouapache commented on a change in pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801#discussion_r593056295 ## File path: test/integration/smoke/test_vm_life_cycle.py ## @@ -1443,6 +1443,12 @@ def setUp(self): if len(self.hosts) < 2:

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

2021-03-12 Thread GitBox
weizhouapache commented on pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801#issuecomment-797381283 @DaanHoogland I checked the result of pr #4582 which are good. ``` Test Deploy Virtual Machine ... === TestName: test_01_deploy_vm | Status : SUCCESS === ok

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4793: systemvmtemplate: new template for 4.15.1

2021-03-12 Thread GitBox
DaanHoogland commented on a change in pull request #4793: URL: https://github.com/apache/cloudstack/pull/4793#discussion_r593048761 ## File path: tools/appliance/systemvmtemplate/template.json ## @@ -36,8 +36,8 @@ "disk_size": 2500, "disk_interface": "virtio",

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4793: systemvmtemplate: new template for 4.15.1

2021-03-12 Thread GitBox
DaanHoogland commented on a change in pull request #4793: URL: https://github.com/apache/cloudstack/pull/4793#discussion_r593047900 ## File path: engine/schema/src/main/java/com/cloud/upgrade/dao/Upgrade41500to41510.java ## @@ -58,6 +66,175 @@ public boolean supportsRollingUpg

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

2021-03-12 Thread GitBox
ravening commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797375897 > @ravening still some issue @shwstppr found the issue.. without the change, if ip address is in free state, its not displayed by `list publicipaddress` command but wi

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

2021-03-12 Thread GitBox
DaanHoogland commented on issue #4778: URL: https://github.com/apache/cloudstack/issues/4778#issuecomment-797372878 agree, just make sure you don't change the semantics or format of any key causing backwards incompatibilities (unless the current filling of a value is obviously a bug, but e

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

2021-03-12 Thread GitBox
Spaceman1984 commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797369613 > @Spaceman1984 can you help manually test/verify this? cc @shwstppr Thanks. I'll test @rhtyd Th

[GitHub] [cloudstack] slavkap commented on pull request #4773: Fix deploy VM from ISOs with UEFI

2021-03-12 Thread GitBox
slavkap commented on pull request #4773: URL: https://github.com/apache/cloudstack/pull/4773#issuecomment-797366632 Hi @DaanHoogland, what should be the expectations of Marvin's tests? I mean, do they have to do an automated install (probably with a kickstart for CentOS) and check if UEFI

[cloudstack-www] branch release-4.15 updated (0634262 -> 2e85b08)

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch release-4.15 in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git. from 0634262 site updates for 4.15 add 2e85b08 added 4.14.1 No new revisions were added by this update

[GitHub] [cloudstack] DK101010 opened a new pull request #4802: version 2.11 is incompatible with java 11

2021-03-12 Thread GitBox
DK101010 opened a new pull request #4802: URL: https://github.com/apache/cloudstack/pull/4802 With 2.11 wire mock tests hanging and mvn can't execute in thread mode. Update to 2.27.2(newest version) ### Types of changes - [ ] Breaking change (fix or f

[GitHub] [cloudstack] ravening removed a comment on pull request #4675: Bug fix in displaying public IP address of shared networks

2021-03-12 Thread GitBox
ravening removed a comment on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797361250 > @ravening still some issue @shwstppr but I dont think the issue is with ip management. This

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

2021-03-12 Thread GitBox
ravening commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797361250 > @ravening still some issue @shwstppr but I dont think the issue is with ip management. This is an a

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

2021-03-12 Thread GitBox
shwstppr commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797360468 @ravening still some issue This is an automated message from the Apache Git Service. To respond to the messa

[cloudstack-www] 01/01: Merge branch 'master' into asf-site

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git commit 7c44b93ccfcf26f0f063938f38594a0bfe118ea9 Merge: 4c49c67 5f3ab83 Author: Daan Hoogland AuthorDate: Fri Mar 12 10:

[cloudstack-www] branch asf-site updated (4c49c67 -> 7c44b93)

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git. from 4c49c67 build master add 604bd98 update issues link add b53f8ff checksum procedures add 5ff

[cloudstack-www] branch master updated: double-quotes

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git The following commit(s) were added to refs/heads/master by this push: new 9d93c94 double-quotes 9d93c94 is describe

[GitHub] [cloudstack] blueorangutan commented on pull request #4775: [Backport] #4698 Fix npe when migrating vm with volume

2021-03-12 Thread GitBox
blueorangutan commented on pull request #4775: URL: https://github.com/apache/cloudstack/pull/4775#issuecomment-797352992 Packaging result: :heavy_check_mark: centos7 :heavy_multiplication_x: centos8 :heavy_check_mark: debian. SL-JID 97

[GitHub] [cloudstack] blueorangutan commented on pull request #4775: [Backport] #4698 Fix npe when migrating vm with volume

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

[GitHub] [cloudstack] rhtyd commented on pull request #4775: [Backport] #4698 Fix npe when migrating vm with volume

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

[GitHub] [cloudstack] rhtyd commented on pull request #4768: display nics deviceid and order nics by deviceid on Nics tab of insta…

2021-03-12 Thread GitBox
rhtyd commented on pull request #4768: URL: https://github.com/apache/cloudstack/pull/4768#issuecomment-797341382 @soreana this looks like a good enhancement. @DaanHoogland @shwstppr should we consider/ask for 4.15.1? This i

[cloudstack-www] 01/01: Merge pull request #79 from apache/DaanHoogland-patch-1

2021-03-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git commit 5ff6d746a324ac796858b363691bc10b7961dd12 Merge: bb435e0 b53f8ff Author: dahn AuthorDate: Fri Mar 12 09:55:41 2021

[cloudstack-www] branch master updated (bb435e0 -> 5ff6d74)

2021-03-12 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-www.git. from bb435e0 downloads: use https://downloads.apache.org/cloudstack/releases links add 604bd98 update issues

[GitHub] [cloudstack] DaanHoogland closed issue #4753: Bug in download page verification instructions

2021-03-12 Thread GitBox
DaanHoogland closed issue #4753: URL: https://github.com/apache/cloudstack/issues/4753 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [cloudstack] DaanHoogland closed issue #4752: Bug: incorrect link to issue tracker

2021-03-12 Thread GitBox
DaanHoogland closed issue #4752: URL: https://github.com/apache/cloudstack/issues/4752 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[cloudstack] branch marvin3 updated (bc09184 -> 25fb029)

2021-03-12 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 bc09184 clean up VMs add 25fb029 remove superfluent arg No new revisions were added by this update. Summa

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

2021-03-12 Thread GitBox
weizhouapache commented on pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801#issuecomment-797338002 > @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. @DaanH

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

2021-03-12 Thread GitBox
shwstppr commented on pull request #4801: URL: https://github.com/apache/cloudstack/pull/4801#issuecomment-797332488 @DaanHoogland there seems some issue with one of the test - test_02_migrate_VM_with_two_data_disks. I found it failing on Ubuntu18 as well, ``` [root@pr4710-t97-kvm

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

2021-03-12 Thread GitBox
rhtyd commented on pull request #4675: URL: https://github.com/apache/cloudstack/pull/4675#issuecomment-797327950 @Spaceman1984 can you help manually test/verify this? cc @shwstppr Thanks. This is an automated message from th

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

2021-03-12 Thread GitBox
DaanHoogland commented on issue #4757: URL: https://github.com/apache/cloudstack/issues/4757#issuecomment-797326810 I've added a PR to skip the tests on centos #4801, of course that does not solve the underlying problem but we can not implement features on hypervisors that do not support t

[GitHub] [cloudstack] DaanHoogland commented on pull request #4701: Added support for removing unused port groups on VMWare

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4701: URL: https://github.com/apache/cloudstack/pull/4701#issuecomment-797320059 code looks good @Spaceman1984 , but i don't see how it is being called. Am I missing something? This is

[GitHub] [cloudstack] DaanHoogland commented on pull request #4773: Fix deploy VM from ISOs with UEFI

2021-03-12 Thread GitBox
DaanHoogland commented on pull request #4773: URL: https://github.com/apache/cloudstack/pull/4773#issuecomment-797316884 In your opinion @slavkap, can we automate those tests in a marvin script, using a smaller linux iso? Th