[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1747 merging --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @karuturi Test passed. Two LGTM received. Looks like it is ready for merging --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Trillian test result (tid-808) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 32193 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1747-t808-kvm-centos7.zip Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.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_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 368.45 | test_vpc_redundant.py test_04_rvpc_privategw_static_routes | `Failure` | 345.44 | test_privategw_acl.py test_01_vpc_site2site_vpn | Success | 160.24 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 66.14 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 256.13 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 269.86 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 543.88 | test_vpc_router_nics.py test_05_rvpc_multi_tiers | Success | 511.03 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | Success | 1398.83 | test_vpc_redundant.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 563.54 | test_vpc_redundant.py test_02_redundant_VPC_default_routes | Success | 760.97 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 156.51 | test_volumes.py test_08_resize_volume | Success | 156.41 | test_volumes.py test_07_resize_fail | Success | 161.61 | test_volumes.py test_06_download_detached_volume | Success | 156.27 | test_volumes.py test_05_detach_volume | Success | 155.82 | test_volumes.py test_04_delete_attached_volume | Success | 151.20 | test_volumes.py test_03_download_attached_volume | Success | 156.32 | test_volumes.py test_02_attach_volume | Success | 94.87 | test_volumes.py test_01_create_volume | Success | 711.13 | test_volumes.py test_03_delete_vm_snapshots | Success | 275.21 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 100.69 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 163.79 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 247.55 | 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.60 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.25 | test_vm_life_cycle.py test_08_migrate_vm | Success | 40.89 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.12 | test_vm_life_cycle.py test_06_destroy_vm | Success | 125.81 | test_vm_life_cycle.py test_03_reboot_vm | Success | 125.84 | test_vm_life_cycle.py test_02_start_vm | Success | 10.17 | test_vm_life_cycle.py test_01_stop_vm | Success | 40.31 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 35.45 | 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 | 5.15 | 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 | 25.32 | test_templates.py test_10_destroy_cpvm | Success | 131.61 | test_ssvm.py test_09_destroy_ssvm | Success | 163.68 | test_ssvm.py test_08_reboot_cpvm | Success | 131.59 | test_ssvm.py test_07_reboot_ssvm | Success | 133.60 | test_ssvm.py test_06_stop_cpvm | Success | 161.79 | test_ssvm.py test_05_stop_ssvm | Success | 133.66 | test_ssvm.py test_04_cpvm_internals | Success | 1.20 | test_ssvm.py test_03_ssvm_internals | Success | 3.32 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.12 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.13 | test_ssvm.py test_01_snapshot_root_disk | Success | 11.22 | test_snapshots.py test_04_change_offering_small | Success | 239.60 | test_service_offerings.py test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py test_02_edit_service_offering | Success | 0.05 | test_service_offerings.py test_01_create_service_offering | Success | 0.11 | test_service_offerings.py test_02_sys_template_ready | Success | 0.12 | test_secondary_storage.py test_01_sys_vm_start | Success | 0.17 | test_secondary_storage.py test_09_reboot_router | Success | 35.29 | test_routers.py test_08_start_router | Success | 30.28 | test_routers.py test_07_stop_router | Success | 10.70 | test_routers.py
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Packaging result: âcentos6 âcentos7 âdebian. JID-476 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1747 @blueorangutan test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1747 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner Do we get you LGTM? @DaanHoogland @rhtyd Can we kick off tests for this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @serg38 got it. Thanks for the explanation --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner This PR is two fold. 1. This is something that was done a long ago for hosts and was never implemented for storage. It transitions location of of storage tags over to dedicated tables (storage_pool_tags) and converts current storage_tags to the new location so there should be no impact on current installation 2. For secondary storage there is no need to show details since they are coming from configurations that now support ImageStore as Scope via PR1615. With that Details tag becomes obsolete. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @nvazquez, @serg38, before checking code any deeper a have a more high-level question. This PR will change the response of an API method, right? It will remove some data that today is being delivered to users. If I understood it properly, this data (storage tags) will be delivered/retrieved using another method. This may break compatibility with current clients that may use this information. If so, this break in compatibility has to be properly documented. Do you guys know how we could do this? Is there a protocol for this? Some alert on the release notes? Or maybe something on documentation pages? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner Can you review 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd @karuturi Can we run tests on this PR and merge on success ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Packaging result: âcentos6 âcentos7 âdebian. JID-455 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1747 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user nvazquez commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd Done, 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1747 @nvazquez can you squash your change and see why Travis failed, in case of intermittent issue, push -f to rekick Travis. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rhtyd Can you kick off tests fro 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 Packaging result: âcentos6 âcentos7 âdebian. JID-180 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1747 @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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1747 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---