)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","display_name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc","status":"Senior Software Quality Engineer @ Red Hat"},"change_message_id":"41f15ec99a130165a5f84b7289f6455ad9a0a1ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cda55b2b_af9b1c1a","updated":"2026-05-15 17:03:56.000000000","message":"Thanks for addressing my comment, LGTM","commit_id":"6becd6f45bd8776defcabdb6d6677325a02ccb79"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4d46be25803d471c31cdb33266cc1a8056906426","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"eeba1d48_23eccd73","updated":"2026-05-15 20:55:24.000000000","message":"nice, good check","commit_id":"6becd6f45bd8776defcabdb6d6677325a02ccb79"}],"tools/check_tests_documented.py":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","display_name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc","status":"Senior Software Quality Engineer @ Red Hat"},"change_message_id":"4b0acc36aebbe183c9b5f65919bc1a3c82ea6eb6","unresolved":true,"context_lines":[{"line_number":55,"context_line":"        for filename in files:"},{"line_number":56,"context_line":"            if not filename.endswith(\u0027.rst\u0027):"},{"line_number":57,"context_line":"                continue"},{"line_number":58,"context_line":"            with open(os.path.join(root, filename)) as f:"},{"line_number":59,"context_line":"                for line in f:"},{"line_number":60,"context_line":"                    line \u003d line.strip()"},{"line_number":61,"context_line":"                    if line.startswith(\u0027.. automodule::\u0027):"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ab913b9_20fe0a69","line":58,"updated":"2026-05-15 14:18:35.000000000","message":"Nit: should the encoding be specified here? for example:\n```suggestion\n            with open(os.path.join(root, filename), encoding\u003d\u0027utf-8\u0027) as f:\n```","commit_id":"6becd6f45bd8776defcabdb6d6677325a02ccb79"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c4174b8607c36038329c133a2449793b8ed1aa1f","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        for filename in files:"},{"line_number":56,"context_line":"            if not filename.endswith(\u0027.rst\u0027):"},{"line_number":57,"context_line":"                continue"},{"line_number":58,"context_line":"            with open(os.path.join(root, filename)) as f:"},{"line_number":59,"context_line":"                for line in f:"},{"line_number":60,"context_line":"                    line \u003d line.strip()"},{"line_number":61,"context_line":"                    if line.startswith(\u0027.. automodule::\u0027):"}],"source_content_type":"text/x-python","patch_set":2,"id":"f2228c40_eda226c5","line":58,"in_reply_to":"3ab913b9_20fe0a69","updated":"2026-05-15 15:10:40.000000000","message":"I don\u0027t think so. We\u0027re not doing it anywhere else in tempest, fwict, and this will default to `utf-8` anyway ([[1]](https://docs.python.org/3/library/functions.html#open), [[2]](https://docs.python.org/3/library/locale.html#locale.getencoding)) unless you do weird things with `LC_CTYPE`.","commit_id":"6becd6f45bd8776defcabdb6d6677325a02ccb79"}]}