)]}'
{"ironic_python_agent/extensions/image.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"24fc1bec2e66480b08bec8fb2280430011fbb6fa","unresolved":false,"context_lines":[{"line_number":403,"context_line":"            try:"},{"line_number":404,"context_line":"                utils.execute(\u0027mount\u0027, \u0027-o\u0027, \u0027bind\u0027, fs, path + fs)"},{"line_number":405,"context_line":"            except Exception:"},{"line_number":406,"context_line":"                pathlist \u003d \u0027. \u0027.join(os.listdir(path))"},{"line_number":407,"context_line":"                log.debug(\u0027Error encountered mounting folders, path \u0027"},{"line_number":408,"context_line":"                          \u0027contents for list may be missing. Contents are: %s\u0027,"},{"line_number":409,"context_line":"                          pathlist)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_65306257","line":406,"updated":"2020-04-02 08:47:34.000000000","message":"let\u0027s use save_and_reraise_exception from oslo here?","commit_id":"874e3ca5a8389448601fd8ea46a642e2ef63875b"},{"author":{"_account_id":11292,"name":"Arne Wiebalck","email":"Arne.Wiebalck@cern.ch","username":"wiebalck"},"change_message_id":"cfd20eb3ed98836f80a522892254533b2df97b9f","unresolved":false,"context_lines":[{"line_number":406,"context_line":"            except Exception as exc:"},{"line_number":407,"context_line":"                with excutils.save_and_reraise_exception():"},{"line_number":408,"context_line":"                    pathlist \u003d \u0027. \u0027.join(os.listdir(path))"},{"line_number":409,"context_line":"                    log.error(\u0027Error encountered mounting folders, path \u0027"},{"line_number":410,"context_line":"                              \u0027contents for list may be missing. Contents \u0027"},{"line_number":411,"context_line":"                              \u0027are: %{pathlist}s. Exception: %(exctype)s, \u0027"},{"line_number":412,"context_line":"                              \u0027Error: %(err)s\u0027,"},{"line_number":413,"context_line":"                              {\u0027pathlist\u0027: pathlist,"}],"source_content_type":"text/x-python","patch_set":2,"id":"df33271e_3612a959","line":410,"range":{"start_line":409,"start_character":67,"end_line":410,"end_character":63},"updated":"2020-04-07 14:26:46.000000000","message":"Couldn\u0027t we check if the content is empty (and then say so)? I\u0027d think that if there is another error, we may confuse the reader, no?","commit_id":"dfbefe7107c0348cb2808c63b044bfaa2324e4cb"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"fe11098960cc3c8e35c650ced16d9516ef77b92a","unresolved":false,"context_lines":[{"line_number":408,"context_line":"                    pathlist \u003d \u0027. \u0027.join(os.listdir(path))"},{"line_number":409,"context_line":"                    log.error(\u0027Error encountered mounting folders, path \u0027"},{"line_number":410,"context_line":"                              \u0027contents for list may be missing. Contents \u0027"},{"line_number":411,"context_line":"                              \u0027are: %{pathlist}s. Exception: %(exctype)s, \u0027"},{"line_number":412,"context_line":"                              \u0027Error: %(err)s\u0027,"},{"line_number":413,"context_line":"                              {\u0027pathlist\u0027: pathlist,"},{"line_number":414,"context_line":"                               \u0027exctype\u0027: type(exc),"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_f72abb29","line":411,"updated":"2020-04-27 10:53:37.000000000","message":"incorrect brackets: {} instead of ()?","commit_id":"dfbefe7107c0348cb2808c63b044bfaa2324e4cb"}]}