)]}'
{"ceilometer/compute/virt/libvirt/inspector.py":[{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"4027f1e2ea3cbf822caf0e95424b27d4fcb8371f","unresolved":false,"context_lines":[{"line_number":149,"context_line":"                    errors\u003dblock_stats[4],"},{"line_number":150,"context_line":"                    wr_total_times\u003dblock_stats_flags[\u0027wr_total_times\u0027],"},{"line_number":151,"context_line":"                    rd_total_times\u003dblock_stats_flags[\u0027rd_total_times\u0027])"},{"line_number":152,"context_line":"            except libvirt.libvirtError:"},{"line_number":153,"context_line":"                # raised error even if lock is acquired while live migration,"},{"line_number":154,"context_line":"                # even it looks normal."},{"line_number":155,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_751d11bc","line":152,"range":{"start_line":152,"start_character":11,"end_line":152,"end_character":40},"updated":"2020-06-19 18:25:49.000000000","message":"I think we need a log here. Try/catch without logging make things pretty complicated when troubleshooting.","commit_id":"cac02e45059b8a157f482ee4ee929daabc023687"},{"author":{"_account_id":6732,"name":"Lingxian Kong","email":"anlin.kong@gmail.com","username":"kong"},"change_message_id":"7f4ea096b50b7c23b8f8855eded3f278df085f58","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            except libvirt.libvirtError as ex:"},{"line_number":153,"context_line":"                # raised error even if lock is acquired while live migration,"},{"line_number":154,"context_line":"                # even it looks normal."},{"line_number":155,"context_line":"                LOG.info(_(\"Error from libvirt while checking blockStats, \""},{"line_number":156,"context_line":"                           \"This may not be harmful, but please check : \""},{"line_number":157,"context_line":"                           \"%(ex)s\") % {\u0027ex\u0027: ex})"},{"line_number":158,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":7,"id":"bf51134e_dd29fbf2","line":155,"updated":"2020-07-01 21:12:13.000000000","message":"Could we change the log level to warning, given it\u0027s something should get cloud operator\u0027s attention?","commit_id":"476a80373e747b2f3f8504e3cba404d0ea6f71a4"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"1adb94639e1320ba0f187220c268a435f2d09592","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            except libvirt.libvirtError as ex:"},{"line_number":153,"context_line":"                # raised error even if lock is acquired while live migration,"},{"line_number":154,"context_line":"                # even it looks normal."},{"line_number":155,"context_line":"                LOG.info(_(\"Error from libvirt while checking blockStats, \""},{"line_number":156,"context_line":"                           \"This may not be harmful, but please check : \""},{"line_number":157,"context_line":"                           \"%(ex)s\") % {\u0027ex\u0027: ex})"},{"line_number":158,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":7,"id":"bf51134e_0fd052c9","line":155,"in_reply_to":"bf51134e_4fcfea56","updated":"2020-07-02 12:18:22.000000000","message":"1+ for changing for warning","commit_id":"476a80373e747b2f3f8504e3cba404d0ea6f71a4"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"5c9e9b64fa82d3a4d8e748e7eac14a8f805f2afc","unresolved":false,"context_lines":[{"line_number":152,"context_line":"            except libvirt.libvirtError as ex:"},{"line_number":153,"context_line":"                # raised error even if lock is acquired while live migration,"},{"line_number":154,"context_line":"                # even it looks normal."},{"line_number":155,"context_line":"                LOG.info(_(\"Error from libvirt while checking blockStats, \""},{"line_number":156,"context_line":"                           \"This may not be harmful, but please check : \""},{"line_number":157,"context_line":"                           \"%(ex)s\") % {\u0027ex\u0027: ex})"},{"line_number":158,"context_line":"                pass"}],"source_content_type":"text/x-python","patch_set":7,"id":"bf51134e_4fcfea56","line":155,"in_reply_to":"bf51134e_dd29fbf2","updated":"2020-07-02 12:11:58.000000000","message":"I agree, this should be a warning.","commit_id":"476a80373e747b2f3f8504e3cba404d0ea6f71a4"}]}