)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c163df94d98c9239076a2ad94e73826563ee9507","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"47e76370_365fa4c4","updated":"2024-07-01 10:18:52.000000000","message":"recheck","commit_id":"f52e888dabaa75c7dde0faf7e121474ee3699f31"}],"openstackclient/volume/v3/service.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4f3887417bd998f193d02faac71521a689888486","unresolved":true,"context_lines":[{"line_number":39,"context_line":"        if service_client.api_version \u003e\u003d api_versions.APIVersion(\u00273.49\u0027):"},{"line_number":40,"context_line":"            columns.append(\"Backend State\")"},{"line_number":41,"context_line":"        if parsed_args.long:"},{"line_number":42,"context_line":"            columns.append(\"Disabled Reason\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        data \u003d service_client.services.list("},{"line_number":45,"context_line":"            parsed_args.host, parsed_args.service"}],"source_content_type":"text/x-python","patch_set":2,"id":"c46ac536_498aa6bc","line":42,"updated":"2024-07-05 06:42:11.000000000","message":"nit: this may change the order of columns compared to before this patch, rather move this append in front of the two others?","commit_id":"f52e888dabaa75c7dde0faf7e121474ee3699f31"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"642e2e40521a2bd191578f3bf0bf88122ff51dcd","unresolved":true,"context_lines":[{"line_number":39,"context_line":"        if service_client.api_version \u003e\u003d api_versions.APIVersion(\u00273.49\u0027):"},{"line_number":40,"context_line":"            columns.append(\"Backend State\")"},{"line_number":41,"context_line":"        if parsed_args.long:"},{"line_number":42,"context_line":"            columns.append(\"Disabled Reason\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        data \u003d service_client.services.list("},{"line_number":45,"context_line":"            parsed_args.host, parsed_args.service"}],"source_content_type":"text/x-python","patch_set":2,"id":"51b89fb2_fc296831","line":42,"in_reply_to":"1cedc428_b38811e7","updated":"2024-07-05 09:18:43.000000000","message":"Yes, this does change order but I think that\u0027s probably a good thing. This field is effectively a free-length string so sticking it to the end makes sense to me intuitively, and the order shouldn\u0027t matter for folks specifying `-f value` since they normally pass `-c \u003ccolumn\u003e` too.","commit_id":"f52e888dabaa75c7dde0faf7e121474ee3699f31"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"c95ce7525422d3ccdf69f37565a29a5b4a39b65d","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        if service_client.api_version \u003e\u003d api_versions.APIVersion(\u00273.49\u0027):"},{"line_number":40,"context_line":"            columns.append(\"Backend State\")"},{"line_number":41,"context_line":"        if parsed_args.long:"},{"line_number":42,"context_line":"            columns.append(\"Disabled Reason\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        data \u003d service_client.services.list("},{"line_number":45,"context_line":"            parsed_args.host, parsed_args.service"}],"source_content_type":"text/x-python","patch_set":2,"id":"29638dcc_93e942ad","line":42,"in_reply_to":"51b89fb2_fc296831","updated":"2024-07-11 13:04:27.000000000","message":"Hmm, I guess that\u0027s ok-ish, then","commit_id":"f52e888dabaa75c7dde0faf7e121474ee3699f31"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"418d3db2f64270eb578ec8e2c3499428c9b342e9","unresolved":true,"context_lines":[{"line_number":39,"context_line":"        if service_client.api_version \u003e\u003d api_versions.APIVersion(\u00273.49\u0027):"},{"line_number":40,"context_line":"            columns.append(\"Backend State\")"},{"line_number":41,"context_line":"        if parsed_args.long:"},{"line_number":42,"context_line":"            columns.append(\"Disabled Reason\")"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"        data \u003d service_client.services.list("},{"line_number":45,"context_line":"            parsed_args.host, parsed_args.service"}],"source_content_type":"text/x-python","patch_set":2,"id":"1cedc428_b38811e7","line":42,"in_reply_to":"c46ac536_498aa6bc","updated":"2024-07-05 06:54:22.000000000","message":"Stephen asked to do this after the microversion check so I\u0027m not sure if he had something in mind. I don\u0027t personally have a preference but your comment makes sense that we should be consistent with the output. not sure what to do here ... will wait for stephen to comment on this.","commit_id":"f52e888dabaa75c7dde0faf7e121474ee3699f31"}]}