)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"466176e5c0cc5d0523cd8521eea90a5ab9db07b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ccb5b8e6_4d84cf7e","updated":"2026-06-24 08:50:58.000000000","message":"looks good for me. Please address Jakub\u0027s comment and once it will not be WIP anymore I can +2 it","commit_id":"7cbf4db9285dad52ee3801f954319bcfe7e170bc"}],"neutron_tempest_plugin/scenario/test_basic.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"33a75ef0c474a447ec4af6fcb285f0be1c2ea7df","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        self.check_connectivity(private_ip,"},{"line_number":82,"context_line":"                                CONF.validation.image_ssh_user,"},{"line_number":83,"context_line":"                                keypair[\u0027private_key\u0027])"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    @testtools.skipUnless("},{"line_number":86,"context_line":"        CONF.neutron_plugin_options.global_ip_address,"},{"line_number":87,"context_line":"        \u0027Global IP address is not defined.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"33568ade_060c6f83","line":84,"updated":"2026-06-22 20:50:14.000000000","message":"We need to ping the remote IP here, no? The IP itself should probably come from the tempest config file?","commit_id":"0b39bf800ef55695c6d4b5f80f77e1c727697562"},{"author":{"_account_id":31291,"name":"Eduardo Olivares","email":"eolivare@redhat.com","username":"eolivare"},"change_message_id":"2b0d69d12fc452aa3a4182cd38630c9a59d18ebc","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        self.check_connectivity(private_ip,"},{"line_number":82,"context_line":"                                CONF.validation.image_ssh_user,"},{"line_number":83,"context_line":"                                keypair[\u0027private_key\u0027])"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    @testtools.skipUnless("},{"line_number":86,"context_line":"        CONF.neutron_plugin_options.global_ip_address,"},{"line_number":87,"context_line":"        \u0027Global IP address is not defined.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f8bba3d_9080c60f","line":84,"in_reply_to":"33568ade_060c6f83","updated":"2026-06-23 11:27:16.000000000","message":"check_connectivity doesn\u0027t run any ping. It simply checks VM can be accessed via ssh:\nhttps://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/scenario/base.py#L353\n\nBut, the test you suggest makes more sense, so I\u0027ll add it.","commit_id":"0b39bf800ef55695c6d4b5f80f77e1c727697562"}]}