)]}'
{"troveclient/v1/instances.py":[{"author":{"_account_id":9782,"name":"Morgan Jones","email":"morgan@tesora.com","username":"vgnbkr"},"change_message_id":"0c1aeaf4539e94d8ea807d1d24a395068f610493","unresolved":false,"context_lines":[{"line_number":283,"context_line":"            metadata_file \u003d log_info.metafile"},{"line_number":284,"context_line":"            return container, prefix, metadata_file"},{"line_number":285,"context_line":"        except client.ClientException:"},{"line_number":286,"context_line":"            raise exceptions.GuestLogNotFoundError()"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"    def log_generator(self, instance, log, publish\u003dNone, lines\u003d50,"},{"line_number":289,"context_line":"                      swift\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_47de3bde","line":286,"updated":"2015-12-07 20:19:17.000000000","message":"We shouldn\u0027t just eat this exception.","commit_id":"6b752d077d91a1f0c46226cdba9daf3e2fba9847"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"28d8f7c70ea581b7ea89e7430fab2538976b90a8","unresolved":false,"context_lines":[{"line_number":283,"context_line":"            metadata_file \u003d log_info.metafile"},{"line_number":284,"context_line":"            return container, prefix, metadata_file"},{"line_number":285,"context_line":"        except client.ClientException:"},{"line_number":286,"context_line":"            raise exceptions.GuestLogNotFoundError()"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"    def log_generator(self, instance, log, publish\u003dNone, lines\u003d50,"},{"line_number":289,"context_line":"                      swift\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_aad6d44c","line":286,"in_reply_to":"7a740942_47de3bde","updated":"2015-12-11 15:09:27.000000000","message":"This code was written either by you or Alex.  Feel free to fix it if you no longer like it.  ;)","commit_id":"6b752d077d91a1f0c46226cdba9daf3e2fba9847"},{"author":{"_account_id":10440,"name":"Alex Tomic","email":"atomic777@gmail.com","username":"atomic77"},"change_message_id":"2feda71f6ab40136a6f9f72b423a16bfd14de720","unresolved":false,"context_lines":[{"line_number":283,"context_line":"            metadata_file \u003d log_info.metafile"},{"line_number":284,"context_line":"            return container, prefix, metadata_file"},{"line_number":285,"context_line":"        except client.ClientException:"},{"line_number":286,"context_line":"            raise exceptions.GuestLogNotFoundError()"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"    def log_generator(self, instance, log, publish\u003dNone, lines\u003d50,"},{"line_number":289,"context_line":"                      swift\u003dNone):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a7b0d38_9995c8e5","line":286,"in_reply_to":"7a740942_aad6d44c","updated":"2015-12-23 19:36:21.000000000","message":"I don\u0027t remember if I wrote or modified any of this module tbh, but I don\u0027t think it really matters -- the real question is whether this is worth a -1. Personally, I don\u0027t think so.","commit_id":"6b752d077d91a1f0c46226cdba9daf3e2fba9847"},{"author":{"_account_id":9782,"name":"Morgan Jones","email":"morgan@tesora.com","username":"vgnbkr"},"change_message_id":"0c1aeaf4539e94d8ea807d1d24a395068f610493","unresolved":false,"context_lines":[{"line_number":335,"context_line":"                                                        log_part[\u0027name\u0027])"},{"line_number":336,"context_line":"                    yield log_obj"},{"line_number":337,"context_line":"            except client.ClientException:"},{"line_number":338,"context_line":"                raise exceptions.GuestLogNotFoundError()"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"        return lambda: _log_generator(instance, log, publish, lines, swift)"},{"line_number":341,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_67db37ed","line":338,"updated":"2015-12-07 20:19:17.000000000","message":"Again, don\u0027t eat the exception.","commit_id":"6b752d077d91a1f0c46226cdba9daf3e2fba9847"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"28d8f7c70ea581b7ea89e7430fab2538976b90a8","unresolved":false,"context_lines":[{"line_number":335,"context_line":"                                                        log_part[\u0027name\u0027])"},{"line_number":336,"context_line":"                    yield log_obj"},{"line_number":337,"context_line":"            except client.ClientException:"},{"line_number":338,"context_line":"                raise exceptions.GuestLogNotFoundError()"},{"line_number":339,"context_line":""},{"line_number":340,"context_line":"        return lambda: _log_generator(instance, log, publish, lines, swift)"},{"line_number":341,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_0ac6089a","line":338,"in_reply_to":"7a740942_67db37ed","updated":"2015-12-11 15:09:27.000000000","message":"ditto","commit_id":"6b752d077d91a1f0c46226cdba9daf3e2fba9847"},{"author":{"_account_id":9782,"name":"Morgan Jones","email":"morgan@tesora.com","username":"vgnbkr"},"change_message_id":"047fc83b31d73792e8cda1edd20aea35d3b799b9","unresolved":false,"context_lines":[{"line_number":246,"context_line":"        common.check_for_exceptions(resp, body, url)"},{"line_number":247,"context_line":"        return [DatastoreLog(self, log, loaded\u003dTrue) for log in body[\u0027logs\u0027]]"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"    def log_action(self, instance, log_name, enable\u003dNone, disable\u003dNone,"},{"line_number":250,"context_line":"                   publish\u003dNone, discard\u003dNone):"},{"line_number":251,"context_line":"        \"\"\"Perform action on guest log."},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9a68dd71_6d55df1b","line":249,"updated":"2016-01-20 17:12:24.000000000","message":"Sorry, I missed this in my first review.  Why are we combining these operations rather than matching the CLI?","commit_id":"949588a91b3390facbbe0ec2486e950929171ace"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"7322535ab275f7e46d9a64211d83b370396a8e35","unresolved":false,"context_lines":[{"line_number":246,"context_line":"        common.check_for_exceptions(resp, body, url)"},{"line_number":247,"context_line":"        return [DatastoreLog(self, log, loaded\u003dTrue) for log in body[\u0027logs\u0027]]"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"    def log_action(self, instance, log_name, enable\u003dNone, disable\u003dNone,"},{"line_number":250,"context_line":"                   publish\u003dNone, discard\u003dNone):"},{"line_number":251,"context_line":"        \"\"\"Perform action on guest log."},{"line_number":252,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9a68dd71_a0c42740","line":249,"in_reply_to":"9a68dd71_6d55df1b","updated":"2016-01-21 01:52:47.000000000","message":"This was to keep the API changes to a minimum when the new commands were added (log-enable, etc.).","commit_id":"949588a91b3390facbbe0ec2486e950929171ace"}],"troveclient/v1/shell.py":[{"author":{"_account_id":10295,"name":"mariam john","email":"mariamj@us.ibm.com","username":"mariamj"},"change_message_id":"292f2c8f53da4da8711797d87f36c6b69e176274","unresolved":false,"context_lines":[{"line_number":1423,"context_line":""},{"line_number":1424,"context_line":""},{"line_number":1425,"context_line":"@utils.arg(\u0027instance\u0027, metavar\u003d\u0027\u003cinstance\u003e\u0027, help\u003d\u0027Id or Name of the instance\u0027)"},{"line_number":1426,"context_line":"@utils.arg(\u0027log_name\u0027, metavar\u003d\u0027\u003clog_name\u003e\u0027, help\u003d\u0027Name of log to publish\u0027)"},{"line_number":1427,"context_line":"@utils.service_type(\u0027database\u0027)"},{"line_number":1428,"context_line":"def do_log_enable(cs, args):"},{"line_number":1429,"context_line":"    \"\"\"Instructs Trove guest to start collecting log details.\"\"\""}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_d40b3e7c","line":1426,"updated":"2016-01-27 14:35:59.000000000","message":"Keeping it consistent with other commands, this could be \u0027ID or Name\u0027. plus help messages need to end with a period.","commit_id":"6e38a0d3833f792620ad34d299aeed1276114a45"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"2292fb84360c9cc125dd33617101c978d974fcc2","unresolved":false,"context_lines":[{"line_number":1423,"context_line":""},{"line_number":1424,"context_line":""},{"line_number":1425,"context_line":"@utils.arg(\u0027instance\u0027, metavar\u003d\u0027\u003cinstance\u003e\u0027, help\u003d\u0027Id or Name of the instance\u0027)"},{"line_number":1426,"context_line":"@utils.arg(\u0027log_name\u0027, metavar\u003d\u0027\u003clog_name\u003e\u0027, help\u003d\u0027Name of log to publish\u0027)"},{"line_number":1427,"context_line":"@utils.service_type(\u0027database\u0027)"},{"line_number":1428,"context_line":"def do_log_enable(cs, args):"},{"line_number":1429,"context_line":"    \"\"\"Instructs Trove guest to start collecting log details.\"\"\""}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_c9563f06","line":1426,"in_reply_to":"7a5de9d1_d40b3e7c","updated":"2016-01-27 16:08:59.000000000","message":"Logs don\u0027t have IDs, but you\u0027re right about the missing \u0027.\u0027 - I\u0027ll add that (since the build failed anyways ...)","commit_id":"6e38a0d3833f792620ad34d299aeed1276114a45"}]}