)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"aba9f71e8ed44d34e74812f7ac194aa5858294c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"656202e0_1c9753f5","updated":"2021-10-13 06:06:49.000000000","message":"-1 is to confirm whether an empty string should be allowed or not.","commit_id":"e91039d53e3160c4c84d04221f327ca0569d2e30"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a8ed79f8e3f26b57e02baf16cd72c845eadaf0f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4b886d4c_e17eae01","updated":"2021-11-15 09:19:51.000000000","message":"recheck","commit_id":"1f4c4031ee8a0dc91149dc6a83c7051db7628f23"}],"neutron_lib/api/definitions/extra_dhcp_opt.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"aba9f71e8ed44d34e74812f7ac194aa5858294c4","unresolved":true,"context_lines":[{"line_number":33,"context_line":"                    \u0027type:values\u0027: [4, 6],"},{"line_number":34,"context_line":"                    \u0027required\u0027: False}},"},{"line_number":35,"context_line":"    # key spec if opt_name not in _VALID_BLANK_EXTRA_DHCP_OPTS"},{"line_number":36,"context_line":"    {\u0027opt_name\u0027: {\u0027type:oneline_string\u0027: DHCP_OPT_NAME_MAX_LEN,"},{"line_number":37,"context_line":"                  \u0027required\u0027: True},"},{"line_number":38,"context_line":"     \u0027opt_value\u0027: {\u0027type:oneline_string_or_none\u0027:"},{"line_number":39,"context_line":"                   DHCP_OPT_VALUE_MAX_LEN,"}],"source_content_type":"text/x-python","patch_set":1,"id":"05b2d132_75f43d82","line":36,"updated":"2021-10-13 06:06:49.000000000","message":"The previous validation does not allow an empty string, but the new validation allows it. It is intentional?\n\nNote that the behavior of the new validator (validate_oneline_string) itself looks good to me.","commit_id":"e91039d53e3160c4c84d04221f327ca0569d2e30"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"4a5afdd0be7e4ec35f0a657b5114b06af3cf1f7e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"                    \u0027type:values\u0027: [4, 6],"},{"line_number":34,"context_line":"                    \u0027required\u0027: False}},"},{"line_number":35,"context_line":"    # key spec if opt_name not in _VALID_BLANK_EXTRA_DHCP_OPTS"},{"line_number":36,"context_line":"    {\u0027opt_name\u0027: {\u0027type:oneline_string\u0027: DHCP_OPT_NAME_MAX_LEN,"},{"line_number":37,"context_line":"                  \u0027required\u0027: True},"},{"line_number":38,"context_line":"     \u0027opt_value\u0027: {\u0027type:oneline_string_or_none\u0027:"},{"line_number":39,"context_line":"                   DHCP_OPT_VALUE_MAX_LEN,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa721e38_0a134041","line":36,"in_reply_to":"05b2d132_75f43d82","updated":"2021-10-25 20:07:00.000000000","message":"You\u0027re right. I changed it.","commit_id":"e91039d53e3160c4c84d04221f327ca0569d2e30"}]}