)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"4267505a7c7fc87b94606c622b18af535a0832bf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2e8a7fe4_bee7aa24","updated":"2021-11-19 10:47:07.000000000","message":"recheck","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"03d9a0a1eb99b7f52dbaf87bc4d1c4f68eb8e656","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2a7d2c66_95e4f374","updated":"2021-11-25 07:13:45.000000000","message":"Thanks Michael, it works fine in my env\n","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"afdea606d9091c76ed8400b3fb7f8d2811247d55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"97f92c20_e00b51f0","updated":"2021-12-01 15:38:45.000000000","message":"recheck\n\nPOST_FAILURE","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"7f09da5260a6da33dd6cbbf2efc3abe756ca3198","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3df3ec68_b9b38016","updated":"2022-01-12 18:16:30.000000000","message":"recheck","commit_id":"9b0103e57dd90c2083b566f8001733538c4efe8b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"a52c0e6f9ec9f8c04d9f7c26821cc9fdbbb5855c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ce3a3077_e10892a5","updated":"2022-01-31 23:46:34.000000000","message":"recheck","commit_id":"9b0103e57dd90c2083b566f8001733538c4efe8b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"426f175ba35d361bb4ce5e7751697376d5f0ee09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e1d7785c_b1460352","updated":"2022-01-13 16:53:48.000000000","message":"recheck","commit_id":"9b0103e57dd90c2083b566f8001733538c4efe8b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"d22f3a1c86339584e09b1994465e7d06a9e0dec9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ef77a77f_6fd1defa","updated":"2021-12-10 10:34:46.000000000","message":"recheck","commit_id":"9b0103e57dd90c2083b566f8001733538c4efe8b"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"bb9d1fd54b3d0dac81552494e7ef7d4ce963743d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"706f21ea_4a34a01f","updated":"2022-02-17 22:50:22.000000000","message":"recheck\nunrelated test issue","commit_id":"9b0103e57dd90c2083b566f8001733538c4efe8b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"97ea929ecc14e3775664681f7a573f21ec9d8271","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"ecbeb2d8_9f048a9b","updated":"2022-10-04 06:53:49.000000000","message":"recheck octavia-v2-dsvm-scenario-stable-wallaby POST_FAILURE","commit_id":"031ecca5132c8491e55d93a9ff88c670fccd7983"}],"octavia_tempest_plugin/tests/barbican_scenario/v2/test_tls_barbican.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bb757a28f4815f8e1aa0d1c7bf7f356818322830","unresolved":true,"context_lines":[{"line_number":1368,"context_line":"        if not self.mem_listener_client.is_version_supported("},{"line_number":1369,"context_line":"                self.api_version, \u00272.8\u0027):"},{"line_number":1370,"context_line":"            raise self.skipException(\u0027Pool re-encryption is only available on \u0027"},{"line_number":1371,"context_line":"                                     \u0027Octavia API version 2.8 or newer.\u0027)"},{"line_number":1372,"context_line":"        pool_name \u003d data_utils.rand_name(\"lb_member_pool1-tls-client-auth\")"},{"line_number":1373,"context_line":"        pool_kwargs \u003d {"},{"line_number":1374,"context_line":"            const.NAME: pool_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf7ea6db_c714aca9","line":1371,"range":{"start_line":1371,"start_character":38,"end_line":1371,"end_character":62},"updated":"2021-04-09 11:54:06.000000000","message":"nit: train provides Octavia API v2.13","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2941cc0fa31c73c6772854405b95bbc70ae76609","unresolved":true,"context_lines":[{"line_number":1368,"context_line":"        if not self.mem_listener_client.is_version_supported("},{"line_number":1369,"context_line":"                self.api_version, \u00272.8\u0027):"},{"line_number":1370,"context_line":"            raise self.skipException(\u0027Pool re-encryption is only available on \u0027"},{"line_number":1371,"context_line":"                                     \u0027Octavia API version 2.8 or newer.\u0027)"},{"line_number":1372,"context_line":"        pool_name \u003d data_utils.rand_name(\"lb_member_pool1-tls-client-auth\")"},{"line_number":1373,"context_line":"        pool_kwargs \u003d {"},{"line_number":1374,"context_line":"            const.NAME: pool_name,"}],"source_content_type":"text/x-python","patch_set":4,"id":"0099d46b_c8db5112","line":1371,"range":{"start_line":1371,"start_character":38,"end_line":1371,"end_character":62},"in_reply_to":"bf7ea6db_c714aca9","updated":"2022-09-22 20:34:05.000000000","message":"I am not sure what this comment means?\nre-encrypt client auth was added in 2.8 per the api-ref in this patch:\nhttps://review.opendev.org/c/openstack/octavia/+/614432","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bb757a28f4815f8e1aa0d1c7bf7f356818322830","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"93c2b775_9e300288","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"updated":"2021-04-09 11:54:06.000000000","message":"This part doesn\u0027t work in my env (I tried both centos and ubuntu amp image), haproxy immediately closes the connection after receiving the request from the client.\n\nbut the part that is below (with client certs) works correctly","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"a2550297813d42fb256171ef8d8f93a3f28afc99","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"c807890d_3d9487c1","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"in_reply_to":"0e256d90_61ef5351","updated":"2021-09-27 18:58:56.000000000","message":"Hmm, this is likely a race condition on when the health checks detect the servers as unreachable. I suspect the \"SD--\" status on your run is during the \"assume up\" window. Maybe we need to define an explicit health monitor with now retry/timeouts here.","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"cd0c2067a9a69d908b0e961df56720eb9e55f297","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"0e256d90_61ef5351","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"in_reply_to":"7018327f_1a498b15","updated":"2021-04-09 13:41:36.000000000","message":"The log line corresponding to this request in the last successful CI job is:\n\nApr  2 22:28:15 amphora-2bd8ed6a-b59e-4157-bc14-a1c4785e2a3e haproxy[3506]: bd6e05430cb349a1b1459a1199eaa47a ac37c0fb-a344-4781-adbd-3fae32b5df39 5ea1fdc1-419c-45df-8046-fbc2e8124f4f 172.24.5.1 38906 02/Apr/2021:22:28:15.049 \"GET / HTTP/1.1\" 503 237 125 - \"\" d987eb09-6a64-4034-bcad-fa73869babe0:5ea1fdc1-419c-45df-8046-fbc2e8124f4f 4d84f2ba-bd9f-4939-922b-c2cfc7491438 140 SC--\n\nSC-- may indicate that 503 was returned to the client because the webserver was not ready to process the request.","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bb11dc53c0209a66a4803b49aaa9673957fb58f6","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"7018327f_1a498b15","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"in_reply_to":"93c2b775_9e300288","updated":"2021-04-09 12:11:17.000000000","message":"This is what I get on my VIP port:\n\n    1   0.000000   172.24.4.1 → 10.1.1.221   TCP 74 48146 → 85 [SYN] Seq\u003d0 Win\u003d29200 Len\u003d0 MSS\u003d1460 SACK_PERM\u003d1 TSval\u003d3540595702 TSecr\u003d0 WS\u003d128\n    2   0.000847   10.1.1.221 → 172.24.4.1   TCP 74 85 → 48146 [SYN, ACK] Seq\u003d0 Ack\u003d1 Win\u003d32154 Len\u003d0 MSS\u003d1410 SACK_PERM\u003d1 TSval\u003d946980836 TSecr\u003d3540595702 WS\u003d16\n    3   0.001124   172.24.4.1 → 10.1.1.221   TCP 66 48146 → 85 [ACK] Seq\u003d1 Ack\u003d1 Win\u003d29312 Len\u003d0 TSval\u003d3540595703 TSecr\u003d946980836\n    4   0.001378   172.24.4.1 → 10.1.1.221   HTTP 212 GET / HTTP/1.1 \n    5   0.044254   10.1.1.221 → 172.24.4.1   TCP 66 85 → 48146 [ACK] Seq\u003d1 Ack\u003d147 Win\u003d32016 Len\u003d0 TSval\u003d946980880 TSecr\u003d3540595703\n    6   0.055830   10.1.1.221 → 172.24.4.1   TCP 66 85 → 48146 [FIN, ACK] Seq\u003d1 Ack\u003d147 Win\u003d32016 Len\u003d0 TSval\u003d946980891 TSecr\u003d3540595703\n    7   0.056081   172.24.4.1 → 10.1.1.221   TCP 66 48146 → 85 [FIN, ACK] Seq\u003d147 Ack\u003d2 Win\u003d29312 Len\u003d0 TSval\u003d3540595758 TSecr\u003d946980891\n\nand the tenant logs:\n\nApr  9 11:35:51 amphora-5a421b9e-3c2c-40a7-a75d-165e60b91ab1 haproxy[1515]: 8177efd6eba84a889c7731bec35111b4 b95693ef-911b-4eef-8361-25680cfff563 2dd280ee-6f88-4108-81e9-d00a65e99fab 172.24.4.1 41012 09/Apr/2021:11:35:51.372 \"GET / HTTP/1.1\" -1 0 146 - \"\" 13b6f4b6-f937-4238-8f79-65c057edfd5f:2dd280ee-6f88-4108-81e9-d00a65e99fab 9e6c90e8-b420-45ec-a841-dd6fa2314de9 30 SD--","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2700f049fb9ded947bdacfc714d39e2e55713583","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"ccaabe1a_a778a94c","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"in_reply_to":"c807890d_3d9487c1","updated":"2021-09-27 19:03:15.000000000","message":"The pcap looks like it might be an issue on the golang server side. I wonder if it was using the wrong golang code?","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"32b02bc8ef2f0b3c33f8cc2ee2cce8a3ee589209","unresolved":true,"context_lines":[{"line_number":1446,"context_line":""},{"line_number":1447,"context_line":"        # Test that there are no members without a client certificate"},{"line_number":1448,"context_line":"        url \u003d \u0027http://{0}:85\u0027.format(self.lb_vip_address)"},{"line_number":1449,"context_line":"        self.validate_URL_response(url, expected_status_code\u003d503)"},{"line_number":1450,"context_line":""},{"line_number":1451,"context_line":"        # Test with client certificates"},{"line_number":1452,"context_line":"        pool_update_kwargs \u003d {"}],"source_content_type":"text/x-python","patch_set":4,"id":"b9e0c8c0_97f9eec5","line":1449,"range":{"start_line":1449,"start_character":8,"end_line":1449,"end_character":65},"in_reply_to":"ccaabe1a_a778a94c","updated":"2021-11-19 13:31:09.000000000","message":"I tested different methods:\n- the test passes when building the test_server.bin app in a Ubuntu bionic environment (ubuntu:bionic container + golang)\n- it fails when bulding the test_server.bin app in a Ubuntu focal or Centos env\n\nI captured the traffic to the member in both cases:\n\nwhen the test passes:\n   21  25.363573    10.2.1.53 → 10.2.1.49    TCP 74 42111 → 9443 [SYN] Seq\u003d0 Win\u003d28040 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d3856723980 TSecr\u003d0 WS\u003d2048\n   22  25.363890    10.2.1.49 → 10.2.1.53    TCP 74 9443 → 42111 [SYN, ACK] Seq\u003d0 Ack\u003d1 Win\u003d65330 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d749253746 TSecr\u003d3856723980 WS\u003d16\n   23  25.364050    10.2.1.53 → 10.2.1.49    TCP 66 42111 → 9443 [ACK] Seq\u003d1 Ack\u003d1 Win\u003d28672 Len\u003d0 TSval\u003d3856723981 TSecr\u003d749253746\n   24  25.364098    10.2.1.53 → 10.2.1.49    TLSv1 583 Client Hello\n   25  25.364892    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42111 [ACK] Seq\u003d1 Ack\u003d518 Win\u003d64816 Len\u003d0 TSval\u003d749253747 TSecr\u003d3856723981\n   26  25.379134    10.2.1.49 → 10.2.1.53    TLSv1.2 1683 Server Hello, Certificate, Server Key Exchange, Certificate Request, Server Hello Done\n   27  25.379387    10.2.1.53 → 10.2.1.49    TCP 66 42111 → 9443 [ACK] Seq\u003d518 Ack\u003d1618 Win\u003d32768 Len\u003d0 TSval\u003d3856723996 TSecr\u003d749253762\n   28  25.380619    10.2.1.53 → 10.2.1.49    TLSv1.2 272 Certificate, Client Key Exchange, Change Cipher Spec, Encrypted Handshake Message\n   29  25.380729    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42111 [ACK] Seq\u003d1618 Ack\u003d724 Win\u003d64624 Len\u003d0 TSval\u003d749253763 TSecr\u003d3856723997\n   30  25.380804    10.2.1.49 → 10.2.1.53    TLSv1.2 73 Alert (Level: Fatal, Description: Bad Certificate)\n   31  25.381012    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42111 [FIN, ACK] Seq\u003d1625 Ack\u003d724 Win\u003d64624 Len\u003d0 TSval\u003d749253764 TSecr\u003d3856723997\n   32  25.381209    10.2.1.53 → 10.2.1.49    TCP 66 42111 → 9443 [RST, ACK] Seq\u003d724 Ack\u003d1626 Win\u003d32768 Len\u003d0 TSval\u003d3856723998 TSecr\u003d749253763\n   33  25.399882    10.2.1.53 → 10.2.1.49    TCP 74 42115 → 9443 [SYN] Seq\u003d0 Win\u003d28040 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d3856724017 TSecr\u003d0 WS\u003d2048\n   34  25.400118    10.2.1.49 → 10.2.1.53    TCP 74 9443 → 42115 [SYN, ACK] Seq\u003d0 Ack\u003d1 Win\u003d65330 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d749253783 TSecr\u003d3856724017 WS\u003d16\n   35  25.400294    10.2.1.53 → 10.2.1.49    TCP 66 42115 → 9443 [ACK] Seq\u003d1 Ack\u003d1 Win\u003d28672 Len\u003d0 TSval\u003d3856724017 TSecr\u003d749253783\n   36  25.400303    10.2.1.53 → 10.2.1.49    TLSv1 583 Client Hello\n   37  25.400402    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42115 [ACK] Seq\u003d1 Ack\u003d518 Win\u003d64816 Len\u003d0 TSval\u003d749253783 TSecr\u003d3856724017\n   38  25.414411    10.2.1.49 → 10.2.1.53    TLSv1.2 1683 Server Hello, Certificate, Server Key Exchange, Certificate Request, Server Hello Done\n   39  25.414640    10.2.1.53 → 10.2.1.49    TCP 66 42115 → 9443 [ACK] Seq\u003d518 Ack\u003d1618 Win\u003d32768 Len\u003d0 TSval\u003d3856724031 TSecr\u003d749253797\n   40  25.415912    10.2.1.53 → 10.2.1.49    TLSv1.2 272 Certificate, Client Key Exchange, Change Cipher Spec, Encrypted Handshake Message\n   41  25.416036    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42115 [ACK] Seq\u003d1618 Ack\u003d724 Win\u003d64624 Len\u003d0 TSval\u003d749253799 TSecr\u003d3856724033\n   42  25.416132    10.2.1.49 → 10.2.1.53    TLSv1.2 73 Alert (Level: Fatal, Description: Bad Certificate)\n   43  25.416349    10.2.1.49 → 10.2.1.53    TCP 66 9443 → 42115 [FIN, ACK] Seq\u003d1625 Ack\u003d724 Win\u003d64624 Len\u003d0 TSval\u003d749253799 TSecr\u003d3856724033\n   44  25.416672    10.2.1.53 → 10.2.1.49    TCP 66 42115 → 9443 [RST, ACK] Seq\u003d724 Ack\u003d1626 Win\u003d32768 Len\u003d0 TSval\u003d3856724034 TSecr\u003d749253799\n\n\nWhen it fails:\n   25  10.069226    10.2.1.85 → 10.2.1.202   TCP 74 40925 → 9443 [SYN] Seq\u003d0 Win\u003d28040 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d1839029739 TSecr\u003d0 WS\u003d2048\n   26  10.069437   10.2.1.202 → 10.2.1.85    TCP 74 9443 → 40925 [SYN, ACK] Seq\u003d0 Ack\u003d1 Win\u003d65330 Len\u003d0 MSS\u003d1402 SACK_PERM\u003d1 TSval\u003d2352920910 TSecr\u003d1839029739 WS\u003d16\n   27  10.069536    10.2.1.85 → 10.2.1.202   TCP 66 40925 → 9443 [ACK] Seq\u003d1 Ack\u003d1 Win\u003d28672 Len\u003d0 TSval\u003d1839029740 TSecr\u003d2352920910\n   28  10.070151    10.2.1.85 → 10.2.1.202   TLSv1 583 Client Hello\n   29  10.070215   10.2.1.202 → 10.2.1.85    TCP 66 9443 → 40925 [ACK] Seq\u003d1 Ack\u003d518 Win\u003d64816 Len\u003d0 TSval\u003d2352920911 TSecr\u003d1839029740\n   30  10.070298   10.2.1.202 → 10.2.1.85    TLSv1.3 159 Hello Retry Request\n   31  10.070336    10.2.1.85 → 10.2.1.202   TCP 66 40925 → 9443 [ACK] Seq\u003d518 Ack\u003d94 Win\u003d28672 Len\u003d0 TSval\u003d1839029741 TSecr\u003d2352920911\n   32  10.070338   10.2.1.202 → 10.2.1.85    TLSv1.3 72 Change Cipher Spec\n   33  10.070384    10.2.1.85 → 10.2.1.202   TCP 66 40925 → 9443 [ACK] Seq\u003d518 Ack\u003d100 Win\u003d28672 Len\u003d0 TSval\u003d1839029741 TSecr\u003d2352920911\n   34  10.070693    10.2.1.85 → 10.2.1.202   TLSv1.3 589 Change Cipher Spec, Client Hello\n   35  10.070796   10.2.1.202 → 10.2.1.85    TCP 66 9443 → 40925 [ACK] Seq\u003d100 Ack\u003d1041 Win\u003d64304 Len\u003d0 TSval\u003d2352920912 TSecr\u003d1839029741\n   36  10.091243   10.2.1.202 → 10.2.1.85    TLSv1.3 1873 Server Hello, Application Data, Application Data, Application Data, Application Data, Application Data\n   37  10.091395    10.2.1.85 → 10.2.1.202   TCP 66 40925 → 9443 [ACK] Seq\u003d1041 Ack\u003d1907 Win\u003d32768 Len\u003d0 TSval\u003d1839029762 TSecr\u003d2352920932\n   38  10.092173    10.2.1.85 → 10.2.1.202   TLSv1.3 154 Application Data, Application Data\n   39  10.092242    10.2.1.85 → 10.2.1.202   TLSv1.3 133 Application Data\n   40  10.092298   10.2.1.202 → 10.2.1.85    TCP 66 9443 → 40925 [ACK] Seq\u003d1907 Ack\u003d1129 Win\u003d64224 Len\u003d0 TSval\u003d2352920933 TSecr\u003d1839029762\n   41  10.092360   10.2.1.202 → 10.2.1.85    TCP 66 9443 → 40925 [ACK] Seq\u003d1907 Ack\u003d1196 Win\u003d64160 Len\u003d0 TSval\u003d2352920933 TSecr\u003d1839029762\n   42  10.092450   10.2.1.202 → 10.2.1.85    TLSv1.3 90 Application Data\n   43  10.092723   10.2.1.202 → 10.2.1.85    TCP 66 9443 → 40925 [FIN, ACK] Seq\u003d1931 Ack\u003d1196 Win\u003d64160 Len\u003d0 TSval\u003d2352920934 TSecr\u003d1839029762\n   44  10.093394    10.2.1.85 → 10.2.1.202   TCP 66 40925 → 9443 [RST, ACK] Seq\u003d1196 Ack\u003d1932 Win\u003d32768 Len\u003d0 TSval\u003d1839029764 TSecr\u003d2352920933","commit_id":"02eeb176ffb8376157ef8dfdbf5cd2ceeafee2f5"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"03d9a0a1eb99b7f52dbaf87bc4d1c4f68eb8e656","unresolved":true,"context_lines":[{"line_number":1377,"context_line":"            const.LOADBALANCER_ID: self.lb_id,"},{"line_number":1378,"context_line":"            const.TLS_ENABLED: True"},{"line_number":1379,"context_line":"        }"},{"line_number":1380,"context_line":"        # Specify an the http/1.x alpn to work around HTTP healthchecks"},{"line_number":1381,"context_line":"        # on older haproxy versions when alpn includes h2"},{"line_number":1382,"context_line":"        if self.mem_listener_client.is_version_supported("},{"line_number":1383,"context_line":"                self.api_version, \u00272.24\u0027):"}],"source_content_type":"text/x-python","patch_set":7,"id":"c42b750d_ae677a88","line":1380,"range":{"start_line":1380,"start_character":18,"end_line":1380,"end_character":24},"updated":"2021-11-25 07:13:45.000000000","message":"nit typo","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"48cffa8cd99a11eb2bafb3aebeb2ea14dd9c3015","unresolved":false,"context_lines":[{"line_number":1377,"context_line":"            const.LOADBALANCER_ID: self.lb_id,"},{"line_number":1378,"context_line":"            const.TLS_ENABLED: True"},{"line_number":1379,"context_line":"        }"},{"line_number":1380,"context_line":"        # Specify an the http/1.x alpn to work around HTTP healthchecks"},{"line_number":1381,"context_line":"        # on older haproxy versions when alpn includes h2"},{"line_number":1382,"context_line":"        if self.mem_listener_client.is_version_supported("},{"line_number":1383,"context_line":"                self.api_version, \u00272.24\u0027):"}],"source_content_type":"text/x-python","patch_set":7,"id":"34bdceba_7e42fefd","line":1380,"range":{"start_line":1380,"start_character":18,"end_line":1380,"end_character":24},"in_reply_to":"c42b750d_ae677a88","updated":"2021-12-02 23:12:12.000000000","message":"Done","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"03d9a0a1eb99b7f52dbaf87bc4d1c4f68eb8e656","unresolved":true,"context_lines":[{"line_number":1438,"context_line":"            member1_kwargs[const.SUBNET_ID] \u003d self.lb_member_1_subnet[const.ID]"},{"line_number":1439,"context_line":""},{"line_number":1440,"context_line":"        self.mem_member_client.create_member(**member1_kwargs)"},{"line_number":1441,"context_line":"        waiters.wait_for_status("},{"line_number":1442,"context_line":"            self.mem_lb_client.show_loadbalancer, self.lb_id,"},{"line_number":1443,"context_line":"            const.PROVISIONING_STATUS, const.ACTIVE,"},{"line_number":1444,"context_line":"            CONF.load_balancer.check_interval,"}],"source_content_type":"text/x-python","patch_set":7,"id":"225f143a_da8e35fa","line":1441,"range":{"start_line":1441,"start_character":16,"end_line":1441,"end_character":31},"updated":"2021-11-25 07:13:45.000000000","message":"shouldn\u0027t we also wait for the operating_status to be DOWN now that we have a HM?","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"48cffa8cd99a11eb2bafb3aebeb2ea14dd9c3015","unresolved":true,"context_lines":[{"line_number":1438,"context_line":"            member1_kwargs[const.SUBNET_ID] \u003d self.lb_member_1_subnet[const.ID]"},{"line_number":1439,"context_line":""},{"line_number":1440,"context_line":"        self.mem_member_client.create_member(**member1_kwargs)"},{"line_number":1441,"context_line":"        waiters.wait_for_status("},{"line_number":1442,"context_line":"            self.mem_lb_client.show_loadbalancer, self.lb_id,"},{"line_number":1443,"context_line":"            const.PROVISIONING_STATUS, const.ACTIVE,"},{"line_number":1444,"context_line":"            CONF.load_balancer.check_interval,"}],"source_content_type":"text/x-python","patch_set":7,"id":"8b7e1f68_cef2b6d4","line":1441,"range":{"start_line":1441,"start_character":16,"end_line":1441,"end_character":31},"in_reply_to":"225f143a_da8e35fa","updated":"2021-12-02 23:12:12.000000000","message":"We could, but it\u0027s a bit duplicate to the 503 check below.","commit_id":"17352972108e394845a1581b66fc1e5fa4b7cfe6"}]}
