)]}'
{"cinder/brick/local_dev/lvm.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"8a1b829277c9a1eac303db4c33f846e9300c0524","unresolved":false,"context_lines":[{"line_number":179,"context_line":"                consumed_space \u003d pool_size / 100 * data_percent"},{"line_number":180,"context_line":"                free_space \u003d pool_size - consumed_space"},{"line_number":181,"context_line":"                free_space \u003d round(free_space, 2)"},{"line_number":182,"context_line":"        except putils.ProcessExecutionError as err:  # noqa"},{"line_number":183,"context_line":"            LOG.exception(\u0027Error querying thin pool about data_percent\u0027)"},{"line_number":184,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":185,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_8d45f746","line":182,"updated":"2019-12-19 18:55:55.000000000","message":"This seems to be a bug in the linting tools. It would error saying the variable err was assigned but never used, but clearly it is used in the last three logging lines below.","commit_id":"222943c3963a6eb074351759696d47fc597a5b8c"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"dc9d5041ecd4be841fb3f8399a85656f0a797050","unresolved":false,"context_lines":[{"line_number":179,"context_line":"                consumed_space \u003d pool_size / 100 * data_percent"},{"line_number":180,"context_line":"                free_space \u003d pool_size - consumed_space"},{"line_number":181,"context_line":"                free_space \u003d round(free_space, 2)"},{"line_number":182,"context_line":"        except putils.ProcessExecutionError as err:  # noqa"},{"line_number":183,"context_line":"            LOG.exception(\u0027Error querying thin pool about data_percent\u0027)"},{"line_number":184,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":185,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_98a773a9","line":182,"in_reply_to":"3fa7e38b_8d45f746","updated":"2019-12-19 20:35:32.000000000","message":"I think this is\n    https://github.com/PyCQA/pyflakes/issues/378\n\nif you\u0027d like to note that somewhere.","commit_id":"222943c3963a6eb074351759696d47fc597a5b8c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"2e318d4283adf8f759f5d05881d0def4c315f4ed","unresolved":false,"context_lines":[{"line_number":179,"context_line":"                consumed_space \u003d pool_size / 100 * data_percent"},{"line_number":180,"context_line":"                free_space \u003d pool_size - consumed_space"},{"line_number":181,"context_line":"                free_space \u003d round(free_space, 2)"},{"line_number":182,"context_line":"        except putils.ProcessExecutionError as err:  # noqa"},{"line_number":183,"context_line":"            LOG.exception(\u0027Error querying thin pool about data_percent\u0027)"},{"line_number":184,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":185,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_d81f2b6f","line":182,"in_reply_to":"3fa7e38b_9843f3b0","updated":"2019-12-19 20:55:48.000000000","message":"I wonder if might be related via the automagic that the logger does in LOG.exception to get whatever exception is in scope.","commit_id":"222943c3963a6eb074351759696d47fc597a5b8c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"0c5d407e6732138efef4b51151bf1475405bde17","unresolved":false,"context_lines":[{"line_number":179,"context_line":"                consumed_space \u003d pool_size / 100 * data_percent"},{"line_number":180,"context_line":"                free_space \u003d pool_size - consumed_space"},{"line_number":181,"context_line":"                free_space \u003d round(free_space, 2)"},{"line_number":182,"context_line":"        except putils.ProcessExecutionError as err:  # noqa"},{"line_number":183,"context_line":"            LOG.exception(\u0027Error querying thin pool about data_percent\u0027)"},{"line_number":184,"context_line":"            LOG.error(\u0027Cmd     :%s\u0027, err.cmd)"},{"line_number":185,"context_line":"            LOG.error(\u0027StdOut  :%s\u0027, err.stdout)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_9843f3b0","line":182,"in_reply_to":"3fa7e38b_98a773a9","updated":"2019-12-19 20:40:50.000000000","message":"That one looks like a little different scenario to this one. That at least appears to be more to do with an inner scope hiding an out scoped variable, resulting in the outer variable not being used. The case here is a little simpler than that.","commit_id":"222943c3963a6eb074351759696d47fc597a5b8c"}]}