)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"85f694961e6d12e88d06d883b043622fb195af7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"68eb0db7_04779ee6","updated":"2024-07-19 18:28:32.000000000","message":"recheck unrelated tempest failure","commit_id":"24a7ce864176421a22ed84914d23140754e12bc4"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"b36e1ae22aced7130c1166867a517de3a4962f94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1d5f321e_5e0c0e66","updated":"2024-07-19 20:07:19.000000000","message":"IMHO the changelog must include the rationale. It is not self-evident why this is needed. A reference to some kind of TC resolution would be welcome.","commit_id":"a8f49cd55223eb96884e103be1f335d0816f8e12"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"84e68e44aeafcbcc9ab368ce2217004f2fa79cde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"85a3d45f_1a9ec23c","updated":"2024-07-19 20:24:56.000000000","message":"what does the TC has to do with it? Release notes are there for functional changes, while here we are modernizing and standardising code basis for easier maintenance (see other changes in the topic). RN is absolutely not applicable here and rationale is given in the first change in the stack. The change is aligned within the Keystone Team.","commit_id":"a8f49cd55223eb96884e103be1f335d0816f8e12"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"cebd0106bbb67daccb5f87f83eda1ea60674190c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3f752538_251534e4","in_reply_to":"85a3d45f_1a9ec23c","updated":"2024-07-20 15:19:28.000000000","message":"One cannot see topics or stack when looking at git. At least put Related-Change-Id in there, make the logs greppable.","commit_id":"a8f49cd55223eb96884e103be1f335d0816f8e12"},{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"cebd0106bbb67daccb5f87f83eda1ea60674190c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2b8b2e65_fa11006d","updated":"2024-07-20 15:19:28.000000000","message":"Commit message looks good, but I have one small question about the tox.ini.","commit_id":"0182728f9ee2b2ebef83a7ec561fa02b5addf764"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"187e194850a79e2683bf17f14436b29d86fc946f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e9ae563b_e3720b9d","updated":"2024-07-22 10:39:27.000000000","message":"I don\u0027t see any changes in keystone/tests/unit/ksfixtures/__init__.py","commit_id":"a00839ca028304946e500cd14ddde322b6303ec8"}],"tox.ini":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"cebd0106bbb67daccb5f87f83eda1ea60674190c","unresolved":true,"context_lines":[{"line_number":121,"context_line":"    keystone/tests/unit/test_backend_ldap.py: E501"},{"line_number":122,"context_line":"    keystone/tests/unit/test_cli.py: E501"},{"line_number":123,"context_line":"    keystone/tests/unit/test_versions.py: E501"},{"line_number":124,"context_line":"    keystone/tests/unit/test_v3_auth.py: E203"},{"line_number":125,"context_line":"    keystone/tests/unit/test_v3_filters.py: E501"},{"line_number":126,"context_line":"    keystone/token/providers/jws/core.py: E501"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"6248d9c7_f4570557","line":124,"updated":"2024-07-20 15:19:28.000000000","message":"Juust 1 silly question: why are yuu adding a E203 here if you also ignore it wholesale in ingnore\u003d above?","commit_id":"0182728f9ee2b2ebef83a7ec561fa02b5addf764"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"424a58492f21ae0454b28d148a554935abc160fa","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    keystone/tests/unit/test_backend_ldap.py: E501"},{"line_number":122,"context_line":"    keystone/tests/unit/test_cli.py: E501"},{"line_number":123,"context_line":"    keystone/tests/unit/test_versions.py: E501"},{"line_number":124,"context_line":"    keystone/tests/unit/test_v3_auth.py: E203"},{"line_number":125,"context_line":"    keystone/tests/unit/test_v3_filters.py: E501"},{"line_number":126,"context_line":"    keystone/token/providers/jws/core.py: E501"},{"line_number":127,"context_line":""}],"source_content_type":"text/x-properties","patch_set":3,"id":"1639dc80_2401520a","line":124,"in_reply_to":"6248d9c7_f4570557","updated":"2024-07-22 07:03:47.000000000","message":"right, this is not necessary here. It was a leftover from different series where things were blackified in batches.","commit_id":"0182728f9ee2b2ebef83a7ec561fa02b5addf764"}]}
