)]}'
{"nova/api/metadata/password.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9b5cb4fd3c9347076276da002109485a083968d7","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            raise exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        im \u003d objects.InstanceMapping.get_by_instance_uuid(ctxt, meta_data.uuid)"},{"line_number":72,"context_line":"        with context.target_cell(ctxt, im.cell_mapping) as cctxt:"},{"line_number":73,"context_line":"            try:"},{"line_number":74,"context_line":"                instance \u003d objects.Instance.get_by_uuid(cctxt, meta_data.uuid)"},{"line_number":75,"context_line":"            except exception.InstanceNotFound as e:"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_02524768","line":72,"range":{"start_line":72,"start_character":21,"end_line":72,"end_character":32},"updated":"2017-06-10 13:28:22.000000000","message":"I don\u0027t know why but the test fails when target_cell context manager is on here. If I remove this, the exception is raised up as expected. It\u0027s almost like it\u0027s swallowing the exception, but I can\u0027t explain it.","commit_id":"5442f8ac172613749bc19bed24d55861c6c616f3"}],"nova/tests/unit/test_metadata.py":[{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"dbac27c6c00df3630bd54fd820bc60ec45ea7978","unresolved":false,"context_lines":[{"line_number":1639,"context_line":"                       return_value\u003dobjects.InstanceMapping(cell_mapping\u003dNone))"},{"line_number":1640,"context_line":"    @mock.patch.object(objects.Instance, \u0027get_by_uuid\u0027)"},{"line_number":1641,"context_line":"    def test_set_password_instance_not_found(self, get_by_uuid, get_mapping):"},{"line_number":1642,"context_line":"        \"\"\"Tests that a 404 is returned if the instance can not be found.\"\"\""},{"line_number":1643,"context_line":"        get_by_uuid.side_effect \u003d exception.InstanceNotFound("},{"line_number":1644,"context_line":"            instance_id\u003dself.instance.uuid)"},{"line_number":1645,"context_line":"        request \u003d webob.Request.blank(\u0027\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f4b6375_98893776","line":1642,"range":{"start_line":1642,"start_character":24,"end_line":1642,"end_character":27},"updated":"2017-10-27 05:02:27.000000000","message":"typo","commit_id":"3bf16c8378e880bd27b109c025410ad06b79eedf"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"3535fa1f98e89cfa8a4a7b2bdc649b5e39f45cc6","unresolved":false,"context_lines":[{"line_number":1644,"context_line":"            instance_id\u003dself.instance.uuid)"},{"line_number":1645,"context_line":"        request \u003d webob.Request.blank(\u0027\u0027)"},{"line_number":1646,"context_line":"        request.method \u003d \u0027POST\u0027"},{"line_number":1647,"context_line":"        request.val \u003d b\u0027foo\u0027"},{"line_number":1648,"context_line":"        self.assertRaises(webob.exc.HTTPBadRequest, password.handle_password,"},{"line_number":1649,"context_line":"                          request, self.mdinst)"},{"line_number":1650,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_57af7e29","line":1647,"range":{"start_line":1647,"start_character":0,"end_line":1647,"end_character":28},"updated":"2017-11-02 04:06:35.000000000","message":"request.content_length \u003d xxx\n\nwe also need add fake content_length in req, otherwise, req len check in handle_password method failed first.","commit_id":"f497cd7ecfaa4ab79ff95b7b83f0e69e409936d8"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5b659b8c8318de84727fe125bc2c9eb38dc8afe0","unresolved":false,"context_lines":[{"line_number":1644,"context_line":"            instance_id\u003dself.instance.uuid)"},{"line_number":1645,"context_line":"        request \u003d webob.Request.blank(\u0027\u0027)"},{"line_number":1646,"context_line":"        request.method \u003d \u0027POST\u0027"},{"line_number":1647,"context_line":"        request.val \u003d b\u0027foo\u0027"},{"line_number":1648,"context_line":"        self.assertRaises(webob.exc.HTTPBadRequest, password.handle_password,"},{"line_number":1649,"context_line":"                          request, self.mdinst)"},{"line_number":1650,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_3a97ec47","line":1647,"range":{"start_line":1647,"start_character":0,"end_line":1647,"end_character":28},"in_reply_to":"3f4b6375_57af7e29","updated":"2017-11-06 22:13:40.000000000","message":"Thanks, nice catch.","commit_id":"f497cd7ecfaa4ab79ff95b7b83f0e69e409936d8"}]}