[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 Yes, we are indeed ready for a merge. Shall we do that? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1955 @karuturi 3 x LGTM, testing successful. Ready for Merge. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 LGTM from the code --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 I just re-depoyed and re-ran the test: Test that libvirt properly created domain with scsi controller ... === TestName: test_01_verify_libvirt | Status : SUCCESS === ok Verify that libvirt settings are as expected after a VM stop / start ... === TestName: test_02_verify_libvirt_after_restart | Status : SUCCESS === ok Verify that libvirt settings are expected after a disk add ... === TestName: test_03_verify_libvirt_attach_disk | Status : SUCCESS === ok Verify that guest sees scsi controller and disks ... === TestName: test_04_verify_guest_lspci | Status : SUCCESS === ok Update os type to Ubuntu, change vm details rootdiskController ... === TestName: test_05_change_vm_ostype_restart | Status : SUCCESS === ok Verify that guest sees scsi controller and disks after switching ostype and rdc ... === TestName: test_06_verify_guest_lspci_again | Status : SUCCESS === ok -- Ran 6 tests in 785.914s OK @wido , I believe all of your concerns should be addressed. trim is enabled, it's using an enum and not a string, and spurious log messages have been eliminated. I have squashed all of the commits into one. I'm at a point where I'm happy with the code. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 @nathanejohnson: Good! I would like to see that being enums. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @wido Actually I was just about to push a commit that changed the string to enum, but builds are failing locally because of this test: assertTrue(QuotaAlertManagerImpl.getDifferenceDays(now, new DateTime(now).plusDays(1).toDate()) == 1L); Daylight savings time strikes again --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 Do we want to pass the unmap not as a string? Shouldn't that be a enum? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @wido I have removed all of the debug logging statements that didn't exist before I touched the code, I have also added the discard option to the disks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 I would like to give a LGTM, but it's only the logging lines which I don't like yet. They seem like personal debug lines, but a admin will never know what they actually mean. In addition, I would like to see discard/unmap enabled by default for Virtio-SCSI disks: https://blog.widodh.nl/2015/12/using-trimdiscard-with-ceph-rbd-and-libvirt/ This will also allow for QCOW2 and RBD trimming. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1955 tag:mergeready --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Trillian test result (tid-938) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 34569 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1955-t938-kvm-centos7.zip Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py Test completed. 47 look ok, 3 have error(s) Test | Result | Time (s) | Test File --- | --- | --- | --- test_02_redundant_VPC_default_routes | `Failure` | 879.31 | test_vpc_redundant.py test_04_rvpc_privategw_static_routes | `Failure` | 362.02 | test_privategw_acl.py test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 873.50 | test_privategw_acl.py test_02_list_snapshots_with_removed_data_store | `Error` | 0.04 | test_snapshots.py test_01_vpc_site2site_vpn | Success | 170.70 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 66.07 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 266.31 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 297.40 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 543.52 | test_vpc_router_nics.py test_05_rvpc_multi_tiers | Success | 521.44 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | Success | 1421.18 | test_vpc_redundant.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 569.99 | test_vpc_redundant.py test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1292.16 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 156.42 | test_volumes.py test_08_resize_volume | Success | 156.48 | test_volumes.py test_07_resize_fail | Success | 161.51 | test_volumes.py test_06_download_detached_volume | Success | 156.29 | test_volumes.py test_05_detach_volume | Success | 155.77 | test_volumes.py test_04_delete_attached_volume | Success | 146.33 | test_volumes.py test_03_download_attached_volume | Success | 156.26 | test_volumes.py test_02_attach_volume | Success | 95.12 | test_volumes.py test_01_create_volume | Success | 714.40 | test_volumes.py test_03_delete_vm_snapshots | Success | 275.21 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 100.75 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 163.76 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 282.81 | test_vm_life_cycle.py test_deploy_vm | Success | 0.03 | test_vm_life_cycle.py test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py test_10_attachAndDetach_iso | Success | 26.68 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.22 | test_vm_life_cycle.py test_08_migrate_vm | Success | 30.86 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.16 | test_vm_life_cycle.py test_06_destroy_vm | Success | 125.79 | test_vm_life_cycle.py test_03_reboot_vm | Success | 125.81 | test_vm_life_cycle.py test_02_start_vm | Success | 10.22 | test_vm_life_cycle.py test_01_stop_vm | Success | 40.33 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 40.50 | test_templates.py test_08_list_system_templates | Success | 0.03 | test_templates.py test_07_list_public_templates | Success | 0.04 | test_templates.py test_05_template_permissions | Success | 0.06 | test_templates.py test_04_extract_template | Success | 5.14 | test_templates.py test_03_delete_template | Success | 5.11 | test_templates.py test_02_edit_template | Success | 90.21 | test_templates.py test_01_create_template | Success | 25.35 | test_templates.py test_10_destroy_cpvm | Success | 161.80 | test_ssvm.py test_09_destroy_ssvm | Success | 193.76 | test_ssvm.py test_08_reboot_cpvm | Success | 131.68 | test_ssvm.py test_07_reboot_ssvm | Success | 133.61 | test_ssvm.py test_06_stop_cpvm | Success | 131.75 | test_ssvm.py test_05_stop_ssvm | Success | 133.78 | test_ssvm.py test_04_cpvm_internals | Success | 1.29 | test_ssvm.py test_03_ssvm_internals | Success | 3.64 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.12 | test_ssvm.py test_01_snapshot_root_disk | Success | 11.19 | test_snapshots.py test_04_change_offering_small | Success | 239.57 | test_service_offerings.py test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py test_02_edit_service_offering | Success | 0.05 | test_service_offerings.py test_01_create_service_offering | Success | 0.11 | test_service_offerings.py test_02_sys_template_ready | Success | 0.13 | test_secondary_storage.py
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Packaging result: âcentos6 âcentos7 âdebian. JID-577 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @nathanejohnson sure, sorry I've missed your comment about the test fix earlier. @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov Whenever you get a second could you kick off a blue orangutan smoke test? I believe I have fixed the issue with ssh'ing into the KVM hosts, I sue the IP now instead of hostname. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 Also, I just made a commit, I have not squashed it because I wanted to get some feedback. I noticed that vmware uses details to pass rootDiskController for the bus type, and there was even some code that inspected this in the kvm plugin. With this latest commit, it will actually honor either OS type "Other PV Virtio-SCSI" or rootDiskController passed in via details, with details overriding OS type. I also made a minor tweak to the UI to allow setting the root disk bus via the add template dialog in the UI:  So this way you can select virtio scsi by one of the following three ways: 1) selecting os type as "Other PV Virtio-SCSI (64 bit)" 2) uploading a template and specifying details[0].rootDiskController=scsi, either from the UI or the API, then creating a VM from this template 3) deploying a VM and specifying details[0].rootDiskController=scsi , currently only available via the API --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov I have reworked the test so that it should now use the host IP versus the hostname. Could you please re-run the blue orangutan test? Thank you! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 I think that's the issue at our environment, it's trying to ssh via hostname instead of IP. Can you change that to IP please @nathanejohnson ? `Trying SSH Connection: Host:pr1955-t929-kvm-centos6-kvm1 User:root Port:22 RetryCnt:3===` --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1955 Tested it manually, works great. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 Thanks @nathanejohnson I will have a look --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov so the good news is it didn't skip the test. the bad news is it didn't seem to have proper credentials for ssh'ing into your hosts. It did pass the test where it ssh'ed into the guest, however. So this at least demonstrates a guest is happy. Do you have any tips on ways of making the smoke test play nice with BO wrt ssh'ing into the kvm hosts? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Trillian test result (tid-921) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36820 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1955-t921-kvm-centos7.zip Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py Test completed. 47 look ok, 3 have error(s) Test | Result | Time (s) | Test File --- | --- | --- | --- test_04_rvpc_privategw_static_routes | `Failure` | 334.38 | test_privategw_acl.py test_02_list_snapshots_with_removed_data_store | `Error` | 0.03 | test_snapshots.py test_03_verify_libvirt_attach_disk | `Error` | 610.88 | test_deploy_virtio_scsi_vm.py test_02_verify_libvirt_after_restart | `Error` | 615.96 | test_deploy_virtio_scsi_vm.py test_01_verify_libvirt | `Error` | 600.73 | test_deploy_virtio_scsi_vm.py test_01_vpc_site2site_vpn | Success | 159.71 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 65.83 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 240.43 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 292.11 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 551.36 | test_vpc_router_nics.py test_05_rvpc_multi_tiers | Success | 514.33 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | Success | 1404.34 | test_vpc_redundant.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 568.61 | test_vpc_redundant.py test_02_redundant_VPC_default_routes | Success | 737.39 | test_vpc_redundant.py test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1271.92 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 156.17 | test_volumes.py test_08_resize_volume | Success | 156.09 | test_volumes.py test_07_resize_fail | Success | 156.08 | test_volumes.py test_06_download_detached_volume | Success | 156.01 | test_volumes.py test_05_detach_volume | Success | 150.53 | test_volumes.py test_04_delete_attached_volume | Success | 150.91 | test_volumes.py test_03_download_attached_volume | Success | 155.98 | test_volumes.py test_02_attach_volume | Success | 94.76 | test_volumes.py test_01_create_volume | Success | 711.91 | test_volumes.py test_03_delete_vm_snapshots | Success | 275.10 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 100.69 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 163.74 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 256.93 | test_vm_life_cycle.py test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py test_10_attachAndDetach_iso | Success | 26.49 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.12 | test_vm_life_cycle.py test_08_migrate_vm | Success | 40.64 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.09 | test_vm_life_cycle.py test_06_destroy_vm | Success | 125.64 | test_vm_life_cycle.py test_03_reboot_vm | Success | 125.79 | test_vm_life_cycle.py test_02_start_vm | Success | 10.12 | test_vm_life_cycle.py test_01_stop_vm | Success | 40.25 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 35.32 | test_templates.py test_08_list_system_templates | Success | 0.02 | test_templates.py test_07_list_public_templates | Success | 0.03 | test_templates.py test_05_template_permissions | Success | 0.04 | test_templates.py test_04_extract_template | Success | 5.11 | test_templates.py test_03_delete_template | Success | 5.09 | test_templates.py test_02_edit_template | Success | 90.09 | test_templates.py test_01_create_template | Success | 45.35 | test_templates.py test_10_destroy_cpvm | Success | 161.57 | test_ssvm.py test_09_destroy_ssvm | Success | 133.41 | test_ssvm.py test_08_reboot_cpvm | Success | 101.46 | test_ssvm.py test_07_reboot_ssvm | Success | 133.47 | test_ssvm.py test_06_stop_cpvm | Success | 131.58 | test_ssvm.py test_05_stop_ssvm | Success | 133.56 | test_ssvm.py test_04_cpvm_internals | Success | 1.17 | test_ssvm.py test_03_ssvm_internals | Success | 3.57 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.08 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py test_01_snapshot_root_disk | Success | 10.92 | test_snapshots.py test_04_change_offering_small | Success | 234.31 | test_service_offerings.py test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov Quick questions, what tags need to be present to get picked up by BO? I'm worried my smoke test might get skipped. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Packaging result: âcentos6 âcentos7 âdebian. JID-552 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 Thanks @nathanejohnson, lets run that with BO @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 @wido I have removed the extraneous log messages. @boris I have added a smoke test for this that tests both virsh output from the host and lspci / lsblk output from the guest. Here is the output from me running this in a bubble: Test that libvirt properly created domain with scsi controller ... === TestName: test_01_verify_libvirt | Status : SUCCESS === ok Verify that libvirt settings are as expected after a VM stop / start ... === TestName: test_02_verify_libvirt_after_restart | Status : SUCCESS === ok Verify that libvirt settings are expected after a disk add ... === TestName: test_03_verify_libvirt_attach_disk | Status : SUCCESS === ok Verify that guest sees scsi controller and disks ... === TestName: test_04_verify_guest_lspci | Status : SUCCESS === ok -- Ran 4 tests in 658.770s OK --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @nathanejohnson thank you, let me know when you're done with the changes so I could kick in smoke tests. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user nathanejohnson commented on the issue: https://github.com/apache/cloudstack/pull/1955 As an update, I'm working on a Marvin test, and I'm also planning to address wido's concerns. Today had many distractions unfortunately. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @nathanejohnson Thanks, for this good enhancement. Can we have a marvin tests that covers the new functionality? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Trillian test result (tid-878) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32944 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1955-t878-kvm-centos7.zip Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py Test completed. 47 look ok, 2 have error(s) Test | Result | Time (s) | Test File --- | --- | --- | --- test_04_rvpc_privategw_static_routes | `Failure` | 329.25 | test_privategw_acl.py test_02_list_snapshots_with_removed_data_store | `Error` | 0.03 | test_snapshots.py test_01_vpc_site2site_vpn | Success | 164.34 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 70.82 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 270.12 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 288.94 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 511.96 | test_vpc_router_nics.py test_05_rvpc_multi_tiers | Success | 515.21 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | Success | 1413.54 | test_vpc_redundant.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 557.28 | test_vpc_redundant.py test_02_redundant_VPC_default_routes | Success | 743.95 | test_vpc_redundant.py test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1282.32 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 156.30 | test_volumes.py test_08_resize_volume | Success | 151.31 | test_volumes.py test_07_resize_fail | Success | 161.21 | test_volumes.py test_06_download_detached_volume | Success | 156.04 | test_volumes.py test_05_detach_volume | Success | 155.69 | test_volumes.py test_04_delete_attached_volume | Success | 151.00 | test_volumes.py test_03_download_attached_volume | Success | 156.04 | test_volumes.py test_02_attach_volume | Success | 124.19 | test_volumes.py test_01_create_volume | Success | 724.31 | test_volumes.py test_03_delete_vm_snapshots | Success | 275.70 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 95.60 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 166.79 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 257.28 | test_vm_life_cycle.py test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py test_10_attachAndDetach_iso | Success | 26.57 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.21 | test_vm_life_cycle.py test_08_migrate_vm | Success | 41.07 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py test_06_destroy_vm | Success | 125.79 | test_vm_life_cycle.py test_03_reboot_vm | Success | 125.68 | test_vm_life_cycle.py test_02_start_vm | Success | 10.13 | test_vm_life_cycle.py test_01_stop_vm | Success | 40.37 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 60.62 | test_templates.py test_08_list_system_templates | Success | 0.02 | test_templates.py test_07_list_public_templates | Success | 0.03 | test_templates.py test_05_template_permissions | Success | 0.04 | test_templates.py test_04_extract_template | Success | 5.13 | test_templates.py test_03_delete_template | Success | 5.08 | test_templates.py test_02_edit_template | Success | 90.14 | test_templates.py test_01_create_template | Success | 25.28 | test_templates.py test_10_destroy_cpvm | Success | 161.61 | test_ssvm.py test_09_destroy_ssvm | Success | 163.57 | test_ssvm.py test_08_reboot_cpvm | Success | 101.43 | test_ssvm.py test_07_reboot_ssvm | Success | 133.41 | test_ssvm.py test_06_stop_cpvm | Success | 131.63 | test_ssvm.py test_05_stop_ssvm | Success | 134.59 | test_ssvm.py test_04_cpvm_internals | Success | 1.17 | test_ssvm.py test_03_ssvm_internals | Success | 3.70 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.09 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.09 | test_ssvm.py test_01_snapshot_root_disk | Success | 10.93 | test_snapshots.py test_04_change_offering_small | Success | 239.36 | test_service_offerings.py test_03_delete_service_offering | Success | 0.06 | test_service_offerings.py test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py test_01_create_service_offering | Success | 0.09 | test_service_offerings.py test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py test_01_sys_vm_start | Success | 0.12 | test_secondary_storage.py test_09_reboot_router | Success | 35.26
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 Packaging result: âcentos6 âcentos7 âdebian. JID-522 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1955 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1955 Thanks for this PR @nathanejohnson, I will build it and run smoketests agains it. @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user dmabry commented on the issue: https://github.com/apache/cloudstack/pull/1955 We are deploying this to our QA environment right now and hope to have it tested in a few days. Great work @kiwiflyer and @nathanejohnson. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1955 Very nice indeed! I will take a look asap. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user remibergsma commented on the issue: https://github.com/apache/cloudstack/pull/1955 Nice work @kiwiflyer @dmabry, thanks! Will give it a go soon. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...
Github user kiwiflyer commented on the issue: https://github.com/apache/cloudstack/pull/1955 @remibergsma @ustcweizhou This might be of interest to you guys as well. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
