)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"eb89a367da637cfd2bf329b43b3c70a75b7a812d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Shnaidman \u003csshnaidm@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-11-27 15:02:15 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for IPv6 for API continue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Reworking all remained tests for IPv6 support."},{"line_number":10,"context_line":"Unifying tenant CIDR and mask bits configuration for all"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"5a890539_136262ad","line":7,"updated":"2014-11-28 11:25:16.000000000","message":"I think \"Support for IPv6 for API (part 2)\" is better or \"Update support for IPv6 for API\"","commit_id":"0332d9f61aff825e2057766c9fb0d6effc6e1713"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"fa1c2eeac34c2614b5a13b59bbf803a5002d8719","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Shnaidman \u003csshnaidm@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-11-27 15:02:15 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for IPv6 for API continue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Reworking all remained tests for IPv6 support."},{"line_number":10,"context_line":"Unifying tenant CIDR and mask bits configuration for all"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"5a890539_aeb76139","line":7,"in_reply_to":"5a890539_136262ad","updated":"2014-11-28 12:07:19.000000000","message":"Done","commit_id":"0332d9f61aff825e2057766c9fb0d6effc6e1713"},{"author":{"_account_id":7249,"name":"Ann Taraday","email":"akamyshnikova@mirantis.com","username":"AKamyshnikova"},"change_message_id":"b3166a85fc6bc339572dea1f564c8af94f5d1a46","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sergey Shnaidman \u003csshnaidm@cisco.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-11-28 12:06:40 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for IPv6 for API (part 2)"},{"line_number":8,"context_line":"Reworking all remained tests for IPv6 support."},{"line_number":9,"context_line":"Unifying tenant CIDR and mask bits configuration for all"},{"line_number":10,"context_line":"network tests."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"5a890539_48027070","line":7,"updated":"2014-11-28 14:16:52.000000000","message":"Missing blank line. pep8 error","commit_id":"97de08c12bbd2e7ccbd8a8b74356375752e5ad23"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e4e9515682800e2688a856d17572aac0a33c5d3f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Reworking all remained tests for IPv6 support"},{"line_number":10,"context_line":"Unifying tenant CIDR and mask bits configuration for all"},{"line_number":11,"context_line":"network tests"},{"line_number":12,"context_line":"Added docstrings to every test in test_networks."},{"line_number":13,"context_line":"Removing a lot of duplicating code and overall cleanup."},{"line_number":14,"context_line":"Removing duplicate coverage tests"},{"line_number":15,"context_line":"Added IPv6 subnet attributes full testing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"3a961159_53ea6fd8","line":12,"updated":"2014-12-24 01:04:02.000000000","message":"nit, I don\u0027t understand why you leave \".\" in some lines.","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"43a8caa410c8895769029a72c424fd6ace4b52cc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Reworking all remained tests for IPv6 support"},{"line_number":10,"context_line":"Unifying tenant CIDR and mask bits configuration for all"},{"line_number":11,"context_line":"network tests"},{"line_number":12,"context_line":"Added docstrings to every test in test_networks."},{"line_number":13,"context_line":"Removing a lot of duplicating code and overall cleanup."},{"line_number":14,"context_line":"Removing duplicate coverage tests"},{"line_number":15,"context_line":"Added IPv6 subnet attributes full testing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"3a961159_c86337a9","line":12,"in_reply_to":"3a961159_53ea6fd8","updated":"2015-01-16 00:09:15.000000000","message":"Done","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"}],"tempest/api/network/test_networks.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e4e9515682800e2688a856d17572aac0a33c5d3f","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_network_subnet(self):"},{"line_number":136,"context_line":"        \"\"\"Test update of network and subnet"},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":"        # Create a network"},{"line_number":139,"context_line":"        name \u003d data_utils.rand_name(\u0027network-\u0027)"},{"line_number":140,"context_line":"        network \u003d self.create_network(network_name\u003dname)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_739dabbb","line":137,"updated":"2014-12-24 01:04:02.000000000","message":"nit,\nI feel closed quotations should be single line for consistency.","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"43a8caa410c8895769029a72c424fd6ace4b52cc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_network_subnet(self):"},{"line_number":136,"context_line":"        \"\"\"Test update of network and subnet"},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":"        # Create a network"},{"line_number":139,"context_line":"        name \u003d data_utils.rand_name(\u0027network-\u0027)"},{"line_number":140,"context_line":"        network \u003d self.create_network(network_name\u003dname)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_a8518bda","line":137,"in_reply_to":"3a961159_739dabbb","updated":"2015-01-16 00:09:15.000000000","message":"Done","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0b6cc249df3b451319cd50d0e8d3676661b28eb2","unresolved":false,"context_lines":[{"line_number":134,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":135,"context_line":"    def test_create_update_delete_network_subnet(self):"},{"line_number":136,"context_line":"        \"\"\"Test update of network and subnet"},{"line_number":137,"context_line":"        \"\"\""},{"line_number":138,"context_line":"        # Create a network"},{"line_number":139,"context_line":"        name \u003d data_utils.rand_name(\u0027network-\u0027)"},{"line_number":140,"context_line":"        network \u003d self.create_network(network_name\u003dname)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_5385f357","line":137,"in_reply_to":"3a961159_739dabbb","updated":"2014-12-25 11:45:48.000000000","message":"This change is not related to this patch at all.","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0b6cc249df3b451319cd50d0e8d3676661b28eb2","unresolved":false,"context_lines":[{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":158,"context_line":"    def test_show_network(self):"},{"line_number":159,"context_line":"        \"\"\"Verify the details of a network \"\"\""},{"line_number":160,"context_line":"        body \u003d self.client.show_network(self.network[\u0027id\u0027])"},{"line_number":161,"context_line":"        network \u003d body[\u0027network\u0027]"},{"line_number":162,"context_line":"        for key in [\u0027id\u0027, \u0027name\u0027]:"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_737aef78","line":159,"updated":"2014-12-25 11:45:48.000000000","message":"ditto","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0b6cc249df3b451319cd50d0e8d3676661b28eb2","unresolved":false,"context_lines":[{"line_number":164,"context_line":""},{"line_number":165,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":166,"context_line":"    def test_show_network_fields(self):"},{"line_number":167,"context_line":"        \"\"\"Verify specific fields in a network query \"\"\""},{"line_number":168,"context_line":"        fields \u003d [\u0027id\u0027, \u0027name\u0027]"},{"line_number":169,"context_line":"        body \u003d self.client.show_network(self.network[\u0027id\u0027],"},{"line_number":170,"context_line":"                                        fields\u003dfields)"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_937f5b66","line":167,"updated":"2014-12-25 11:45:48.000000000","message":"ditto","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0b6cc249df3b451319cd50d0e8d3676661b28eb2","unresolved":false,"context_lines":[{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":177,"context_line":"    def test_list_networks(self):"},{"line_number":178,"context_line":"        \"\"\"Verify the network exists in the list of all networks \"\"\""},{"line_number":179,"context_line":"        body \u003d self.client.list_networks()"},{"line_number":180,"context_line":"        networks \u003d [network[\u0027id\u0027] for network in body[\u0027networks\u0027]"},{"line_number":181,"context_line":"                    if network[\u0027id\u0027] \u003d\u003d self.network[\u0027id\u0027]]"}],"source_content_type":"text/x-python","patch_set":14,"id":"3a961159_d3900399","line":178,"updated":"2014-12-25 11:45:48.000000000","message":"ditto, there is a lot of this kind of change in this patch.","commit_id":"dfe75927f6165ee3294edadeb53d25974581417b"},{"author":{"_account_id":13372,"name":"Eugene Bagdasaryan","email":"ebagdasaryan@me.com","username":"ebagdasa"},"change_message_id":"494b986cc7391ce9a9f97be501d3b220b830bc68","unresolved":false,"context_lines":[{"line_number":586,"context_line":""},{"line_number":587,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":588,"context_line":"    def test_create_delete_subnet_with_dhcp_disabled(self):"},{"line_number":589,"context_line":"        \"\"\"Test network and subnet with disabled DHCP and IPv6 attributes \"\"\""},{"line_number":590,"context_line":"        self.assertRaisesRegexp("},{"line_number":591,"context_line":"            exceptions.BadRequest,"},{"line_number":592,"context_line":"            \"ipv6_ra_mode or ipv6_address_mode cannot be set when \""}],"source_content_type":"text/x-python","patch_set":15,"id":"3a961159_1799c16c","line":589,"updated":"2015-01-16 16:49:38.000000000","message":"Good!","commit_id":"ede0c6a07a9d231ac491ac1fefd81f83862c4e34"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"39b4c90cf8b6502e1ac1133eb3d9635dd48ad16d","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"import itertools"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import netaddr"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest.api.network import base"}],"source_content_type":"text/x-python","patch_set":19,"id":"da86d52c_08cfa3b1","side":"PARENT","line":16,"updated":"2015-02-09 01:12:54.000000000","message":"Is this removing needed for this patch?","commit_id":"afb6517890918e1cf92cd291044087476ecbf64a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"26add27692bbbef5a863a315a9bcdb4a71d80528","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"import itertools"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import netaddr"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest.api.network import base"}],"source_content_type":"text/x-python","patch_set":19,"id":"da86d52c_da670d24","side":"PARENT","line":16,"in_reply_to":"da86d52c_08cfa3b1","updated":"2015-02-09 08:12:34.000000000","message":"no, I can remove it.","commit_id":"afb6517890918e1cf92cd291044087476ecbf64a"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"39b4c90cf8b6502e1ac1133eb3d9635dd48ad16d","unresolved":false,"context_lines":[{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":179,"context_line":"    def test_show_network(self):"},{"line_number":180,"context_line":"        \"\"\"Verify the details of a network \"\"\""},{"line_number":181,"context_line":"        body \u003d self.client.show_network(self.network[\u0027id\u0027])"},{"line_number":182,"context_line":"        network \u003d body[\u0027network\u0027]"},{"line_number":183,"context_line":"        for key in [\u0027id\u0027, \u0027name\u0027]:"}],"source_content_type":"text/x-python","patch_set":19,"id":"da86d52c_c8510b10","line":180,"updated":"2015-02-09 01:12:54.000000000","message":"I think these documents fixings (\u0027#\u0027 -\u003e \u0027\"\"\"\u0027) are not necessary in this patch\u0027s subject. These should be split into another patch.\nThat will be simplified this patch.","commit_id":"faa355b4e45547d70e7fdc9eeb052a816243d994"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"26add27692bbbef5a863a315a9bcdb4a71d80528","unresolved":false,"context_lines":[{"line_number":177,"context_line":""},{"line_number":178,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":179,"context_line":"    def test_show_network(self):"},{"line_number":180,"context_line":"        \"\"\"Verify the details of a network \"\"\""},{"line_number":181,"context_line":"        body \u003d self.client.show_network(self.network[\u0027id\u0027])"},{"line_number":182,"context_line":"        network \u003d body[\u0027network\u0027]"},{"line_number":183,"context_line":"        for key in [\u0027id\u0027, \u0027name\u0027]:"}],"source_content_type":"text/x-python","patch_set":19,"id":"da86d52c_da354df8","line":180,"in_reply_to":"da86d52c_c8510b10","updated":"2015-02-09 08:12:34.000000000","message":"I think it\u0027s automatically was fixed by my IDE and I agreed with it, because it\u0027s not code and not even comment, only punctuation. As a reviewer I would just skip it. However if you are so against this, I can change it back.","commit_id":"faa355b4e45547d70e7fdc9eeb052a816243d994"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"fb26031c38afc9e1ec18b8e632d7a74b664fa20b","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                                \u0027new_dns_nameservers\u0027: [\u00277.8.8.8\u0027, \u00277.8.4.4\u0027]}}"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @classmethod"},{"line_number":98,"context_line":"    def _create_subnet_with_last_subnet_block(cls, network, ip_version):"},{"line_number":99,"context_line":"        \"\"\"Derive last subnet CIDR block from tenant CIDR and"},{"line_number":100,"context_line":"           create the subnet with that derived CIDR"},{"line_number":101,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":22,"id":"da86d52c_42de1d6e","side":"PARENT","line":98,"updated":"2015-02-19 16:49:26.000000000","message":"You are changing this method but above removed the only call to it. If it was intended that there no longer be a call to this then it should be removed.","commit_id":"10bbdac2130101a6c00ed509ef0ee9fcb312bed0"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"b3acb5420e1808f66a258ff8890d0de41ca5340d","unresolved":false,"context_lines":[{"line_number":95,"context_line":"                                \u0027new_dns_nameservers\u0027: [\u00277.8.8.8\u0027, \u00277.8.4.4\u0027]}}"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @classmethod"},{"line_number":98,"context_line":"    def _create_subnet_with_last_subnet_block(cls, network, ip_version):"},{"line_number":99,"context_line":"        \"\"\"Derive last subnet CIDR block from tenant CIDR and"},{"line_number":100,"context_line":"           create the subnet with that derived CIDR"},{"line_number":101,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":22,"id":"ba7be1f8_87fc4740","side":"PARENT","line":98,"in_reply_to":"da86d52c_42de1d6e","updated":"2015-02-22 07:41:56.000000000","message":"great catch, thanks!","commit_id":"10bbdac2130101a6c00ed509ef0ee9fcb312bed0"}],"tempest/api/network/test_ports.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8b3f028c8cc06c23ba4f3856fb4814ec2559aac1","unresolved":false,"context_lines":[{"line_number":312,"context_line":"                         show_port[\u0027binding:vif_details\u0027])"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"class PortsIpV6TestJSON(PortsTestJSON):"},{"line_number":316,"context_line":"    _ip_version \u003d 6"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"3a961159_a35453fe","line":315,"updated":"2014-12-22 12:24:51.000000000","message":"Why do you leave the suffix of \"JSON\" in this file?","commit_id":"b14bc34484bc8993cd39577c12cea80c9e997829"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"f0dab2dcf5517c172b6e4db527d99903740128cd","unresolved":false,"context_lines":[{"line_number":312,"context_line":"                         show_port[\u0027binding:vif_details\u0027])"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"class PortsIpV6TestJSON(PortsTestJSON):"},{"line_number":316,"context_line":"    _ip_version \u003d 6"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"3a961159_831837d4","line":315,"in_reply_to":"3a961159_a35453fe","updated":"2014-12-22 13:15:12.000000000","message":"I\u0027ve removed them only in test_networks.py file. Actually I thought DKranz removed everything with JSON in his patch. No problem to remove it, but I\u0027m afraid I\u0027ve already exceeded all limits of unrelated changes in this patch :)","commit_id":"b14bc34484bc8993cd39577c12cea80c9e997829"}]}