)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"3ddeb34d5706203037d4496ec82df36d91fad58b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"In this particular case, dhcp agent is deleting the dhcp port when"},{"line_number":16,"context_line":"an auto addr subnet is getting created."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This patch catches PortNotFound exception."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ia1bb178720461decd21d983543b7281e146e6535"},{"line_number":21,"context_line":"Closes-bug: #1550278"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aed3d3a_7993eb4f","line":18,"updated":"2016-02-27 02:41:34.000000000","message":"what\u0027s the evidence of this happening in the gate besides the ovn plugin?","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"3d95a84c996b2b1086cba23e5d5d6768b9786f5d","unresolved":false,"context_lines":[{"line_number":15,"context_line":"In this particular case, dhcp agent is deleting the dhcp port when"},{"line_number":16,"context_line":"an auto addr subnet is getting created."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"This patch catches PortNotFound exception."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ia1bb178720461decd21d983543b7281e146e6535"},{"line_number":21,"context_line":"Closes-bug: #1550278"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aed3d3a_2f455ab2","line":18,"in_reply_to":"9aed3d3a_7993eb4f","updated":"2016-02-29 14:48:06.000000000","message":"I suspect it\u0027s worked around in the ML2 plugin.  It\u0027s not the first case we\u0027ve hit a race condition that the ML2 plugin works around with some while loops and retries.","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"3ddeb34d5706203037d4496ec82df36d91fad58b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"This patch catches PortNotFound exception."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ia1bb178720461decd21d983543b7281e146e6535"},{"line_number":21,"context_line":"Closes-bug: #1550278"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aed3d3a_99b7cfd7","line":21,"updated":"2016-02-27 02:41:34.000000000","message":"that\u0027s the type of change that would need a test.","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"}],"neutron/db/db_base_plugin_v2.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"3ddeb34d5706203037d4496ec82df36d91fad58b","unresolved":false,"context_lines":[{"line_number":594,"context_line":"                port_info \u003d {\u0027port\u0027: {\u0027id\u0027: port_id}}"},{"line_number":595,"context_line":"                try:"},{"line_number":596,"context_line":"                    self.update_port(context, port_id, port_info)"},{"line_number":597,"context_line":"                except n_exc.PortNotFound:"},{"line_number":598,"context_line":"                    # Its possible that port could have been deleted"},{"line_number":599,"context_line":"                    # by an api/rpc worker."},{"line_number":600,"context_line":"                    LOG.debug(\u0027Port update unsuccessful. Port %s\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_d99af762","line":597,"updated":"2016-02-27 02:41:34.000000000","message":"I am honestly not convinced that catching this exception and unconditionally pass is the right approach. This may mask real issues underlying the disappearance of a port.","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"242da3cc52b29a31630a54f045b88efd52ebb0a7","unresolved":false,"context_lines":[{"line_number":594,"context_line":"                port_info \u003d {\u0027port\u0027: {\u0027id\u0027: port_id}}"},{"line_number":595,"context_line":"                try:"},{"line_number":596,"context_line":"                    self.update_port(context, port_id, port_info)"},{"line_number":597,"context_line":"                except n_exc.PortNotFound:"},{"line_number":598,"context_line":"                    # Its possible that port could have been deleted"},{"line_number":599,"context_line":"                    # by an api/rpc worker."},{"line_number":600,"context_line":"                    LOG.debug(\u0027Port update unsuccessful. Port %s\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_dc0ae520","line":597,"in_reply_to":"9aed3d3a_d99af762","updated":"2016-02-27 03:03:22.000000000","message":"disappearance or absence","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"665e5261e403595cf6528e2ad9f73aa0098d4d33","unresolved":false,"context_lines":[{"line_number":598,"context_line":"                    # Its possible that port could have been deleted"},{"line_number":599,"context_line":"                    # by an api/rpc worker."},{"line_number":600,"context_line":"                    LOG.debug(\u0027Port update unsuccessful. Port %s\u0027"},{"line_number":601,"context_line":"                              \u0027 not found \u0027, port_id)"},{"line_number":602,"context_line":""},{"line_number":603,"context_line":"        return self._make_subnet_dict(subnet, context\u003dcontext)"},{"line_number":604,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_6cf507ca","line":601,"updated":"2016-02-26 16:11:22.000000000","message":"I wonder if the debug log entry is really needed if this is something we know can happen under \"normal\" circumstances","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"3ddeb34d5706203037d4496ec82df36d91fad58b","unresolved":false,"context_lines":[{"line_number":598,"context_line":"                    # Its possible that port could have been deleted"},{"line_number":599,"context_line":"                    # by an api/rpc worker."},{"line_number":600,"context_line":"                    LOG.debug(\u0027Port update unsuccessful. Port %s\u0027"},{"line_number":601,"context_line":"                              \u0027 not found \u0027, port_id)"},{"line_number":602,"context_line":""},{"line_number":603,"context_line":"        return self._make_subnet_dict(subnet, context\u003dcontext)"},{"line_number":604,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_f99ffb4d","line":601,"in_reply_to":"9aed3d3a_6cf507ca","updated":"2016-02-27 02:41:34.000000000","message":"To be fair, I am not sure this can be \u0027normal\u0027 in all cases, I\u0027d be wary of making such a change, especially when there is not hard evidence that this race is not showing up besides OVN","commit_id":"7bd03efae11e103aae9f148af57e4110ae2e9994"}]}