)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5972197a9c783aaa20e46103ec73d2f8be981f0d","unresolved":false,"context_lines":[{"line_number":4389,"context_line":"            return b\u0027\\n\u0027.join(log.split(b\u0027\\n\u0027)[-int(length):])"},{"line_number":4390,"context_line":""},{"line_number":4391,"context_line":"    def _get_token_and_access_url(self, context, ctype_debug, instance,"},{"line_number":4392,"context_line":"                                  ctype_enabel, console_type, ctype_str,"},{"line_number":4393,"context_line":"                                  proxy_base_url):"},{"line_number":4394,"context_line":"        _context \u003d context.elevated()"},{"line_number":4395,"context_line":"        LOG.debug(\"Getting \"+str(ctype_debug)+\" console\", instance\u003dinstance)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_6ede4b6e","line":4392,"range":{"start_line":4392,"start_character":34,"end_line":4392,"end_character":46},"updated":"2016-07-12 09:42:00.000000000","message":"s/enabel/enable/","commit_id":"52be14abdd053ae95ec9189252363647a8f868b5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5972197a9c783aaa20e46103ec73d2f8be981f0d","unresolved":false,"context_lines":[{"line_number":4392,"context_line":"                                  ctype_enabel, console_type, ctype_str,"},{"line_number":4393,"context_line":"                                  proxy_base_url):"},{"line_number":4394,"context_line":"        _context \u003d context.elevated()"},{"line_number":4395,"context_line":"        LOG.debug(\"Getting \"+str(ctype_debug)+\" console\", instance\u003dinstance)"},{"line_number":4396,"context_line":"        token \u003d str(uuid.uuid4())"},{"line_number":4397,"context_line":""},{"line_number":4398,"context_line":"        if not ctype_enabel:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_ceee5f21","line":4395,"range":{"start_line":4395,"start_character":18,"end_line":4395,"end_character":56},"updated":"2016-07-12 09:42:00.000000000","message":"could you use string formatting instead? Something like this, I guess?\n\n    LOG.debug(\u0027Getting %s console\u0027, str(ctype_debug), instance\u003dinstance)","commit_id":"52be14abdd053ae95ec9189252363647a8f868b5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5972197a9c783aaa20e46103ec73d2f8be981f0d","unresolved":false,"context_lines":[{"line_number":4420,"context_line":"            if instance.vm_state !\u003d vm_states.BUILDING:"},{"line_number":4421,"context_line":"                raise"},{"line_number":4422,"context_line":"            raise exception.InstanceNotReady(instance_id\u003dinstance.uuid)"},{"line_number":4423,"context_line":"            pass"},{"line_number":4424,"context_line":""},{"line_number":4425,"context_line":"        return connect_info"},{"line_number":4426,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_ce63ffaf","line":4423,"range":{"start_line":4423,"start_character":12,"end_line":4423,"end_character":16},"updated":"2016-07-12 09:42:00.000000000","message":"this does nothing - remove it","commit_id":"52be14abdd053ae95ec9189252363647a8f868b5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5972197a9c783aaa20e46103ec73d2f8be981f0d","unresolved":false,"context_lines":[{"line_number":4502,"context_line":"    def get_mks_console(self, context, console_type, instance):"},{"line_number":4503,"context_line":"        \"\"\"Return connection information for a MKS console.\"\"\""},{"line_number":4504,"context_line":"        return self._get_rdp_and_mks_console("},{"line_number":4505,"context_line":"            context, \"MKS\", instance, CONF.mks.enabled, \u0027webmks\u0027, console_type,"},{"line_number":4506,"context_line":"            CONF.mks.mksproxy_base_url, self.driver.get_mks_console)"},{"line_number":4507,"context_line":""},{"line_number":4508,"context_line":"    @messaging.expected_exceptions("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_8e1457e9","line":4505,"range":{"start_line":4505,"start_character":21,"end_line":4505,"end_character":26},"updated":"2016-07-12 09:42:00.000000000","message":"nit: could you use one quote style consistently?","commit_id":"52be14abdd053ae95ec9189252363647a8f868b5"}]}
