)]}'
{"nova/tests/consoleauth/test_consoleauth.py":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"45e95adfd9220815509a98ba2f4c7f9336399035","unresolved":false,"context_lines":[{"line_number":81,"context_line":"            self.manager.authorize_console(self.context, token, \u0027novnc\u0027,"},{"line_number":82,"context_line":"                                          \u0027127.0.0.1\u0027, \u00278080\u0027, \u0027host\u0027,"},{"line_number":83,"context_line":"                                          self.instance[\u0027uuid\u0027])"},{"line_number":84,"context_line":"        instance_uuid \u003d self.instance[\u0027uuid\u0027]"},{"line_number":85,"context_line":"        self.manager.delete_tokens_for_instance(self.context, instance_uuid)"},{"line_number":86,"context_line":"        stored_tokens \u003d self.manager._get_tokens_for_instance(instance_uuid)"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPn%2F%2FjYc%3D","line":84,"updated":"2013-06-27 21:59:12.000000000","message":"Super nitpick: I think you could move this above the loop and use instance_uuid there instead.","commit_id":"907b85fbb948cd642e58ed52021e95dcf890078e"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"242283c20cc42e2e4b083f357235125079fe4710","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        super(ConsoleauthTestCase, self).setUp()"},{"line_number":36,"context_line":"        self.manager \u003d manager.ConsoleAuthManager()"},{"line_number":37,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":38,"context_line":"        self.instance \u003d db.instance_create(self.context, {})"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def test_tokens_expire(self):"},{"line_number":41,"context_line":"        # Test that tokens expire correctly."}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPn%2F%2FQFs%3D","line":38,"updated":"2013-07-06 03:38:56.000000000","message":"Somehow I\u0027m failing to understand how this worked before, since  in the local conductor case (which we use in tests), the behavior should be identical.  Why was the instance not necessary to be created in the DB before? :)","commit_id":"13a6574987ee91068ffc46f2653256c57e86cced"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"f9092af64a8014b9f08f3c00b72bc71b096a1308","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        super(ConsoleauthTestCase, self).setUp()"},{"line_number":36,"context_line":"        self.manager \u003d manager.ConsoleAuthManager()"},{"line_number":37,"context_line":"        self.context \u003d context.get_admin_context()"},{"line_number":38,"context_line":"        self.instance \u003d db.instance_create(self.context, {})"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def test_tokens_expire(self):"},{"line_number":41,"context_line":"        # Test that tokens expire correctly."}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2F%2BvQ%3D","line":38,"in_reply_to":"AAAAPn%2F%2FQFs%3D","updated":"2013-07-08 06:34:04.000000000","message":"This is because it wasn\u0027t using conductor, and instead going over RPC the fake driver\u0027s call() method which returns None if there is no consumer\n\n... which all seems pretty broken. I\u0027ve filed bug #1198809","commit_id":"13a6574987ee91068ffc46f2653256c57e86cced"}]}