You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by nvazquez <gi...@git.apache.org> on 2016/11/18 03:29:52 UTC
[GitHub] cloudstack pull request #1767: CLOUDSTACK-9457: Allow retrieval and modifica...
GitHub user nvazquez opened a pull request:
https://github.com/apache/cloudstack/pull/1767
CLOUDSTACK-9457: Allow retrieval and modification of VM and template details via API and UI
JIRA TICKET: https://issues.apache.org/jira/browse/CLOUDSTACK-9457
### Goal
This PR proposes list/add/update/delete user vm and vm template details via API and UI.
### New API methods
| Method name| Desc | Params|
| ------------- |:-------------:|:-----:|
| `listTemplateDetails` / `listVirtualMachineDetails` | List all details | `templateid` / `virtualmachineid` (required) |
| | | detailid (optional) |
| Method name| Desc | Params|
| ------------- |:-------------:|:-----:|
| `addTemplateDetail` / `addVirtualMachineDetail` | Add detail | `templateid` / `virtualmachineid` (required) |
| | | `detailname` (required) |
| | | `detailvalue` (required) |
| Method name| Desc | Params|
| ------------- |:-------------:|:-----:|
| `updateTemplateDetail` / `updateVirtualMachineDetail` | Update detail | `templateid` / `virtualmachineid` (required) |
| | | `detailid` (required) |
| | | `detailname` (required) |
| | | `detailvalue` (required) |
| Method name| Desc | Params|
| ------------- |:-------------:|:-----:|
| `deleteTemplateDetail` / `deleteVirtualMachineDetail` | Delete detail | `templateid` / `virtualmachineid` (required) |
| | | detailid (required) |
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nvazquez/cloudstack userVmAndTemplatesDetails
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1767.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 #1767
----
commit 04bd4a49cd71da4096f9fe05d2cb17b903cb72cb
Author: nvazquez <ni...@gmail.com>
Date: 2016-11-18T02:18:16Z
CLOUDSTACK-9457: Allow retrieval and modification of VM and template details via API and UI
----
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modifica...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1767
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@serg38 sure
@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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by nvazquez <gi...@git.apache.org>.
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@serg38 @koushik-das sure, I can work on it, the actual way is passing details as key value pairs on the update methods but I think it would require to define a way to delete details (for example the actual behaviour of `updateTemplates` overrides any existing detail for the ones provided under `details` field while on `updateVirtualMachines` they are added to the existing ones, not allowing to delete any detail)
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@nvazquez @koushik-das If we go this route we might want to change update API to behave consistently. It would make sense to allow overriding any details.
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by nvazquez <gi...@git.apache.org>.
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1767
Hi @koushik-das @ustcweizhou,
For this new methods as they require the entity id (VM or template) I added a basic validation to check if the entity is found on DB, but I realize that I should also validate that entity is not destroyed. This data is to be stored in *details tables.
About existing API methods, I have checked `updateTemplate` and `updateVirtualMachines` methods before adding this new methods, and they provide a way to add/update details (although updateTemplate overrides existing details). Also, details can be listed on `listTemplates` and `listVirtualMachines` but thought the best approach was introducing this new methods to reduce overhead and treat entity details separately instead of updating/listing the entity every time. What do you think of this approach?
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by koushik-das <gi...@git.apache.org>.
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@serg38 @nvazquez Thanks for the update. As I understand form some of the detail parameter examples, these are tied to the lifecycle of the entity (create/destroy or start/stop of the VM). If thats the case then it make sense to pass on these details along with the corresponding lifecycle APIs rather than creating new APIs. Do you see any use case where the details are not tied to entity lifecycle?
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by nvazquez <gi...@git.apache.org>.
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@karuturi @rafaelweingartner thanks! I closed Jira ticket 9379
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by rhtyd <gi...@git.apache.org>.
Github user rhtyd commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@karuturi Can we merge this one ?
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by koushik-das <gi...@git.apache.org>.
Github user koushik-das commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@nvazquez Can you add use case details that will be addressed by these new methods? Also what kind of validation will be performed on the inputs and the state of the entity (VM or template)? Will it be metadata for book keeping only or will impact the entity during runtime? I am assuming that data will be stored in the corresponding *details table in DB. How will these APIs impact already existing data that are created using some other APIs and stored in the details table?
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@koushik-das No I don't. I tend to agree with you but obviously the implementation will be messier since configs are random. @nvazquez Do you think it can be changes so that modification is done via detail tag of updateVirtuaMachine and updateTemplate APIs.
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by nvazquez <gi...@git.apache.org>.
Github user nvazquez commented on the issue:
https://github.com/apache/cloudstack/pull/1767
Hi @rhtyd @koushik-das @ustcweizhou,
I refactored this PR removing previously created API commands, only adding an optional parameter (`cleanupdetails`) on `updateVirtualMachine` and `updateTemplate` commands to allow cleaning up details. Besides that change, add/edit/remove are now performed by `updateVirtualMachine` or `updateTemplate` as you suggested. I'll post screenshots on Description to show changes on the UI.
Happy new year!
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
<b>Trillian test result (tid-776)</b>
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43663 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr1767-t776-kvm-centos7.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_templates.py
Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Test completed. 45 look ok, 4 have error(s)
Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_redundant_VPC_default_routes | `Failure` | 869.57 | test_vpc_redundant.py
test_04_rvpc_privategw_static_routes | `Failure` | 391.44 | test_privategw_acl.py
test_01_create_volume | `Error` | 279.98 | test_volumes.py
test_03_delete_template | `Error` | 5.18 | test_templates.py
test_01_vpc_site2site_vpn | Success | 195.95 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.47 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 352.03 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 453.60 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 827.87 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 537.38 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1336.10 | test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 580.77 | test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1292.65 | test_vpc_redundant.py
test_09_delete_detached_volume | Success | 156.61 | test_volumes.py
test_08_resize_volume | Success | 156.44 | test_volumes.py
test_07_resize_fail | Success | 161.53 | test_volumes.py
test_06_download_detached_volume | Success | 156.58 | test_volumes.py
test_05_detach_volume | Success | 241.36 | test_volumes.py
test_04_delete_attached_volume | Success | 151.27 | test_volumes.py
test_03_download_attached_volume | Success | 156.33 | test_volumes.py
test_02_attach_volume | Success | 185.35 | test_volumes.py
test_deploy_vm_multiple | Success | 338.47 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.04 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.03 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.72 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.22 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.98 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.14 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 125.93 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 126.01 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.20 | test_vm_life_cycle.py
test_01_stop_vm | Success | 125.91 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 70.76 | test_templates.py
test_08_list_system_templates | Success | 0.03 | test_templates.py
test_07_list_public_templates | Success | 0.04 | test_templates.py
test_05_template_permissions | Success | 0.06 | test_templates.py
test_04_extract_template | Success | 5.18 | test_templates.py
test_02_edit_template | Success | 90.16 | test_templates.py
test_01_create_template | Success | 50.54 | test_templates.py
test_10_destroy_cpvm | Success | 161.84 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.62 | test_ssvm.py
test_08_reboot_cpvm | Success | 131.73 | test_ssvm.py
test_07_reboot_ssvm | Success | 133.64 | test_ssvm.py
test_06_stop_cpvm | Success | 131.94 | test_ssvm.py
test_05_stop_ssvm | Success | 133.80 | test_ssvm.py
test_04_cpvm_internals | Success | 1.21 | test_ssvm.py
test_03_ssvm_internals | Success | 3.48 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.14 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.15 | test_ssvm.py
test_01_snapshot_root_disk | Success | 11.32 | test_snapshots.py
test_04_change_offering_small | Success | 242.88 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.04 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.06 | test_service_offerings.py
test_01_create_service_offering | Success | 0.12 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.14 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.20 | test_secondary_storage.py
test_09_reboot_router | Success | 45.40 | test_routers.py
test_08_start_router | Success | 30.31 | test_routers.py
test_07_stop_router | Success | 10.19 | test_routers.py
test_06_router_advanced | Success | 0.06 | test_routers.py
test_05_router_basic | Success | 0.05 | test_routers.py
test_04_restart_network_wo_cleanup | Success | 5.72 | test_routers.py
test_03_restart_network_cleanup | Success | 60.53 | test_routers.py
test_02_router_internal_adv | Success | 1.04 | test_routers.py
test_01_router_internal_basic | Success | 0.59 | test_routers.py
test_router_dns_guestipquery | Success | 76.94 | test_router_dns.py
test_router_dns_externalipquery | Success | 0.09 | test_router_dns.py
test_router_dhcphosts | Success | 392.96 | test_router_dhcphosts.py
test_router_dhcp_opts | Success | 21.86 | test_router_dhcphosts.py
test_01_updatevolumedetail | Success | 0.09 | test_resource_detail.py
test_01_reset_vm_on_reboot | Success | 131.02 | test_reset_vm_on_reboot.py
test_createRegion | Success | 0.05 | test_regions.py
test_create_pvlan_network | Success | 5.26 | test_pvlan.py
test_dedicatePublicIpRange | Success | 0.54 | test_public_ip_range.py
test_03_vpc_privategw_restart_vpc_cleanup | Success | 687.07 | test_privategw_acl.py
test_02_vpc_privategw_static_routes | Success | 547.01 | test_privategw_acl.py
test_01_vpc_privategw_acl | Success | 97.44 | test_privategw_acl.py
test_01_primary_storage_nfs | Success | 35.99 | test_primary_storage.py
test_createPortablePublicIPRange | Success | 15.22 | test_portable_publicip.py
test_createPortablePublicIPAcquire | Success | 15.58 | test_portable_publicip.py
test_isolate_network_password_server | Success | 86.63 | test_password_server.py
test_UpdateStorageOverProvisioningFactor | Success | 0.15 | test_over_provisioning.py
test_oobm_zchange_password | Success | 31.03 | test_outofbandmanagement.py
test_oobm_multiple_mgmt_server_ownership | Success | 16.37 | test_outofbandmanagement.py
test_oobm_issue_power_status | Success | 10.28 | test_outofbandmanagement.py
test_oobm_issue_power_soft | Success | 15.44 | test_outofbandmanagement.py
test_oobm_issue_power_reset | Success | 15.41 | test_outofbandmanagement.py
test_oobm_issue_power_on | Success | 15.44 | test_outofbandmanagement.py
test_oobm_issue_power_off | Success | 15.70 | test_outofbandmanagement.py
test_oobm_issue_power_cycle | Success | 15.78 | test_outofbandmanagement.py
test_oobm_enabledisable_across_clusterzones | Success | 92.87 | test_outofbandmanagement.py
test_oobm_enable_feature_valid | Success | 5.18 | test_outofbandmanagement.py
test_oobm_enable_feature_invalid | Success | 0.14 | test_outofbandmanagement.py
test_oobm_disable_feature_valid | Success | 0.23 | test_outofbandmanagement.py
test_oobm_disable_feature_invalid | Success | 0.15 | test_outofbandmanagement.py
test_oobm_configure_invalid_driver | Success | 0.12 | test_outofbandmanagement.py
test_oobm_configure_default_driver | Success | 0.09 | test_outofbandmanagement.py
test_oobm_background_powerstate_sync | Success | 23.48 | test_outofbandmanagement.py
test_extendPhysicalNetworkVlan | Success | 15.37 | test_non_contigiousvlan.py
test_01_nic | Success | 525.38 | test_nic.py
test_releaseIP | Success | 243.06 | test_network.py
test_reboot_router | Success | 539.68 | test_network.py
test_public_ip_user_account | Success | 10.29 | test_network.py
test_public_ip_admin_account | Success | 40.33 | test_network.py
test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Success | 67.11 | test_network.py
test_network_rules_acquired_public_ip_2_nat_rule | Success | 61.80 | test_network.py
test_network_rules_acquired_public_ip_1_static_nat_rule | Success | 124.26 | test_network.py
test_delete_account | Success | 303.17 | test_network.py
test_02_port_fwd_on_non_src_nat | Success | 55.72 | test_network.py
test_01_port_fwd_on_src_nat | Success | 111.85 | test_network.py
test_nic_secondaryip_add_remove | Success | 207.84 | test_multipleips_per_nic.py
login_test_saml_user | Success | 19.37 | test_login.py
test_assign_and_removal_lb | Success | 133.82 | test_loadbalance.py
test_02_create_lb_rule_non_nat | Success | 187.38 | test_loadbalance.py
test_01_create_lb_rule_src_nat | Success | 217.92 | test_loadbalance.py
test_03_list_snapshots | Success | 0.10 | 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.07 | test_iso.py
test_05_iso_permissions | Success | 0.07 | test_iso.py
test_04_extract_Iso | Success | 5.36 | test_iso.py
test_03_delete_iso | Success | 95.18 | test_iso.py
test_02_edit_iso | Success | 0.07 | test_iso.py
test_01_create_iso | Success | 21.11 | test_iso.py
test_04_rvpc_internallb_haproxy_stats_on_all_interfaces | Success | 213.52 | test_internal_lb.py
test_03_vpc_internallb_haproxy_stats_on_all_interfaces | Success | 188.21 | test_internal_lb.py
test_02_internallb_roundrobin_1RVPC_3VM_HTTP_port80 | Success | 526.53 | test_internal_lb.py
test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Success | 471.60 | test_internal_lb.py
test_dedicateGuestVlanRange | Success | 10.40 | test_guest_vlan_range.py
test_UpdateConfigParamWithScope | Success | 0.17 | test_global_settings.py
test_rolepermission_lifecycle_update | Success | 6.28 | test_dynamicroles.py
test_rolepermission_lifecycle_list | Success | 6.18 | test_dynamicroles.py
test_rolepermission_lifecycle_delete | Success | 5.93 | test_dynamicroles.py
test_rolepermission_lifecycle_create | Success | 5.95 | test_dynamicroles.py
test_rolepermission_lifecycle_concurrent_updates | Success | 6.09 | test_dynamicroles.py
test_role_lifecycle_update_role_inuse | Success | 5.96 | test_dynamicroles.py
test_role_lifecycle_update | Success | 11.09 | test_dynamicroles.py
test_role_lifecycle_list | Success | 5.96 | test_dynamicroles.py
test_role_lifecycle_delete | Success | 11.00 | test_dynamicroles.py
test_role_lifecycle_create | Success | 5.97 | test_dynamicroles.py
test_role_inuse_deletion | Success | 5.93 | test_dynamicroles.py
test_role_account_acls_multiple_mgmt_servers | Success | 8.64 | test_dynamicroles.py
test_role_account_acls | Success | 8.77 | test_dynamicroles.py
test_default_role_deletion | Success | 6.04 | test_dynamicroles.py
test_04_create_fat_type_disk_offering | Success | 0.10 | test_disk_offerings.py
test_03_delete_disk_offering | Success | 0.05 | test_disk_offerings.py
test_02_edit_disk_offering | Success | 0.07 | test_disk_offerings.py
test_02_create_sparse_type_disk_offering | Success | 0.09 | test_disk_offerings.py
test_01_create_disk_offering | Success | 0.14 | test_disk_offerings.py
test_deployvm_userdispersing | Success | 20.90 | test_deploy_vms_with_varied_deploymentplanners.py
test_deployvm_userconcentrated | Success | 50.95 | test_deploy_vms_with_varied_deploymentplanners.py
test_deployvm_firstfit | Success | 55.84 | test_deploy_vms_with_varied_deploymentplanners.py
test_deployvm_userdata_post | Success | 10.43 | test_deploy_vm_with_userdata.py
test_deployvm_userdata | Success | 86.02 | test_deploy_vm_with_userdata.py
test_02_deploy_vm_root_resize | Success | 6.21 | test_deploy_vm_root_resize.py
test_01_deploy_vm_root_resize | Success | 6.10 | test_deploy_vm_root_resize.py
test_00_deploy_vm_root_resize | Success | 227.97 | test_deploy_vm_root_resize.py
test_deploy_vm_from_iso | Success | 218.91 | test_deploy_vm_iso.py
test_DeployVmAntiAffinityGroup | Success | 127.53 | test_affinity_groups.py
test_03_delete_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
test_02_revert_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
test_01_test_vm_volume_snapshot | Skipped | 0.00 | test_vm_snapshots.py
test_01_create_vm_snapshots | Skipped | 0.00 | test_vm_snapshots.py
test_06_copy_template | Skipped | 0.00 | test_templates.py
test_static_role_account_acls | Skipped | 0.03 | test_staticroles.py
test_11_ss_nfs_version_on_ssvm | Skipped | 0.02 | test_ssvm.py
test_01_scale_vm | Skipped | 0.00 | test_scale_vm.py
test_01_primary_storage_iscsi | Skipped | 0.04 | test_primary_storage.py
test_nested_virtualization_vmware | Skipped | 0.00 | test_nested_virtualization.py
test_06_copy_iso | Skipped | 0.00 | test_iso.py
test_deploy_vgpu_enabled_vm | Skipped | 0.04 | test_deploy_vgpu_enabled_vm.py
test_3d_gpu_support | Skipped | 0.04 | 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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@rhtyd 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@rhtyd Can you kick off tests 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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by ustcweizhou <gi...@git.apache.org>.
Github user ustcweizhou commented on the issue:
https://github.com/apache/cloudstack/pull/1767
I have the same concerns as @koushik-das .
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-451
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by serg38 <gi...@git.apache.org>.
Github user serg38 commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@koushik-das @ustcweizhou Just want to add on the use case. There is growing need to change template/VM details for after the deployment e.g. to switch root disk controller or change ratio of cores/socket with cpuid.coresPerSocket or with adding vGPU startup parameters or switch boot firmware from bios to efi as required by some OS. Currently users are unable to easily do it with API and/or UI. As implemented now, all settings with display=1 in user_vm_details and vm_template_details will be returned by API and allowed to changed. If desired we can open another PR to switch details created by other API in automatic fashion to have display=0 so they will not show up in the response of these APIs and users wont be able to edit/delete them. We thought this to be out of scope for now.
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by rafaelweingartner <gi...@git.apache.org>.
Github user rafaelweingartner commented on the issue:
https://github.com/apache/cloudstack/pull/1767
I have just reviewed the PR. There are only small suggestions I made.
BTW: from the Jira ticket https://issues.apache.org/jira/browse/CLOUDSTACK-9457, I ended up at https://issues.apache.org/jira/browse/CLOUDSTACK-9379, which is already done. I do not have permission to mark it as done. I believe @nvazquez might have.
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
Packaging result: \u2714centos6 \u2714centos7 \u2714debian. JID-166
---
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 #1767: CLOUDSTACK-9457: Allow retrieval and modification of...
Posted by blueorangutan <gi...@git.apache.org>.
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1767
@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.
---