)]}'
{"doc/api_samples/os-volumes/snapshot-create-resp.json":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"35c7f2044ca0160fc303a08fbd5b78fda23055e4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"        \"size\": 100,"},{"line_number":8,"context_line":"        \"status\": \"available\","},{"line_number":9,"context_line":"        \"volumeId\": {"},{"line_number":10,"context_line":"            \"attach_status\": \"attached\","},{"line_number":11,"context_line":"            \"availability_zone\": \"fakeaz\","},{"line_number":12,"context_line":"            \"created_at\": \"1999-01-01T01:01:01.000000\","},{"line_number":13,"context_line":"            \"display_description\": \"displaydesc\","}],"source_content_type":"application/json","patch_set":3,"id":"AAAAOn%2F%2FVvc%3D","side":"PARENT","line":10,"updated":"2013-05-14 08:53:45.000000000","message":"This looks pretty bogus - we clearly weren\u0027t just returning the ID in the volumeId field","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb"}],"doc/api_samples/os-volumes/snapshot-create-resp.xml":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"35c7f2044ca0160fc303a08fbd5b78fda23055e4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"\u003c?xml version\u003d\u00271.0\u0027 encoding\u003d\u0027UTF-8\u0027?\u003e"},{"line_number":2,"context_line":"\u003csnapshot status\u003d\"available\" displayDescription\u003d\"Daily backup\" displayName\u003d\"snap-001\" volumeId\u003d\"{\u0027instance_uuid\u0027: \u0027fakeuuid\u0027, \u0027status\u0027: \u0027fakestatus\u0027, \u0027user_id\u0027: \u0027fakeuser\u0027, \u0027name\u0027: \u0027vol name\u0027, \u0027display_name\u0027: \u0027displayname\u0027, \u0027availability_zone\u0027: \u0027fakeaz\u0027, \u0027created_at\u0027: datetime.datetime(1999, 1, 1, 1, 1, 1), \u0027attach_status\u0027: \u0027attached\u0027, \u0027display_description\u0027: \u0027displaydesc\u0027, \u0027host\u0027: \u0027fakehost\u0027, \u0027volume_type_id\u0027: \u0027fakevoltype\u0027, \u0027volume_metadata\u0027: [], \u0027volume_type\u0027: {\u0027name\u0027: \u0027vol_type_name\u0027}, \u0027snapshot_id\u0027: None, \u0027mountpoint\u0027: \u0027/\u0027, \u0027project_id\u0027: \u0027fakeproject\u0027, \u0027id\u0027: u\u0027521752a6-acf6-4b2d-bc7a-119f9148cd8c\u0027, \u0027size\u0027: 1}\" id\u003d\"100\" createdAt\u003d\"2013-02-25 16:27:36.840121\" size\u003d\"100\"/\u003e"}],"source_content_type":"application/xml","patch_set":3,"id":"AAAAOn%2F%2FVvU%3D","side":"PARENT","line":2,"updated":"2013-05-14 08:53:45.000000000","message":"Looks bogus here too","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb"}],"nova/tests/api/ec2/test_cinder_cloud.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"e76c1cc90b81c60312b8fdd0b7fe356d14fe908b","unresolved":false,"context_lines":[{"line_number":371,"context_line":"                                                        **kwargs)"},{"line_number":372,"context_line":"                if \u0027snapshot_id\u0027 in values:"},{"line_number":373,"context_line":"                    self.volume_api.create_snapshot(self.context,"},{"line_number":374,"context_line":"                                                    bdm[\u0027volume_id\u0027],"},{"line_number":375,"context_line":"                                                    \u0027snapshot-bdm\u0027,"},{"line_number":376,"context_line":"                                                    \u0027fake snap for bdm tests\u0027,"},{"line_number":377,"context_line":"                                                    values[\u0027snapshot_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2Fq7w%3D","line":374,"updated":"2013-04-29 19:06:45.000000000","message":"Shouldn\u0027t these be \"vol[\u0027id\u0027]\", since you have to get the volume anyway?","commit_id":"d7f3537e810387ba051ca5e94aefe12665981a1e"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"42c41c14b36ff710830b935478ef2d25f2c0411c","unresolved":false,"context_lines":[{"line_number":371,"context_line":"                                                        **kwargs)"},{"line_number":372,"context_line":"                if \u0027snapshot_id\u0027 in values:"},{"line_number":373,"context_line":"                    self.volume_api.create_snapshot(self.context,"},{"line_number":374,"context_line":"                                                    bdm[\u0027volume_id\u0027],"},{"line_number":375,"context_line":"                                                    \u0027snapshot-bdm\u0027,"},{"line_number":376,"context_line":"                                                    \u0027fake snap for bdm tests\u0027,"},{"line_number":377,"context_line":"                                                    values[\u0027snapshot_id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FqIM%3D","line":374,"in_reply_to":"AAAAOn%2F%2Fq7w%3D","updated":"2013-04-30 11:40:03.000000000","message":"Done","commit_id":"d7f3537e810387ba051ca5e94aefe12665981a1e"}],"nova/tests/api/openstack/fakes.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"35c7f2044ca0160fc303a08fbd5b78fda23055e4","unresolved":false,"context_lines":[{"line_number":602,"context_line":""},{"line_number":603,"context_line":""},{"line_number":604,"context_line":"def stub_snapshot_create(self, context, volume_id, name, description):"},{"line_number":605,"context_line":"    return stub_snapshot(100, volume_id\u003dvolume_id, display_name\u003dname,"},{"line_number":606,"context_line":"                         display_description\u003ddescription)"},{"line_number":607,"context_line":""},{"line_number":608,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FVtE%3D","side":"PARENT","line":605,"updated":"2013-05-14 08:53:45.000000000","message":"This is the bug that was causing the tests to be screwed up - the actual signature of snapshot_create() was:\n\n  def create_snapshot(self, context, volume, name, description):","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb"}],"nova/tests/integrated/api_samples/os-volumes/snapshot-create-resp.xml.tpl":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"e76c1cc90b81c60312b8fdd0b7fe356d14fe908b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"\u003c?xml version\u003d\u00271.0\u0027 encoding\u003d\u0027UTF-8\u0027?\u003e"},{"line_number":2,"context_line":"\u003csnapshot status\u003d\"available\" displayDescription\u003d\"%(description)s\" displayName\u003d\"%(snapshot_name)s\" volumeId\u003d\"521752a6-acf6-4b2d-bc7a-119f9148cd8c\" id\u003d\"100\" createdAt\u003d\"%(timestamp)s\" size\u003d\"100\"/\u003e"}],"source_content_type":"text/x-smarty","patch_set":2,"id":"AAAAOn%2F%2Fq7U%3D","line":2,"updated":"2013-04-29 19:06:45.000000000","message":"Heh.  We should never have let the \u0027volumeId\u0027 attribute become a raw dictionary anyway, so +1 just for this :)","commit_id":"d7f3537e810387ba051ca5e94aefe12665981a1e"}],"nova/volume/cinder.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"35c7f2044ca0160fc303a08fbd5b78fda23055e4","unresolved":false,"context_lines":[{"line_number":204,"context_line":"class API(base.Base):"},{"line_number":205,"context_line":"    \"\"\"API for interacting with the volume manager.\"\"\""},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"    @translate_volume_exception"},{"line_number":208,"context_line":"    def get(self, context, volume_id):"},{"line_number":209,"context_line":"        item \u003d cinderclient(context).volumes.get(volume_id)"},{"line_number":210,"context_line":"        return _untranslate_volume_summary_view(context, item)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FVvA%3D","line":207,"updated":"2013-05-14 08:53:45.000000000","message":"I think it would have been better (i.e. easier for reviewers) to do the exception translating stuff as a separate commit before this one ... but I can see how they\u0027re related","commit_id":"b417d83338a762f1ae0aeb8e1b88b8b18cc817c6"}]}