)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1a3cb6626392f010bb3d60b605d631ec5c15c395","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"030b4877_57df1759","updated":"2023-01-03 17:09:01.000000000","message":"Thanks for your patch, This isn\u0027t really a \"critical\" bug or a bug at all; but perhaps an opportunity for enhancement. Please see comment inline","commit_id":"3903caabd442098d4ff3dda491f5ede07cea6881"}],"manila/test.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1a3cb6626392f010bb3d60b605d631ec5c15c395","unresolved":true,"context_lines":[{"line_number":211,"context_line":"            CONF.set_override(k, v)"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    def start_service(self, name, host\u003dNone, **kwargs):"},{"line_number":214,"context_line":"        host \u003d host and host or uuidutils.generate_uuid()"},{"line_number":215,"context_line":"        kwargs.setdefault(\u0027host\u0027, host)"},{"line_number":216,"context_line":"        kwargs.setdefault(\u0027binary\u0027, \u0027manila-%s\u0027 % name)"},{"line_number":217,"context_line":"        svc \u003d service.Service.create(**kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"8636674e_9426b087","side":"PARENT","line":214,"range":{"start_line":214,"start_character":15,"end_line":214,"end_character":28},"updated":"2023-01-03 17:09:01.000000000","message":"it\u0027s not really a mistake - i.e., if host is None, this does work like you\u0027re fixing it. If you\u0027d like to make it more readable, you can change this to:\n\n host \u003d host or uuidutils.generate_uuid()","commit_id":"c81ff020f39eed6722de4b284f6b8468b3ca80c1"}]}