)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"4453f329438dad7d712bb36970134556bd391c9d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Currently test_large_ops is derived from NetworkScenarioTest and used"},{"line_number":12,"context_line":"to create security group from Nova which is expected."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This test supposed to ask Nova for required networking services and"},{"line_number":15,"context_line":"Nova proxy the network service for neutron in case neutron is enabled."},{"line_number":16,"context_line":"SO this test should be derived from OfficialClientTest instead of"},{"line_number":17,"context_line":"NetworkScenarioTest. This is needed to migrate this test from official"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa98f980_bdf25a04","line":14,"updated":"2014-09-08 10:17:58.000000000","message":"I thinkd the comment is wrong here:\nThis test asks Tempest to avoid creating any network resources, and\nthus doesn\u0027t use Neutron or nova network at all\n\nHowever, since the VMs are booted without network \nconnection, I believe it\u0027s better to make sure they are booted \nwithout secgroup as well (since it\u0027s irrelevant) and you can remove\n the secgroup creation altogether.","commit_id":"0655019ff423973a978e190922f5605302a787e3"},{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"0f317c85429c9f26ce587d7eefb5ce7ecfb642bd","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Currently test_large_ops is derived from NetworkScenarioTest and used"},{"line_number":12,"context_line":"to create security group from Nova which is expected."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This test supposed to ask Nova for required networking services and"},{"line_number":15,"context_line":"Nova proxy the network service for neutron in case neutron is enabled."},{"line_number":16,"context_line":"SO this test should be derived from OfficialClientTest instead of"},{"line_number":17,"context_line":"NetworkScenarioTest. This is needed to migrate this test from official"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa98f980_f2ba2bbf","line":14,"in_reply_to":"fa98f980_3c3acc36","updated":"2014-09-09 06:36:21.000000000","message":"all is well, but as far as Neutron testing is concerned, creating secgroup doesn\u0027t do anything when no network port is involved, so unless you connect all VMs to network (I\u0027m working on a patch for that), there\u0027s really no point for messing about with secgroups here...","commit_id":"0655019ff423973a978e190922f5605302a787e3"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"701451788326038dfca473cda658a6199832ae5e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Currently test_large_ops is derived from NetworkScenarioTest and used"},{"line_number":12,"context_line":"to create security group from Nova which is expected."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This test supposed to ask Nova for required networking services and"},{"line_number":15,"context_line":"Nova proxy the network service for neutron in case neutron is enabled."},{"line_number":16,"context_line":"SO this test should be derived from OfficialClientTest instead of"},{"line_number":17,"context_line":"NetworkScenarioTest. This is needed to migrate this test from official"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fa98f980_3c3acc36","line":14,"in_reply_to":"fa98f980_bdf25a04","updated":"2014-09-09 04:53:17.000000000","message":"Agree currently VM are being booted without network resources but i think it is valid to create VM with sec grp. It has been added to tests sec grp performance - https://review.openstack.org/#/c/50514/ \n\nThis tests is currently for gate-tempest-dsvm-large-ops job too.","commit_id":"0655019ff423973a978e190922f5605302a787e3"}]}