{"action":"created","issue":{"url":"https://api.github.com/repos/apache/cloudstack/issues/280","labels_url":"https://api.github.com/repos/apache/cloudstack/issues/280/labels{/name}","comments_url":"https://api.github.com/repos/apache/cloudstack/issues/280/comments","events_url":"https://api.github.com/repos/apache/cloudstack/issues/280/events","html_url":"https://github.com/apache/cloudstack/pull/280","id":78973020,"number":280,"title":"CLOUDSTACK-8502 Improvement/annotations kvm citrix","user":{"login":"wilderrodrigues","id":5129209,"avatar_url":"https://avatars.githubusercontent.com/u/5129209?v=3","gravatar_id":"","url":"https://api.github.com/users/wilderrodrigues","html_url":"https://github.com/wilderrodrigues","followers_url":"https://api.github.com/users/wilderrodrigues/followers","following_url":"https://api.github.com/users/wilderrodrigues/following{/other_user}","gists_url":"https://api.github.com/users/wilderrodrigues/gists{/gist_id}","starred_url":"https://api.github.com/users/wilderrodrigues/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wilderrodrigues/subscriptions","organizations_url":"https://api.github.com/users/wilderrodrigues/orgs","repos_url":"https://api.github.com/users/wilderrodrigues/repos","events_url":"https://api.github.com/users/wilderrodrigues/events{/privacy}","received_events_url":"https://api.github.com/users/wilderrodrigues/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-21T12:25:59Z","updated_at":"2015-05-22T10:15:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/cloudstack/pulls/280","html_url":"https://github.com/apache/cloudstack/pull/280","diff_url":"https://github.com/apache/cloudstack/pull/280.diff","patch_url":"https://github.com/apache/cloudstack/pull/280.patch"},"body":"@mlsorensen, I implemented the Annotations as you suggested. Thanks for the hints. ;)\r\n\r\n@bhaisaab, I'm now testing Xen 6.2 and KVM with the changes in this PR.\r\n\r\nAll unit tests (262 for both plugins) are green! Once the integration tests are done, I will send the report.\r\n\r\nCheers,\r\nWilder"},"comment":{"url":"https://api.github.com/repos/apache/cloudstack/issues/comments/104608654","html_url":"https://github.com/apache/cloudstack/pull/280#issuecomment-104608654","issue_url":"https://api.github.com/repos/apache/cloudstack/issues/280","id":104608654,"user":{"login":"wilderrodrigues","id":5129209,"avatar_url":"https://avatars.githubusercontent.com/u/5129209?v=3","gravatar_id":"","url":"https://api.github.com/users/wilderrodrigues","html_url":"https://github.com/wilderrodrigues","followers_url":"https://api.github.com/users/wilderrodrigues/followers","following_url":"https://api.github.com/users/wilderrodrigues/following{/other_user}","gists_url":"https://api.github.com/users/wilderrodrigues/gists{/gist_id}","starred_url":"https://api.github.com/users/wilderrodrigues/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wilderrodrigues/subscriptions","organizations_url":"https://api.github.com/users/wilderrodrigues/orgs","repos_url":"https://api.github.com/users/wilderrodrigues/repos","events_url":"https://api.github.com/users/wilderrodrigues/events{/privacy}","received_events_url":"https://api.github.com/users/wilderrodrigues/received_events","type":"User","site_admin":false},"created_at":"2015-05-22T10:15:12Z","updated_at":"2015-05-22T10:15:12Z","body":"@bhaisaab @mlsorensen\r\n\r\nIf you are okay with the result, I will merge the PR this afternoon.\r\n\r\nEnvironments:\r\n\r\nCentos7 + KVM + Qemu\r\nMySQL/MariaDB 5.5.41\r\nManagement Server running CentOS 7\r\nComponent/Smoke tests: https://github.com/apache/cloudstack (test/integration/component and smoke)\r\nStorage type: NFS shared\r\nIsolation method: VLAN\r\nAgent: 4.6.0\r\n\r\nXenServer 6.2\r\nMySQL/MariaDB 5.5.41\r\nManagement Server running CentOS 7\r\nComponent/Smoke tests: https://github.com/apache/cloudstack (test/integration/component and smoke)\r\nStorage type: NFS shared\r\nIsolation method: VLAN\r\n\r\n\r\n================== KVM tests ==================\r\n\r\nTest advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===\r\nok\r\nTest Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===\r\nok\r\nTest Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===\r\nok\r\nTest Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===\r\nok\r\nTest Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===\r\nok\r\nTest Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===\r\nok\r\nTest destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===\r\nok\r\nTest recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===\r\nok\r\nTest migrate VM ... SKIP: At least two hosts should be present in the zone for migration\r\nTest destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===\r\nok\r\n\r\n----------------------------------------------------------------------\r\nRan 10 tests in 2698.252s\r\n\r\nOK (SKIP=1)\r\n/tmp//MarvinLogs/test_vm_life_cycle_DZNVX5/results.txt (END)\r\n\r\n\r\nTest Create Account and user for that account ... === TestName: test_01_create_account | Status : SUCCESS ===\r\nok\r\nTest Sub domain allowed to launch VM  when a Domain ... === TestName: test_01_add_vm_to_subdomain | Status : SUCCESS ===\r\nok\r\nTest delete domain without force option ... === TestName: test_DeleteDomain | Status : SUCCESS ===\r\nok\r\nTest delete domain with force option ... === TestName: test_forceDeleteDomain | Status : SUCCESS ===\r\nok\r\nTest to verify Non Root admin previleges ... === TestName: test_01_non_root_admin_Privileges | Status : SUCCESS ===\r\nok\r\nTest Remove one user from the account ... === TestName: test_01_user_remove_VM_running | Status : SUCCESS ===\r\nok\r\nTest to verify service offerings at same level in hierarchy ... === TestName: test_01_service_offering_hierarchy | Status : SUCCESS ===\r\nok\r\nTest to verify service offerings at same level in hierarchy ... === TestName: test_01_service_offering_siblings | Status : SUCCESS ===\r\nok\r\nTest to verify template at same level in hierarchy ... === TestName: test_01_template_hierarchy | Status : SUCCESS ===\r\nok\r\nTest update admin details ... === TestName: test_updateAdminDetails | Status : SUCCESS ===\r\nok\r\nTest update domain admin details ... === TestName: test_updateDomainAdminDetails | Status : SUCCESS ===\r\nok\r\nTest user update API ... === TestName: test_updateUserDetails | Status : SUCCESS ===\r\nok\r\nTest login API with domain ... === TestName: test_LoginApiDomain | Status : SUCCESS ===\r\nok\r\nTest if Login API does not return UUID's ... === TestName: test_LoginApiUuidResponse | Status : SUCCESS ===\r\nok\r\n\r\n----------------------------------------------------------------------\r\nRan 14 tests in 6499.442s\r\n\r\nOK\r\n/tmp//MarvinLogs/test_accounts_8RTDXH/results.txt (END)\r\n\r\n\r\n================== XenServer 6.2 tests ==================\r\n\r\nTest advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===\r\nok\r\nTest Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===\r\nok\r\nTest Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===\r\nok\r\nTest Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===\r\nok\r\nTest Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===\r\nok\r\nTest Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===\r\nok\r\nTest destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===\r\nok\r\nTest recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===\r\nok\r\nTest migrate VM ... SKIP: At least two hosts should be present in the zone for migration\r\nTest destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===\r\nok\r\n\r\n----------------------------------------------------------------------\r\nRan 10 tests in 872.307s\r\n\r\nOK (SKIP=1)\r\n/tmp//MarvinLogs/test_vm_life_cycle_D6HQHB/results.txt (END)\r\n\r\n\r\nTest Create Account and user for that account ... === TestName: test_01_create_account | Status : SUCCESS ===\r\nok\r\nTest Sub domain allowed to launch VM  when a Domain ... === TestName: test_01_add_vm_to_subdomain | Status : SUCCESS ===\r\nok\r\nTest delete domain without force option ... === TestName: test_DeleteDomain | Status : SUCCESS ===\r\nok\r\nTest delete domain with force option ... === TestName: test_forceDeleteDomain | Status : SUCCESS ===\r\nok\r\nTest to verify Non Root admin previleges ... === TestName: test_01_non_root_admin_Privileges | Status : SUCCESS ===\r\nok\r\nTest Remove one user from the account ... === TestName: test_01_user_remove_VM_running | Status : SUCCESS ===\r\nok\r\nTest to verify service offerings at same level in hierarchy ... === TestName: test_01_service_offering_hierarchy | Status : SUCCESS ===\r\nok\r\nTest to verify service offerings at same level in hierarchy ... === TestName: test_01_service_offering_siblings | Status : SUCCESS ===\r\nok\r\nTest to verify template at same level in hierarchy ... === TestName: test_01_template_hierarchy | Status : SUCCESS ===\r\nok\r\nTest update admin details ... === TestName: test_updateAdminDetails | Status : SUCCESS ===\r\nok\r\nTest update domain admin details ... === TestName: test_updateDomainAdminDetails | Status : SUCCESS ===\r\nok\r\nTest user update API ... === TestName: test_updateUserDetails | Status : SUCCESS ===\r\nok\r\nTest login API with domain ... === TestName: test_LoginApiDomain | Status : SUCCESS ===\r\nok\r\nTest if Login API does not return UUID's ... === TestName: test_LoginApiUuidResponse | Status : SUCCESS ===\r\nok\r\n\r\n----------------------------------------------------------------------\r\nRan 14 tests in 1320.486s\r\n\r\nOK\r\n/tmp//MarvinLogs/test_accounts_1TSTJ1/results.txt (END)"}}