)]}'
{"heat/engine/resources/nova_utils.py":[{"author":{"_account_id":7135,"name":"Liang Chen","email":"liangchen.linux@gmail.com","username":"Liang"},"change_message_id":"d8bd3cad224d9bfef3cdd8e11816048191942fad","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    \u0027\u0027\u0027"},{"line_number":55,"context_line":"    try:"},{"line_number":56,"context_line":"        server.get()"},{"line_number":57,"context_line":"    except clients.novaclient.exceptions.OverLimit as exc:"},{"line_number":58,"context_line":"        msg \u003d _(\"Server %(name)s (%(id)s) received an OverLimit \""},{"line_number":59,"context_line":"                \"response during server.get(): %(exception)s\")"},{"line_number":60,"context_line":"        logger.warning(msg % {\u0027name\u0027: server.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2B7p4%3D","line":57,"updated":"2014-02-18 05:45:11.000000000","message":"Will it just keep failing and eating users\u0027 quota if the next try doesn\u0027t wait for a certain period of time as indicated by retry_after field of OverLimit?","commit_id":"a5b42c515236319834cff2512c18d971d4ba273c"},{"author":{"_account_id":7135,"name":"Liang Chen","email":"liangchen.linux@gmail.com","username":"Liang"},"change_message_id":"8afdb49e5140c21fe0b188ae7ef64542922e7728","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    \u0027\u0027\u0027"},{"line_number":55,"context_line":"    try:"},{"line_number":56,"context_line":"        server.get()"},{"line_number":57,"context_line":"    except clients.novaclient.exceptions.OverLimit as exc:"},{"line_number":58,"context_line":"        msg \u003d _(\"Server %(name)s (%(id)s) received an OverLimit \""},{"line_number":59,"context_line":"                \"response during server.get(): %(exception)s\")"},{"line_number":60,"context_line":"        logger.warning(msg % {\u0027name\u0027: server.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2BwTU%3D","line":57,"in_reply_to":"AAAAUX%2F%2B1Oc%3D","updated":"2014-02-19 09:36:48.000000000","message":"Thanks for the information. As long as this doesn\u0027t count toward the quota, making some extra requests doesn\u0027t seems very harmful. -1 taken out;)","commit_id":"a5b42c515236319834cff2512c18d971d4ba273c"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"35d177982ee9a142b579f7be0639a409cd8643dd","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    \u0027\u0027\u0027"},{"line_number":55,"context_line":"    try:"},{"line_number":56,"context_line":"        server.get()"},{"line_number":57,"context_line":"    except clients.novaclient.exceptions.OverLimit as exc:"},{"line_number":58,"context_line":"        msg \u003d _(\"Server %(name)s (%(id)s) received an OverLimit \""},{"line_number":59,"context_line":"                \"response during server.get(): %(exception)s\")"},{"line_number":60,"context_line":"        logger.warning(msg % {\u0027name\u0027: server.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUX%2F%2B1Oc%3D","line":57,"in_reply_to":"AAAAUX%2F%2B7p4%3D","updated":"2014-02-18 18:30:30.000000000","message":"In my tests, the OverLimit responses did not count toward the quota.  So the responses looked something like:\n\n | +++++---------- | +++++---------- |\n\nWhere \u0027+\u0027 is a successful server.get(), \u0027-\u0027 is an OverLimit response, and \u0027|\u0027 is the time frame boundary.","commit_id":"a5b42c515236319834cff2512c18d971d4ba273c"},{"author":{"_account_id":6488,"name":"Clint Byrum","email":"clint@fewbar.com","username":"clint-fewbar"},"change_message_id":"630229504f5634af11f483d9aa75ab5832d5e441","unresolved":false,"context_lines":[{"line_number":59,"context_line":"                \"response during server.get(): %(exception)s\")"},{"line_number":60,"context_line":"        logger.warning(msg % {\u0027name\u0027: server.name,"},{"line_number":61,"context_line":"                              \u0027id\u0027: server.id,"},{"line_number":62,"context_line":"                              \u0027exception\u0027: str(exc)})"},{"line_number":63,"context_line":"    except clients.novaclient.exceptions.ClientException as exc:"},{"line_number":64,"context_line":"        if exc.code \u003d\u003d 500:"},{"line_number":65,"context_line":"            msg \u003d _(\u0027Server \"%(name)s\" (%(id)s) received the following \u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAVX%2F%2FuQM%3D","line":62,"updated":"2014-02-28 19:11:57.000000000","message":"\\o/","commit_id":"8528b203fa5d111cbbb733fda2e75e3e0b42d317"}],"heat/tests/test_nova_utils.py":[{"author":{"_account_id":9189,"name":"Anderson Mesquita","email":"andersonvom@gmail.com","username":"andersonvom"},"change_message_id":"5328ba75a2863ede2c5467753509d794743a0450","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        self.m.VerifyAll()"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"    def test_overlimit_error(self):"},{"line_number":122,"context_line":"        server \u003d self.m.CreateMockAnything()"},{"line_number":123,"context_line":"        kwargs \u003d {"},{"line_number":124,"context_line":"            \u0027code\u0027: 413,"},{"line_number":125,"context_line":"            \u0027method\u0027: None,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAU3%2F%2FZGA%3D","line":122,"updated":"2014-02-26 21:42:36.000000000","message":"We should probably be mocking this with mock, right?","commit_id":"837953eefbf02c16b06328f012b8f2280255083c"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"2265adea94a8d01f89a2870091420c07a25136f7","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        self.m.VerifyAll()"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"    def test_overlimit_error(self):"},{"line_number":122,"context_line":"        server \u003d self.m.CreateMockAnything()"},{"line_number":123,"context_line":"        kwargs \u003d {"},{"line_number":124,"context_line":"            \u0027code\u0027: 413,"},{"line_number":125,"context_line":"            \u0027method\u0027: None,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAVX%2F%2F%2F%2B8%3D","line":122,"in_reply_to":"AAAAU3%2F%2FZGA%3D","updated":"2014-02-26 22:30:02.000000000","message":"Done","commit_id":"837953eefbf02c16b06328f012b8f2280255083c"},{"author":{"_account_id":9189,"name":"Anderson Mesquita","email":"andersonvom@gmail.com","username":"andersonvom"},"change_message_id":"4a7d2741f4d9cb22a95623b5e62a42598532406c","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        self.m.VerifyAll()"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def test_overlimit_error(self):"},{"line_number":123,"context_line":"        kwargs \u003d {"},{"line_number":124,"context_line":"            \u0027code\u0027: 413,"},{"line_number":125,"context_line":"            \u0027method\u0027: None,"},{"line_number":126,"context_line":"            \u0027url\u0027: \"http://foo.com\","}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVX%2F%2F4ks%3D","line":123,"updated":"2014-02-27 15:58:36.000000000","message":"Sorry, I totally missed this the first time.\n\nkwargs is not being used for anything, so we could simplify the test getting rid of it and just initializing OverLimit exception with:\n\n    clients.novaclient.exceptions.OverLimit(413, \"limit reached\")","commit_id":"2a7334794c022317dd1ce45ed3d349503b0e9726"},{"author":{"_account_id":7253,"name":"Jason Dunsmore","email":"jasondunsmore@gmail.com","username":"jbd"},"change_message_id":"355db916bcc1f0e75a98bafe99ead8e888269549","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        self.m.VerifyAll()"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def test_overlimit_error(self):"},{"line_number":123,"context_line":"        kwargs \u003d {"},{"line_number":124,"context_line":"            \u0027code\u0027: 413,"},{"line_number":125,"context_line":"            \u0027method\u0027: None,"},{"line_number":126,"context_line":"            \u0027url\u0027: \"http://foo.com\","}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAVX%2F%2F4cA%3D","line":123,"in_reply_to":"AAAAVX%2F%2F4ks%3D","updated":"2014-02-27 16:08:47.000000000","message":"Done","commit_id":"2a7334794c022317dd1ce45ed3d349503b0e9726"}]}