)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"d79af19b8d6e6a095f4cb68b6feda6db64f7dfbf","unresolved":false,"context_lines":[{"line_number":21,"context_line":"   network\u0027s subnet as this subnet don\u0027t belongs to tenant user"},{"line_number":22,"context_line":"   so regular client gets 404 error while doing subnet_show"},{"line_number":23,"context_line":"   command."},{"line_number":24,"context_line":"   "},{"line_number":25,"context_line":"Change-Id: Ibebb20b29dd6ae902d194fd26ba1ea728a976286"},{"line_number":26,"context_line":"Closes-bug: #1861670"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3fa7e38b_0f66f157","line":24,"updated":"2020-02-04 11:24:46.000000000","message":"Nit: whitespace","commit_id":"7700227ea50958984efd5e362876357043b7e67a"}],"neutron_tempest_plugin/scenario/test_connectivity.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"19f1155209670262b4556660abebc8846e288ab3","unresolved":false,"context_lines":[{"line_number":185,"context_line":"        Test ensures that both 10.1.0.1 and 10.1.0.x IP addresses are"},{"line_number":186,"context_line":"        reachable from VM."},{"line_number":187,"context_line":"        \"\"\""},{"line_number":188,"context_line":"        ext_network \u003d self.client.show_network(self.external_network_id)"},{"line_number":189,"context_line":"        ext_subnet_id \u003d ext_network[\u0027network\u0027][\u0027subnets\u0027][0][\u0027id\u0027]"},{"line_number":190,"context_line":"        ext_subnet \u003d self.client.show_subnet(ext_subnet_id)"},{"line_number":191,"context_line":"        ext_cidr \u003d ext_subnet[\u0027subnet\u0027][\u0027cidr\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_5e1a6e19","line":188,"updated":"2020-02-03 11:45:42.000000000","message":"Thanks! I didn\u0027t check the non-voting CI jobs. Let\u0027s do this in this patch.","commit_id":"eae033d325010921411a81d13993c7f0059e0cd2"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7668e0bc8efad109dd3167ed92778568d867c41b","unresolved":false,"context_lines":[{"line_number":186,"context_line":"        reachable from VM."},{"line_number":187,"context_line":"        \"\"\""},{"line_number":188,"context_line":"        ext_network \u003d self.client.show_network(self.external_network_id)"},{"line_number":189,"context_line":"        ext_subnet_id \u003d ext_network[\u0027network\u0027][\u0027subnets\u0027][0][\u0027id\u0027]"},{"line_number":190,"context_line":"        ext_subnet \u003d self.client.show_subnet(ext_subnet_id)"},{"line_number":191,"context_line":"        ext_cidr \u003d ext_subnet[\u0027subnet\u0027][\u0027cidr\u0027]"},{"line_number":192,"context_line":"        subnet_cidr \u003d ip_utils.find_valid_cidr(used_cidr\u003dext_cidr)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_9abf8fc9","line":189,"range":{"start_line":189,"start_character":60,"end_line":189,"end_character":66},"updated":"2020-02-03 13:52:49.000000000","message":"Slawek, this is wrong: \"subnets\" stored in \"network\" is a list of ID.\n\nThis line should be:\n  ext_subnet_id \u003d ext_network[\u0027network\u0027][\u0027subnets\u0027][0]\n\n\n        Body: b\u0027{\"network\":{\"id\":\"de7a61aa-b218-4480-af86-0d556d5860fa\",\"name\":\"public\",\"tenant_id\":\"230e4a610c344d7c9fee3980e67172e6\",\"admin_state_up\":true,\"mtu\":1450,\"status\":\"ACTIVE\",\"subnets\":[\"9675aed6-9f03-47ff-926c-0d7b44ab37be\",\"b9c8ba65-56fb-462b-82ed-862f5910cd8d\"],\"shared\":false,\"availability_zone_hints\":[],\"availability_zones\":[\"nova\"],\"ipv4_address_scope\":null,\"ipv6_address_scope\":null,\"router:external\":true,\"description\":\"\",\"port_security_enabled\":true,\"qos_policy_id\":null,\"dns_domain\":\"\",\"l2_adjacency\":true,\"is_default\":true,\"tags\":[],\"created_at\":\"2020-02-03T12:15:16Z\",\"updated_at\":\"2020-02-03T12:15:36Z\",\"revision_number\":3,\"project_id\":\"230e4a610c344d7c9fee3980e67172e6\"}}\u0027","commit_id":"eae033d325010921411a81d13993c7f0059e0cd2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9659e947bb16353ce29695415012f69dd183aed3","unresolved":false,"context_lines":[{"line_number":190,"context_line":"        for ext_subnetid in ext_network[\u0027network\u0027][\u0027subnets\u0027]:"},{"line_number":191,"context_line":"            ext_subnet \u003d self.os_admin.network_client.show_subnet(ext_subnetid)"},{"line_number":192,"context_line":"            ext_cidr \u003d ext_subnet[\u0027subnet\u0027][\u0027cidr\u0027]"},{"line_number":193,"context_line":"            if ip_utils.get_ip_version(ext_cidr) \u003d\u003d constants.IP_VERSION_4:"},{"line_number":194,"context_line":"                break"},{"line_number":195,"context_line":"        else:"},{"line_number":196,"context_line":"            self.fail(\u0027No IPv4 subnet was found in external network %s\u0027 %"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_c02d06ae","line":193,"updated":"2020-02-04 14:39:44.000000000","message":"You can just use ext_subnet[\u0027subnet\u0027][\u0027ip_version\u0027] right?","commit_id":"e9daddc535f58eabc6f5d75d1c272644ad94a958"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"c568fc61cf1b582e2df125ed6b725d34b757cbe6","unresolved":false,"context_lines":[{"line_number":190,"context_line":"        for ext_subnetid in ext_network[\u0027network\u0027][\u0027subnets\u0027]:"},{"line_number":191,"context_line":"            ext_subnet \u003d self.os_admin.network_client.show_subnet(ext_subnetid)"},{"line_number":192,"context_line":"            ext_cidr \u003d ext_subnet[\u0027subnet\u0027][\u0027cidr\u0027]"},{"line_number":193,"context_line":"            if ip_utils.get_ip_version(ext_cidr) \u003d\u003d constants.IP_VERSION_4:"},{"line_number":194,"context_line":"                break"},{"line_number":195,"context_line":"        else:"},{"line_number":196,"context_line":"            self.fail(\u0027No IPv4 subnet was found in external network %s\u0027 %"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_66c54a90","line":193,"in_reply_to":"3fa7e38b_263af2c4","updated":"2020-02-04 15:37:43.000000000","message":"Now it\u0027s working [1] !!!\n\n[1]https://4986b2318f7a379866e9-7f68e4d0971e68dd5f62a722346ed27d.ssl.cf5.rackcdn.com/705413/5/check/neutron-tempest-plugin-dvr-multinode-scenario/137d13d/testr_results.html","commit_id":"e9daddc535f58eabc6f5d75d1c272644ad94a958"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"92e3b70ffa2d20b86b700f3a3ce7407a9ed52611","unresolved":false,"context_lines":[{"line_number":190,"context_line":"        for ext_subnetid in ext_network[\u0027network\u0027][\u0027subnets\u0027]:"},{"line_number":191,"context_line":"            ext_subnet \u003d self.os_admin.network_client.show_subnet(ext_subnetid)"},{"line_number":192,"context_line":"            ext_cidr \u003d ext_subnet[\u0027subnet\u0027][\u0027cidr\u0027]"},{"line_number":193,"context_line":"            if ip_utils.get_ip_version(ext_cidr) \u003d\u003d constants.IP_VERSION_4:"},{"line_number":194,"context_line":"                break"},{"line_number":195,"context_line":"        else:"},{"line_number":196,"context_line":"            self.fail(\u0027No IPv4 subnet was found in external network %s\u0027 %"}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_263af2c4","line":193,"in_reply_to":"3fa7e38b_c02d06ae","updated":"2020-02-04 15:32:30.000000000","message":"Right! I\u0027ll push a new PS to change this but I would like to check the CI first.","commit_id":"e9daddc535f58eabc6f5d75d1c272644ad94a958"}]}