You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by borisstoyanov <gi...@git.apache.org> on 2016/11/30 09:52:11 UTC

[GitHub] cloudstack pull request #1800: CLOUDSTACK-9633:test_snapshot is failing due ...

GitHub user borisstoyanov opened a pull request:

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

    CLOUDSTACK-9633:test_snapshot is failing due to incorrect string cons\u2026

    \u2026truction in utils.py

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

    $ git pull https://github.com/shapeblue/cloudstack 4.8-marvin-fix-for-snapshots-xenserver

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

    https://github.com/apache/cloudstack/pull/1800.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 #1800
    
----
commit dfae673e7fae02ccb9d3b8a5d7d0fb202f4d6d1d
Author: Boris <bs...@gmail.com>
Date:   2016-11-30T09:38:12Z

    CLOUDSTACK-9633:test_snapshot is failing due to incorrect string construction in utils.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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) 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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by abhinandanprateek <gi...@git.apache.org>.

Github user abhinandanprateek commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @rhtyd this is good.


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by jburwell <gi...@git.apache.org>.

Github user jburwell commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @borisstoyanov there is a Jenkins failure, but the results page returns a 404.  Could you please re-push this PR to re-trigger the Jenkins/Travis run by running `git commit --amend --no-edit && git push -f`?   


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

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


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @blueorangutan test centos7 xenserver-65sp1


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    <b>Trillian test result (tid-509)</b>
    Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7
    Total time taken: 33518 seconds
    Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1800-t509-xenserver-65sp1.zip
    Test completed. 40 look ok, 3 have error(s)
    
    
    Test | Result | Time (s) | Test File
    --- | --- | --- | ---
    test_05_rvpc_multi_tiers | `Failure` | 504.06 | test_vpc_redundant.py
    test_04_rvpc_network_garbage_collector_nics | `Failure` | 1342.50 | test_vpc_redundant.py
    test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 517.75 | test_vpc_redundant.py
    test_01_snapshot_root_disk | `Failure` | 27.27 | test_snapshots.py
    test_router_dhcp_opts | `Failure` | 21.96 | test_router_dhcphosts.py
    test_01_vpc_site2site_vpn | Success | 318.25 | test_vpc_vpn.py
    test_01_vpc_remote_access_vpn | Success | 142.09 | test_vpc_vpn.py
    test_01_redundant_vpc_site2site_vpn | Success | 540.15 | test_vpc_vpn.py
    test_02_VPC_default_routes | Success | 333.11 | test_vpc_router_nics.py
    test_01_VPC_nics_after_destroy | Success | 686.85 | test_vpc_router_nics.py
    test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 819.09 | test_vpc_redundant.py
    test_02_redundant_VPC_default_routes | Success | 1014.05 | test_vpc_redundant.py
    test_09_delete_detached_volume | Success | 20.79 | test_volumes.py
    test_08_resize_volume | Success | 106.57 | test_volumes.py
    test_07_resize_fail | Success | 111.50 | test_volumes.py
    test_06_download_detached_volume | Success | 30.48 | test_volumes.py
    test_05_detach_volume | Success | 105.41 | test_volumes.py
    test_04_delete_attached_volume | Success | 15.40 | test_volumes.py
    test_03_download_attached_volume | Success | 20.43 | test_volumes.py
    test_02_attach_volume | Success | 20.97 | test_volumes.py
    test_01_create_volume | Success | 393.48 | test_volumes.py
    test_03_delete_vm_snapshots | Success | 280.31 | test_vm_snapshots.py
    test_02_revert_vm_snapshots | Success | 186.63 | test_vm_snapshots.py
    test_01_create_vm_snapshots | Success | 137.33 | test_vm_snapshots.py
    test_deploy_vm_multiple | Success | 295.38 | test_vm_life_cycle.py
    test_deploy_vm | Success | 0.05 | test_vm_life_cycle.py
    test_advZoneVirtualRouter | Success | 0.07 | test_vm_life_cycle.py
    test_10_attachAndDetach_iso | Success | 42.10 | test_vm_life_cycle.py
    test_09_expunge_vm | Success | 125.22 | test_vm_life_cycle.py
    test_08_migrate_vm | Success | 66.41 | test_vm_life_cycle.py
    test_07_restore_vm | Success | 0.11 | test_vm_life_cycle.py
    test_06_destroy_vm | Success | 10.18 | test_vm_life_cycle.py
    test_03_reboot_vm | Success | 10.33 | test_vm_life_cycle.py
    test_02_start_vm | Success | 15.31 | test_vm_life_cycle.py
    test_01_stop_vm | Success | 30.36 | test_vm_life_cycle.py
    test_CreateTemplateWithDuplicateName | Success | 151.83 | test_templates.py
    test_08_list_system_templates | Success | 0.05 | test_templates.py
    test_07_list_public_templates | Success | 0.05 | test_templates.py
    test_05_template_permissions | Success | 0.09 | test_templates.py
    test_04_extract_template | Success | 5.24 | test_templates.py
    test_03_delete_template | Success | 5.14 | test_templates.py
    test_02_edit_template | Success | 90.09 | test_templates.py
    test_01_create_template | Success | 91.15 | test_templates.py
    test_10_destroy_cpvm | Success | 232.00 | test_ssvm.py
    test_09_destroy_ssvm | Success | 199.28 | test_ssvm.py
    test_08_reboot_cpvm | Success | 181.95 | test_ssvm.py
    test_07_reboot_ssvm | Success | 154.18 | test_ssvm.py
    test_06_stop_cpvm | Success | 141.85 | test_ssvm.py
    test_05_stop_ssvm | Success | 144.23 | test_ssvm.py
    test_04_cpvm_internals | Success | 1.24 | test_ssvm.py
    test_03_ssvm_internals | Success | 3.88 | test_ssvm.py
    test_02_list_cpvm_vm | Success | 0.16 | test_ssvm.py
    test_01_list_sec_storage_vm | Success | 0.15 | test_ssvm.py
    test_04_change_offering_small | Success | 64.15 | 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.09 | test_service_offerings.py
    test_02_sys_template_ready | Success | 0.12 | test_secondary_storage.py
    test_01_sys_vm_start | Success | 0.21 | test_secondary_storage.py
    test_01_scale_vm | Success | 5.37 | test_scale_vm.py
    test_09_reboot_router | Success | 80.90 | test_routers.py
    test_08_start_router | Success | 65.76 | test_routers.py
    test_07_stop_router | Success | 15.28 | test_routers.py
    test_06_router_advanced | Success | 0.08 | test_routers.py
    test_05_router_basic | Success | 0.04 | test_routers.py
    test_04_restart_network_wo_cleanup | Success | 5.60 | test_routers.py
    test_03_restart_network_cleanup | Success | 141.82 | test_routers.py
    test_02_router_internal_adv | Success | 0.81 | test_routers.py
    test_01_router_internal_basic | Success | 0.45 | test_routers.py
    test_router_dhcphosts | Success | 103.30 | test_router_dhcphosts.py
    test_01_updatevolumedetail | Success | 5.13 | test_resource_detail.py
    test_01_reset_vm_on_reboot | Success | 106.18 | test_reset_vm_on_reboot.py
    test_createRegion | Success | 0.04 | test_regions.py
    test_create_pvlan_network | Success | 5.60 | test_pvlan.py
    test_dedicatePublicIpRange | Success | 1.30 | test_public_ip_range.py
    test_04_rvpc_privategw_static_routes | Success | 1011.20 | test_privategw_acl.py
    test_03_vpc_privategw_restart_vpc_cleanup | Success | 949.36 | test_privategw_acl.py
    test_02_vpc_privategw_static_routes | Success | 572.31 | test_privategw_acl.py
    test_01_vpc_privategw_acl | Success | 170.00 | test_privategw_acl.py
    test_01_primary_storage_nfs | Success | 39.50 | test_primary_storage.py
    test_01_primary_storage_iscsi | Success | 83.54 | test_primary_storage.py
    test_createPortablePublicIPRange | Success | 10.43 | test_portable_publicip.py
    test_createPortablePublicIPAcquire | Success | 15.85 | test_portable_publicip.py
    test_isolate_network_password_server | Success | 36.48 | test_password_server.py
    test_UpdateStorageOverProvisioningFactor | Success | 0.30 | test_over_provisioning.py
    test_extendPhysicalNetworkVlan | Success | 15.49 | test_non_contigiousvlan.py
    test_01_nic | Success | 840.15 | test_nic.py
    test_releaseIP | Success | 290.21 | test_network.py
    test_reboot_router | Success | 617.35 | test_network.py
    test_public_ip_user_account | Success | 10.31 | test_network.py
    test_public_ip_admin_account | Success | 40.79 | test_network.py
    test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 77.18 | test_network.py
    test_network_rules_acquired_public_ip_2_nat_rule | Success | 62.12 | test_network.py
    test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 93.03 | test_network.py
    test_delete_account | Success | 296.01 | test_network.py
    test_02_port_fwd_on_non_src_nat | Success | 60.92 | test_network.py
    test_01_port_fwd_on_src_nat | Success | 79.12 | test_network.py
    test_nic_secondaryip_add_remove | Success | 195.36 | test_multipleips_per_nic.py
    login_test_saml_user | Success | 28.26 | test_login.py
    test_assign_and_removal_lb | Success | 149.83 | test_loadbalance.py
    test_02_create_lb_rule_non_nat | Success | 208.11 | test_loadbalance.py
    test_01_create_lb_rule_src_nat | Success | 208.73 | test_loadbalance.py
    test_07_list_default_iso | Success | 0.08 | test_iso.py
    test_05_iso_permissions | Success | 0.11 | test_iso.py
    test_04_extract_Iso | Success | 5.30 | test_iso.py
    test_03_delete_iso | Success | 95.18 | test_iso.py
    test_02_edit_iso | Success | 0.08 | test_iso.py
    test_01_create_iso | Success | 22.21 | test_iso.py
    test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 553.28 | test_internal_lb.py
    test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 325.58 | test_internal_lb.py
    test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 1119.53 | test_internal_lb.py
    test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 836.39 | test_internal_lb.py
    test_dedicateGuestVlanRange | Success | 10.50 | test_guest_vlan_range.py
    test_UpdateConfigParamWithScope | Success | 0.28 | test_global_settings.py
    test_04_create_fat_type_disk_offering | Success | 0.30 | test_disk_offerings.py
    test_03_delete_disk_offering | Success | 0.12 | 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.18 | test_disk_offerings.py
    test_01_create_disk_offering | Success | 0.21 | test_disk_offerings.py
    test_deployvm_userdispersing | Success | 76.50 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userconcentrated | Success | 137.38 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_firstfit | Success | 212.71 | test_deploy_vms_with_varied_deploymentplanners.py
    test_deployvm_userdata_post | Success | 21.27 | test_deploy_vm_with_userdata.py
    test_deployvm_userdata | Success | 254.69 | test_deploy_vm_with_userdata.py
    test_02_deploy_vm_root_resize | Success | 7.07 | test_deploy_vm_root_resize.py
    test_01_deploy_vm_root_resize | Success | 7.00 | test_deploy_vm_root_resize.py
    test_00_deploy_vm_root_resize | Success | 8.19 | test_deploy_vm_root_resize.py
    test_deploy_vm_from_iso | Success | 194.13 | test_deploy_vm_iso.py
    test_DeployVmAntiAffinityGroup | Success | 421.03 | test_affinity_groups.py
    test_01_test_vm_volume_snapshot | Skipped | 0.00 | test_vm_snapshots.py
    test_06_copy_template | Skipped | 0.00 | test_templates.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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    Packaging result: \u2716centos6 \u2716centos7 \u2716debian. JID-295


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @blueorangutan test centos7 xenserver-65sp1


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    I guess that needs to go only in 4.10 then @rhtyd 


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @mike-tutkowski @nvazquez @syed ping, it seems there were some changes on master due to which snapshot file names on xenserver now have `.vhd` extension which was not seen in 4.8, 4.9 branches. I think the snapshot file name should indeed have the extension, but I wanted to check with you if this is something that you may have introduced/wanted? Any thoughts on this? Thanks.


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    LGTM, we found the issue happens only in master branch (ref #1754). The fix is applicable only on master, the PR's base branch has been fixed and test has been verified manually by @borisstoyanov .


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    Thanks @syed I'll go ahead and merge this, as the changes confirm. I'm okay that moving (4.10+) fwd we've `.vhd` extension in snapshot files.


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    LGTM.
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

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


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by blueorangutan <gi...@git.apache.org>.

Github user blueorangutan commented on the issue:

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


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by rhtyd <gi...@git.apache.org>.

Github user rhtyd commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @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.
---

Re: [GitHub] cloudstack issue #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by Syed Ahmed <sa...@cloudops.com>.

I'll take a look at this. It's been some moths but afaik I added that
extension because previously that code wasn't working. I'll take a look in
detail and let you guys know
On Thu, Dec 1, 2016 at 11:34 mike-tutkowski <gi...@git.apache.org> wrote:

> Github user mike-tutkowski commented on the issue:
>
>     https://github.com/apache/cloudstack/pull/1800
>
>     Thanks, @rhtyd I've been trying to get test_snapshots.py to fail in my
> environment, but haven't been able to.
>
>     I'll let @syed answer your question, Rohit, since that is a change he
> put in.
>
>     If we do need/want to change that, let's do it in #1749 as I have that
> PR open to fix a couple issues I found with regards to snapshots on managed
> storage (specifically with regards to storing them on secondary storage, if
> that option is passed into the command).
>
>
> ---
> 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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by mike-tutkowski <gi...@git.apache.org>.

Github user mike-tutkowski commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    Thanks, @rhtyd I've been trying to get test_snapshots.py to fail in my environment, but haven't been able to.
    
    I'll let @syed answer your question, Rohit, since that is a change he put in.
    
    If we do need/want to change that, let's do it in #1749 as I have that PR open to fix a couple issues I found with regards to snapshots on managed storage (specifically with regards to storing them on secondary storage, if that option is passed into the command).


---
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 #1800: CLOUDSTACK-9633:test_snapshot is failing due to inco...

Posted by borisstoyanov <gi...@git.apache.org>.

Github user borisstoyanov commented on the issue:

    https://github.com/apache/cloudstack/pull/1800
  
    @rhtyd I've spend some time and found out what was wrong with the test_01_snapshot_root_disk 
    the issue come from the way snapshot name has been stored in the DB:
    
    - in 4.8: snapshots/2/7/84ab1c82-13e9-44cb-8978-bc4c8c38f78f
    
    - in 4.10: snapshots/89/188/dc59f069-752e-4fb3-bcdf-2dfa15b31674.vhd
    
    so this is why we're experiencing the failure at the first place with 4.10 
    I'm building a 4.9 env now to check this there as well. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---