)]}'
{"neutron/tests/unit/agent/linux/openvswitch_firewall/test_firewall.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f75e4653f81caf07db3031f9cb32feb2f171b6b8","unresolved":true,"context_lines":[{"line_number":1054,"context_line":"                                             constants.IPv4, 100, [0])"},{"line_number":1055,"context_line":"            _, kwargs \u003d mock_delete_flows.call_args"},{"line_number":1056,"context_line":"            self.assertIn(\u0027cookie\u0027, kwargs)"},{"line_number":1057,"context_line":"            self.assertIs(ovs_lib.COOKIE_ANY, kwargs[\u0027cookie\u0027])"},{"line_number":1058,"context_line":""},{"line_number":1059,"context_line":""},{"line_number":1060,"context_line":"class TestCookieContext(base.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"b0de3f12_ecd00283","line":1057,"range":{"start_line":1057,"start_character":17,"end_line":1057,"end_character":25},"updated":"2020-12-11 21:12:01.000000000","message":"nit: I wonder if this should be assertEqual since the in-tree code is only ever doing comparisons like \"if cookie \u003d\u003d ovs_lib.COOKIE_ANY\", or maybe those should be using \u0027is\u0027, it\u0027s unclear to me what the intent was when this was introduced.  Either way this test should pass I think.","commit_id":"f4b64e519cdb9fd9c5046f21bc9f325341fd367f"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5b0fe523415c4e39f8a743521c11b1b709793666","unresolved":true,"context_lines":[{"line_number":1054,"context_line":"                                             constants.IPv4, 100, [0])"},{"line_number":1055,"context_line":"            _, kwargs \u003d mock_delete_flows.call_args"},{"line_number":1056,"context_line":"            self.assertIn(\u0027cookie\u0027, kwargs)"},{"line_number":1057,"context_line":"            self.assertIs(ovs_lib.COOKIE_ANY, kwargs[\u0027cookie\u0027])"},{"line_number":1058,"context_line":""},{"line_number":1059,"context_line":""},{"line_number":1060,"context_line":"class TestCookieContext(base.BaseTestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"a42e2337_35266a9b","line":1057,"range":{"start_line":1057,"start_character":17,"end_line":1057,"end_character":25},"in_reply_to":"b0de3f12_ecd00283","updated":"2020-12-14 09:15:19.000000000","message":"according to the python documentation a is b implies a\u003d\u003db so I think it\u0027s fine to left it as it is now.","commit_id":"f4b64e519cdb9fd9c5046f21bc9f325341fd367f"}]}