)]}'
{"trove/backup/service.py":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"249eb616566c0c5e449294d5c6af3ae4b7483278","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        name \u003d data[\u0027name\u0027]"},{"line_number":65,"context_line":"        desc \u003d data.get(\u0027description\u0027)"},{"line_number":66,"context_line":"        parent \u003d data.get(\u0027parent_id\u0027)"},{"line_number":67,"context_line":"        incremental \u003d data.get(\u0027incremental\u0027)"},{"line_number":68,"context_line":"        context.notification \u003d notification.DBaaSBackupCreate(context,"},{"line_number":69,"context_line":"                                                              request\u003dreq)"},{"line_number":70,"context_line":"        with StartNotification(context, name\u003dname, instance_id\u003dinstance,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_37516c0f","line":67,"range":{"start_line":67,"start_character":32,"end_line":67,"end_character":43},"updated":"2016-08-16 19:15:45.000000000","message":"you should add this to trove/common/apischema - not sure why this didn\u0027t cause the tests to fail ... ah right, there are no tests. :P   Need to add those too. :)","commit_id":"c712760f611542b87edfb08eb68a22224b9df3fa"},{"author":{"_account_id":21668,"name":"Ali Asgar Adil","email":"aadil@tesora.com","username":"aliaadil"},"change_message_id":"7d8597cbbd8afc6a61eac10ffac12f198c5e500d","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        name \u003d data[\u0027name\u0027]"},{"line_number":65,"context_line":"        desc \u003d data.get(\u0027description\u0027)"},{"line_number":66,"context_line":"        parent \u003d data.get(\u0027parent_id\u0027)"},{"line_number":67,"context_line":"        incremental \u003d data.get(\u0027incremental\u0027)"},{"line_number":68,"context_line":"        context.notification \u003d notification.DBaaSBackupCreate(context,"},{"line_number":69,"context_line":"                                                              request\u003dreq)"},{"line_number":70,"context_line":"        with StartNotification(context, name\u003dname, instance_id\u003dinstance,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3ac371cc_85c6e797","line":67,"range":{"start_line":67,"start_character":32,"end_line":67,"end_character":43},"in_reply_to":"3ac371cc_37516c0f","updated":"2016-08-17 20:25:58.000000000","message":"Added to apischema and added test.","commit_id":"c712760f611542b87edfb08eb68a22224b9df3fa"}],"trove/tests/scenario/runners/backup_runners.py":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"56237de10ca3a73e529ff8c8d650044909aea0ff","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def assert_backup_create(self, name, desc, instance_id, parent_id\u003dNone,"},{"line_number":111,"context_line":"                             incremental\u003dFalse):"},{"line_number":112,"context_line":"        if incremental:"},{"line_number":113,"context_line":"            result \u003d self.auth_client.backups.create("},{"line_number":114,"context_line":"                name, instance_id, desc, incremental\u003dincremental)"},{"line_number":115,"context_line":"        else:"},{"line_number":116,"context_line":"            result \u003d self.auth_client.backups.create("},{"line_number":117,"context_line":"                name, instance_id, desc, parent_id\u003dparent_id)"},{"line_number":118,"context_line":"        self.assert_equal(name, result.name,"},{"line_number":119,"context_line":"                          \u0027Unexpected backup name\u0027)"},{"line_number":120,"context_line":"        self.assert_equal(desc, result.description,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a89bdaa_efe2912e","line":117,"range":{"start_line":112,"start_character":0,"end_line":117,"end_character":61},"updated":"2016-09-06 21:28:09.000000000","message":"You could probably just pass both down (incremental and parent_id) and so avoid the \u0027if\u0027 but not worth holding up the patchset for.","commit_id":"4b54ab30bb9a178a8c41bf5e2b3f23c943a4b8c1"}]}