)]}'
{"octavia_tempest_plugin/contrib/test_server/test_server.go":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"32432f3cc8c21505e1c93de2c47df8cd16535fba","unresolved":true,"context_lines":[{"line_number":191,"context_line":"\t\t\t\ttls.TLS_RSA_WITH_AES_256_GCM_SHA384,"},{"line_number":192,"context_line":"\t\t\t\ttls.TLS_RSA_WITH_AES_256_CBC_SHA,"},{"line_number":193,"context_line":"\t\t\t},"},{"line_number":194,"context_line":"\t\t\tNextProtos: []string{\"http/1.1\", \"http/1.0\"},"},{"line_number":195,"context_line":"\t\t}"},{"line_number":196,"context_line":"\t}"},{"line_number":197,"context_line":"\ttlsConfig.Rand \u003d rand.Reader"}],"source_content_type":"text/x-go","patch_set":17,"id":"3bf10751_62ddb988","line":194,"range":{"start_line":194,"start_character":3,"end_line":194,"end_character":47},"updated":"2021-04-02 09:02:37.000000000","message":"It looks like we have http2 configuration issues in the golang server, NextProtos indicates that it\u0027s supported, but CipherSuites don\u0027t contain mandatory Ciphers for http2 (at least they are required by the golang net/http module).\nThen the TLSNextProto attribute in http.Server (line 203) disables http2. So the server may announce that it supports http2 but cannot handle an http2 connexion.","commit_id":"11ccf1559d19217de2e4a64bf384f5f7809fae48"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c780ecbeac473eaa23aa3a872f8306ae80cb2e63","unresolved":true,"context_lines":[{"line_number":191,"context_line":"\t\t\t\ttls.TLS_RSA_WITH_AES_256_GCM_SHA384,"},{"line_number":192,"context_line":"\t\t\t\ttls.TLS_RSA_WITH_AES_256_CBC_SHA,"},{"line_number":193,"context_line":"\t\t\t},"},{"line_number":194,"context_line":"\t\t\tNextProtos: []string{\"http/1.1\", \"http/1.0\"},"},{"line_number":195,"context_line":"\t\t}"},{"line_number":196,"context_line":"\t}"},{"line_number":197,"context_line":"\ttlsConfig.Rand \u003d rand.Reader"}],"source_content_type":"text/x-go","patch_set":17,"id":"a4c22b47_30157f15","line":194,"range":{"start_line":194,"start_character":3,"end_line":194,"end_character":47},"in_reply_to":"3bf10751_62ddb988","updated":"2021-04-02 09:05:23.000000000","message":"This is weird because I did a lot of https testing this week and I didn\u0027t hit that issue but now I reproduce it in my env","commit_id":"11ccf1559d19217de2e4a64bf384f5f7809fae48"}],"octavia_tempest_plugin/tests/barbican_scenario/v2/test_tls_barbican.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"7b6c6886c898c86d98ea58acaf788cdf9a5c0c3f","unresolved":true,"context_lines":[{"line_number":198,"context_line":"        lb_kwargs \u003d {const.PROVIDER: CONF.load_balancer.provider,"},{"line_number":199,"context_line":"                     const.NAME: lb_name}"},{"line_number":200,"context_line":""},{"line_number":201,"context_line":"        # TODO(johnsom) Update for IPv6"},{"line_number":202,"context_line":"        cls._setup_lb_network_kwargs(lb_kwargs, 4)"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"        lb \u003d cls.mem_lb_client.create_loadbalancer(**lb_kwargs)"}],"source_content_type":"text/x-python","patch_set":21,"id":"f979c92b_68a48e48","line":201,"updated":"2021-04-05 19:51:12.000000000","message":"😭 some day...","commit_id":"74b6f2ff50380ae12fe7a93ea932b9deff71bcf1"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"b088d08b68e538a04ca2e5f9c6936c87af9a215b","unresolved":true,"context_lines":[{"line_number":1222,"context_line":"        if not self.mem_listener_client.is_version_supported("},{"line_number":1223,"context_line":"                self.api_version, \u00272.8\u0027):"},{"line_number":1224,"context_line":"            raise self.skipException(\u0027Pool re-encryption is only available on \u0027"},{"line_number":1225,"context_line":"                                     \u0027Octavia API version 2.8 or newer.\u0027)"},{"line_number":1226,"context_line":"        pool_name \u003d data_utils.rand_name(\"lb_member_pool1-tls-reencrypt\")"},{"line_number":1227,"context_line":"        pool_kwargs \u003d {"},{"line_number":1228,"context_line":"            const.NAME: pool_name,"}],"source_content_type":"text/x-python","patch_set":21,"id":"081a0636_b311af9a","line":1225,"range":{"start_line":1225,"start_character":38,"end_line":1225,"end_character":71},"updated":"2021-04-09 09:12:52.000000000","message":"I think we can remove this block, train provides v2.13","commit_id":"74b6f2ff50380ae12fe7a93ea932b9deff71bcf1"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"8f00b9ad91c27c5a70c414e16eb89be20d5c6bff","unresolved":true,"context_lines":[{"line_number":1222,"context_line":"        if not self.mem_listener_client.is_version_supported("},{"line_number":1223,"context_line":"                self.api_version, \u00272.8\u0027):"},{"line_number":1224,"context_line":"            raise self.skipException(\u0027Pool re-encryption is only available on \u0027"},{"line_number":1225,"context_line":"                                     \u0027Octavia API version 2.8 or newer.\u0027)"},{"line_number":1226,"context_line":"        pool_name \u003d data_utils.rand_name(\"lb_member_pool1-tls-reencrypt\")"},{"line_number":1227,"context_line":"        pool_kwargs \u003d {"},{"line_number":1228,"context_line":"            const.NAME: pool_name,"}],"source_content_type":"text/x-python","patch_set":21,"id":"36324581_1b75042a","line":1225,"range":{"start_line":1225,"start_character":38,"end_line":1225,"end_character":71},"in_reply_to":"081a0636_b311af9a","updated":"2021-04-09 15:26:52.000000000","message":"Nope, tempest plugin is branch-less, so there could be a chance someone would run it against something older than we currently test upstream. It\u0027s best to just leave these IMO.","commit_id":"74b6f2ff50380ae12fe7a93ea932b9deff71bcf1"}]}
