)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"5329b7f617985b67000a9dcf2a045d32df8b53b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"33f66b47_4edf6267","updated":"2025-05-21 16:00:36.000000000","message":"That is a \"privileged\" port, so I think you are going to run into security issues","commit_id":"72672173accbbb427d2078cd4e2f89a31718d618"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"30bd88c463fe9d1d1d4784100f3dac8307a1fceb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"188fbbe6_c74da68a","updated":"2025-06-10 14:52:15.000000000","message":"LGTM\nI like this approach of incrementing ports as necessary.","commit_id":"13cb2ec83d8457189030ab1be209c28186d85c69"}],"octavia_tempest_plugin/common/decorators.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9d7dae6c5b125fd85369c6f45d6f59b18b6c4a71","unresolved":true,"context_lines":[{"line_number":92,"context_line":"                    raise testtools.TestCase.skipException(message)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"                except Exception as e:"},{"line_number":95,"context_line":"                    if \"not begin passing traffic\" in str(e):"},{"line_number":96,"context_line":"                        last_exception \u003d e"},{"line_number":97,"context_line":"                        port +\u003d 1"},{"line_number":98,"context_line":"                    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"c9be39f0_f71915fb","line":95,"range":{"start_line":95,"start_character":24,"end_line":95,"end_character":49},"updated":"2025-05-22 08:30:44.000000000","message":"I think I would search for \"Address already in use\" which should happen only when using an already used port.\n\"not begin passing traffic\" can be caused by some other random issues.","commit_id":"f4e59c95a0c608d3da88db3680014fe5c1bb9a0e"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"e9dc59166d6232dc85d46e5b12df7513420634ab","unresolved":true,"context_lines":[{"line_number":92,"context_line":"                    raise testtools.TestCase.skipException(message)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"                except Exception as e:"},{"line_number":95,"context_line":"                    if \"not begin passing traffic\" in str(e):"},{"line_number":96,"context_line":"                        last_exception \u003d e"},{"line_number":97,"context_line":"                        port +\u003d 1"},{"line_number":98,"context_line":"                    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"7b0b8529_03e5e684","line":95,"range":{"start_line":95,"start_character":24,"end_line":95,"end_character":49},"in_reply_to":"47d9673d_5b0906c9","updated":"2025-05-22 09:39:37.000000000","message":"giving a try to propagate up the Errno 98, let see what gates think about it 😊","commit_id":"f4e59c95a0c608d3da88db3680014fe5c1bb9a0e"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"5b7094cee9f767fa038c9191ef891661d7b4e794","unresolved":true,"context_lines":[{"line_number":92,"context_line":"                    raise testtools.TestCase.skipException(message)"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"                except Exception as e:"},{"line_number":95,"context_line":"                    if \"not begin passing traffic\" in str(e):"},{"line_number":96,"context_line":"                        last_exception \u003d e"},{"line_number":97,"context_line":"                        port +\u003d 1"},{"line_number":98,"context_line":"                    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"47d9673d_5b0906c9","line":95,"range":{"start_line":95,"start_character":24,"end_line":95,"end_character":49},"in_reply_to":"c9be39f0_f71915fb","updated":"2025-05-22 08:51:04.000000000","message":"Yeah, I already tried that way, but to get \"Address already in use\" in the decorator scope, we need to raise on https://opendev.org/openstack/octavia-tempest-plugin/src/branch/master/octavia_tempest_plugin/tests/validators.py#L116 adding an specific control for the \u0027HTTPConnectionPool(host\u003d\u0027172.24.4.19\u0027, port\u003d60092): Max retries exceeded with url: / (Caused by NewConnectionError(\u0027\u003curllib3.connection.HTTPConnection object at 0x7f7d050cffb0\u003e: Failed to establish a new connection: [Errno 98] Address already in use\u0027)). Retrying.\u0027 and also the methods in the middle capturing it to propagate so dont appear a track easy to follow. wdyt?","commit_id":"f4e59c95a0c608d3da88db3680014fe5c1bb9a0e"}],"octavia_tempest_plugin/tests/scenario/v2/test_traffic_ops.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"3977a6f3e5310b98c13baa302f02c0711aaf0d80","unresolved":true,"context_lines":[{"line_number":920,"context_line":"        # handle this test case special."},{"line_number":921,"context_line":"        try:"},{"line_number":922,"context_line":"            pool_id \u003d self._listener_pool_create("},{"line_number":923,"context_line":"                const.HTTP, port,"},{"line_number":924,"context_line":"                pool_algorithm\u003dconst.LB_ALGORITHM_SOURCE_IP_PORT)[1]"},{"line_number":925,"context_line":"            self._test_basic_traffic("},{"line_number":926,"context_line":"                const.HTTP, port, pool_id,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5c362f09_faaba3e4","line":923,"range":{"start_line":923,"start_character":28,"end_line":923,"end_character":32},"updated":"2025-06-13 11:43:46.000000000","message":"nit: we could have only used a dynamic port for the source_port of _test_basic_traffic, because the port of the listener is not impacted by the issue. in that case it\u0027s not required that we create a new listener at each retry.","commit_id":"13cb2ec83d8457189030ab1be209c28186d85c69"}]}