)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6685,"name":"Baodong (Robert) Li","email":"baoli@cisco.com","username":"baoli"},"change_message_id":"f0e4e5573ba249fb822bc6fffe097a575f450859","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_3ad7f2ee","line":12,"updated":"2015-06-03 12:30:57.000000000","message":"When this notification is added, the VIF_DELETED event will be generated regardless if it\u0027s a result of deleting a VM (by Nova) or of deleting a port associated with a running instance (by Neutron). In the former case (which is the majority of cases), the event will be dropped as it\u0027s very much likely the instance no longer exists in nova by the time the event is received. This is actually similar to VIF_UNPLUGGED event, which would be dropped by nova due to unknown instance.\n\nThis is also the main reason that I didn\u0027t put the two patches as dependent to each other.","commit_id":"6575db592c92791a51540134192bc86465940283"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2e60a3eca4c729720d6412ee21d234eb5cfc946f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_7ade7c40","line":12,"in_reply_to":"5afe65bd_18067ece","updated":"2015-06-03 16:55:24.000000000","message":"I honestly don\u0027t think it\u0027s a matter of making people edgy. It\u0027s about doing the right thing :)\n\nWe should strive to avoid causing unnecessary noise. The noise is caused by the fact that nova doesn\u0027t recognize the event when validating it.","commit_id":"6575db592c92791a51540134192bc86465940283"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"5fe64af948ab620a6e306caaaacd8c578ed6c58b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_9ad49847","line":12,"in_reply_to":"5afe65bd_3ad7f2ee","updated":"2015-06-03 16:50:26.000000000","message":"Right, the issue is that the nova support isn\u0027t there so it throws an exception for every port delete because it doesn\u0027t recognize the event type.","commit_id":"6575db592c92791a51540134192bc86465940283"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"0647c2cdce802401bf2ee9a2a0408534427bc283","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_18067ece","line":12,"in_reply_to":"5afe65bd_3ad7f2ee","updated":"2015-06-03 13:05:26.000000000","message":"Robert, I agree, but the new log messages are apparently making people nervous. Let\u0027s wait for the nova-side patch to merge and then we can revert this revert. No big deal.","commit_id":"6575db592c92791a51540134192bc86465940283"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"9b4c7f44c231e8504520ef1da6d44f8919a5ce93","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3afb71cf_b36e7f94","line":12,"in_reply_to":"5afe65bd_8e6e96ff","updated":"2015-06-08 03:00:12.000000000","message":"check out the logstash link in the bug report.","commit_id":"6575db592c92791a51540134192bc86465940283"},{"author":{"_account_id":6685,"name":"Baodong (Robert) Li","email":"baoli@cisco.com","username":"baoli"},"change_message_id":"3d7f1bb7d7afb03e0d2d1621a24609e952b2b88d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We need to wait until the nova support is added in"},{"line_number":10,"context_line":"I998b6bb80cc0a81d665b61b8c4a424d7219c666f. Otherwise"},{"line_number":11,"context_line":"this generates a ton of error messages in the nova api"},{"line_number":12,"context_line":"log as well as on the neutron side."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This reverts commit 0ace88fd4a75ff213dc36fd16c1f8e7080ab7d6d."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_8e6e96ff","line":12,"in_reply_to":"5afe65bd_9ad49847","updated":"2015-06-03 18:43:03.000000000","message":"I may have missed it. Can you point to the log where Nova throws exception? Searching the strings vif-unplugged and vif-deleted in this log: http://logs.openstack.org/66/178666/7/check/check-tempest-dsvm-neutron-full/6cf0279/logs/screen-n-api.txt.gz\n\nNot intended to introduce extra noise, though. As commented earlier, dropping the event cannot be avoided in the case that Nova deletes the port as a result of deleting the instance.\n\nI have been thinking how to avoid this given the same API is used to delete a port by both the user and Nova. But given that it doesn\u0027t seem to be possible to differentiate between the two cases in Neutron, Neutron sends the event whenever a port is deleted. Any idea?","commit_id":"6575db592c92791a51540134192bc86465940283"}]}