{"action":"created","issue":{"url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/69","labels_url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/69/labels{/name}","comments_url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/69/comments","events_url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/69/events","html_url":"https://github.com/dice-cyfronet/atmosphere/issues/69","id":53376197,"number":69,"title":"Move destroy VM on compute site from VM into VmDestroyWorker","user":{"login":"mkasztelnik","id":1265430,"avatar_url":"https://avatars.githubusercontent.com/u/1265430?v=3","gravatar_id":"","url":"https://api.github.com/users/mkasztelnik","html_url":"https://github.com/mkasztelnik","followers_url":"https://api.github.com/users/mkasztelnik/followers","following_url":"https://api.github.com/users/mkasztelnik/following{/other_user}","gists_url":"https://api.github.com/users/mkasztelnik/gists{/gist_id}","starred_url":"https://api.github.com/users/mkasztelnik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkasztelnik/subscriptions","organizations_url":"https://api.github.com/users/mkasztelnik/orgs","repos_url":"https://api.github.com/users/mkasztelnik/repos","events_url":"https://api.github.com/users/mkasztelnik/events{/privacy}","received_events_url":"https://api.github.com/users/mkasztelnik/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dice-cyfronet/atmosphere/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/dice-cyfronet/atmosphere/labels/question","name":"question","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-05T09:01:29Z","updated_at":"2015-01-05T12:57:33Z","closed_at":null,"body":"Right now all logic responsible for destroying VM on compute site is located inside `Atmosphere::VirtualMachine` model class. I propose to move this logic from `Atmosphere::VirtualMachine` into `Atmosphere::Cloud::VmDestroyWorker`.  As a conclusion ORM VM model manipulation will not have any side effects.\r\n\r\n@tbartynski, @nowakowski, @bwilk  - what do you think?"},"comment":{"url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/comments/68704335","html_url":"https://github.com/dice-cyfronet/atmosphere/issues/69#issuecomment-68704335","issue_url":"https://api.github.com/repos/dice-cyfronet/atmosphere/issues/69","id":68704335,"user":{"login":"nowakowski","id":4568385,"avatar_url":"https://avatars.githubusercontent.com/u/4568385?v=3","gravatar_id":"","url":"https://api.github.com/users/nowakowski","html_url":"https://github.com/nowakowski","followers_url":"https://api.github.com/users/nowakowski/followers","following_url":"https://api.github.com/users/nowakowski/following{/other_user}","gists_url":"https://api.github.com/users/nowakowski/gists{/gist_id}","starred_url":"https://api.github.com/users/nowakowski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nowakowski/subscriptions","organizations_url":"https://api.github.com/users/nowakowski/orgs","repos_url":"https://api.github.com/users/nowakowski/repos","events_url":"https://api.github.com/users/nowakowski/events{/privacy}","received_events_url":"https://api.github.com/users/nowakowski/received_events","type":"User","site_admin":false},"created_at":"2015-01-05T12:57:33Z","updated_at":"2015-01-05T12:57:33Z","body":"Yes, this seems sensible."}}