)]}'
{"cinder/tests/unit/api/v3/test_default_types.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"3494f772548cd3e38562ec8028ab6a34825134b1","unresolved":true,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from unittest import mock"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from cinder.api import microversions as mv"},{"line_number":21,"context_line":"from cinder.api.v3 import default_types"}],"source_content_type":"text/x-python","patch_set":1,"id":"1d3c642f_e421de2c","line":18,"in_reply_to":"0ff7ae91_700d0ded","updated":"2021-09-03 06:18:18.000000000","message":"Not sure what you mean by other patches but here we are removing tests which make it forbidden for a project admin to alter default types for other projects. This was done keeping in mind system scope but we are not implementing that in Xena. In summary, we don\u0027t need webob here anymore and i don\u0027t think a similar case occurs in other RBAC patches.","commit_id":"05d4f3b02f594e3c3bb72cbd169099bafef1fed2"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f1265de7c2f7d3d98a7e7497c5c2d55f1b743e57","unresolved":true,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from unittest import mock"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from cinder.api import microversions as mv"},{"line_number":21,"context_line":"from cinder.api.v3 import default_types"}],"source_content_type":"text/x-python","patch_set":1,"id":"0ff7ae91_700d0ded","line":18,"in_reply_to":"5ffdc214_0ab2aa45","updated":"2021-09-02 20:42:22.000000000","message":"\u003e pep8: F401 \u0027webob\u0027 imported but unused\n\nmmh.. Why is this not showing in other patches results..","commit_id":"05d4f3b02f594e3c3bb72cbd169099bafef1fed2"}]}