)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ea43b336b1815b0e9ef2abe9bbcb44279f72f51a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"It has been some time since we marked the security group tests in"},{"line_number":10,"context_line":"fullstack as unstable.  Now that mark is impeding the fullstack tests"},{"line_number":11,"context_line":"for stateless security groups.  It\u0027s time to revisit if the unstable"},{"line_number":12,"context_line":"mark is still needed."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib1ef0763496aa536ba80f1c34c64d8a6c313fd0b"},{"line_number":15,"context_line":"Needed-By: https://review.opendev.org/710252"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1fa4df85_3008b82a","line":12,"updated":"2020-03-02 14:32:55.000000000","message":"Unfortunatelly I think it\u0027s still needed and I think that I may have some clue what may be wrong with it. IMO problem is that iptables rules modifications may somehow interfere between 2 tests and that may cause some problems. But to confirm that I think that indeed it would be good to maybe mark this test as stable now and get some fresh data to debug it.","commit_id":"f9af5356006831e0ac4db4a3fc575a437e1667a4"}],"neutron/tests/fullstack/test_securitygroup.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"21b65d6ed0213a4ccca066083901e7e763bba81e","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron.cmd.sanity import checks"},{"line_number":20,"context_line":"from neutron.common import utils as common_utils"},{"line_number":21,"context_line":"from neutron.tests import base as tests_base"},{"line_number":22,"context_line":"from neutron.tests.common import net_helpers"},{"line_number":23,"context_line":"from neutron.tests.fullstack import base"},{"line_number":24,"context_line":"from neutron.tests.fullstack.resources import environment"}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_5b31d5eb","line":21,"updated":"2020-03-02 14:35:08.000000000","message":"I think You will need to remove this import too as it\u0027s probably not used anymore in this file","commit_id":"f9af5356006831e0ac4db4a3fc575a437e1667a4"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"0ab5efefba8bd8dee54e00d99a48392eb1b1f917","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron.cmd.sanity import checks"},{"line_number":20,"context_line":"from neutron.common import utils as common_utils"},{"line_number":21,"context_line":"from neutron.tests import base as tests_base"},{"line_number":22,"context_line":"from neutron.tests.common import net_helpers"},{"line_number":23,"context_line":"from neutron.tests.fullstack import base"},{"line_number":24,"context_line":"from neutron.tests.fullstack.resources import environment"}],"source_content_type":"text/x-python","patch_set":1,"id":"1fa4df85_d25cb5ee","line":21,"in_reply_to":"1fa4df85_5b31d5eb","updated":"2020-03-03 14:25:51.000000000","message":"Done.  Will this fix be needed in https://review.opendev.org/#/c/710964/ as well?","commit_id":"f9af5356006831e0ac4db4a3fc575a437e1667a4"},{"author":{"_account_id":13995,"name":"Nate Johnston","email":"nate.johnston@redhat.com","username":"natejohnston"},"change_message_id":"7b2b00072df5c81395085da570f8091175b58d37","unresolved":false,"context_lines":[{"line_number":95,"context_line":"            \u0027firewall_driver\u0027: \u0027openvswitch\u0027,"},{"line_number":96,"context_line":"            \u0027l2_agent_type\u0027: constants.AGENT_TYPE_OVS,"},{"line_number":97,"context_line":"            \u0027num_hosts\u0027: 2})]"},{"line_number":98,"context_line":"        # Linux Bridge Iptables tests are unstable"},{"line_number":99,"context_line":"        #(\u0027linuxbridge-iptables\u0027, {"},{"line_number":100,"context_line":"        #    \u0027firewall_driver\u0027: \u0027iptables\u0027,"},{"line_number":101,"context_line":"        #    \u0027l2_agent_type\u0027: constants.AGENT_TYPE_LINUXBRIDGE,"},{"line_number":102,"context_line":"        #    \u0027num_hosts\u0027: 2})]"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    index_to_sg \u003d [0, 0, 1, 2]"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1fa4df85_8e4cccc8","line":102,"range":{"start_line":98,"start_character":0,"end_line":102,"end_character":30},"updated":"2020-03-20 12:54:28.000000000","message":"All of the errors seen in these test runs were in the linuxbridge-iptables scenario here.  As I said in the revised commit message, I think it\u0027s better to drop this and get the benefit of the other two scenarios, rather than not get any.","commit_id":"aed9d7c57fe78c6574e692719a3e28371b494b84"}]}