{"action":"created","issue":{"url":"https://api.github.com/repos/juju/juju/issues/3273","labels_url":"https://api.github.com/repos/juju/juju/issues/3273/labels{/name}","comments_url":"https://api.github.com/repos/juju/juju/issues/3273/comments","events_url":"https://api.github.com/repos/juju/juju/issues/3273/events","html_url":"https://github.com/juju/juju/pull/3273","id":106291323,"number":3273,"title":"{api/apiserver}/storage: update StorageDetails to match model","user":{"login":"axw","id":843579,"avatar_url":"https://avatars.githubusercontent.com/u/843579?v=3","gravatar_id":"","url":"https://api.github.com/users/axw","html_url":"https://github.com/axw","followers_url":"https://api.github.com/users/axw/followers","following_url":"https://api.github.com/users/axw/following{/other_user}","gists_url":"https://api.github.com/users/axw/gists{/gist_id}","starred_url":"https://api.github.com/users/axw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/axw/subscriptions","organizations_url":"https://api.github.com/users/axw/orgs","repos_url":"https://api.github.com/users/axw/repos","events_url":"https://api.github.com/users/axw/events{/privacy}","received_events_url":"https://api.github.com/users/axw/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-09-14T08:10:05Z","updated_at":"2015-09-15T03:06:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/juju/juju/pulls/3273","html_url":"https://github.com/juju/juju/pull/3273","diff_url":"https://github.com/juju/juju/pull/3273.diff","patch_url":"https://github.com/juju/juju/pull/3273.patch"},"body":"(NOTE: the first commit is a cherry-pick from master, fixing up the VolumeDetails params struct. There were no conflicts, this does not need to be reviewed. I suggest reviewing the individual commits, rather than the full diff on RB.)\r\n\r\nThe second commit is a drive-by cleanup, removing a redundant type (params.StorageInfo).\r\n\r\nThe third commit is where the meat is. This is the first in a series of changes to fix the \"storage\" API facade and storage CLI commands to better reflect the model. This is necessary to cater for future changes, to support shared storage.\r\n\r\nThis change in particular renames the existing apiserver/params.StorageDetails to LegacyStorageDetails, and introduces a new StorageDetails. On the server we always populate both legacy and new. A follow-up will change the CLI to use the new details structure, falling back to legacy if\r\nthe new one is not available (i.e. old server).\n\n(Review request: http://reviews.vapour.ws/r/2653/)"},"comment":{"url":"https://api.github.com/repos/juju/juju/issues/comments/140265203","html_url":"https://github.com/juju/juju/pull/3273#issuecomment-140265203","issue_url":"https://api.github.com/repos/juju/juju/issues/3273","id":140265203,"user":{"login":"jujubot","id":7779494,"avatar_url":"https://avatars.githubusercontent.com/u/7779494?v=3","gravatar_id":"","url":"https://api.github.com/users/jujubot","html_url":"https://github.com/jujubot","followers_url":"https://api.github.com/users/jujubot/followers","following_url":"https://api.github.com/users/jujubot/following{/other_user}","gists_url":"https://api.github.com/users/jujubot/gists{/gist_id}","starred_url":"https://api.github.com/users/jujubot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jujubot/subscriptions","organizations_url":"https://api.github.com/users/jujubot/orgs","repos_url":"https://api.github.com/users/jujubot/repos","events_url":"https://api.github.com/users/jujubot/events{/privacy}","received_events_url":"https://api.github.com/users/jujubot/received_events","type":"User","site_admin":false},"created_at":"2015-09-15T03:06:51Z","updated_at":"2015-09-15T03:06:51Z","body":"Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju"}}