)]}'
{"neutron_tempest_plugin/api/test_routers.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"51986a398c0d5273a019bc04342a7aacce99b6da","unresolved":true,"context_lines":[{"line_number":487,"context_line":"            network, name\u003ddata_utils.rand_name(\"port\"),"},{"line_number":488,"context_line":"            security_groups\u003d[self.secgroup[\u0027id\u0027]])"},{"line_number":489,"context_line":"        self.create_floatingip(port\u003dport)"},{"line_number":490,"context_line":"        self.assertRaises(lib_exc.Conflict, self.delete_router, self.router)"},{"line_number":491,"context_line":"        self.client.delete_port(port[\u0027id\u0027])"},{"line_number":492,"context_line":"        self.delete_router(self.router)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1bbb85cb_b3c16614","line":490,"updated":"2021-02-01 08:29:52.000000000","message":"Isn\u0027t this part the same as https://github.com/openstack/neutron-tempest-plugin/blob/eb84955bf723b25b5b83030b889955f80861ce07/neutron_tempest_plugin/api/test_routers_negative.py#L44 ? If yes, maybe we can remove that old test as this one provides a bit more coverage IMO. Wdyt?","commit_id":"def2e68cff2047c620606313d4fb5311f4cfffac"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0c8d7778d43a59be9292dacf98f2ac874eeba089","unresolved":true,"context_lines":[{"line_number":488,"context_line":"        self.create_floatingip(port\u003dport)"},{"line_number":489,"context_line":"        self.assertRaises(lib_exc.Conflict, self.delete_router, self.router)"},{"line_number":490,"context_line":"        self.client.delete_port(port[\u0027id\u0027])"},{"line_number":491,"context_line":"        self.delete_router(self.router)"}],"source_content_type":"text/x-python","patch_set":8,"id":"8b1c9193_8b757ef6","line":491,"range":{"start_line":491,"start_character":8,"end_line":491,"end_character":26},"updated":"2021-02-03 11:40:04.000000000","message":"Same comment as in PS5 [1]: when a router having a port attached is deleted, a ConflictException should be raised [2].\n\nBut now I see [3] that \"self.delete_router\" removes first the router interfaces.\n\nThis test should also check what happens when a router is being deleted without removing those interfaces. The method BaseNetworkTest.delete_router should not be used here but just the client call \"client.delete_router(router[\u0027id\u0027])\".\n\n[1]https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/763072/5/neutron_tempest_plugin/scenario/test_routers.py#72\n[2]http://paste.openstack.org/show/802275/\n[3]https://github.com/openstack/neutron-tempest-plugin/blob/eb84955bf723b25b5b83030b889955f80861ce07/neutron_tempest_plugin/api/base.py#L790-L795","commit_id":"149fe2d564c0bcf232f4d74555c9c82e4a018712"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"4c179ebee3f731b261c29f544d7d9d078a7ea2cf","unresolved":true,"context_lines":[{"line_number":488,"context_line":"        self.create_floatingip(port\u003dport)"},{"line_number":489,"context_line":"        self.assertRaises(lib_exc.Conflict, self.delete_router, self.router)"},{"line_number":490,"context_line":"        self.client.delete_port(port[\u0027id\u0027])"},{"line_number":491,"context_line":"        self.delete_router(self.router)"}],"source_content_type":"text/x-python","patch_set":8,"id":"6bf046df_330401a1","line":491,"range":{"start_line":491,"start_character":8,"end_line":491,"end_character":26},"in_reply_to":"8b1c9193_8b757ef6","updated":"2021-02-04 08:26:53.000000000","message":"Good catch. Thx :)","commit_id":"149fe2d564c0bcf232f4d74555c9c82e4a018712"}],"neutron_tempest_plugin/scenario/test_routers.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"6efef71782e5571ca91f3e16ad8207bcce15b728","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        fip \u003d self.create_floatingip(port\u003dport)"},{"line_number":75,"context_line":"        self.check_connectivity(fip[\u0027floating_ip_address\u0027],"},{"line_number":76,"context_line":"                                CONF.validation.image_ssh_user,"},{"line_number":77,"context_line":"                                self.keypair[\u0027private_key\u0027])"},{"line_number":78,"context_line":"        # Try to delete router, should fail"},{"line_number":79,"context_line":"        self.assertRaises(lib_exc.Conflict, self.delete_router, router)"},{"line_number":80,"context_line":"        # Delete the VM and try to delete router, should fail."}],"source_content_type":"text/x-python","patch_set":1,"id":"fffc6b78_a7c7d845","line":77,"updated":"2020-11-19 08:37:08.000000000","message":"I wonder if we really need to spawn vm and check connectivity here. If You would just create port (unbound), effect would be the same from the Neutron PoV but test could be in API tests and should be much faster and more stable.\n\nIf that\u0027s not true, and You really need to have VM, maybe You don\u0027t need to ssh to it at least? Just wait until it will be active in Nova - would that be enough?","commit_id":"7c224f755d2003840457559fefd4fc0bfb8aa6c0"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"795715291801f7e9b63b10a50b6a1a091373926d","unresolved":false,"context_lines":[{"line_number":74,"context_line":"        fip \u003d self.create_floatingip(port\u003dport)"},{"line_number":75,"context_line":"        self.check_connectivity(fip[\u0027floating_ip_address\u0027],"},{"line_number":76,"context_line":"                                CONF.validation.image_ssh_user,"},{"line_number":77,"context_line":"                                self.keypair[\u0027private_key\u0027])"},{"line_number":78,"context_line":"        # Try to delete router, should fail"},{"line_number":79,"context_line":"        self.assertRaises(lib_exc.Conflict, self.delete_router, router)"},{"line_number":80,"context_line":"        # Delete the VM and try to delete router, should fail."}],"source_content_type":"text/x-python","patch_set":1,"id":"fffc6b78_b9cb8070","line":77,"in_reply_to":"fffc6b78_a7c7d845","updated":"2020-11-19 10:08:23.000000000","message":"In this test we do want all the stuff to be UP and running as expected before getting into the point (attempts to delete router), for example we don\u0027t want to continue the test if SSH to FIP fails (could be failure in router functionality). Also test\u0027s logic is planned to validate that the router fails to be deleted after removing VM (won\u0027t be covered without having the VM actually) \n\nAs for time needed for SSH connectivity it took me (on my setup): 0.18047285079956055 and I\u0027m not sure that changing test\u0027s logic just to reduce Zero seconds from test execution time will make sense.","commit_id":"7c224f755d2003840457559fefd4fc0bfb8aa6c0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"31d9486821e67d424adc760e68693127afc02964","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    def resource_setup(cls):"},{"line_number":32,"context_line":"        super(RoutersTest, cls).resource_setup()"},{"line_number":33,"context_line":"        # setup basic topology for servers we can log into it"},{"line_number":34,"context_line":"        cls.router \u003d cls.create_router_by_client()"},{"line_number":35,"context_line":"        cls.keypair \u003d cls.create_keypair()"},{"line_number":36,"context_line":"        cls.secgroup \u003d cls.create_security_group("},{"line_number":37,"context_line":"            name\u003ddata_utils.rand_name(\"test_port_secgroup\"))"},{"line_number":38,"context_line":"        cls.create_loginable_secgroup_rule("},{"line_number":39,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":40,"context_line":"        cls.create_pingable_secgroup_rule("},{"line_number":41,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":42,"context_line":"        cls.network \u003d cls.create_network()"},{"line_number":43,"context_line":"        cls.subnet \u003d cls.create_subnet(cls.network)"},{"line_number":44,"context_line":"        cls.create_router_interface(cls.router[\u0027id\u0027], cls.subnet[\u0027id\u0027])"},{"line_number":45,"context_line":"        cls.port \u003d cls.create_port(cls.network,"},{"line_number":46,"context_line":"                                   name\u003ddata_utils.rand_name(\"port\"),"},{"line_number":47,"context_line":"                                   security_groups\u003d[cls.secgroup[\u0027id\u0027]])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @decorators.idempotent_id(\u00275b766a72-28d7-11eb-8253-74e5f9e2a801\u0027)"},{"line_number":50,"context_line":"    def test_delete_router(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"832a8f56_2237ac67","line":47,"range":{"start_line":34,"start_character":8,"end_line":47,"end_character":72},"updated":"2020-12-21 10:17:58.000000000","message":"what is the point to create all those resources if we are not using them and instead creating new ones in the test below?","commit_id":"807fd6ed853a8d0edcc4038754b02b4178662eb7"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"97105a96d7d948cb224bfafeca29d6474ebfbacb","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    def resource_setup(cls):"},{"line_number":32,"context_line":"        super(RoutersTest, cls).resource_setup()"},{"line_number":33,"context_line":"        # setup basic topology for servers we can log into it"},{"line_number":34,"context_line":"        cls.router \u003d cls.create_router_by_client()"},{"line_number":35,"context_line":"        cls.keypair \u003d cls.create_keypair()"},{"line_number":36,"context_line":"        cls.secgroup \u003d cls.create_security_group("},{"line_number":37,"context_line":"            name\u003ddata_utils.rand_name(\"test_port_secgroup\"))"},{"line_number":38,"context_line":"        cls.create_loginable_secgroup_rule("},{"line_number":39,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":40,"context_line":"        cls.create_pingable_secgroup_rule("},{"line_number":41,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":42,"context_line":"        cls.network \u003d cls.create_network()"},{"line_number":43,"context_line":"        cls.subnet \u003d cls.create_subnet(cls.network)"},{"line_number":44,"context_line":"        cls.create_router_interface(cls.router[\u0027id\u0027], cls.subnet[\u0027id\u0027])"},{"line_number":45,"context_line":"        cls.port \u003d cls.create_port(cls.network,"},{"line_number":46,"context_line":"                                   name\u003ddata_utils.rand_name(\"port\"),"},{"line_number":47,"context_line":"                                   security_groups\u003d[cls.secgroup[\u0027id\u0027]])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @decorators.idempotent_id(\u00275b766a72-28d7-11eb-8253-74e5f9e2a801\u0027)"},{"line_number":50,"context_line":"    def test_delete_router(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1b0381f0_86b14871","line":47,"range":{"start_line":34,"start_character":8,"end_line":47,"end_character":72},"in_reply_to":"56f3b2b1_9cca20e6","updated":"2021-01-25 22:33:55.000000000","message":"For now You don\u0027t need those resources. If there will be other tests, You can add creation of them then.","commit_id":"807fd6ed853a8d0edcc4038754b02b4178662eb7"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"8007b0d4427499d1022b91afd846b71d10703ba4","unresolved":true,"context_lines":[{"line_number":31,"context_line":"    def resource_setup(cls):"},{"line_number":32,"context_line":"        super(RoutersTest, cls).resource_setup()"},{"line_number":33,"context_line":"        # setup basic topology for servers we can log into it"},{"line_number":34,"context_line":"        cls.router \u003d cls.create_router_by_client()"},{"line_number":35,"context_line":"        cls.keypair \u003d cls.create_keypair()"},{"line_number":36,"context_line":"        cls.secgroup \u003d cls.create_security_group("},{"line_number":37,"context_line":"            name\u003ddata_utils.rand_name(\"test_port_secgroup\"))"},{"line_number":38,"context_line":"        cls.create_loginable_secgroup_rule("},{"line_number":39,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":40,"context_line":"        cls.create_pingable_secgroup_rule("},{"line_number":41,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":42,"context_line":"        cls.network \u003d cls.create_network()"},{"line_number":43,"context_line":"        cls.subnet \u003d cls.create_subnet(cls.network)"},{"line_number":44,"context_line":"        cls.create_router_interface(cls.router[\u0027id\u0027], cls.subnet[\u0027id\u0027])"},{"line_number":45,"context_line":"        cls.port \u003d cls.create_port(cls.network,"},{"line_number":46,"context_line":"                                   name\u003ddata_utils.rand_name(\"port\"),"},{"line_number":47,"context_line":"                                   security_groups\u003d[cls.secgroup[\u0027id\u0027]])"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    @decorators.idempotent_id(\u00275b766a72-28d7-11eb-8253-74e5f9e2a801\u0027)"},{"line_number":50,"context_line":"    def test_delete_router(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"56f3b2b1_9cca20e6","line":47,"range":{"start_line":34,"start_character":8,"end_line":47,"end_character":72},"in_reply_to":"832a8f56_2237ac67","updated":"2020-12-21 10:42:00.000000000","message":"Currently we have only one single test in this script and this test cannot run in parallel with the other tests (not yet implemented router related tests, but may be added in the future) as this test will delete the common used router, and it will make other tests fail.\n\nIn order to prevent this, test_delete_router - is using its own resources created inside the test.\n\nSlawek - what would be you proposal?","commit_id":"807fd6ed853a8d0edcc4038754b02b4178662eb7"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"aca2f0055509c4915aa0298a36b7bdec7156ec34","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        cls.create_loginable_secgroup_rule("},{"line_number":38,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":39,"context_line":"        cls.create_pingable_secgroup_rule("},{"line_number":40,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @decorators.idempotent_id(\u00275b766a72-28d7-11eb-8253-74e5f9e2a801\u0027)"},{"line_number":43,"context_line":"    def test_delete_router(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"98caaf57_7e34ca6f","line":40,"updated":"2021-01-27 09:37:58.000000000","message":"You don\u0027t need those things as You don\u0027t try to ping or ssh vm, do You?","commit_id":"93291757eeeaae1b616d40517819ebd5ca5dac95"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"ea7f230ee66d89dcd30711a25a3610ef78d3cb65","unresolved":true,"context_lines":[{"line_number":37,"context_line":"        cls.create_loginable_secgroup_rule("},{"line_number":38,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":39,"context_line":"        cls.create_pingable_secgroup_rule("},{"line_number":40,"context_line":"            secgroup_id\u003dcls.secgroup[\u0027id\u0027])"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @decorators.idempotent_id(\u00275b766a72-28d7-11eb-8253-74e5f9e2a801\u0027)"},{"line_number":43,"context_line":"    def test_delete_router(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"969f3f3f_5b712687","line":40,"in_reply_to":"98caaf57_7e34ca6f","updated":"2021-01-27 09:43:00.000000000","message":"Agree, will be removed on next patch.","commit_id":"93291757eeeaae1b616d40517819ebd5ca5dac95"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"3b6fadc14acc07bbae3094c63dcab40d533ae552","unresolved":true,"context_lines":[{"line_number":56,"context_line":"            \u0027key_name\u0027: self.keypair[\u0027name\u0027],"},{"line_number":57,"context_line":"            \u0027networks\u0027: [{\u0027port\u0027: port[\u0027id\u0027]}]"},{"line_number":58,"context_line":"        }"},{"line_number":59,"context_line":"        server \u003d self.create_server(**server_args)"},{"line_number":60,"context_line":"        waiters.wait_for_server_status("},{"line_number":61,"context_line":"            self.os_primary.servers_client, server[\u0027server\u0027][\u0027id\u0027],"},{"line_number":62,"context_line":"            const.SERVER_STATUS_ACTIVE)"}],"source_content_type":"text/x-python","patch_set":5,"id":"64e48523_9f1957a7","line":59,"updated":"2021-01-27 10:21:44.000000000","message":"one more though about this patch - do we really need to spawn server here? Isn\u0027t enough to create port and then remove it?","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"6517c16a63aad4f3f0d0fb672810972c03671f48","unresolved":true,"context_lines":[{"line_number":56,"context_line":"            \u0027key_name\u0027: self.keypair[\u0027name\u0027],"},{"line_number":57,"context_line":"            \u0027networks\u0027: [{\u0027port\u0027: port[\u0027id\u0027]}]"},{"line_number":58,"context_line":"        }"},{"line_number":59,"context_line":"        server \u003d self.create_server(**server_args)"},{"line_number":60,"context_line":"        waiters.wait_for_server_status("},{"line_number":61,"context_line":"            self.os_primary.servers_client, server[\u0027server\u0027][\u0027id\u0027],"},{"line_number":62,"context_line":"            const.SERVER_STATUS_ACTIVE)"}],"source_content_type":"text/x-python","patch_set":5,"id":"d5cebab9_db3a14e3","line":59,"in_reply_to":"0e83ffb0_fbdd19f5","updated":"2021-01-27 13:54:09.000000000","message":"1) The initial patch #1 was including SSH check to VM, but then it was removed to reduce the execution time.\n\n2) @Lajos OPTION2 - the existing API test is simply adds router into the cleanup and the only indication we can have about the potential problem to \"delete router\" will be raised by cleanup (as Error in log I guess) and not by this API test, unless the scenario test in this patch!\nImportant: the router used in this API test wasn\u0027t actually in use, I mean it wasn\u0027t associated to Port/VM ... so it\u0027s a \"Free Router\" and a chance that something will go wrong to delete such a router is pretty low. It\u0027s absolutely not the same coverage implemented in this Scenario patch.","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c6370349c342108ec097d6c8b0193f2b12005f68","unresolved":true,"context_lines":[{"line_number":56,"context_line":"            \u0027key_name\u0027: self.keypair[\u0027name\u0027],"},{"line_number":57,"context_line":"            \u0027networks\u0027: [{\u0027port\u0027: port[\u0027id\u0027]}]"},{"line_number":58,"context_line":"        }"},{"line_number":59,"context_line":"        server \u003d self.create_server(**server_args)"},{"line_number":60,"context_line":"        waiters.wait_for_server_status("},{"line_number":61,"context_line":"            self.os_primary.servers_client, server[\u0027server\u0027][\u0027id\u0027],"},{"line_number":62,"context_line":"            const.SERVER_STATUS_ACTIVE)"}],"source_content_type":"text/x-python","patch_set":5,"id":"0e83ffb0_fbdd19f5","line":59,"in_reply_to":"64e48523_9f1957a7","updated":"2021-01-27 12:44:38.000000000","message":"Agree with Slaweq, generally I would think this is not a scenario test (IMO scenarios in tempest are for end-to-end with VMs to check if the VM has connectivity, image, volume whatever), so \n\n* OPTION1: under neutron_tempest_plugin.api.test_routers it has a role (https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/api/test_routers.py )\n\n* OPTION2: as I see under tempest there\u0027s coverage for this (https://opendev.org/openstack/tempest/src/branch/master/tempest/api/network/test_routers.py#L56 ) so no need for this extra test.","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a90240a15057e12d8c2483130c9854d9c6fbc1d4","unresolved":true,"context_lines":[{"line_number":56,"context_line":"            \u0027key_name\u0027: self.keypair[\u0027name\u0027],"},{"line_number":57,"context_line":"            \u0027networks\u0027: [{\u0027port\u0027: port[\u0027id\u0027]}]"},{"line_number":58,"context_line":"        }"},{"line_number":59,"context_line":"        server \u003d self.create_server(**server_args)"},{"line_number":60,"context_line":"        waiters.wait_for_server_status("},{"line_number":61,"context_line":"            self.os_primary.servers_client, server[\u0027server\u0027][\u0027id\u0027],"},{"line_number":62,"context_line":"            const.SERVER_STATUS_ACTIVE)"}],"source_content_type":"text/x-python","patch_set":5,"id":"3faab639_1ace8d01","line":59,"in_reply_to":"d5cebab9_db3a14e3","updated":"2021-01-28 07:59:56.000000000","message":"I can accept that this adds to the coverage, but I still think it\u0027s not a scenario, but more an API test","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"816ab22bc88194037e3edf4b333b1cc7b9d93627","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        # Delete port associated with router and try to delete"},{"line_number":70,"context_line":"        # router, should PASS"},{"line_number":71,"context_line":"        self.client.delete_port(port[\u0027id\u0027])"},{"line_number":72,"context_line":"        self.delete_router(router)"}],"source_content_type":"text/x-python","patch_set":5,"id":"46242c8b_02325fd0","line":72,"updated":"2021-01-27 11:59:32.000000000","message":"A router can\u0027t be deleted because it has an interface in a subnet. For example, the interface created in L48. Or because there is an assigned FIP in the external network (L63).\n\nWhat I don\u0027t understand is why this is passing; it should fail because the interface in the private network is still present. \"port\" (created in L49) is not the router interface but the port used in the VM. If \"port\" is present in the system should not interfere in the router deletion.\n\nBut the interface created in L48 should and it isn\u0027t. This is something that must be investigated: why if this port (owned by the router, in the private subnet) is still present and the router deletion succeeds.\n\n(or maybe I\u0027m missing something here, but I tested those operations manually in an env).","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"},{"author":{"_account_id":28609,"name":"Arkady Shtempler","email":"ashtempl@redhat.com","username":"ashtempl"},"change_message_id":"6517c16a63aad4f3f0d0fb672810972c03671f48","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        # Delete port associated with router and try to delete"},{"line_number":70,"context_line":"        # router, should PASS"},{"line_number":71,"context_line":"        self.client.delete_port(port[\u0027id\u0027])"},{"line_number":72,"context_line":"        self.delete_router(router)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ec69ba42_5618e77e","line":72,"in_reply_to":"46242c8b_02325fd0","updated":"2021-01-27 13:54:09.000000000","message":"I\u0027ll try to figure out manually on my host what\u0027s going on there, but this was how the things did worked for me when I was developing this test case. Anyway I\u0027ll take a look.","commit_id":"be1744e9f26a96ab2d22617c0a738e1e74d05e16"}]}