{"action":"created","issue":{"url":"https://api.github.com/repos/apache/cloudstack/issues/873","repository_url":"https://api.github.com/repos/apache/cloudstack","labels_url":"https://api.github.com/repos/apache/cloudstack/issues/873/labels{/name}","comments_url":"https://api.github.com/repos/apache/cloudstack/issues/873/comments","events_url":"https://api.github.com/repos/apache/cloudstack/issues/873/events","html_url":"https://github.com/apache/cloudstack/pull/873","id":107677870,"number":873,"title":"CLOUDSTACK-8896: allocated percentage of storage pool going beyond 100%","user":{"login":"karuturi","id":186833,"avatar_url":"https://avatars.githubusercontent.com/u/186833?v=3","gravatar_id":"","url":"https://api.github.com/users/karuturi","html_url":"https://github.com/karuturi","followers_url":"https://api.github.com/users/karuturi/followers","following_url":"https://api.github.com/users/karuturi/following{/other_user}","gists_url":"https://api.github.com/users/karuturi/gists{/gist_id}","starred_url":"https://api.github.com/users/karuturi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karuturi/subscriptions","organizations_url":"https://api.github.com/users/karuturi/orgs","repos_url":"https://api.github.com/users/karuturi/repos","events_url":"https://api.github.com/users/karuturi/events{/privacy}","received_events_url":"https://api.github.com/users/karuturi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":15,"created_at":"2015-09-22T09:39:49Z","updated_at":"2016-12-01T07:27:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/cloudstack/pulls/873","html_url":"https://github.com/apache/cloudstack/pull/873","diff_url":"https://github.com/apache/cloudstack/pull/873.diff","patch_url":"https://github.com/apache/cloudstack/pull/873.patch"},"body":"This issue occurs when a volume in Ready state is moved across storage\npools.\n\nWhile finding if the storage pool has enough space, it has a check to\nconsider the size of non Ready volumes only. This is true if the volume\nto be attached to a vm is in the same storage pool. But, if the volume\nis in another storage pool and has to be moved to a vm's storage pool,\nthe size of the volume should be considered in doing the space check.\n\ncomputing the asking size when volume is not in ready state or when the \nvolume is on a different storage pool.\n\nTesting:\nI couldnt write unittests for it. This class is not in a unittestable state.\n\nmanually tested in the below environment\n1. xenserver 6.5 setup with 2 clusters and a host each in each of them.\n2. added storage tags for the primary storage.\n3. created two service offerings with the storage tags.\n4. deployed two vms using newly created offerings in step 3.\n5. at this stage, there are two vms one on each host with root disks on the corresponding primary.\n6. create a data disk and attach it to vm1\n7. detach the data disk. now the data disk is in the primary storage of the cluster of vm1 (let us say primary1)\n8. attach this data disk to vm2(running on a host in different cluster)\n9. the volume should be moved to the primary storage of another cluster and op_host_capacity should be accordingly updated.\n"},"comment":{"url":"https://api.github.com/repos/apache/cloudstack/issues/comments/264097907","html_url":"https://github.com/apache/cloudstack/pull/873#issuecomment-264097907","issue_url":"https://api.github.com/repos/apache/cloudstack/issues/873","id":264097907,"user":{"login":"rhtyd","id":95203,"avatar_url":"https://avatars.githubusercontent.com/u/95203?v=3","gravatar_id":"","url":"https://api.github.com/users/rhtyd","html_url":"https://github.com/rhtyd","followers_url":"https://api.github.com/users/rhtyd/followers","following_url":"https://api.github.com/users/rhtyd/following{/other_user}","gists_url":"https://api.github.com/users/rhtyd/gists{/gist_id}","starred_url":"https://api.github.com/users/rhtyd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhtyd/subscriptions","organizations_url":"https://api.github.com/users/rhtyd/orgs","repos_url":"https://api.github.com/users/rhtyd/repos","events_url":"https://api.github.com/users/rhtyd/events{/privacy}","received_events_url":"https://api.github.com/users/rhtyd/received_events","type":"User","site_admin":false},"created_at":"2016-12-01T07:27:43Z","updated_at":"2016-12-01T07:27:43Z","body":"@blueorangutan package"}}
{"actor":{"display_login":"rhtyd"}}