)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f393204a91fb5cfc396fa53556246b3bbb0d6421","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the BEFORE_UPDATE event handler rather than waiting for"},{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff5d4b2f_62f8e5c3","line":14,"updated":"2017-09-19 19:50:07.000000000","message":"I am thinking we may want to add a api test where we would delete default secgroup, then update port, and check it doesn\u0027t spill 500 onto us. Will it trigger the error? Do you think we can add it to the patch?","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"4370308fdb393de8666583df1f0d035c94deca96","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the BEFORE_UPDATE event handler rather than waiting for"},{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df6247ed_ac34abdf","line":14,"updated":"2017-09-19 22:00:09.000000000","message":"Oh, possibly. Are tenants shared between tests?","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"48274899728d12cb4e2f67d9af492f532cb92d8e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the BEFORE_UPDATE event handler rather than waiting for"},{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bf57530f_af1dbabe","line":14,"in_reply_to":"df6247ed_ac34abdf","updated":"2017-09-20 05:18:01.000000000","message":"I think they are at least shared between test cases of the same test class. I also believe tempest reuses some tenants.","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"0393452f3ade77b886df2e7d5acf5431af0e5476","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the BEFORE_UPDATE event handler rather than waiting for"},{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff5d4b2f_e27855f9","line":14,"in_reply_to":"ff5d4b2f_427a8124","updated":"2017-09-19 20:03:39.000000000","message":"I am not sure that it explicitly deletes the group though: https://review.openstack.org/#/c/504021/1/neutron/tests/tempest/scenario/test_security_groups.py Could it be that in that test run, the group was deleted when tearing down tenant resources after another test case?","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"358fd42080eb845b06a43a890cb25e8a0fb82339","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the BEFORE_UPDATE event handler rather than waiting for"},{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff5d4b2f_427a8124","line":14,"in_reply_to":"ff5d4b2f_62f8e5c3","updated":"2017-09-19 19:55:38.000000000","message":"I think that\u0027s exactly what that test was that you sent me the traceback from. It deleted the default security group and then updated the port.","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f393204a91fb5cfc396fa53556246b3bbb0d6421","unresolved":false,"context_lines":[{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff5d4b2f_e25f35e3","line":15,"updated":"2017-09-19 19:50:07.000000000","message":"We need a bug report for the issue. Tell me if you would like me to create it.","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"0393452f3ade77b886df2e7d5acf5431af0e5476","unresolved":false,"context_lines":[{"line_number":12,"context_line":"it to happen inside of the port update transaction which"},{"line_number":13,"context_line":"violates the transaction semantics of the security group"},{"line_number":14,"context_line":"callbacks."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I1ce8b558b0a831adcebead512d97554173423955"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff5d4b2f_e22a95e5","line":15,"in_reply_to":"ff5d4b2f_e25f35e3","updated":"2017-09-19 20:03:39.000000000","message":"The bug: https://bugs.launchpad.net/neutron/+bug/1718282","commit_id":"1ed36631cd5ec41c7695bcd0478926b76ecc5c7f"}]}