)]}'
{"trove/common/configurations.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"1b765839d84dbbbdcaa923a3add0811f34584da5","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    LOG.debug(\"rules?: %s\" % rules)"},{"line_number":40,"context_line":"    for key in overrides.keys():"},{"line_number":41,"context_line":"        rule \u003d _get_item(key, rules[\u0027configuration-parameters\u0027])"},{"line_number":42,"context_line":"        LOG.debug(\"checking the rule: %s\" % rule)"},{"line_number":43,"context_line":"        if rule.get(\u0027restart_required\u0027):"},{"line_number":44,"context_line":"            return True"},{"line_number":45,"context_line":"    return False"}],"source_content_type":"text/x-python","patch_set":7,"id":"baada198_894e45c0","side":"PARENT","line":42,"updated":"2014-07-01 12:06:19.000000000","message":"why this logging message was ripped off ?","commit_id":"cc9eb44564d629f871c8ccd1621ce28fcb4a9c93"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"1b765839d84dbbbdcaa923a3add0811f34584da5","unresolved":false,"context_lines":[{"line_number":81,"context_line":"            elif line_clean.startswith(\u0027[\u0027) and line_clean.endswith(\u0027]\u0027):"},{"line_number":82,"context_line":"                ret.append(line_clean)"},{"line_number":83,"context_line":"            elif line_clean and \"\u003d\" not in line_clean:"},{"line_number":84,"context_line":"                LOG.debug(\"fixing line without \u0027\u003d\u0027 in it: %s\" % line_clean)"},{"line_number":85,"context_line":"                ret.append(line_clean + \" \u003d 1\")"},{"line_number":86,"context_line":"            else:"},{"line_number":87,"context_line":"                ret.append(line_clean)"}],"source_content_type":"text/x-python","patch_set":7,"id":"baada198_e9cc2138","side":"PARENT","line":84,"updated":"2014-07-01 12:06:19.000000000","message":"same here","commit_id":"cc9eb44564d629f871c8ccd1621ce28fcb4a9c93"}],"trove/common/debug_utils.py":[{"author":{"_account_id":9746,"name":"Doug Shelley","email":"dougshelley66@gmail.com","username":"dougshelley"},"change_message_id":"f1a6d77b45389d454f48591650a02a4dff6ad7e9","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        import pydevd"},{"line_number":129,"context_line":"        LOG.debug(\"pydevd module was imported from system path\")"},{"line_number":130,"context_line":"    except ImportError:"},{"line_number":131,"context_line":"        LOG.debug(\"Can\u0027t load pydevd module from system path. Try load it \""},{"line_number":132,"context_line":"                  \"from pydev_path: %s\", pydev_path)"},{"line_number":133,"context_line":"        assert pydev_path, \"pydev_path is not set\""},{"line_number":134,"context_line":"        if pydev_path not in sys.path:"}],"source_content_type":"text/x-python","patch_set":5,"id":"dab895d6_a6a6e1bf","line":131,"updated":"2014-06-30 12:11:43.000000000","message":"...Try loadING it...","commit_id":"bdf0742c0bfd6573167e14dabac651607ba8e410"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"79d30fd36ca935fc058d25a3b224318656708353","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        import pydevd"},{"line_number":129,"context_line":"        LOG.debug(\"pydevd module was imported from system path\")"},{"line_number":130,"context_line":"    except ImportError:"},{"line_number":131,"context_line":"        LOG.debug(\"Can\u0027t load pydevd module from system path. Try load it \""},{"line_number":132,"context_line":"                  \"from pydev_path: %s\", pydev_path)"},{"line_number":133,"context_line":"        assert pydev_path, \"pydev_path is not set\""},{"line_number":134,"context_line":"        if pydev_path not in sys.path:"}],"source_content_type":"text/x-python","patch_set":5,"id":"baada198_5be7b7e4","line":131,"in_reply_to":"dab895d6_a6a6e1bf","updated":"2014-06-30 14:35:19.000000000","message":"got it, thanks!","commit_id":"bdf0742c0bfd6573167e14dabac651607ba8e410"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":101,"context_line":"            pydev_path\u003dpydev_path)"},{"line_number":102,"context_line":"    except Exception as e:"},{"line_number":103,"context_line":"        LOG.info(_(\"Can\u0027t connect to remote debug server. Continue working in \""},{"line_number":104,"context_line":"                 \"standard mode. Error: %s\"), e)"},{"line_number":105,"context_line":"        return False"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_b0703041","line":104,"updated":"2014-07-05 18:00:53.000000000","message":"This looks like it should be a warning (IMO).  Also the message might be clearer if it said \u0027Continuing to work in standard mode\u0027","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"}],"trove/common/exception.py":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"class DBConstraintError(TroveError):"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    message \u003d _(\"Failed to save %(model_name)s because: %(error)s\")"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"class InvalidRPCConnectionReuse(TroveError):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_7093f85a","line":55,"updated":"2014-07-05 18:00:53.000000000","message":"Most messages end with punctuation (i.e. \u0027.\u0027).  Do we want this to be the standard, or is it to pedantic?","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":146,"context_line":""},{"line_number":147,"context_line":"class NoUniqueMatch(TroveError):"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    message \u003d _(\"Multiple matches found for \u0027%(name)s\u0027, i\""},{"line_number":150,"context_line":"                \"use an UUID to be more specific.\")"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_909ecc61","line":149,"updated":"2014-07-05 18:00:53.000000000","message":"Typo: the \u0027i\u0027 should be removed (looks like a vi mistake ;))","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":213,"context_line":""},{"line_number":214,"context_line":"class CannotResizeToSameSize(TroveError):"},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"    message \u003d _(\"When resizing, instances must change size!\")"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"class VolumeAttachmentsNotFound(NotFound):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_b099107b","line":216,"updated":"2014-07-05 18:00:53.000000000","message":"Is there a reason this needs an exclamation mark at the end?  Is this a surprising error?  ;)","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":345,"context_line":""},{"line_number":346,"context_line":"    message \u003d _(\"Error creating security group rules.\""},{"line_number":347,"context_line":"                \" Malformed port(s). Port(s) is not integer.\""},{"line_number":348,"context_line":"                \" FromPort \u003d %(from)s greater than ToPort \u003d %(to)s\")"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"class BackupNotCompleteError(TroveError):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_d08be41c","line":348,"updated":"2014-07-05 18:00:53.000000000","message":"Most multi-line messages have the space at the end of the line","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":362,"context_line":"class BackupDatastoreMismatchError(TroveError):"},{"line_number":363,"context_line":"    message \u003d _(\"The datastore from which the backup was taken, \""},{"line_number":364,"context_line":"                \"%(datastore1)s, does not match the destination\""},{"line_number":365,"context_line":"                \" datastore of %(datastore2)s\")"},{"line_number":366,"context_line":""},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"class SwiftAuthError(TroveError):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_f08e282d","line":365,"updated":"2014-07-05 18:00:53.000000000","message":"Most multi-line messages have the space at the end of the line (this one has a mix)","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":410,"context_line":""},{"line_number":411,"context_line":"class ConfigurationParameterDeleted(TroveError):"},{"line_number":412,"context_line":"    message \u003d _(\"%(parameter_name)s parameter can no longer be \""},{"line_number":413,"context_line":"                \" set as of %(parameter_deleted_at)s\")"},{"line_number":414,"context_line":""},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"class ConfigurationAlreadyAttached(TroveError):"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_1082bcff","line":413,"updated":"2014-07-05 18:00:53.000000000","message":"Typo: Most multi-line messages have the space at the end of the line (this one has both, i.e. two spaces)","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":424,"context_line":""},{"line_number":425,"context_line":""},{"line_number":426,"context_line":"class RegionAmbiguity(TroveError):"},{"line_number":427,"context_line":"    \"\"\"Found more than one matching endpoint in Service Catalog.\"\"\""},{"line_number":428,"context_line":"    message \u003d _(\"Multiple matches for service_type\u003d%(service_type)s and \""},{"line_number":429,"context_line":"                \"endpoint_region\u003d%(endpoint_region)s. This generally means \""},{"line_number":430,"context_line":"                \"that a region is required and you have not supplied one.\")"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_30850007","line":427,"updated":"2014-07-05 18:00:53.000000000","message":"This might be out of scope, but it seems redundant that the last few exceptions have help comments as well as message strings","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"}],"trove/common/template.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"1b765839d84dbbbdcaa923a3add0811f34584da5","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        template_obj \u003d ENV.get_template(template_filename)"},{"line_number":120,"context_line":"        return template_obj"},{"line_number":121,"context_line":"    except jinja2.TemplateNotFound:"},{"line_number":122,"context_line":"        msg \u003d _(\"Missing heat template for %(s_datastore_manager)s\")"},{"line_number":123,"context_line":"        LOG.error(msg % {\"s_datastore_manager\": datastore_manager})"},{"line_number":124,"context_line":"        raise exception.TroveError("},{"line_number":125,"context_line":"            msg % {\"s_datastore_manager\": datastore_manager})"}],"source_content_type":"text/x-python","patch_set":7,"id":"baada198_a99069ed","line":122,"updated":"2014-07-01 12:06:19.000000000","message":"rendering could be done once and for all, no need to do rendering twice.","commit_id":"5e4ffc414e6908344486e1622262ff6182adddd1"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"b974edaffe956d72cfc7a5841040b5e9365c7ebe","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        template_obj \u003d ENV.get_template(template_filename)"},{"line_number":120,"context_line":"        return template_obj"},{"line_number":121,"context_line":"    except jinja2.TemplateNotFound:"},{"line_number":122,"context_line":"        msg \u003d _(\"Missing heat template for %(s_datastore_manager)s\")"},{"line_number":123,"context_line":"        LOG.error(msg % {\"s_datastore_manager\": datastore_manager})"},{"line_number":124,"context_line":"        raise exception.TroveError("},{"line_number":125,"context_line":"            msg % {\"s_datastore_manager\": datastore_manager})"}],"source_content_type":"text/x-python","patch_set":7,"id":"baada198_3583bc05","line":122,"in_reply_to":"baada198_a99069ed","updated":"2014-07-01 17:11:08.000000000","message":"Done","commit_id":"5e4ffc414e6908344486e1622262ff6182adddd1"}],"trove/common/utils.py":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"44adb7f45a0aad2bd3ae7f67e560f2d3fa2aa458","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    #              \" %(kwargs)s\") % {\u0027time\u0027: time, \u0027args\u0027: args,"},{"line_number":280,"context_line":"    #                                \u0027kwargs\u0027: kwargs})"},{"line_number":281,"context_line":"    #     LOG.error(msg)"},{"line_number":282,"context_line":"    #     raise exception.ProcessExecutionError(msg)"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"    timeout \u003d Timeout(time)"},{"line_number":285,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_6ad01954","line":282,"range":{"start_line":282,"start_character":10,"end_line":282,"end_character":15},"updated":"2014-06-30 21:56:25.000000000","message":"I\u0027d prefer to remove dead code instead of just commenting it out. Thanks!","commit_id":"821da3975cbba3721c6c2bbdff4037bb5d7e7d0b"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"b3424f33f41778d4d1010a8b35dadb221f95c802","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    #              \" %(kwargs)s\") % {\u0027time\u0027: time, \u0027args\u0027: args,"},{"line_number":280,"context_line":"    #                                \u0027kwargs\u0027: kwargs})"},{"line_number":281,"context_line":"    #     LOG.error(msg)"},{"line_number":282,"context_line":"    #     raise exception.ProcessExecutionError(msg)"},{"line_number":283,"context_line":""},{"line_number":284,"context_line":"    timeout \u003d Timeout(time)"},{"line_number":285,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"baada198_6979f12e","line":282,"in_reply_to":"baada198_6ad01954","updated":"2014-07-01 11:55:42.000000000","message":"Done","commit_id":"821da3975cbba3721c6c2bbdff4037bb5d7e7d0b"}],"trove/common/wsgi.py":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            # If action_result is not a Fault then there really was a"},{"line_number":302,"context_line":"            # serialization error which we log. Otherwise return the Fault."},{"line_number":303,"context_line":"            if not isinstance(action_result, Fault):"},{"line_number":304,"context_line":"                LOG.exception(_(\"unserializable result detected.\"))"},{"line_number":305,"context_line":"                raise"},{"line_number":306,"context_line":"            return action_result"},{"line_number":307,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_10e71ce5","line":304,"updated":"2014-07-05 18:00:53.000000000","message":"Should this message be capitalized?","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"0fbe508994daeaa6ecd922078003685ef4a719ab","unresolved":false,"context_lines":[{"line_number":467,"context_line":"        name \u003d exc.__class__.__name__"},{"line_number":468,"context_line":"        if name in named_exceptions:"},{"line_number":469,"context_line":"            return named_exceptions[name]"},{"line_number":470,"context_line":"            # If the exception isn\u0027t in our list, at least strip off the"},{"line_number":471,"context_line":"        # HTTP from the name, and then drop the case on the first letter."},{"line_number":472,"context_line":"        name \u003d name.split(\"HTTP\").pop()"},{"line_number":473,"context_line":"        name \u003d name[:1].lower() + name[1:]"}],"source_content_type":"text/x-python","patch_set":8,"id":"baada198_70cf7856","line":470,"updated":"2014-07-05 18:00:53.000000000","message":"This comment is indented too much (out of scope?)","commit_id":"089ec2651f25c4cf8d04077c733e6d6aebe835b6"}]}