)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e39a64f99d7c18d40011c159f0559a6015efbd67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"166a264f_9b38ced0","updated":"2022-09-15 17:58:13.000000000","message":"LGTM, thanks Liron!","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"402c6b4f88d37c2a82527f94a453c14c87c29e59","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8534c031_87cedc1c","updated":"2022-09-20 13:19:37.000000000","message":"Lets get this patch merged since it is causing manila_tempest_tests failing ","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"10e73b07bc93a4d281d3e7bf0bb8cb4bb2d68b4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"816178ef_1d772d79","updated":"2022-09-15 10:38:05.000000000","message":"recheck","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"41c1cb2aa0f81f033f0a6057eaf9b2b154b770b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8d123f19_4b622d10","updated":"2022-10-11 09:11:32.000000000","message":"recheck","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"5a4a70e4ff44973b969e2d2995e4dba8faffbb15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"df04f118_08d66925","updated":"2022-09-15 12:51:18.000000000","message":"recheck","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"cbec0e647fcb280ea98431d93d3c2d7950d64118","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ef8c127c_f1d9c645","in_reply_to":"8534c031_87cedc1c","updated":"2022-09-20 18:56:51.000000000","message":"Soniya, I caught the chat in the tripleo OFTC channels that the issue could be that fedora needs to package a newer version of ddt; If you think we could revert https://review.opendev.org/#/q/Iaf39f4a83e383fb9db7f519136b9169352ff84af in the meanwhile, let me know - else, we\u0027ll get this change reviewed and merged ; and wait on the packaging issues to be resolved..","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"7da672a9effaffba09135a7078818c703f32e67c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3a7b2f27_4d83859e","in_reply_to":"ef8c127c_f1d9c645","updated":"2022-10-11 09:11:20.000000000","message":"since then nobody has asked for this issue so IMHO we dont need to revert.\nlet\u0027s get this change reviewed and merged and wait for issue to be resolved :)","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"}],"manila_tempest_tests/tests/api/admin/test_share_types.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"71bb332a6e4cd87c58c58300d81ad3904631584a","unresolved":true,"context_lines":[{"line_number":122,"context_line":"         None),"},{"line_number":123,"context_line":"        (\u00272_50_public\u0027, \u00272.50\u0027, None, None, True),"},{"line_number":124,"context_line":"        (\u00272_50\u0027, \u00272.50\u0027, None, None, False),"},{"line_number":125,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name_description_public\u0027,"},{"line_number":126,"context_line":"         LATEST_MICROVERSION, data_utils.rand_name(\"type_updated\"),"},{"line_number":127,"context_line":"         \u0027description_updated\u0027, True),"},{"line_number":128,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name\u0027, LATEST_MICROVERSION,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa829b22_14c41c0e","line":125,"range":{"start_line":125,"start_character":12,"end_line":125,"end_character":31},"updated":"2022-09-15 17:22:38.000000000","message":"this will have a period, how about:\n\nf\u0027{LATEST_MICROVERSION}_name_description_public\u0027.replace(\u0027.\u0027, \u0027_\u0027)","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e39a64f99d7c18d40011c159f0559a6015efbd67","unresolved":true,"context_lines":[{"line_number":122,"context_line":"         None),"},{"line_number":123,"context_line":"        (\u00272_50_public\u0027, \u00272.50\u0027, None, None, True),"},{"line_number":124,"context_line":"        (\u00272_50\u0027, \u00272.50\u0027, None, None, False),"},{"line_number":125,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name_description_public\u0027,"},{"line_number":126,"context_line":"         LATEST_MICROVERSION, data_utils.rand_name(\"type_updated\"),"},{"line_number":127,"context_line":"         \u0027description_updated\u0027, True),"},{"line_number":128,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name\u0027, LATEST_MICROVERSION,"}],"source_content_type":"text/x-python","patch_set":1,"id":"e613cccb_f9c97264","line":125,"range":{"start_line":125,"start_character":12,"end_line":125,"end_character":31},"in_reply_to":"25b0a307_f486cd6b","updated":"2022-09-15 17:58:13.000000000","message":"true true","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"a43f056e91e9d686a27c9f149b323d2321f3d0b1","unresolved":true,"context_lines":[{"line_number":122,"context_line":"         None),"},{"line_number":123,"context_line":"        (\u00272_50_public\u0027, \u00272.50\u0027, None, None, True),"},{"line_number":124,"context_line":"        (\u00272_50\u0027, \u00272.50\u0027, None, None, False),"},{"line_number":125,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name_description_public\u0027,"},{"line_number":126,"context_line":"         LATEST_MICROVERSION, data_utils.rand_name(\"type_updated\"),"},{"line_number":127,"context_line":"         \u0027description_updated\u0027, True),"},{"line_number":128,"context_line":"        (f\u0027{LATEST_MICROVERSION}_name\u0027, LATEST_MICROVERSION,"}],"source_content_type":"text/x-python","patch_set":1,"id":"25b0a307_f486cd6b","line":125,"range":{"start_line":125,"start_character":12,"end_line":125,"end_character":31},"in_reply_to":"fa829b22_14c41c0e","updated":"2022-09-15 17:36:00.000000000","message":"But what does it matter ? ddt replaces it to underscore\nhttps://github.com/datadriventests/ddt/blob/master/ddt.py#L210\n\nI think it\u0027s redundant to do it for a variable.","commit_id":"8125ca48c33e987cb7ddf726a33390f01a30ccc6"}]}