)]}'
{"cinder/tests/unit/api/contrib/test_scheduler_hints.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from http import client as http_client"},{"line_number":17,"context_line":"import datetime"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import ddt"},{"line_number":20,"context_line":"from oslo_serialization import jsonutils"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_3f1cfd64","line":17,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (http.client, datetime)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}],"cinder/tests/unit/api/contrib/test_types_manage.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1c8507f8137209c777d6905730b1743af0224a23","unresolved":false,"context_lines":[{"line_number":42,"context_line":"             \"key4\": \"value4\","},{"line_number":43,"context_line":"             \"key5\": \"value5\"}"},{"line_number":44,"context_line":"    return dict(id\u003did,"},{"line_number":45,"context_line":"                name\u003d\u0027vol_type_%s\u0027 % str(id),"},{"line_number":46,"context_line":"                description\u003d\u0027vol_type_desc_%s\u0027 % str(id),"},{"line_number":47,"context_line":"                extra_specs\u003dspecs)"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_ace0dbf9","line":45,"range":{"start_line":45,"start_character":37,"end_line":45,"end_character":40},"updated":"2020-10-09 00:05:48.000000000","message":"I don\u0027t think you need the str() on any of these.","commit_id":"4df8739b277b19f737cb83edf1d5c597d649432d"}],"cinder/tests/unit/api/contrib/test_volume_manage.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from http import client as http_client"},{"line_number":17,"context_line":"from urllib.parse import urlencode"},{"line_number":18,"context_line":"from unittest import mock"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import ddt"},{"line_number":21,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_1f21399f","line":18,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (urllib.parse.urlencode, unittest.mock)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}],"cinder/tests/unit/api/v2/test_limits.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1c8507f8137209c777d6905730b1743af0224a23","unresolved":false,"context_lines":[{"line_number":348,"context_line":"                \u0027(POST, /bar*, /bar.*, 5, second);\u0027"},{"line_number":349,"context_line":"                \u0027(Say, /derp*, /derp.*, 1, day)\u0027)"},{"line_number":350,"context_line":"        except ValueError as e:"},{"line_number":351,"context_line":"            self.assertFalse(str(e))"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        # Make sure the number of returned limits are correct"},{"line_number":354,"context_line":"        self.assertEqual(4, len(test_limits))"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_acc57b3f","line":351,"updated":"2020-10-09 00:05:48.000000000","message":"I guess this is OK, but I have no idea why this try/except block is here.","commit_id":"4df8739b277b19f737cb83edf1d5c597d649432d"}],"cinder/tests/unit/api/v2/test_snapshots.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import datetime"},{"line_number":17,"context_line":"from http import client as http_client"},{"line_number":18,"context_line":"from urllib import parse as urllib"},{"line_number":19,"context_line":"from unittest import mock"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"import ddt"},{"line_number":22,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_5f3611e9","line":19,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (urllib.parse, unittest.mock)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}],"cinder/tests/unit/api/v2/test_types.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1c8507f8137209c777d6905730b1743af0224a23","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    }"},{"line_number":40,"context_line":"    return dict("},{"line_number":41,"context_line":"        id\u003did,"},{"line_number":42,"context_line":"        name\u003d\u0027vol_type_%s\u0027 % str(id),"},{"line_number":43,"context_line":"        description\u003d\u0027vol_type_desc_%s\u0027 % str(id),"},{"line_number":44,"context_line":"        extra_specs\u003dspecs,"},{"line_number":45,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_4cdadf1c","line":42,"range":{"start_line":42,"start_character":29,"end_line":42,"end_character":32},"updated":"2020-10-09 00:05:48.000000000","message":"I don\u0027t think you need the string conversions.","commit_id":"4df8739b277b19f737cb83edf1d5c597d649432d"}],"cinder/tests/unit/api/v2/test_volumes.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from http import client as http_client"},{"line_number":19,"context_line":"import json"},{"line_number":20,"context_line":"import urllib"},{"line_number":21,"context_line":"from unittest import mock"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import ddt"},{"line_number":24,"context_line":"import fixtures"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_7f3955d6","line":21,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (urllib, unittest.mock)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}],"cinder/tests/unit/api/v3/test_group_types.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1c8507f8137209c777d6905730b1743af0224a23","unresolved":false,"context_lines":[{"line_number":45,"context_line":"    }"},{"line_number":46,"context_line":"    return dict("},{"line_number":47,"context_line":"        id\u003did,"},{"line_number":48,"context_line":"        name\u003d\u0027group_type_%s\u0027 % str(id),"},{"line_number":49,"context_line":"        description\u003d\u0027group_type_desc_%s\u0027 % str(id),"},{"line_number":50,"context_line":"        group_specs\u003dspecs,"},{"line_number":51,"context_line":"    )"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_8ce077cd","line":48,"range":{"start_line":48,"start_character":31,"end_line":48,"end_character":34},"updated":"2020-10-09 00:05:48.000000000","message":"I don\u0027t think you need the string conversions here.","commit_id":"4df8739b277b19f737cb83edf1d5c597d649432d"}],"cinder/tests/unit/api/v3/test_resource_filters.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        with mock.patch(\u0027cinder.api.common._FILTERS_COLLECTION\u0027, filters):"},{"line_number":56,"context_line":"            result \u003d self.controller.index(req)"},{"line_number":57,"context_line":"            self.assertCountEqual(self,"},{"line_number":58,"context_line":"                                 list(result),"},{"line_number":59,"context_line":"                                 [\u0027resource_filters\u0027])"},{"line_number":60,"context_line":"            self.assertCountEqual(self,"},{"line_number":61,"context_line":"                                 expected_filters,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_7f103550","line":58,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":56,"context_line":"            result \u003d self.controller.index(req)"},{"line_number":57,"context_line":"            self.assertCountEqual(self,"},{"line_number":58,"context_line":"                                 list(result),"},{"line_number":59,"context_line":"                                 [\u0027resource_filters\u0027])"},{"line_number":60,"context_line":"            self.assertCountEqual(self,"},{"line_number":61,"context_line":"                                 expected_filters,"},{"line_number":62,"context_line":"                                 result[\u0027resource_filters\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_5f0d7133","line":59,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                                 list(result),"},{"line_number":59,"context_line":"                                 [\u0027resource_filters\u0027])"},{"line_number":60,"context_line":"            self.assertCountEqual(self,"},{"line_number":61,"context_line":"                                 expected_filters,"},{"line_number":62,"context_line":"                                 result[\u0027resource_filters\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_bf164d49","line":61,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":59,"context_line":"                                 [\u0027resource_filters\u0027])"},{"line_number":60,"context_line":"            self.assertCountEqual(self,"},{"line_number":61,"context_line":"                                 expected_filters,"},{"line_number":62,"context_line":"                                 result[\u0027resource_filters\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_9f138959","line":62,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"7486ef13440b7560b1f7b65d0532b91903628bc9","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        with mock.patch(\u0027cinder.api.common._FILTERS_COLLECTION\u0027, filters):"},{"line_number":56,"context_line":"            result \u003d self.controller.index(req)"},{"line_number":57,"context_line":"            self.assertCountEqual(self, list(result), [\u0027resource_filters\u0027])"},{"line_number":58,"context_line":"            self.assertCountEqual(self, expected_filters, "},{"line_number":59,"context_line":"                                  result[\u0027resource_filters\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_a81575e8","line":58,"updated":"2020-10-08 08:05:46.000000000","message":"pep8: W291 trailing whitespace","commit_id":"df7634cc9d6498a51640e97cac6051a810b7a269"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6c15d718060b728d77bbf014abe5c97e01408e81","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        with mock.patch(\u0027cinder.api.common._FILTERS_COLLECTION\u0027, filters):"},{"line_number":56,"context_line":"            result \u003d self.controller.index(req)"},{"line_number":57,"context_line":"            self.assertCountEqual(self, list(result), [\u0027resource_filters\u0027])"},{"line_number":58,"context_line":"            self.assertCountEqual(self, expected_filters,"},{"line_number":59,"context_line":"                                  result[\u0027resource_filters\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_8c5357de","line":57,"range":{"start_line":57,"start_character":34,"end_line":57,"end_character":39},"updated":"2020-10-08 23:31:00.000000000","message":"need to remove this argument here and in the next line","commit_id":"4df8739b277b19f737cb83edf1d5c597d649432d"}],"cinder/tests/unit/api/v3/test_snapshot_manage.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from http import client as http_client"},{"line_number":16,"context_line":"from urllib.parse import urlencode"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import ddt"},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_df2a217c","line":17,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (urllib.parse.urlencode, unittest.mock)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}],"cinder/tests/unit/api/v3/test_volume_manage.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"657b9d28dd05aa8c599049642ad7549f4e406d35","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"from http import client as http_client"},{"line_number":16,"context_line":"from urllib.parse import urlencode"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import ddt"},{"line_number":20,"context_line":"from oslo_config import cfg"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_ff2565af","line":17,"updated":"2020-10-08 03:34:09.000000000","message":"pep8: H306: imports not in alphabetical order (urllib.parse.urlencode, unittest.mock)","commit_id":"db0626ea0f3081a9d4ea93c477dfd8df88dca3a9"}]}