You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by jayapalu <gi...@git.apache.org> on 2016/12/28 05:35:53 UTC

[GitHub] cloudstack pull request #1873: CLOUDSTACK-9709: Updated the vm ip fetch task...

GitHub user jayapalu opened a pull request:

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

    CLOUDSTACK-9709: Updated the vm ip fetch task to use the correct the \u2026

     Updated the vm ip fetch task to use the seperate  thread executor
    
    Also fixed in this
    If a vm with iso attached is restarted it would not have the attached iso after restart.
    Detect vmware tools mount status ahead to restore after reboot.

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

    $ git pull https://github.com/Accelerite/cloudstack dhcpOffloadFix

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

    https://github.com/apache/cloudstack/pull/1873.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 #1873
    
----
commit 36658a60d85e6a8926fd1367b138508fefdcb9f1
Author: Jayapal <ja...@accelerite.com>
Date:   2016-12-27T09:04:28Z

    CLOUDSTACK-9709: Updated the vm ip fetch task to use the correct the thread
    
    Also fixed in this
    If a vm with iso attached is restarted it would not have the attached iso after restart.
    Detect vmware tools mount status ahead to restore after reboot.

----


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    ### ACS CI BVT Run
     **Sumarry:**
     Build Number 389
     Hypervisor xenserver
     NetworkType Advanced
     Passed=104
     Failed=1
     Skipped=7
    
    _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
    
    
    **Failed tests:**
    * test_non_contigiousvlan.py
    
     * test_extendPhysicalNetworkVlan Failed
    
    
    **Skipped tests:**
    test_01_test_vm_volume_snapshot
    test_vm_nic_adapter_vmxnet3
    test_static_role_account_acls
    test_11_ss_nfs_version_on_ssvm
    test_nested_virtualization_vmware
    test_3d_gpu_support
    test_deploy_vgpu_enabled_vm
    
    **Passed test suits:**
    test_deploy_vm_with_userdata.py
    test_affinity_groups_projects.py
    test_portable_publicip.py
    test_over_provisioning.py
    test_global_settings.py
    test_scale_vm.py
    test_service_offerings.py
    test_routers_iptables_default_policy.py
    test_loadbalance.py
    test_routers.py
    test_reset_vm_on_reboot.py
    test_deploy_vms_with_varied_deploymentplanners.py
    test_network.py
    test_router_dns.py
    test_login.py
    test_deploy_vm_iso.py
    test_list_ids_parameter.py
    test_public_ip_range.py
    test_multipleips_per_nic.py
    test_regions.py
    test_affinity_groups.py
    test_network_acl.py
    test_pvlan.py
    test_volumes.py
    test_nic.py
    test_deploy_vm_root_resize.py
    test_resource_detail.py
    test_secondary_storage.py
    test_vm_life_cycle.py
    test_routers_network_ops.py
    test_disk_offerings.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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @blueorangutan test centos7 vmware-60u1


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    <b>Trillian test result (tid-895)</b>
    Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 43388 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1873-t895-vmware-60u2.zip
    Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
    Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.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` | 813.33 | test_privategw_acl.py
    test_02_list_snapshots_with_removed_data_store | `Error` | 70.84 | test_snapshots.py
    test_02_list_snapshots_with_removed_data_store | `Error` | 75.94 | test_snapshots.py
    test_01_vpc_site2site_vpn | Success | 341.75 | test_vpc_vpn.py
    test_01_vpc_remote_access_vpn | Success | 186.88 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | Success | 568.89 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 350.35 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 694.44 | test_vpc_router_nics.py
    test_05_rvpc_multi_tiers | Success | 597.79 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | Success | 1508.27 | test_vpc_redundant.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 704.25 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 619.57 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1306.33 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 20.75 | test_volumes.py
    test_06_download_detached_volume | Success | 40.42 | test_volumes.py
    test_05_detach_volume | Success | 100.23 | test_volumes.py
    test_04_delete_attached_volume | Success | 10.18 | test_volumes.py
    test_03_download_attached_volume | Success | 15.29 | test_volumes.py
    test_02_attach_volume | Success | 48.90 | test_volumes.py
    test_01_create_volume | Success | 509.64 | test_volumes.py
    test_change_service_offering_for_vm_with_snapshots | Success | 537.08 | test_vm_snapshots.py
    test_03_delete_vm_snapshots | Success | 275.21 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 225.17 | test_vm_snapshots.py
    test_01_test_vm_volume_snapshot | Success | 176.48 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 161.64 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 257.62 | 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.86 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.22 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 56.00 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.10 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 5.12 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 5.13 | test_vm_life_cycle.py
    test_02_start_vm | Success | 20.22 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 5.11 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 206.47 | 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.05 | test_templates.py
    test_04_extract_template | Success | 10.20 | test_templates.py
    test_03_delete_template | Success | 5.10 | test_templates.py
    test_02_edit_template | Success | 90.18 | test_templates.py
    test_01_create_template | Success | 105.79 | test_templates.py
    test_10_destroy_cpvm | Success | 196.70 | test_ssvm.py
    test_09_destroy_ssvm | Success | 238.62 | test_ssvm.py
    test_08_reboot_cpvm | Success | 156.49 | test_ssvm.py
    test_07_reboot_ssvm | Success | 188.44 | test_ssvm.py
    test_06_stop_cpvm | Success | 166.76 | test_ssvm.py
    test_05_stop_ssvm | Success | 203.69 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.20 | test_ssvm.py
    test_03_ssvm_internals | Success | 3.36 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.11 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py
    test_01_snapshot_root_disk | Success | 61.33 | test_snapshots.py
    test_04_change_offering_small | Success | 116.84 | test_service_offerings.py
    test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
    test_02_edit_service_offering | Success | 0.08 | test_service_offerings.py
    test_01_create_service_offering | Success | 0.10 | 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 | 135.88 | test_routers.py
    test_08_start_router | Success | 110.77 | test_routers.py
    test_07_stop_router | Success | 25.24 | test_routers.py
    test_06_router_advanced | Success | 0.05 | test_routers.py
    test_05_router_basic | Success | 0.04 | test_routers.py
    test_04_restart_network_wo_cleanup | Success | 5.62 | test_routers.py
    test_03_restart_network_cleanup | Success | 131.02 | test_routers.py
    test_02_router_internal_adv | Success | 1.01 | test_routers.py
    test_01_router_internal_basic | Success | 0.55 | test_routers.py
    test_router_dns_guestipquery | Success | 76.76 | test_router_dns.py
    test_router_dns_externalipquery | Success | 0.07 | test_router_dns.py
    test_router_dhcphosts | Success | 169.00 | test_router_dhcphosts.py
    test_router_dhcp_opts | Success | 21.54 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 0.08 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 35.39 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.20 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 0.42 | test_public_ip_range.py
    test_03_vpc_privategw_restart_vpc_cleanup | Success | 988.28 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 612.02 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 202.84 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 36.38 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 15.19 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.44 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 97.12 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.14 | test_over_provisioning.py
    test_oobm_zchange_password | Success | 30.76 | test_outofbandmanagement.py
    test_oobm_multiple_mgmt_server_ownership | Success | 16.34 | test_outofbandmanagement.py
    test_oobm_issue_power_status | Success | 10.23 | test_outofbandmanagement.py
    test_oobm_issue_power_soft | Success | 15.31 | test_outofbandmanagement.py
    test_oobm_issue_power_reset | Success | 15.38 | test_outofbandmanagement.py
    test_oobm_issue_power_on | Success | 15.39 | test_outofbandmanagement.py
    test_oobm_issue_power_off | Success | 15.34 | test_outofbandmanagement.py
    test_oobm_issue_power_cycle | Success | 15.35 | test_outofbandmanagement.py
    test_oobm_enabledisable_across_clusterzones | Success | 92.85 | test_outofbandmanagement.py
    test_oobm_enable_feature_valid | Success | 5.15 | test_outofbandmanagement.py
    test_oobm_enable_feature_invalid | Success | 0.09 | test_outofbandmanagement.py
    test_oobm_disable_feature_valid | Success | 5.18 | test_outofbandmanagement.py
    test_oobm_disable_feature_invalid | Success | 0.11 | test_outofbandmanagement.py
    test_oobm_configure_invalid_driver | Success | 0.08 | test_outofbandmanagement.py
    test_oobm_configure_default_driver | Success | 0.07 | test_outofbandmanagement.py
    test_oobm_background_powerstate_sync | Success | 23.41 | test_outofbandmanagement.py
    test_extendPhysicalNetworkVlan | Success | 15.35 | test_non_contigiousvlan.py
    test_01_nic | Success | 505.12 | test_nic.py
    test_releaseIP | Success | 278.20 | test_network.py
    test_reboot_router | Success | 670.84 | test_network.py
    test_public_ip_user_account | Success | 10.31 | test_network.py
    test_public_ip_admin_account | Success | 40.28 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 76.73 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.58 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 125.65 | test_network.py
    test_delete_account | Success | 262.92 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 55.62 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 111.72 | test_network.py
    test_nested_virtualization_vmware | Success | 358.62 | test_nested_virtualization.py
    test_nic_secondaryip_add_remove | Success | 182.63 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 19.21 | test_login.py
    test_assign_and_removal_lb | Success | 149.21 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 207.67 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 208.02 | test_loadbalance.py
    test_03_list_snapshots | Success | 0.13 | 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.12 | test_iso.py
    test_04_extract_Iso | Success | 5.13 | test_iso.py
    test_03_delete_iso | Success | 95.21 | test_iso.py
    test_02_edit_iso | Success | 0.08 | test_iso.py
    test_01_create_iso | Success | 21.00 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 490.61 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 364.24 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 1067.59 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 828.69 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.42 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.13 | test_global_settings.py
    test_rolepermission_lifecycle_update | Success | 6.13 | test_dynamicroles.py
    test_rolepermission_lifecycle_list | Success | 5.98 | test_dynamicroles.py
    test_rolepermission_lifecycle_delete | Success | 5.87 | test_dynamicroles.py
    test_rolepermission_lifecycle_create | Success | 5.94 | test_dynamicroles.py
    test_rolepermission_lifecycle_concurrent_updates | Success | 6.03 | test_dynamicroles.py
    test_role_lifecycle_update_role_inuse | Success | 5.89 | test_dynamicroles.py
    test_role_lifecycle_update | Success | 10.96 | test_dynamicroles.py
    test_role_lifecycle_list | Success | 5.90 | test_dynamicroles.py
    test_role_lifecycle_delete | Success | 10.92 | test_dynamicroles.py
    test_role_lifecycle_create | Success | 5.88 | test_dynamicroles.py
    test_role_inuse_deletion | Success | 5.86 | test_dynamicroles.py
    test_role_account_acls_multiple_mgmt_servers | Success | 8.39 | test_dynamicroles.py
    test_role_account_acls | Success | 8.29 | test_dynamicroles.py
    test_default_role_deletion | Success | 5.99 | test_dynamicroles.py
    test_04_create_fat_type_disk_offering | Success | 0.07 | 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.06 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.10 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 50.85 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 75.91 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 176.40 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 50.68 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 201.86 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 5.84 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 5.84 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 5.99 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 242.68 | test_deploy_vm_iso.py
    test_3d_gpu_support | Success | 395.08 | test_deploy_vgpu_enabled_vm.py
    test_DeployVmAntiAffinityGroup | Success | 161.86 | test_affinity_groups.py
    test_08_resize_volume | Skipped | 5.10 | test_volumes.py
    test_07_resize_fail | Skipped | 10.29 | 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_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
    test_01_scale_vm | Skipped | 66.33 | 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 | 1.09 | 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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    ### ACS CI BVT Run
     **Sumarry:**
     Build Number 351
     Hypervisor xenserver
     NetworkType Advanced
     Passed=105
     Failed=0
     Skipped=7
    
    _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
    
    
    **Failed tests:**
    
    **Skipped tests:**
    test_01_test_vm_volume_snapshot
    test_vm_nic_adapter_vmxnet3
    test_static_role_account_acls
    test_11_ss_nfs_version_on_ssvm
    test_nested_virtualization_vmware
    test_3d_gpu_support
    test_deploy_vgpu_enabled_vm
    
    **Passed test suits:**
    test_deploy_vm_with_userdata.py
    test_affinity_groups_projects.py
    test_portable_publicip.py
    test_over_provisioning.py
    test_global_settings.py
    test_scale_vm.py
    test_service_offerings.py
    test_routers_iptables_default_policy.py
    test_loadbalance.py
    test_routers.py
    test_reset_vm_on_reboot.py
    test_deploy_vms_with_varied_deploymentplanners.py
    test_network.py
    test_router_dns.py
    test_non_contigiousvlan.py
    test_login.py
    test_deploy_vm_iso.py
    test_list_ids_parameter.py
    test_public_ip_range.py
    test_multipleips_per_nic.py
    test_regions.py
    test_affinity_groups.py
    test_network_acl.py
    test_pvlan.py
    test_volumes.py
    test_nic.py
    test_deploy_vm_root_resize.py
    test_resource_detail.py
    test_secondary_storage.py
    test_vm_life_cycle.py
    test_routers_network_ops.py
    test_disk_offerings.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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @blueorangutan test centos7 vmware-60u2


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) 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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @jayapalu Thanks. Have you made changes for kvm ? I cannot find GetVmIpAddressCommand in plugins/hypervisors/kvm(actually only find it in plugins/hypervisors/xenserver/)


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @ustcweizhou changes for kvm were in new file. It is missed while commiting, added now. Thanks for catching. 


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @ustcweizhou  I will add disable and CLOUDSTACK-9413 in new feature.


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    Jenkins and travis are failing because of its internal failures.
    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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    ### ACS CI BVT Run
     **Sumarry:**
     Build Number 404
     Hypervisor xenserver
     NetworkType Advanced
     Passed=105
     Failed=0
     Skipped=7
    
    _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
    
    
    **Failed tests:**
    
    **Skipped tests:**
    test_01_test_vm_volume_snapshot
    test_vm_nic_adapter_vmxnet3
    test_static_role_account_acls
    test_11_ss_nfs_version_on_ssvm
    test_nested_virtualization_vmware
    test_3d_gpu_support
    test_deploy_vgpu_enabled_vm
    
    **Passed test suits:**
    test_deploy_vm_with_userdata.py
    test_affinity_groups_projects.py
    test_portable_publicip.py
    test_over_provisioning.py
    test_global_settings.py
    test_scale_vm.py
    test_service_offerings.py
    test_routers_iptables_default_policy.py
    test_loadbalance.py
    test_routers.py
    test_reset_vm_on_reboot.py
    test_deploy_vms_with_varied_deploymentplanners.py
    test_network.py
    test_router_dns.py
    test_non_contigiousvlan.py
    test_login.py
    test_deploy_vm_iso.py
    test_list_ids_parameter.py
    test_public_ip_range.py
    test_multipleips_per_nic.py
    test_regions.py
    test_affinity_groups.py
    test_network_acl.py
    test_pvlan.py
    test_volumes.py
    test_nic.py
    test_deploy_vm_root_resize.py
    test_resource_detail.py
    test_secondary_storage.py
    test_vm_life_cycle.py
    test_routers_network_ops.py
    test_disk_offerings.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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @jayapalu  LGTM
    
    I have two questions
    (1) Is it possible to disable the check ?
    (2) Could you please fix CLOUDSTACK-9413 (with this PR or in new 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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    There are not failed test cases on CI run.


---
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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @borisstoyanov 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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @ustcweizhou can you review the updated 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    Two LGTMs and no test failures from cloudmonger  test results.
    @ustcweizhou waited for your comments for this, I am putting the mergeready tag.  If you have anything to bring up let me know.
    
    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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    code LGTM
    
    one small suggestion: 
    Could you modify line 2253 (_vmIpFetchThreadExecutor.execute) to be executed only when hypervisor is XenServer(already supported) or Vmware (supported by 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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    @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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

    https://github.com/apache/cloudstack/pull/1873
  
    ### ACS CI BVT Run
     **Sumarry:**
     Build Number 437
     Hypervisor xenserver
     NetworkType Advanced
     Passed=105
     Failed=0
     Skipped=7
    
    _Link to logs Folder (search by build_no):_ https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
    
    
    **Failed tests:**
    
    **Skipped tests:**
    test_01_test_vm_volume_snapshot
    test_vm_nic_adapter_vmxnet3
    test_static_role_account_acls
    test_11_ss_nfs_version_on_ssvm
    test_nested_virtualization_vmware
    test_3d_gpu_support
    test_deploy_vgpu_enabled_vm
    
    **Passed test suits:**
    test_deploy_vm_with_userdata.py
    test_affinity_groups_projects.py
    test_portable_publicip.py
    test_over_provisioning.py
    test_global_settings.py
    test_scale_vm.py
    test_service_offerings.py
    test_routers_iptables_default_policy.py
    test_loadbalance.py
    test_routers.py
    test_reset_vm_on_reboot.py
    test_deploy_vms_with_varied_deploymentplanners.py
    test_network.py
    test_router_dns.py
    test_non_contigiousvlan.py
    test_login.py
    test_deploy_vm_iso.py
    test_list_ids_parameter.py
    test_public_ip_range.py
    test_multipleips_per_nic.py
    test_regions.py
    test_affinity_groups.py
    test_network_acl.py
    test_pvlan.py
    test_volumes.py
    test_nic.py
    test_deploy_vm_root_resize.py
    test_resource_detail.py
    test_secondary_storage.py
    test_vm_life_cycle.py
    test_routers_network_ops.py
    test_disk_offerings.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 #1873: CLOUDSTACK-9709: Updated the vm ip fetch task to use...

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

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