)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"db485eb9d764ce02b87422d770584e5f70460de8","unresolved":false,"context_lines":[{"line_number":561,"context_line":"                     else \u0027false\u0027)"},{"line_number":562,"context_line":"            for ls in self._nb_idl.ls_list().execute(check_error\u003dTrue):"},{"line_number":563,"context_line":"                if (ls.other_config.get(ovn_const.MCAST_SNOOP,"},{"line_number":564,"context_line":"                                        None) \u003d\u003d value and not ls.name):"},{"line_number":565,"context_line":"                    continue"},{"line_number":566,"context_line":"                txn.add(self._nb_idl.db_set("},{"line_number":567,"context_line":"                    \u0027Logical_Switch\u0027, ls.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"fffc6b78_ea7d166f","line":564,"range":{"start_line":564,"start_character":55,"end_line":564,"end_character":58},"updated":"2020-11-20 09:12:19.000000000","message":"shouldn\u0027t be \"or\" here?","commit_id":"028a77df250ccd44012588feeda2435a2850541b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3439ab6e2d7260454164e820bd4ccedec6d01119","unresolved":false,"context_lines":[{"line_number":561,"context_line":"                     else \u0027false\u0027)"},{"line_number":562,"context_line":"            for ls in self._nb_idl.ls_list().execute(check_error\u003dTrue):"},{"line_number":563,"context_line":"                if (ls.other_config.get(ovn_const.MCAST_SNOOP,"},{"line_number":564,"context_line":"                                        None) \u003d\u003d value and not ls.name):"},{"line_number":565,"context_line":"                    continue"},{"line_number":566,"context_line":"                txn.add(self._nb_idl.db_set("},{"line_number":567,"context_line":"                    \u0027Logical_Switch\u0027, ls.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"fffc6b78_0ac21a86","line":564,"range":{"start_line":564,"start_character":55,"end_line":564,"end_character":58},"in_reply_to":"fffc6b78_ea7d166f","updated":"2020-11-20 09:18:53.000000000","message":"Of course, but I missed the lecture about boolean logic in the University hehehehe\n\nThanks!","commit_id":"028a77df250ccd44012588feeda2435a2850541b"}],"neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"821fe885fe7cf058e435a30276995e86e8fa8ccd","unresolved":true,"context_lines":[{"line_number":334,"context_line":"                          self.periodic.check_for_igmp_snoop_support)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"        # \"ls2\" is not part of the transaction because it already"},{"line_number":337,"context_line":"        # have the right value set; \"ls3\" does not have a name set."},{"line_number":338,"context_line":"        expected_calls \u003d ["},{"line_number":339,"context_line":"            mock.call(\u0027Logical_Switch\u0027, \u0027ls0\u0027,"},{"line_number":340,"context_line":"                      (\u0027other_config\u0027, {"}],"source_content_type":"text/x-python","patch_set":4,"id":"43d77b77_09dd89f6","line":337,"updated":"2020-11-24 09:45:17.000000000","message":"nit: I think You missed \"ls4\" which also isn\u0027t part of transaction because don\u0027t have name set","commit_id":"5a31611a6a85bc7ba737d79c519518c0b11ba7be"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"14ba4d4dae7cec269f489e3449fd93bb7b3f88ab","unresolved":false,"context_lines":[{"line_number":334,"context_line":"                          self.periodic.check_for_igmp_snoop_support)"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"        # \"ls2\" is not part of the transaction because it already"},{"line_number":337,"context_line":"        # have the right value set; \"ls3\" does not have a name set."},{"line_number":338,"context_line":"        expected_calls \u003d ["},{"line_number":339,"context_line":"            mock.call(\u0027Logical_Switch\u0027, \u0027ls0\u0027,"},{"line_number":340,"context_line":"                      (\u0027other_config\u0027, {"}],"source_content_type":"text/x-python","patch_set":4,"id":"993110e4_c52251ad","line":337,"in_reply_to":"43d77b77_09dd89f6","updated":"2020-11-24 09:46:46.000000000","message":"Done","commit_id":"5a31611a6a85bc7ba737d79c519518c0b11ba7be"}]}