)]}'
{"tempest/api/compute/admin/test_quotas.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"14021340d3a773bf2a0dd56fd3f0caf30049ab04","unresolved":false,"context_lines":[{"line_number":224,"context_line":"            # there is a real chance that we go from -1 (unlimited)"},{"line_number":225,"context_line":"            # to a very small number which causes issues."},{"line_number":226,"context_line":"            body[quota] \u003d default + 100"},{"line_number":227,"context_line":"        LOG.debug(\"update limits for the default quota class set\")"},{"line_number":228,"context_line":"        update_body \u003d self.adm_client.update_quota_class_set("},{"line_number":229,"context_line":"            \u0027default\u0027, **body)[\u0027quota_class_set\u0027]"},{"line_number":230,"context_line":"        LOG.debug(\"assert that the response has all of the changed values\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffb9cba7_c534374d","line":227,"range":{"start_line":227,"start_character":8,"end_line":227,"end_character":66},"updated":"2019-04-24 02:54:10.000000000","message":"Unrelated, but log statements like this are pretty unusual. They look more like \"hey I\u0027m debugging this while I\u0027m implementing this test\" than an actual useful LOG debug statement.","commit_id":"6ff2b5a113162449ea2b822b471bfb2938f95e09"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"b61e633c759b627f3416196fc253950bbb6dbca2","unresolved":false,"context_lines":[{"line_number":224,"context_line":"            # there is a real chance that we go from -1 (unlimited)"},{"line_number":225,"context_line":"            # to a very small number which causes issues."},{"line_number":226,"context_line":"            body[quota] \u003d default + 100"},{"line_number":227,"context_line":"        LOG.debug(\"update limits for the default quota class set\")"},{"line_number":228,"context_line":"        update_body \u003d self.adm_client.update_quota_class_set("},{"line_number":229,"context_line":"            \u0027default\u0027, **body)[\u0027quota_class_set\u0027]"},{"line_number":230,"context_line":"        LOG.debug(\"assert that the response has all of the changed values\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffb9cba7_25367305","line":227,"range":{"start_line":227,"start_character":8,"end_line":227,"end_character":66},"in_reply_to":"ffb9cba7_c534374d","updated":"2019-04-24 03:12:25.000000000","message":"Yes I have the same feeling!\nMaybe these statements should be used as comments?","commit_id":"6ff2b5a113162449ea2b822b471bfb2938f95e09"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"14021340d3a773bf2a0dd56fd3f0caf30049ab04","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        LOG.debug(\"update limits for the default quota class set\")"},{"line_number":228,"context_line":"        update_body \u003d self.adm_client.update_quota_class_set("},{"line_number":229,"context_line":"            \u0027default\u0027, **body)[\u0027quota_class_set\u0027]"},{"line_number":230,"context_line":"        LOG.debug(\"assert that the response has all of the changed values\")"},{"line_number":231,"context_line":"        self.assertThat(update_body.items(),"},{"line_number":232,"context_line":"                        matchers.ContainsAll(body.items()))"},{"line_number":233,"context_line":"        # check quota values are changed"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffb9cba7_853e3f27","line":230,"range":{"start_line":230,"start_character":8,"end_line":230,"end_character":75},"updated":"2019-04-24 02:54:10.000000000","message":"Ditto.","commit_id":"6ff2b5a113162449ea2b822b471bfb2938f95e09"}]}