)]}'
{"cue/api/controllers/v1/cluster.py":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"9b7d51afb1d0083df6095610c89cfc40e81616ba","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        if data.size \u003c\u003d 0:"},{"line_number":216,"context_line":"            raise exception.Invalid(_(\"Invalid cluster size provided\"))"},{"line_number":217,"context_line":"        elif data.size \u003e CONF.api.max_cluster_size:"},{"line_number":218,"context_line":"            raise exception.Invalid(_(\"Invalid cluster size, max size is: %d\")"},{"line_number":219,"context_line":"            % CONF.api.max_cluster_size)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"        # create new cluster object with required data from user"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_5b5f6d41","line":218,"updated":"2015-03-27 17:14:54.000000000","message":"413 would be a more appropriate error response than 400 here\n\nhttp://en.wikipedia.org/wiki/List_of_HTTP_status_codes\n\npython-cueclient can then use that to provide a more informative error to the user.","commit_id":"feb70da31f635799c162cbacd35e6b5d27995c33"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"70a977e10190705e4ae14d42db716a59a352d438","unresolved":false,"context_lines":[{"line_number":215,"context_line":"        if data.size \u003c\u003d 0:"},{"line_number":216,"context_line":"            raise exception.Invalid(_(\"Invalid cluster size provided\"))"},{"line_number":217,"context_line":"        elif data.size \u003e CONF.api.max_cluster_size:"},{"line_number":218,"context_line":"            raise exception.Invalid(_(\"Invalid cluster size, max size is: %d\")"},{"line_number":219,"context_line":"            % CONF.api.max_cluster_size)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"        # create new cluster object with required data from user"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_ecc1e22f","line":218,"in_reply_to":"fa963961_5b5f6d41","updated":"2015-03-27 19:37:33.000000000","message":"Done","commit_id":"feb70da31f635799c162cbacd35e6b5d27995c33"}],"cue/tests/api/v1/test_clusters.py":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"9133d4e9da169888bebc14c2da2b0eba6748c2ff","unresolved":false,"context_lines":[{"line_number":136,"context_line":"                            expect_errors\u003dTrue)"},{"line_number":137,"context_line":"        self.assertEqual(400, data.status_code,"},{"line_number":138,"context_line":"                         \u0027Invalid status code value received.\u0027)"},{"line_number":139,"context_line":"        self.assertEqual(\u0027400 Bad Request\u0027, data.status,"},{"line_number":140,"context_line":"                         \u0027Invalid status value received.\u0027)"},{"line_number":141,"context_line":"        self.assertIn(\u0027Invalid cluster size, max size is: \u0027 +"},{"line_number":142,"context_line":"                      str(CONF.api.max_cluster_size),"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_2f9467e0","line":139,"updated":"2015-03-27 15:35:13.000000000","message":"Is this a necessary test if we\u0027re already checking status_code?  Who\u0027s supplying \"Bad Request\"?  If it\u0027s pecan, does pecan have localization for the string?","commit_id":"feb70da31f635799c162cbacd35e6b5d27995c33"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"70a977e10190705e4ae14d42db716a59a352d438","unresolved":false,"context_lines":[{"line_number":136,"context_line":"                            expect_errors\u003dTrue)"},{"line_number":137,"context_line":"        self.assertEqual(400, data.status_code,"},{"line_number":138,"context_line":"                         \u0027Invalid status code value received.\u0027)"},{"line_number":139,"context_line":"        self.assertEqual(\u0027400 Bad Request\u0027, data.status,"},{"line_number":140,"context_line":"                         \u0027Invalid status value received.\u0027)"},{"line_number":141,"context_line":"        self.assertIn(\u0027Invalid cluster size, max size is: \u0027 +"},{"line_number":142,"context_line":"                      str(CONF.api.max_cluster_size),"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa963961_8ce7f6e3","line":139,"in_reply_to":"fa963961_2f9467e0","updated":"2015-03-27 19:37:33.000000000","message":"Done","commit_id":"feb70da31f635799c162cbacd35e6b5d27995c33"}]}
