You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by murali-reddy <gi...@git.apache.org> on 2016/09/21 06:16:16 UTC

[GitHub] cloudstack pull request #1681: CLOUDSTACK-9491: incorrect parsing of device ...

GitHub user murali-reddy opened a pull request:

    https://github.com/apache/cloudstack/pull/1681

    CLOUDSTACK-9491: incorrect parsing of device list to find ethernet index of plugged NIC

    
    In VmwareResource, findRouterEthDeviceIndex() method find ethernet interface index given
    the mac address. This method is used, once a nic is plugged to determine ethernet interface.
    "/proc/sys/net/ipv4/conf" from the VR and looped through the devices to find the right
    ethernet interface. Howver current logic read it once, and loops through the device list.
    Its observerd device may not show up '/proc/sys/net/ipv4/conf' immediatly once NIC is plugged
    in the VM from vCenter.
    
    Fix ensured, while waiting for 15 sec in the loop, read the latest content from /proc/sys/net/ipv4/conf
    , so that right device list is processed.
    
    Manual tested VPC scenarios of adding new tiers which uses findRouterEthDeviceIndex, to find the guest/public network ethernet index.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/murali-reddy/cloudstack router_eth_device_index

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1681.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1681
    
----
commit be7ade3153d3b29c18d904d5327c1af139e1c037
Author: Murali Reddy <mu...@gmail.com>
Date:   2016-09-21T06:08:47Z

    CLOUDSTACK-9491: incorrect parsing of device list to find ethener index of plugged nic
    
    In VmwareResource, findRouterEthDeviceIndex() method find ethernet interface index given
    the mac address. This method is used, once a nic is plugged to determine ethernet interface.
    "/proc/sys/net/ipv4/conf" from the VR and looped through the devices to find the right
    ethernet interface. Howver current logic read it once, and loops through the device list.
    Its observerd device may not show up '/proc/sys/net/ipv4/conf' immediatly once NIC is plugged
    in the VM from vCenter.
    
    Fix ensured, while waiting for 15 sec in the loop, read the latest content from /proc/sys/net/ipv4/conf
    , so that right device list is processed.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @rhtyd unsupported parameters provided. Supported mgmt server os are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6, kvm-centos7, kvm-ubuntu, xenserver-65sp1, xenserver-62sp1, vmware-60u2, vmware-55u3, vmware-51u1, vmware-50u1`


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-25


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware55u3


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy have you had a chance to investigate the [test failures](https://github.com/apache/cloudstack/pull/1681#issuecomment-257717551)?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy This issue might be related to PR1660. Do you mind testing against latest master to see if you it is addressed already?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-226


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @rhtyd 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @jburwell a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    <b>Trillian test result (tid-232)</b>
    Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 37364 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1681-t232-vmware-55u3.zip
    Test completed. 45 look ok, 3 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Failure` | 333.42 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Failure` | 121.34 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 156.35 | test_internal_lb.py
    test_01_vpc_site2site_vpn | `Error` | 495.89 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | `Error` | 807.67 | test_vpc_vpn.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Error` | 1295.34 | test_vpc_redundant.py
    test_01_vpc_remote_access_vpn | Success | 166.30 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 399.32 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 775.75 | test_vpc_router_nics.py
    test_05_rvpc_multi_tiers | Success | 759.90 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | Success | 1619.27 | test_vpc_redundant.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 817.78 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 688.69 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 36.51 | test_volumes.py
    test_06_download_detached_volume | Success | 85.74 | test_volumes.py
    test_05_detach_volume | Success | 110.73 | test_volumes.py
    test_04_delete_attached_volume | Success | 20.21 | test_volumes.py
    test_03_download_attached_volume | Success | 26.01 | test_volumes.py
    test_02_attach_volume | Success | 64.50 | test_volumes.py
    test_01_create_volume | Success | 530.34 | test_volumes.py
    test_03_delete_vm_snapshots | Success | 280.23 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 227.15 | test_vm_snapshots.py
    test_01_test_vm_volume_snapshot | Success | 231.41 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 161.71 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 288.30 | 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.76 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.19 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 80.82 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.07 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 10.14 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 5.11 | test_vm_life_cycle.py
    test_02_start_vm | Success | 25.20 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 10.11 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 276.43 | 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 | 15.34 | test_templates.py
    test_03_delete_template | Success | 5.08 | test_templates.py
    test_02_edit_template | Success | 90.16 | test_templates.py
    test_01_create_template | Success | 130.78 | test_templates.py
    test_10_destroy_cpvm | Success | 261.57 | test_ssvm.py
    test_09_destroy_ssvm | Success | 263.50 | test_ssvm.py
    test_08_reboot_cpvm | Success | 156.37 | test_ssvm.py
    test_07_reboot_ssvm | Success | 158.21 | test_ssvm.py
    test_06_stop_cpvm | Success | 231.67 | test_ssvm.py
    test_05_stop_ssvm | Success | 203.50 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.03 | test_ssvm.py
    test_03_ssvm_internals | Success | 3.36 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.09 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.14 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 81.27 | test_snapshots.py
    test_04_change_offering_small | Success | 126.97 | test_service_offerings.py
    test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
    test_02_edit_service_offering | Success | 0.06 | test_service_offerings.py
    test_01_create_service_offering | Success | 0.08 | test_service_offerings.py
    test_02_sys_template_ready | Success | 0.10 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.16 | test_secondary_storage.py
    test_09_reboot_router | Success | 176.00 | test_routers.py
    test_08_start_router | Success | 155.90 | test_routers.py
    test_07_stop_router | Success | 20.24 | test_routers.py
    test_06_router_advanced | Success | 0.05 | test_routers.py
    test_05_router_basic | Success | 0.07 | test_routers.py
    test_04_restart_network_wo_cleanup | Success | 5.56 | test_routers.py
    test_03_restart_network_cleanup | Success | 186.00 | test_routers.py
    test_02_router_internal_adv | Success | 0.89 | test_routers.py
    test_01_router_internal_basic | Success | 0.50 | test_routers.py
    test_router_dns_guestipquery | Success | 77.02 | test_router_dns.py
    test_router_dns_externalipquery | Success | 0.05 | test_router_dns.py
    test_router_dhcphosts | Success | 155.63 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 0.06 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 65.45 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.19 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.31 | test_public_ip_range.py
    test_04_rvpc_privategw_static_routes | Success | 1294.48 | test_privategw_acl.py
    test_03_vpc_privategw_restart_vpc_cleanup | Success | 1177.77 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 802.10 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 197.78 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 37.58 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 10.16 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.33 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 94.12 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.10 | test_over_provisioning.py
    test_oobm_zchange_password | Success | 15.51 | test_outofbandmanagement.py
    test_oobm_multiple_mgmt_server_ownership | Success | 16.52 | test_outofbandmanagement.py
    test_oobm_issue_power_status | Success | 10.51 | test_outofbandmanagement.py
    test_oobm_issue_power_soft | Success | 15.51 | test_outofbandmanagement.py
    test_oobm_issue_power_reset | Success | 10.51 | test_outofbandmanagement.py
    test_oobm_issue_power_on | Success | 15.51 | test_outofbandmanagement.py
    test_oobm_issue_power_off | Success | 15.53 | test_outofbandmanagement.py
    test_oobm_issue_power_cycle | Success | 15.51 | test_outofbandmanagement.py
    test_oobm_enabledisable_across_clusterzones | Success | 57.30 | test_outofbandmanagement.py
    test_oobm_enable_feature_valid | Success | 5.14 | test_outofbandmanagement.py
    test_oobm_enable_feature_invalid | Success | 0.09 | test_outofbandmanagement.py
    test_oobm_disable_feature_valid | Success | 5.15 | test_outofbandmanagement.py
    test_oobm_disable_feature_invalid | Success | 0.09 | test_outofbandmanagement.py
    test_oobm_configure_invalid_driver | Success | 0.07 | test_outofbandmanagement.py
    test_oobm_configure_default_driver | Success | 0.18 | test_outofbandmanagement.py
    test_oobm_background_powerstate_sync | Success | 23.71 | test_outofbandmanagement.py
    test_extendPhysicalNetworkVlan | Success | 15.33 | test_non_contigiousvlan.py
    test_01_nic | Success | 817.03 | test_nic.py
    test_releaseIP | Success | 494.18 | test_network.py
    test_reboot_router | Success | 690.23 | test_network.py
    test_public_ip_user_account | Success | 10.18 | test_network.py
    test_public_ip_admin_account | Success | 40.24 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 76.51 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.70 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 125.58 | test_network.py
    test_delete_account | Success | 343.29 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.49 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 111.70 | test_network.py
    test_nic_secondaryip_add_remove | Success | 293.25 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 21.70 | test_login.py
    test_assign_and_removal_lb | Success | 149.04 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 207.59 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 208.70 | test_loadbalance.py
    test_03_list_snapshots | Success | 0.07 | test_list_ids_parameter.py
    test_02_list_templates | Success | 0.04 | test_list_ids_parameter.py
    test_01_list_volumes | Success | 0.03 | test_list_ids_parameter.py
    test_07_list_default_iso | Success | 0.05 | test_iso.py
    test_05_iso_permissions | Success | 0.05 | test_iso.py
    test_04_extract_Iso | Success | 5.16 | test_iso.py
    test_03_delete_iso | Success | 95.18 | test_iso.py
    test_02_edit_iso | Success | 0.05 | test_iso.py
    test_01_create_iso | Success | 21.32 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 732.30 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.23 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.12 | test_global_settings.py
    test_rolepermission_lifecycle_update | Success | 6.62 | test_dynamicroles.py
    test_rolepermission_lifecycle_list | Success | 6.44 | test_dynamicroles.py
    test_rolepermission_lifecycle_delete | Success | 6.41 | test_dynamicroles.py
    test_rolepermission_lifecycle_create | Success | 6.25 | test_dynamicroles.py
    test_rolepermission_lifecycle_concurrent_updates | Success | 6.37 | test_dynamicroles.py
    test_role_lifecycle_update_role_inuse | Success | 6.23 | test_dynamicroles.py
    test_role_lifecycle_update | Success | 11.32 | test_dynamicroles.py
    test_role_lifecycle_list | Success | 6.38 | test_dynamicroles.py
    test_role_lifecycle_delete | Success | 6.26 | test_dynamicroles.py
    test_role_lifecycle_create | Success | 6.28 | test_dynamicroles.py
    test_role_inuse_deletion | Success | 6.22 | test_dynamicroles.py
    test_role_account_acls_multiple_mgmt_servers | Success | 7.70 | test_dynamicroles.py
    test_role_account_acls | Success | 8.01 | test_dynamicroles.py
    test_default_role_deletion | Success | 6.30 | test_dynamicroles.py
    test_04_create_fat_type_disk_offering | Success | 0.11 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.04 | test_disk_offerings.py
    test_02_edit_disk_offering | Success | 0.05 | test_disk_offerings.py
    test_02_create_sparse_type_disk_offering | Success | 0.07 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.11 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 65.71 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 56.55 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 211.43 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 65.68 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 232.26 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 6.41 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 6.20 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 6.28 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 237.60 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 221.73 | test_affinity_groups.py
    test_08_resize_volume | Skipped | 10.13 | test_volumes.py
    test_07_resize_fail | Skipped | 15.30 | test_volumes.py
    test_06_copy_template | Skipped | 0.00 | test_templates.py
    test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
    test_01_scale_vm | Skipped | 66.34 | test_scale_vm.py
    test_01_primary_storage_iscsi | Skipped | 0.03 | test_primary_storage.py
    test_06_copy_iso | Skipped | 0.00 | test_iso.py
    test_deploy_vgpu_enabled_vm | Skipped | 0.01 | test_deploy_vgpu_enabled_vm.py



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy could you please investigate the following test case failures:
    
    * ``test_03_vpc_internallb_haproxy_stats_on_all_interfaces``
    * ``test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80``
    * ``test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80``
    * ``test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL``


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @rhtyd 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1681: CLOUDSTACK-9491: incorrect parsing of device ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/1681


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @sateesh-chodapuneedi  added optimization for now, but its better we eliminate the whole logic of finding device index from the VR.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy could you please squash your commits?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    LGTM, failures related to vpc/rvr are known intermittent failures. We'll kick another test round on #1753, #1754 etc. and work towards fixing them for vmware and xenserver separately.  Merging this based on code reviews and 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy can you squash your commits
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware55u3


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    <b>Trillian test result (tid-58)</b>
    Environment: vmware-55u3 (x2), Advanced Networking
    Total time taken: 33200 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1681-t58-vmware-55u3.zip
    Test completed. 41 look ok, 12 have errors
    
    
    Test | Result | Time (s)
    --- | --- | ---
    test_oobm_zchange_password | `Failure` | 20.361
    test_04_rvpc_privategw_static_routes | `Failure` | 383.174
    test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 212.325
    test_02_vpc_privategw_static_routes | `Failure` | 232.360
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | `Failure` | 585.633
    test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Failure` | 459.765
    test_01_vpc_privategw_acl | `Failure` | 106.756
    test_01_redundant_vpc_site2site_vpn | `Failure` | 412.476
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | `Failure` | 425.171
    test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Failure` | 399.796
    test_router_dhcphosts | `Error` | 827.549
    test_isolate_network_password_server | `Error` | 825.097
    test_extendPhysicalNetworkVlan | `Error` | 0.031
    test_dedicatePublicIpRange | `Error` | 0.213
    test_04_rvpc_network_garbage_collector_nics | `Error` | 227.392
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 455.683
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | `Error` | 370.529
    test_02_routervm_iptables_policies | `Error` | 920.696
    test_01_vpc_site2site_vpn | `Error` | 491.171
    test_01_vpc_remote_access_vpn | `Error` | 0.094
    test_01_test_vm_volume_snapshot | `Error` | 151.079
    test_01_single_VPC_iptables_policies | `Error` | 1010.381
    ContextSuite context=TestRouterDHCPHosts>:teardown | `Error` | 843.716
    ContextSuite context=TestDedicateGuestVlanRange>:setup | `Error` | 0.000
    test_router_dns_guestipquery | Success | 76.635
    test_router_dns_externalipquery | Success | 0.070
    test_rolepermission_lifecycle_update | Success | 6.566
    test_rolepermission_lifecycle_list | Success | 6.263
    test_rolepermission_lifecycle_delete | Success | 6.158
    test_rolepermission_lifecycle_create | Success | 6.179
    test_rolepermission_lifecycle_concurrent_updates | Success | 6.302
    test_role_lifecycle_update_role_inuse | Success | 6.197
    test_role_lifecycle_update | Success | 11.258
    test_role_lifecycle_list | Success | 6.282
    test_role_lifecycle_delete | Success | 11.247
    test_role_lifecycle_create | Success | 6.212
    test_role_inuse_deletion | Success | 6.184
    test_role_account_acls_multiple_mgmt_servers | Success | 7.661
    test_role_account_acls | Success | 7.971
    test_releaseIP | Success | 288.086
    test_reboot_router | Success | 599.543
    test_public_ip_user_account | Success | 10.186
    test_public_ip_admin_account | Success | 40.235
    test_oobm_multiple_mgmt_server_ownership | Success | 14.558
    test_oobm_issue_power_status | Success | 10.557
    test_oobm_issue_power_soft | Success | 15.557
    test_oobm_issue_power_reset | Success | 15.549
    test_oobm_issue_power_on | Success | 15.593
    test_oobm_issue_power_off | Success | 15.590
    test_oobm_issue_power_cycle | Success | 15.658
    test_oobm_enabledisable_across_clusterzones | Success | 57.971
    test_oobm_enable_feature_valid | Success | 5.150
    test_oobm_enable_feature_invalid | Success | 0.089
    test_oobm_disable_feature_valid | Success | 5.158
    test_oobm_disable_feature_invalid | Success | 5.168
    test_oobm_configure_invalid_driver | Success | 0.077
    test_oobm_configure_default_driver | Success | 0.076
    test_oobm_background_powerstate_sync | Success | 19.591
    test_nic_secondaryip_add_remove | Success | 212.565
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 76.621
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.475
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 125.136
    test_network_acl | Success | 201.411
    test_deployvm_userdispersing | Success | 60.629
    test_deployvm_userdata_post | Success | 30.423
    test_deployvm_userdata | Success | 186.248
    test_deployvm_userconcentrated | Success | 55.702
    test_deployvm_firstfit | Success | 196.124
    test_deploy_vm_multiple | Success | 222.462
    test_deploy_vm_from_iso | Success | 432.595
    test_deploy_vm | Success | 0.023
    test_delete_account | Success | 318.066
    test_default_role_deletion | Success | 6.285
    test_create_pvlan_network | Success | 5.163
    test_createRegion | Success | 0.031
    test_createPortablePublicIPRange | Success | 15.274
    test_createPortablePublicIPAcquire | Success | 15.482
    test_assign_and_removal_lb | Success | 148.732
    test_advZoneVirtualRouter | Success | 0.019
    test_UpdateStorageOverProvisioningFactor | Success | 0.098
    test_UpdateConfigParamWithScope | Success | 0.099
    test_DeployVmAntiAffinityGroup_in_project | Success | 261.861
    test_DeployVmAntiAffinityGroup | Success | 176.286
    test_CreateTemplateWithDuplicateName | Success | 221.386
    test_10_destroy_cpvm | Success | 271.790
    test_10_attachAndDetach_iso | Success | 71.825
    test_09_reboot_router | Success | 171.122
    test_09_expunge_vm | Success | 125.215
    test_09_destroy_ssvm | Success | 268.768
    test_09_delete_detached_volume | Success | 26.076
    test_08_start_router | Success | 156.080
    test_08_reboot_cpvm | Success | 156.594
    test_08_migrate_vm | Success | 85.962
    test_08_list_system_templates | Success | 0.031
    test_07_stop_router | Success | 20.251
    test_07_restore_vm | Success | 0.072
    test_07_reboot_ssvm | Success | 158.292
    test_07_list_public_templates | Success | 0.039
    test_07_list_default_iso | Success | 0.040
    test_06_stop_cpvm | Success | 226.725
    test_06_router_advanced | Success | 0.078
    test_06_download_detached_volume | Success | 60.450
    test_06_destroy_vm | Success | 10.116
    test_05_template_permissions | Success | 0.062
    test_05_stop_ssvm | Success | 208.743
    test_05_rvpc_multi_tiers | Success | 670.737
    test_05_router_basic | Success | 0.053
    test_05_iso_permissions | Success | 0.044
    test_05_detach_volume | Success | 110.256
    test_04_restart_network_wo_cleanup | Success | 5.793
    test_04_extract_template | Success | 10.814
    test_04_extract_Iso | Success | 5.113
    test_04_delete_attached_volume | Success | 20.200
    test_04_create_fat_type_disk_offering | Success | 0.054
    test_04_cpvm_internals | Success | 1.131
    test_04_change_offering_small | Success | 97.976
    test_03_ssvm_internals | Success | 3.486
    test_03_restart_network_cleanup | Success | 150.913
    test_03_reboot_vm | Success | 5.134
    test_03_list_snapshots | Success | 0.051
    test_03_download_attached_volume | Success | 15.223
    test_03_delete_vm_snapshots | Success | 275.219
    test_03_delete_template | Success | 5.107
    test_03_delete_service_offering | Success | 0.037
    test_03_delete_iso | Success | 95.094
    test_03_delete_disk_offering | Success | 0.030
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 747.514
    test_03_RVR_Network_check_router_state | Success | 370.433
    test_02_sys_template_ready | Success | 0.112
    test_02_start_vm | Success | 20.186
    test_02_router_internal_adv | Success | 1.014
    test_02_revert_vm_snapshots | Success | 229.115
    test_02_redundant_VPC_default_routes | Success | 773.247
    test_02_port_fwd_on_non_src_nat | Success | 55.519
    test_02_list_templates | Success | 0.032
    test_02_list_cpvm_vm | Success | 0.106
    test_02_isolate_network_FW_PF_default_routes_egress_false | Success | 256.184
    test_02_edit_template | Success | 90.088
    test_02_edit_service_offering | Success | 0.067
    test_02_edit_iso | Success | 0.058
    test_02_edit_disk_offering | Success | 0.034
    test_02_deploy_vm_root_resize | Success | 6.231
    test_02_create_sparse_type_disk_offering | Success | 0.050
    test_02_create_lb_rule_non_nat | Success | 207.287
    test_02_attach_volume | Success | 60.452
    test_02_VPC_default_routes | Success | 358.088
    test_01_updatevolumedetail | Success | 5.090
    test_01_sys_vm_start | Success | 0.147
    test_01_stop_vm | Success | 10.119
    test_01_snapshot_root_disk | Success | 181.873
    test_01_router_internal_basic | Success | 0.504
    test_01_reset_vm_on_reboot | Success | 25.239
    test_01_primary_storage_nfs | Success | 38.521
    test_01_port_fwd_on_src_nat | Success | 111.912
    test_01_nic | Success | 700.705
    test_01_list_volumes | Success | 0.021
    test_01_list_sec_storage_vm | Success | 0.107
    test_01_isolate_network_FW_PF_default_routes_egress_true | Success | 311.392
    test_01_deploy_vm_root_resize | Success | 6.186
    test_01_create_volume | Success | 525.905
    test_01_create_vm_snapshots | Success | 161.916
    test_01_create_template | Success | 140.821
    test_01_create_service_offering | Success | 0.107
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1417.998
    test_01_create_lb_rule_src_nat | Success | 207.774
    test_01_create_iso | Success | 66.269
    test_01_create_disk_offering | Success | 0.086
    test_01_VPC_nics_after_destroy | Success | 676.058
    test_00_deploy_vm_root_resize | Success | 6.308
    login_test_saml_user | Success | 21.636
    test_vm_nic_adapter_vmxnet3 | Skipped | 0.001
    test_static_role_account_acls | Skipped | 0.017
    test_deploy_vgpu_enabled_vm | Skipped | 0.005
    test_08_resize_volume | Skipped | 5.105
    test_07_resize_fail | Skipped | 10.241
    test_06_copy_template | Skipped | 0.000
    test_06_copy_iso | Skipped | 0.000
    test_01_scale_vm | Skipped | 66.423
    test_01_primary_storage_iscsi | Skipped | 0.051



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    LGTM on tests, the failures wrt VPC are known intermittent issues. I'm not sure about the internal lb failures, I'll kick another test round since there have been several changes on the base branch. 
    
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy is there a way to create or update a Marvin test case to verify this fix?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by sateesh-chodapuneedi <gi...@git.apache.org>.
Github user sateesh-chodapuneedi commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Code LGTM, \U0001f44d +1


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Invalid params were provided, the hypervisor to be passed was vmware-55u3. I'll fix this in blueorangutan as it currently does not check/validate the hypervisor name properly. Kicked a new job manually from Jenkins.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware55u3


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-86


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy Thanks for the explanation. LGTM for code review


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by abhinandanprateek <gi...@git.apache.org>.
Github user abhinandanprateek commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    The fix looks like the correct thing to do. 
    @muali-reddy LGTM for code review.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware-55u3



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    <b>Trillian test result (tid-378)</b>
    Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 35100 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1681-t378-vmware-55u3.zip
    Test completed. 45 look ok, 3 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_04_rvpc_privategw_static_routes | `Failure` | 881.18 | test_privategw_acl.py
    test_01_vpc_site2site_vpn | `Error` | 491.85 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | `Error` | 692.86 | test_vpc_vpn.py
    test_03_restart_network_cleanup | `Error` | 141.19 | test_routers.py
    test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 679.12 | test_privategw_acl.py
    test_03_vpc_privategw_restart_vpc_cleanup | `Error` | 729.79 | test_privategw_acl.py
    test_01_vpc_remote_access_vpn | Success | 166.72 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 380.57 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 712.43 | test_vpc_router_nics.py
    test_05_rvpc_multi_tiers | Success | 692.84 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | Success | 1665.06 | test_vpc_redundant.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 720.00 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 735.66 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1377.56 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 25.86 | test_volumes.py
    test_06_download_detached_volume | Success | 85.81 | test_volumes.py
    test_05_detach_volume | Success | 105.23 | test_volumes.py
    test_04_delete_attached_volume | Success | 10.19 | test_volumes.py
    test_03_download_attached_volume | Success | 20.38 | test_volumes.py
    test_02_attach_volume | Success | 48.73 | test_volumes.py
    test_01_create_volume | Success | 477.51 | test_volumes.py
    test_03_delete_vm_snapshots | Success | 280.24 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 242.30 | test_vm_snapshots.py
    test_01_test_vm_volume_snapshot | Success | 337.50 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 159.40 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 228.68 | 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 | 27.19 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.17 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 71.37 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.11 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 10.15 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 5.14 | test_vm_life_cycle.py
    test_02_start_vm | Success | 20.24 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 5.12 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 261.64 | 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 | 20.59 | test_templates.py
    test_03_delete_template | Success | 5.11 | test_templates.py
    test_02_edit_template | Success | 90.13 | test_templates.py
    test_01_create_template | Success | 125.87 | test_templates.py
    test_10_destroy_cpvm | Success | 236.74 | test_ssvm.py
    test_09_destroy_ssvm | Success | 268.77 | test_ssvm.py
    test_08_reboot_cpvm | Success | 156.56 | test_ssvm.py
    test_07_reboot_ssvm | Success | 158.70 | test_ssvm.py
    test_06_stop_cpvm | Success | 176.78 | test_ssvm.py
    test_05_stop_ssvm | Success | 213.78 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.29 | test_ssvm.py
    test_03_ssvm_internals | Success | 3.96 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.14 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.14 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 31.23 | test_snapshots.py
    test_04_change_offering_small | Success | 91.85 | 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.15 | test_service_offerings.py
    test_02_sys_template_ready | Success | 0.13 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.17 | test_secondary_storage.py
    test_09_reboot_router | Success | 125.88 | test_routers.py
    test_08_start_router | Success | 105.86 | test_routers.py
    test_07_stop_router | Success | 25.35 | test_routers.py
    test_06_router_advanced | Success | 0.10 | test_routers.py
    test_05_router_basic | Success | 0.06 | test_routers.py
    test_04_restart_network_wo_cleanup | Success | 117.79 | test_routers.py
    test_02_router_internal_adv | Success | 1.24 | test_routers.py
    test_01_router_internal_basic | Success | 0.60 | test_routers.py
    test_router_dns_guestipquery | Success | 79.68 | test_router_dns.py
    test_router_dns_externalipquery | Success | 0.05 | test_router_dns.py
    test_router_dhcphosts | Success | 185.26 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 5.11 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 75.90 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.23 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.43 | test_public_ip_range.py
    test_02_vpc_privategw_static_routes | Success | 759.17 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 183.97 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 38.28 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 15.20 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.44 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 89.14 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.16 | test_over_provisioning.py
    test_oobm_zchange_password | Success | 20.52 | test_outofbandmanagement.py
    test_oobm_multiple_mgmt_server_ownership | Success | 16.50 | test_outofbandmanagement.py
    test_oobm_issue_power_status | Success | 10.53 | test_outofbandmanagement.py
    test_oobm_issue_power_soft | Success | 15.52 | test_outofbandmanagement.py
    test_oobm_issue_power_reset | Success | 10.51 | test_outofbandmanagement.py
    test_oobm_issue_power_on | Success | 15.52 | test_outofbandmanagement.py
    test_oobm_issue_power_off | Success | 15.52 | test_outofbandmanagement.py
    test_oobm_issue_power_cycle | Success | 15.52 | test_outofbandmanagement.py
    test_oobm_enabledisable_across_clusterzones | Success | 57.23 | test_outofbandmanagement.py
    test_oobm_enable_feature_valid | Success | 5.16 | test_outofbandmanagement.py
    test_oobm_enable_feature_invalid | Success | 0.10 | test_outofbandmanagement.py
    test_oobm_disable_feature_valid | Success | 0.21 | test_outofbandmanagement.py
    test_oobm_disable_feature_invalid | Success | 0.13 | test_outofbandmanagement.py
    test_oobm_configure_invalid_driver | Success | 0.09 | test_outofbandmanagement.py
    test_oobm_configure_default_driver | Success | 0.09 | test_outofbandmanagement.py
    test_oobm_background_powerstate_sync | Success | 29.58 | test_outofbandmanagement.py
    test_extendPhysicalNetworkVlan | Success | 15.29 | test_non_contigiousvlan.py
    test_01_nic | Success | 652.35 | test_nic.py
    test_releaseIP | Success | 228.69 | test_network.py
    test_reboot_router | Success | 605.97 | test_network.py
    test_public_ip_user_account | Success | 10.27 | test_network.py
    test_public_ip_admin_account | Success | 40.51 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 76.81 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.95 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 124.89 | test_network.py
    test_delete_account | Success | 243.46 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.71 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 111.71 | test_network.py
    test_nic_secondaryip_add_remove | Success | 243.80 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 24.64 | test_login.py
    test_assign_and_removal_lb | Success | 137.83 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 207.28 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 204.28 | test_loadbalance.py
    test_03_list_snapshots | Success | 0.09 | test_list_ids_parameter.py
    test_02_list_templates | Success | 0.04 | test_list_ids_parameter.py
    test_01_list_volumes | Success | 0.03 | test_list_ids_parameter.py
    test_07_list_default_iso | Success | 0.06 | test_iso.py
    test_05_iso_permissions | Success | 0.06 | test_iso.py
    test_04_extract_Iso | Success | 5.15 | test_iso.py
    test_03_delete_iso | Success | 95.12 | test_iso.py
    test_02_edit_iso | Success | 0.05 | test_iso.py
    test_01_create_iso | Success | 21.89 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 625.89 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 454.97 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 1180.26 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 836.80 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.25 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.13 | test_global_settings.py
    test_rolepermission_lifecycle_update | Success | 7.05 | test_dynamicroles.py
    test_rolepermission_lifecycle_list | Success | 6.99 | test_dynamicroles.py
    test_rolepermission_lifecycle_delete | Success | 6.82 | test_dynamicroles.py
    test_rolepermission_lifecycle_create | Success | 6.80 | test_dynamicroles.py
    test_rolepermission_lifecycle_concurrent_updates | Success | 6.99 | test_dynamicroles.py
    test_role_lifecycle_update_role_inuse | Success | 6.80 | test_dynamicroles.py
    test_role_lifecycle_update | Success | 11.96 | test_dynamicroles.py
    test_role_lifecycle_list | Success | 6.83 | test_dynamicroles.py
    test_role_lifecycle_delete | Success | 11.96 | test_dynamicroles.py
    test_role_lifecycle_create | Success | 6.96 | test_dynamicroles.py
    test_role_inuse_deletion | Success | 6.89 | test_dynamicroles.py
    test_role_account_acls_multiple_mgmt_servers | Success | 9.25 | test_dynamicroles.py
    test_role_account_acls | Success | 9.32 | test_dynamicroles.py
    test_default_role_deletion | Success | 7.02 | test_dynamicroles.py
    test_04_create_fat_type_disk_offering | Success | 0.09 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.04 | test_disk_offerings.py
    test_02_edit_disk_offering | Success | 0.06 | test_disk_offerings.py
    test_02_create_sparse_type_disk_offering | Success | 0.07 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.11 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 86.22 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 131.30 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 201.51 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 30.55 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 322.41 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 6.77 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 6.75 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 6.86 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 243.65 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 227.14 | test_affinity_groups.py
    test_08_resize_volume | Skipped | 10.14 | test_volumes.py
    test_07_resize_fail | Skipped | 10.30 | test_volumes.py
    test_06_copy_template | Skipped | 0.00 | test_templates.py
    test_static_role_account_acls | Skipped | 0.02 | test_staticroles.py
    test_01_scale_vm | Skipped | 33.88 | test_scale_vm.py
    test_01_primary_storage_iscsi | Skipped | 0.04 | test_primary_storage.py
    test_06_copy_iso | Skipped | 0.00 | test_iso.py
    test_deploy_vgpu_enabled_vm | Skipped | 0.00 | test_deploy_vgpu_enabled_vm.py



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-114


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware55u3


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    LGTM, some test failures have been addressed here: https://github.com/apache/cloudstack/pull/1692


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy have you had a chance to address @sateesh-chodapuneedi comments?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by jburwell <gi...@git.apache.org>.
Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @rhtyd could you check into the status of the test run for this PR?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @jburwell function 'findRouterEthDeviceIndex' which is modified, is invoked when ever a new tier to VPC is added (virtual NIC is hot-plugged). Many VPC tests run againest the VMware environment will cover this scenario.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by sateesh-chodapuneedi <gi...@git.apache.org>.
Github user sateesh-chodapuneedi commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy Makes sense to get fresh listing of virtual NICs in next iteration as the first iteration might have missed the hot-plugged virtual NIC (for VPC tier being configured). Though one side effect is we will loop through all the virtual NICs, except the newly hot-plugged NIC, at least one more time. 
    Can you please see if we can avoid repeating the command 'ip address show' for the virtual NICs that were already verified to be not matching.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @blueorangutan test centos7 vmware55u3


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by murali-reddy <gi...@git.apache.org>.
Github user murali-reddy commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @serg38 Though same code path, i ran into different issue. When i was adding new tier to VPC, eth3 was plugged into VR from vCenter. As commented in the function there is delay for ethernet device to appear in the guest instance.
    
    I could see code loops through eth0, eth1, eth2 only, because it read the '/proc/sys/net/ipv4/conf' too early.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @murali-reddy a Trillian-Jenkins test job (centos7 mgmt + vmware55u3) 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1681: CLOUDSTACK-9491: incorrect parsing of device list to...

Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1681
  
    @jburwell 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---