)]}'
{"ovsdbapp/backend/ovs_idl/__init__.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6729d079476508493e6a38da932c04e8a1d1c3c4","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                idx \u003d self.idl.tables[table].rows.index_create(index_name)"},{"line_number":80,"context_line":"            except ValueError:"},{"line_number":81,"context_line":"                LOG.info(\"lookup_table index %s.%s already exists\", table,"},{"line_number":82,"context_line":"                         index_name)"},{"line_number":83,"context_line":"            else:"},{"line_number":84,"context_line":"                idx.add_column(col)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff570b3c_b4692bf9","line":81,"range":{"start_line":81,"start_character":20,"end_line":81,"end_character":24},"updated":"2020-06-03 08:23:39.000000000","message":"nit: maybe \"info\" here and in L100 is too much, debug could be enough. I think this is something that could happen frequently and I don\u0027t want to populate excessively the logs.","commit_id":"1f6a5aa343ddddba463c2d57e931a3f5c2e76e86"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"df5b028260c7ef45c8d4557f7267528e0d7e3330","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                idx \u003d self.idl.tables[table].rows.index_create(index_name)"},{"line_number":80,"context_line":"            except ValueError:"},{"line_number":81,"context_line":"                LOG.info(\"lookup_table index %s.%s already exists\", table,"},{"line_number":82,"context_line":"                         index_name)"},{"line_number":83,"context_line":"            else:"},{"line_number":84,"context_line":"                idx.add_column(col)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff570b3c_79ab00d5","line":81,"range":{"start_line":81,"start_character":20,"end_line":81,"end_character":24},"in_reply_to":"ff570b3c_9447c777","updated":"2020-06-03 17:46:53.000000000","message":"This should rarely happen, and ultimately I\u0027d like to see it *never* happen w/ only a single Backend instance per Connection; either because the connection is an instance attribute (ML2/OVN) or Backend/IdlImpl class being a Singleton if using a global/class-level connection (ML2/OVS). If ML2/OVS didn\u0027t currently hit this, I would have made these a LOG.warning.\n\nI\u0027m fine with switching to debug until https://review.opendev.org/#/c/732523/3 if you really want, though.","commit_id":"1f6a5aa343ddddba463c2d57e931a3f5c2e76e86"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"4ea0a6df26c2bf0cd81422163529b8106fe032d6","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            try:"},{"line_number":79,"context_line":"                idx \u003d self.idl.tables[table].rows.index_create(index_name)"},{"line_number":80,"context_line":"            except ValueError:"},{"line_number":81,"context_line":"                LOG.info(\"lookup_table index %s.%s already exists\", table,"},{"line_number":82,"context_line":"                         index_name)"},{"line_number":83,"context_line":"            else:"},{"line_number":84,"context_line":"                idx.add_column(col)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff570b3c_9447c777","line":81,"range":{"start_line":81,"start_character":20,"end_line":81,"end_character":24},"in_reply_to":"ff570b3c_b4692bf9","updated":"2020-06-03 08:25:24.000000000","message":"+1 and I think we can change that as zuul failed on this patch already so there will be need for another run.","commit_id":"1f6a5aa343ddddba463c2d57e931a3f5c2e76e86"}]}