{"action":"created","issue":{"url":"https://api.github.com/repos/kimchi-project/ginger/issues/133","labels_url":"https://api.github.com/repos/kimchi-project/ginger/issues/133/labels{/name}","comments_url":"https://api.github.com/repos/kimchi-project/ginger/issues/133/comments","events_url":"https://api.github.com/repos/kimchi-project/ginger/issues/133/events","html_url":"https://github.com/kimchi-project/ginger/issues/133","id":125200778,"number":133,"title":"Delete physical volume fails with 500 status code","user":{"login":"sureshab","id":12499230,"avatar_url":"https://avatars.githubusercontent.com/u/12499230?v=3","gravatar_id":"","url":"https://api.github.com/users/sureshab","html_url":"https://github.com/sureshab","followers_url":"https://api.github.com/users/sureshab/followers","following_url":"https://api.github.com/users/sureshab/following{/other_user}","gists_url":"https://api.github.com/users/sureshab/gists{/gist_id}","starred_url":"https://api.github.com/users/sureshab/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sureshab/subscriptions","organizations_url":"https://api.github.com/users/sureshab/orgs","repos_url":"https://api.github.com/users/sureshab/repos","events_url":"https://api.github.com/users/sureshab/events{/privacy}","received_events_url":"https://api.github.com/users/sureshab/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/kimchi-project/ginger/labels/backend","name":"backend","color":"000000"},{"url":"https://api.github.com/repos/kimchi-project/ginger/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"danielhb","id":866780,"avatar_url":"https://avatars.githubusercontent.com/u/866780?v=3","gravatar_id":"","url":"https://api.github.com/users/danielhb","html_url":"https://github.com/danielhb","followers_url":"https://api.github.com/users/danielhb/followers","following_url":"https://api.github.com/users/danielhb/following{/other_user}","gists_url":"https://api.github.com/users/danielhb/gists{/gist_id}","starred_url":"https://api.github.com/users/danielhb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielhb/subscriptions","organizations_url":"https://api.github.com/users/danielhb/orgs","repos_url":"https://api.github.com/users/danielhb/repos","events_url":"https://api.github.com/users/danielhb/events{/privacy}","received_events_url":"https://api.github.com/users/danielhb/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/kimchi-project/ginger/milestones/3","html_url":"https://github.com/kimchi-project/ginger/milestones/Ginger2.1","labels_url":"https://api.github.com/repos/kimchi-project/ginger/milestones/3/labels","id":1473668,"number":3,"title":"Ginger2.1","description":"Next Ginger release to be available in the end of February/2016 (date can vary).","creator":{"login":"danielhb","id":866780,"avatar_url":"https://avatars.githubusercontent.com/u/866780?v=3","gravatar_id":"","url":"https://api.github.com/users/danielhb","html_url":"https://github.com/danielhb","followers_url":"https://api.github.com/users/danielhb/followers","following_url":"https://api.github.com/users/danielhb/following{/other_user}","gists_url":"https://api.github.com/users/danielhb/gists{/gist_id}","starred_url":"https://api.github.com/users/danielhb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielhb/subscriptions","organizations_url":"https://api.github.com/users/danielhb/orgs","repos_url":"https://api.github.com/users/danielhb/repos","events_url":"https://api.github.com/users/danielhb/events{/privacy}","received_events_url":"https://api.github.com/users/danielhb/received_events","type":"User","site_admin":false},"open_issues":19,"closed_issues":9,"state":"open","created_at":"2015-12-24T13:15:18Z","updated_at":"2016-01-07T13:01:41Z","due_on":"2016-02-26T03:00:00Z","closed_at":null},"comments":2,"created_at":"2016-01-06T15:39:45Z","updated_at":"2016-01-07T16:52:48Z","closed_at":null,"body":"Selected a pv which vg and executed delete api.\r\n\r\n[root@zfwcec164 ~]# pvdisplay /dev/mapper/36005076802810d504800000000002ef2 \r\n  --- Physical volume ---\r\n  PV Name               /dev/mapper/36005076802810d504800000000002ef2\r\n  VG Name               testvg3\r\n  PV Size               1.00 GiB / not usable 4.00 MiB\r\n  Allocatable           yes \r\n  PE Size               4.00 MiB\r\n  Total PE              255\r\n  Free PE               255\r\n  Allocated PE          0\r\n  PV UUID               rcnxbt-KQIS-lCbu-EbDy-K68N-K1QM-og4Wes\r\n   \r\n[root@zfwcec164 ~]# curl -k -i -u root -H \"Content-Type: application/json\" -H \"Accept: application/json\" -X DELETE 'https://127.0.0.1:8001/plugins/ginger/pvs/%2fdev%2fmapper%2f36005076802810d504800000000002ef2'\r\nEnter host password for user 'root':\r\nHTTP/1.1 500 Internal Server Error\r\n{\r\n  \"reason\":\"GINPV00005E: Failed to delete PV\",\r\n  \"code\":\"500 Internal Server Error\"\r\n}                                                                                                                                                               \r\n"},"comment":{"url":"https://api.github.com/repos/kimchi-project/ginger/issues/comments/169726212","html_url":"https://github.com/kimchi-project/ginger/issues/133#issuecomment-169726212","issue_url":"https://api.github.com/repos/kimchi-project/ginger/issues/133","id":169726212,"user":{"login":"danielhb","id":866780,"avatar_url":"https://avatars.githubusercontent.com/u/866780?v=3","gravatar_id":"","url":"https://api.github.com/users/danielhb","html_url":"https://github.com/danielhb","followers_url":"https://api.github.com/users/danielhb/followers","following_url":"https://api.github.com/users/danielhb/following{/other_user}","gists_url":"https://api.github.com/users/danielhb/gists{/gist_id}","starred_url":"https://api.github.com/users/danielhb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielhb/subscriptions","organizations_url":"https://api.github.com/users/danielhb/orgs","repos_url":"https://api.github.com/users/danielhb/repos","events_url":"https://api.github.com/users/danielhb/events{/privacy}","received_events_url":"https://api.github.com/users/danielhb/received_events","type":"User","site_admin":false},"created_at":"2016-01-07T16:52:48Z","updated_at":"2016-01-07T16:52:48Z","body":"The problem here is not that it will return 500 even if the volume doesn't exist (we return a 404 in this case) but also the error message is not informing why the operation failed. \r\n\r\nI'll fix both of these issues and send a patch to the ML."}}