{"action":"created","issue":{"url":"https://api.github.com/repos/juju/juju/issues/3813","labels_url":"https://api.github.com/repos/juju/juju/issues/3813/labels{/name}","comments_url":"https://api.github.com/repos/juju/juju/issues/3813/comments","events_url":"https://api.github.com/repos/juju/juju/issues/3813/events","html_url":"https://github.com/juju/juju/pull/3813","id":118686562,"number":3813,"title":"fix bug 1517344","user":{"login":"natefinch","id":3185864,"avatar_url":"https://avatars.githubusercontent.com/u/3185864?v=3","gravatar_id":"","url":"https://api.github.com/users/natefinch","html_url":"https://github.com/natefinch","followers_url":"https://api.github.com/users/natefinch/followers","following_url":"https://api.github.com/users/natefinch/following{/other_user}","gists_url":"https://api.github.com/users/natefinch/gists{/gist_id}","starred_url":"https://api.github.com/users/natefinch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natefinch/subscriptions","organizations_url":"https://api.github.com/users/natefinch/orgs","repos_url":"https://api.github.com/users/natefinch/repos","events_url":"https://api.github.com/users/natefinch/events{/privacy}","received_events_url":"https://api.github.com/users/natefinch/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-24T19:35:39Z","updated_at":"2015-11-25T02:52:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/juju/juju/pulls/3813","html_url":"https://github.com/juju/juju/pull/3813","diff_url":"https://github.com/juju/juju/pull/3813.diff","patch_url":"https://github.com/juju/juju/pull/3813.patch"},"body":"This fixes https://pad.lv/1517344 - storage not getting assigned to first unit of deployed service.\r\n\r\nTurns out the code to assign storage was reading the storage information out of the database... but because the service hasn't been deployed yet, it would just return an empty set of storage constraints.\r\n\r\nThis fix just passes in the storage constraints from outside the function addUnitOps function.  Also updated a storage test so that we actually check that a unit deployed with a service gets the correct storage assigned.  This was not previously being checked.\n\n(Review request: http://reviews.vapour.ws/r/3224/)"},"comment":{"url":"https://api.github.com/repos/juju/juju/issues/comments/159471885","html_url":"https://github.com/juju/juju/pull/3813#issuecomment-159471885","issue_url":"https://api.github.com/repos/juju/juju/issues/3813","id":159471885,"user":{"login":"natefinch","id":3185864,"avatar_url":"https://avatars.githubusercontent.com/u/3185864?v=3","gravatar_id":"","url":"https://api.github.com/users/natefinch","html_url":"https://github.com/natefinch","followers_url":"https://api.github.com/users/natefinch/followers","following_url":"https://api.github.com/users/natefinch/following{/other_user}","gists_url":"https://api.github.com/users/natefinch/gists{/gist_id}","starred_url":"https://api.github.com/users/natefinch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natefinch/subscriptions","organizations_url":"https://api.github.com/users/natefinch/orgs","repos_url":"https://api.github.com/users/natefinch/repos","events_url":"https://api.github.com/users/natefinch/events{/privacy}","received_events_url":"https://api.github.com/users/natefinch/received_events","type":"User","site_admin":false},"created_at":"2015-11-25T02:52:53Z","updated_at":"2015-11-25T02:52:53Z","body":"$$merge$$"}}