)]}'
{"tempest/api/network/test_security_groups.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"33185b33a70f9bad8c0fb5915983af72227a5ddf","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    def test_create_show_delete_security_group_rule_names(self):"},{"line_number":146,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":147,"context_line":"            [\u0027ah\u0027, \u0027pgm\u0027, \u0027tcp\u0027, \u0027dccp\u0027, \u0027igmp\u0027, \u0027icmp\u0027, \u0027esp\u0027, \u0027vrrp\u0027, \u0027gre\u0027,"},{"line_number":148,"context_line":"             \u0027sctp\u0027, \u0027rsvp\u0027, \u0027udp\u0027, \u0027udplite\u0027, \u0027egp\u0027, \u0027ospf\u0027])"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u002766e47f1f-20b6-4417-8839-3cc671c7afa3\u0027)"},{"line_number":151,"context_line":"    def test_create_show_delete_security_group_rule_numbers(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_150f02e2","line":148,"updated":"2017-07-31 19:33:01.000000000","message":"Is it easier to import neutron_lib and use:\n\nconstants.IP_PROTOCOL_MAP.values() ?\n\nI think at a minimum they should be in the alphabetized order they are in neutron_lib.","commit_id":"56ddedfe8dc8f68a4f7f38247f022ad50e757f88"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"5d57179e77b5a12e31a44155c43f4b8857425a93","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    def test_create_show_delete_security_group_rule_names(self):"},{"line_number":146,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":147,"context_line":"            [\u0027ah\u0027, \u0027pgm\u0027, \u0027tcp\u0027, \u0027dccp\u0027, \u0027igmp\u0027, \u0027icmp\u0027, \u0027esp\u0027, \u0027vrrp\u0027, \u0027gre\u0027,"},{"line_number":148,"context_line":"             \u0027sctp\u0027, \u0027rsvp\u0027, \u0027udp\u0027, \u0027udplite\u0027, \u0027egp\u0027, \u0027ospf\u0027])"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u002766e47f1f-20b6-4417-8839-3cc671c7afa3\u0027)"},{"line_number":151,"context_line":"    def test_create_show_delete_security_group_rule_numbers(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_d5ed83f0","line":148,"in_reply_to":"df3967d1_150f02e2","updated":"2017-08-03 20:24:44.000000000","message":"I was hesitant because neutron_lib is not a requirement  for tempest.. I guess I can propose this to the neutron project and see what happens.","commit_id":"56ddedfe8dc8f68a4f7f38247f022ad50e757f88"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"d6f87dd6197812aa7361759665fd2941f864ade7","unresolved":false,"context_lines":[{"line_number":140,"context_line":"            self.assertIn(rule_create_body[\u0027security_group_rule\u0027][\u0027id\u0027],"},{"line_number":141,"context_line":"                          rule_list)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    @decorators.attr(type\u003d\u0027smoke\u0027)"},{"line_number":144,"context_line":"    @decorators.idempotent_id(\u0027282e3681-aa6e-42a7-b05c-c341aa1e3cdf\u0027)"},{"line_number":145,"context_line":"    def test_create_show_delete_security_group_rule_names(self):"},{"line_number":146,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":147,"context_line":"            [\u0027ah\u0027, \u0027pgm\u0027, \u0027tcp\u0027, \u0027dccp\u0027, \u0027igmp\u0027, \u0027icmp\u0027, \u0027esp\u0027, \u0027vrrp\u0027, \u0027gre\u0027,"},{"line_number":148,"context_line":"             \u0027sctp\u0027, \u0027rsvp\u0027, \u0027udp\u0027, \u0027udplite\u0027, \u0027egp\u0027, \u0027ospf\u0027])"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u002766e47f1f-20b6-4417-8839-3cc671c7afa3\u0027)"},{"line_number":151,"context_line":"    def test_create_show_delete_security_group_rule_numbers(self):"},{"line_number":152,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":153,"context_line":"            [51, 113, 6, 33, 2, 1, 50, 112, 47, 132, 46, 17, 136, 8, 89])"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    @decorators.idempotent_id(\u002787dfbcf9-1849-43ea-b1e4-efa3eeae9f71\u0027)"},{"line_number":156,"context_line":"    def test_create_security_group_rule_with_additional_args(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_a16ff3c3","line":153,"range":{"start_line":143,"start_character":0,"end_line":153,"end_character":73},"updated":"2017-07-31 02:36:44.000000000","message":"i guess not all backends accept these protocols.\ni suppose some feature discovery mechanism is necessary.","commit_id":"56ddedfe8dc8f68a4f7f38247f022ad50e757f88"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"33185b33a70f9bad8c0fb5915983af72227a5ddf","unresolved":false,"context_lines":[{"line_number":140,"context_line":"            self.assertIn(rule_create_body[\u0027security_group_rule\u0027][\u0027id\u0027],"},{"line_number":141,"context_line":"                          rule_list)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    @decorators.attr(type\u003d\u0027smoke\u0027)"},{"line_number":144,"context_line":"    @decorators.idempotent_id(\u0027282e3681-aa6e-42a7-b05c-c341aa1e3cdf\u0027)"},{"line_number":145,"context_line":"    def test_create_show_delete_security_group_rule_names(self):"},{"line_number":146,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":147,"context_line":"            [\u0027ah\u0027, \u0027pgm\u0027, \u0027tcp\u0027, \u0027dccp\u0027, \u0027igmp\u0027, \u0027icmp\u0027, \u0027esp\u0027, \u0027vrrp\u0027, \u0027gre\u0027,"},{"line_number":148,"context_line":"             \u0027sctp\u0027, \u0027rsvp\u0027, \u0027udp\u0027, \u0027udplite\u0027, \u0027egp\u0027, \u0027ospf\u0027])"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u002766e47f1f-20b6-4417-8839-3cc671c7afa3\u0027)"},{"line_number":151,"context_line":"    def test_create_show_delete_security_group_rule_numbers(self):"},{"line_number":152,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":153,"context_line":"            [51, 113, 6, 33, 2, 1, 50, 112, 47, 132, 46, 17, 136, 8, 89])"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    @decorators.idempotent_id(\u002787dfbcf9-1849-43ea-b1e4-efa3eeae9f71\u0027)"},{"line_number":156,"context_line":"    def test_create_security_group_rule_with_additional_args(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_9502d2b3","line":153,"range":{"start_line":143,"start_character":0,"end_line":153,"end_character":73},"in_reply_to":"df3967d1_a16ff3c3","updated":"2017-07-31 19:33:01.000000000","message":"IP_PROTOCOL_MAP.keys() ?","commit_id":"56ddedfe8dc8f68a4f7f38247f022ad50e757f88"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"5d57179e77b5a12e31a44155c43f4b8857425a93","unresolved":false,"context_lines":[{"line_number":140,"context_line":"            self.assertIn(rule_create_body[\u0027security_group_rule\u0027][\u0027id\u0027],"},{"line_number":141,"context_line":"                          rule_list)"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    @decorators.attr(type\u003d\u0027smoke\u0027)"},{"line_number":144,"context_line":"    @decorators.idempotent_id(\u0027282e3681-aa6e-42a7-b05c-c341aa1e3cdf\u0027)"},{"line_number":145,"context_line":"    def test_create_show_delete_security_group_rule_names(self):"},{"line_number":146,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":147,"context_line":"            [\u0027ah\u0027, \u0027pgm\u0027, \u0027tcp\u0027, \u0027dccp\u0027, \u0027igmp\u0027, \u0027icmp\u0027, \u0027esp\u0027, \u0027vrrp\u0027, \u0027gre\u0027,"},{"line_number":148,"context_line":"             \u0027sctp\u0027, \u0027rsvp\u0027, \u0027udp\u0027, \u0027udplite\u0027, \u0027egp\u0027, \u0027ospf\u0027])"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    @decorators.idempotent_id(\u002766e47f1f-20b6-4417-8839-3cc671c7afa3\u0027)"},{"line_number":151,"context_line":"    def test_create_show_delete_security_group_rule_numbers(self):"},{"line_number":152,"context_line":"        self._test_create_show_delete_security_group_rule("},{"line_number":153,"context_line":"            [51, 113, 6, 33, 2, 1, 50, 112, 47, 132, 46, 17, 136, 8, 89])"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    @decorators.idempotent_id(\u002787dfbcf9-1849-43ea-b1e4-efa3eeae9f71\u0027)"},{"line_number":156,"context_line":"    def test_create_security_group_rule_with_additional_args(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_55b3d30a","line":153,"range":{"start_line":143,"start_character":0,"end_line":153,"end_character":73},"in_reply_to":"df3967d1_a16ff3c3","updated":"2017-08-03 20:24:44.000000000","message":"Oh is this what zhufl was saying?","commit_id":"56ddedfe8dc8f68a4f7f38247f022ad50e757f88"}]}