)]}'
{"octavia_tempest_plugin/common/constants.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9da872b8428fe5161fb8863e7bddfc56f628f4e4","unresolved":false,"context_lines":[{"line_number":64,"context_line":"SNI_CONTAINER_REFS \u003d \u0027sni_container_refs\u0027"},{"line_number":65,"context_line":"DEFAULT_POOL_ID \u003d \u0027default_pool_id\u0027"},{"line_number":66,"context_line":"L7_POLICIES \u003d \u0027l7_policies\u0027"},{"line_number":67,"context_line":"ALLOWED_CIDRS \u003d \u0027allowed_cidrs\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"LB_ALGORITHM \u003d \u0027lb_algorithm\u0027"},{"line_number":70,"context_line":"LB_ALGORITHM_ROUND_ROBIN \u003d \u0027ROUND_ROBIN\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_9059802a","line":67,"updated":"2020-01-17 17:27:55.000000000","message":"nit: should probably be above to make alphabetical, the jobboard change did a lot of changes there.","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"41d83e68dff0c1e4b2f7946d64f84bc3ef6ce51e","unresolved":false,"context_lines":[{"line_number":64,"context_line":"SNI_CONTAINER_REFS \u003d \u0027sni_container_refs\u0027"},{"line_number":65,"context_line":"DEFAULT_POOL_ID \u003d \u0027default_pool_id\u0027"},{"line_number":66,"context_line":"L7_POLICIES \u003d \u0027l7_policies\u0027"},{"line_number":67,"context_line":"ALLOWED_CIDRS \u003d \u0027allowed_cidrs\u0027"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"LB_ALGORITHM \u003d \u0027lb_algorithm\u0027"},{"line_number":70,"context_line":"LB_ALGORITHM_ROUND_ROBIN \u003d \u0027ROUND_ROBIN\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_b60d9437","line":67,"in_reply_to":"3fa7e38b_9059802a","updated":"2020-01-17 19:30:00.000000000","message":"Done","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"}],"octavia_tempest_plugin/tests/api/v2/test_listener.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9da872b8428fe5161fb8863e7bddfc56f628f4e4","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":610,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":611,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":612,"context_line":"                allowed_cidrs \u003d [\u00272a02:8071:69e::/64\u0027]"},{"line_number":613,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_b0927c9f","line":612,"updated":"2020-01-17 17:27:55.000000000","message":"should use the 2001:db8 doc prefix, same below, looks like an ses.com prefix?","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"41d83e68dff0c1e4b2f7946d64f84bc3ef6ce51e","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":610,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":611,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":612,"context_line":"                allowed_cidrs \u003d [\u00272a02:8071:69e::/64\u0027]"},{"line_number":613,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_9659782a","line":612,"in_reply_to":"3fa7e38b_b0927c9f","updated":"2020-01-17 19:30:00.000000000","message":"Done","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f2cb4767af5f5eae8081e735ff4cb2a506aa2832","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"            allowed_cidrs \u003d [\u0027192.0.1.0/24\u0027]"},{"line_number":116,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":117,"context_line":"                allowed_cidrs \u003d [\u00272001:db8:a0b:12f0::/64\u0027]"},{"line_number":118,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        # Test that a user without the load balancer role cannot"}],"source_content_type":"text/x-python","patch_set":10,"id":"df33271e_93f0ec69","line":117,"updated":"2020-03-25 16:42:26.000000000","message":"I\u0027ll just throw it out there that this could be in resource_setup() to avoid all the code duplication, or even outside this if() block since you\u0027re doing the same thing on L180-2 below.","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"508067c10a314e792d35055830c28e30b0e7c8e8","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"            allowed_cidrs \u003d [\u0027192.0.1.0/24\u0027]"},{"line_number":116,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":117,"context_line":"                allowed_cidrs \u003d [\u00272001:db8:a0b:12f0::/64\u0027]"},{"line_number":118,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        # Test that a user without the load balancer role cannot"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_d6e76ae2","line":117,"in_reply_to":"bf51134e_8de6ea15","updated":"2020-07-14 20:35:34.000000000","message":"Done","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"00a7964ff9a8f8f3da132df41e8a5dcfb9327e98","unresolved":false,"context_lines":[{"line_number":114,"context_line":""},{"line_number":115,"context_line":"            allowed_cidrs \u003d [\u0027192.0.1.0/24\u0027]"},{"line_number":116,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":117,"context_line":"                allowed_cidrs \u003d [\u00272001:db8:a0b:12f0::/64\u0027]"},{"line_number":118,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        # Test that a user without the load balancer role cannot"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_8de6ea15","line":117,"in_reply_to":"df33271e_93f0ec69","updated":"2020-07-07 20:13:40.000000000","message":"I agree, this definition seems to be used multiple times.","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f2cb4767af5f5eae8081e735ff4cb2a506aa2832","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":610,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":611,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":612,"context_line":"                allowed_cidrs \u003d [\u00272001:db8::/64\u0027]"},{"line_number":613,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":10,"id":"df33271e_d39f1401","line":612,"updated":"2020-03-25 16:42:26.000000000","message":"cidr change from above tests?","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"508067c10a314e792d35055830c28e30b0e7c8e8","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":610,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":611,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":612,"context_line":"                allowed_cidrs \u003d [\u00272001:db8::/64\u0027]"},{"line_number":613,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_36d22601","line":612,"in_reply_to":"df33271e_d39f1401","updated":"2020-07-14 20:35:34.000000000","message":"Done","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"}],"octavia_tempest_plugin/tests/scenario/v2/test_ipv6_traffic_ops.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":201,"context_line":"        }"},{"line_number":202,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"},{"line_number":203,"context_line":"        listener_id \u003d listener[const.ID]"},{"line_number":204,"context_line":"        self.addClassResourceCleanup("},{"line_number":205,"context_line":"            self.mem_listener_client.cleanup_listener,"},{"line_number":206,"context_line":"            listener_id,"},{"line_number":207,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_8dfa94b8","line":204,"range":{"start_line":204,"start_character":13,"end_line":204,"end_character":36},"updated":"2020-01-22 09:19:52.000000000","message":"you could use addCleanup here","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":201,"context_line":"        }"},{"line_number":202,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"},{"line_number":203,"context_line":"        listener_id \u003d listener[const.ID]"},{"line_number":204,"context_line":"        self.addClassResourceCleanup("},{"line_number":205,"context_line":"            self.mem_listener_client.cleanup_listener,"},{"line_number":206,"context_line":"            listener_id,"},{"line_number":207,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_8dd294fd","line":204,"range":{"start_line":204,"start_character":13,"end_line":204,"end_character":36},"in_reply_to":"3fa7e38b_8dfa94b8","updated":"2020-01-22 09:40:22.000000000","message":"Oops! Well spotted from your side, bad copy-pasta from my side :D","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":221,"context_line":"        }"},{"line_number":222,"context_line":"        pool \u003d self.mem_pool_client.create_pool(**pool_kwargs)"},{"line_number":223,"context_line":"        pool_id \u003d pool[const.ID]"},{"line_number":224,"context_line":"        self.addClassResourceCleanup("},{"line_number":225,"context_line":"            self.mem_pool_client.cleanup_pool,"},{"line_number":226,"context_line":"            pool_id,"},{"line_number":227,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_6d2f183e","line":224,"range":{"start_line":224,"start_character":13,"end_line":224,"end_character":36},"updated":"2020-01-22 09:19:52.000000000","message":"ditto","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":221,"context_line":"        }"},{"line_number":222,"context_line":"        pool \u003d self.mem_pool_client.create_pool(**pool_kwargs)"},{"line_number":223,"context_line":"        pool_id \u003d pool[const.ID]"},{"line_number":224,"context_line":"        self.addClassResourceCleanup("},{"line_number":225,"context_line":"            self.mem_pool_client.cleanup_pool,"},{"line_number":226,"context_line":"            pool_id,"},{"line_number":227,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_adcf5052","line":224,"range":{"start_line":224,"start_character":13,"end_line":224,"end_character":36},"in_reply_to":"3fa7e38b_6d2f183e","updated":"2020-01-22 09:40:22.000000000","message":"Done","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                                CONF.load_balancer.build_timeout)"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        # Assert that the server is unavailable (HTTP 503)"},{"line_number":298,"context_line":"        url_for_vip \u003d \u0027http://[{}]/\u0027.format(self.lb_vip_address)"},{"line_number":299,"context_line":"        self.assertConsistentResponse((503, None), url_for_vip)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_0daea4a0","line":298,"range":{"start_line":298,"start_character":23,"end_line":298,"end_character":34},"updated":"2020-01-22 09:19:52.000000000","message":":8080 (listener port) is missing from the url","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                                CONF.load_balancer.build_timeout)"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        # Assert that the server is unavailable (HTTP 503)"},{"line_number":298,"context_line":"        url_for_vip \u003d \u0027http://[{}]/\u0027.format(self.lb_vip_address)"},{"line_number":299,"context_line":"        self.assertConsistentResponse((503, None), url_for_vip)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_4db49cd4","line":298,"range":{"start_line":298,"start_character":23,"end_line":298,"end_character":34},"in_reply_to":"3fa7e38b_0daea4a0","updated":"2020-01-22 09:40:22.000000000","message":"Done","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9258966deabe058d023d3b5932e2df8641556f14","unresolved":false,"context_lines":[{"line_number":315,"context_line":"        listener_port \u003d 8080"},{"line_number":316,"context_line":"        listener_kwargs \u003d {"},{"line_number":317,"context_line":"            const.NAME: listener_name,"},{"line_number":318,"context_line":"            const.PROTOCOL: self.protocol,"},{"line_number":319,"context_line":"            const.PROTOCOL_PORT: listener_port,"},{"line_number":320,"context_line":"            const.LOADBALANCER_ID: self.lb_id,"},{"line_number":321,"context_line":"            const.ALLOWED_CIDRS: [\u0027::/0\u0027]"}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_e41928d7","line":318,"range":{"start_line":318,"start_character":12,"end_line":318,"end_character":41},"updated":"2020-08-06 13:47:59.000000000","message":"nit: this test uses ipv4 members with ipv6 VIP, it would fail if self.protocol is UDP","commit_id":"7413a6e85f943fea05263a8aea3e3e87d1a7caf7"}],"octavia_tempest_plugin/tests/scenario/v2/test_listener.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9da872b8428fe5161fb8863e7bddfc56f628f4e4","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":221,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":222,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":223,"context_line":"                allowed_cidrs \u003d [\u00272a02:8071:69e::/64\u0027]"},{"line_number":224,"context_line":"            listener_update_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        listener \u003d self.mem_listener_client.update_listener("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_30320cc2","line":223,"updated":"2020-01-17 17:27:55.000000000","message":"should use doc prefix","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"41d83e68dff0c1e4b2f7946d64f84bc3ef6ce51e","unresolved":false,"context_lines":[{"line_number":220,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":221,"context_line":"            allowed_cidrs \u003d [\u0027192.0.2.0/24\u0027]"},{"line_number":222,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":223,"context_line":"                allowed_cidrs \u003d [\u00272a02:8071:69e::/64\u0027]"},{"line_number":224,"context_line":"            listener_update_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        listener \u003d self.mem_listener_client.update_listener("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_369684a2","line":223,"in_reply_to":"3fa7e38b_30320cc2","updated":"2020-01-17 19:30:00.000000000","message":"Done","commit_id":"c6ccf2b99c494ebe2622bd31b2b20d077354615c"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"f2cb4767af5f5eae8081e735ff4cb2a506aa2832","unresolved":false,"context_lines":[{"line_number":138,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":139,"context_line":"            allowed_cidrs \u003d [\u0027192.0.1.0/24\u0027]"},{"line_number":140,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":141,"context_line":"                allowed_cidrs \u003d [\u00272001:db8:a0b:12f0::/64\u0027]"},{"line_number":142,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":10,"id":"df33271e_338e40d0","line":141,"updated":"2020-03-25 16:42:26.000000000","message":"Same comment here.","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"508067c10a314e792d35055830c28e30b0e7c8e8","unresolved":false,"context_lines":[{"line_number":138,"context_line":"                self.api_version, \u00272.12\u0027):"},{"line_number":139,"context_line":"            allowed_cidrs \u003d [\u0027192.0.1.0/24\u0027]"},{"line_number":140,"context_line":"            if CONF.load_balancer.test_with_ipv6:"},{"line_number":141,"context_line":"                allowed_cidrs \u003d [\u00272001:db8:a0b:12f0::/64\u0027]"},{"line_number":142,"context_line":"            listener_kwargs.update({const.ALLOWED_CIDRS: allowed_cidrs})"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_56bebaad","line":141,"in_reply_to":"df33271e_338e40d0","updated":"2020-07-14 20:35:34.000000000","message":"Done","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"}],"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":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":726,"context_line":"        }"},{"line_number":727,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"},{"line_number":728,"context_line":"        listener_id \u003d listener[const.ID]"},{"line_number":729,"context_line":"        self.addClassResourceCleanup("},{"line_number":730,"context_line":"            self.mem_listener_client.cleanup_listener,"},{"line_number":731,"context_line":"            listener_id,"},{"line_number":732,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_8d137402","line":729,"range":{"start_line":729,"start_character":13,"end_line":729,"end_character":36},"updated":"2020-01-22 09:19:52.000000000","message":"addCleanup","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":726,"context_line":"        }"},{"line_number":727,"context_line":"        listener \u003d self.mem_listener_client.create_listener(**listener_kwargs)"},{"line_number":728,"context_line":"        listener_id \u003d listener[const.ID]"},{"line_number":729,"context_line":"        self.addClassResourceCleanup("},{"line_number":730,"context_line":"            self.mem_listener_client.cleanup_listener,"},{"line_number":731,"context_line":"            listener_id,"},{"line_number":732,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_0df8e47b","line":729,"range":{"start_line":729,"start_character":13,"end_line":729,"end_character":36},"in_reply_to":"3fa7e38b_8d137402","updated":"2020-01-22 09:40:22.000000000","message":"Done","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":746,"context_line":"        }"},{"line_number":747,"context_line":"        pool \u003d self.mem_pool_client.create_pool(**pool_kwargs)"},{"line_number":748,"context_line":"        pool_id \u003d pool[const.ID]"},{"line_number":749,"context_line":"        self.addClassResourceCleanup("},{"line_number":750,"context_line":"            self.mem_pool_client.cleanup_pool,"},{"line_number":751,"context_line":"            pool_id,"},{"line_number":752,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_ad10b0f5","line":749,"range":{"start_line":749,"start_character":13,"end_line":749,"end_character":36},"updated":"2020-01-22 09:19:52.000000000","message":"ditto","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":746,"context_line":"        }"},{"line_number":747,"context_line":"        pool \u003d self.mem_pool_client.create_pool(**pool_kwargs)"},{"line_number":748,"context_line":"        pool_id \u003d pool[const.ID]"},{"line_number":749,"context_line":"        self.addClassResourceCleanup("},{"line_number":750,"context_line":"            self.mem_pool_client.cleanup_pool,"},{"line_number":751,"context_line":"            pool_id,"},{"line_number":752,"context_line":"            lb_client\u003dself.mem_lb_client, lb_id\u003dself.lb_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_2dfda06c","line":749,"range":{"start_line":749,"start_character":13,"end_line":749,"end_character":36},"in_reply_to":"3fa7e38b_ad10b0f5","updated":"2020-01-22 09:40:22.000000000","message":"Done","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"c7521383061b56ce7c35a4df4535b203cfe3f47a","unresolved":false,"context_lines":[{"line_number":820,"context_line":"                                CONF.load_balancer.build_timeout)"},{"line_number":821,"context_line":""},{"line_number":822,"context_line":"        # Assert that the server is unavailable (HTTP 503)"},{"line_number":823,"context_line":"        url_for_vip \u003d \u0027http://{}/\u0027.format(self.lb_vip_address)"},{"line_number":824,"context_line":"        self.assertConsistentResponse((503, None), url_for_vip)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_edc8e875","line":823,"range":{"start_line":823,"start_character":23,"end_line":823,"end_character":33},"updated":"2020-01-22 09:19:52.000000000","message":"listener port is missing","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"9b6476d6a4e413c9b36f36b07b3efa86acfc0ee2","unresolved":false,"context_lines":[{"line_number":820,"context_line":"                                CONF.load_balancer.build_timeout)"},{"line_number":821,"context_line":""},{"line_number":822,"context_line":"        # Assert that the server is unavailable (HTTP 503)"},{"line_number":823,"context_line":"        url_for_vip \u003d \u0027http://{}/\u0027.format(self.lb_vip_address)"},{"line_number":824,"context_line":"        self.assertConsistentResponse((503, None), url_for_vip)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_6d2cf800","line":823,"range":{"start_line":823,"start_character":23,"end_line":823,"end_character":33},"in_reply_to":"3fa7e38b_edc8e875","updated":"2020-01-22 09:40:22.000000000","message":"Done","commit_id":"f7428cdca998e811cee72d44ec76b906bb05bc01"}],"octavia_tempest_plugin/tests/test_base.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"00a7964ff9a8f8f3da132df41e8a5dcfb9327e98","unresolved":false,"context_lines":[{"line_number":504,"context_line":"                lb_kwargs[const.VIP_NETWORK_ID] \u003d ("},{"line_number":505,"context_line":"                    cls.lb_member_vip_net[const.ID])"},{"line_number":506,"context_line":"                if ip_version \u003d\u003d 6:"},{"line_number":507,"context_line":"                    lb_kwargs[const.VIP_ADDRESS] \u003d lb_vip_address"},{"line_number":508,"context_line":"        else:"},{"line_number":509,"context_line":"            lb_kwargs[const.VIP_NETWORK_ID] \u003d cls.lb_member_vip_net[const.ID]"},{"line_number":510,"context_line":"            lb_kwargs[const.VIP_SUBNET_ID] \u003d None"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_2d9cfe5b","line":507,"updated":"2020-07-07 20:13:40.000000000","message":"Why is this different than line 501-502? It seems like it should only be valid if fixed IPs are being used.","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"508067c10a314e792d35055830c28e30b0e7c8e8","unresolved":false,"context_lines":[{"line_number":504,"context_line":"                lb_kwargs[const.VIP_NETWORK_ID] \u003d ("},{"line_number":505,"context_line":"                    cls.lb_member_vip_net[const.ID])"},{"line_number":506,"context_line":"                if ip_version \u003d\u003d 6:"},{"line_number":507,"context_line":"                    lb_kwargs[const.VIP_ADDRESS] \u003d lb_vip_address"},{"line_number":508,"context_line":"        else:"},{"line_number":509,"context_line":"            lb_kwargs[const.VIP_NETWORK_ID] \u003d cls.lb_member_vip_net[const.ID]"},{"line_number":510,"context_line":"            lb_kwargs[const.VIP_SUBNET_ID] \u003d None"}],"source_content_type":"text/x-python","patch_set":10,"id":"bf51134e_9167bc25","line":507,"in_reply_to":"bf51134e_2d9cfe5b","updated":"2020-07-14 20:35:34.000000000","message":"Otherwise the LB VIP is an IPv4 address. L501-502 is conditional on using fixed IP which is false (L499).\n\nCommenting L506-507 and testing with noop drivers, result in 3 test failures. Example:\n\nCaptured pythonlogging:\n~~~~~~~~~~~~~~~~~~~~~~~\n    2020-07-14 20:27:35,035 31843 INFO     [tempest.lib.common.rest_client] Request (ListenerAPITest:test_listener_update): 400 POST http://192.168.121.16/load-balancer/v2.0/lbaas/listeners 0.056s\n2020-07-14 20:27:35,035 31843 DEBUG    [tempest.lib.common.rest_client] Request - Headers: {\u0027Content-Type\u0027: \u0027application/json\u0027, \u0027Accept\u0027: \u0027application/json\u0027, \u0027X-Auth-Token\u0027: \u0027\u003comitted\u003e\u0027}\n        Body: {\"listener\": {\"allowed_cidrs\": [\"2001:db8:a0b:12f0::/64\"], \"insert_headers\": {\"X-Forwarded-For\": \"true\", \"X-Forwarded-Port\": \"true\"}, \"timeout_tcp_inspect\": 50, \"timeout_member_data\": 1000, \"timeout_member_connect\": 1000, \"timeout_client_data\": 1000, \"connection_limit\": 200, \"admin_state_up\": false, \"description\": \"testtesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttesttes\", \"name\": \"tempest-lb_member_listener1-update-1695918742\", \"loadbalancer_id\": \"5934078d-2778-46a6-aa98-142bbf65e31a\", \"protocol_port\": 82, \"protocol\": \"HTTP\"}}\n    Response - Headers: {\u0027date\u0027: \u0027Tue, 14 Jul 2020 20:27:34 GMT\u0027, \u0027server\u0027: \u0027Apache/2.4.29 (Ubuntu)\u0027, \u0027content-length\u0027: \u0027166\u0027, \u0027content-type\u0027: \u0027application/json\u0027, \u0027x-openstack-request-id\u0027: \u0027req-fcc1a7a9-0955-4ecf-a241-175a32354d20\u0027, \u0027connection\u0027: \u0027close\u0027, \u0027status\u0027: \u0027400\u0027, \u0027content-location\u0027: \u0027http://192.168.121.16/load-balancer/v2.0/lbaas/listeners\u0027}\n        Body: b\u0027{\"faultcode\": \"Client\", \"faultstring\": \"Validation failure: CIDR 2001:db8:a0b:12f0::/64 IP version incompatible with VIP 198.51.100.1 IP version.\", \"debuginfo\": null}\u0027","commit_id":"0d29a7162e7ae99db77a5ba2b8a57a8fb70c84c2"}],"octavia_tempest_plugin/tests/waiters.py":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"9258966deabe058d023d3b5932e2df8641556f14","unresolved":false,"context_lines":[{"line_number":227,"context_line":"                return"},{"line_number":228,"context_line":"        except Exception as e:"},{"line_number":229,"context_line":"            LOG.error(e)"},{"line_number":230,"context_line":"            pass"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        if int(time.time()) - start \u003e\u003d timeout:"},{"line_number":233,"context_line":"            message \u003d \"Timed out after {timeout} seconds waiting\".format("}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_646a381a","line":230,"range":{"start_line":230,"start_character":12,"end_line":230,"end_character":16},"updated":"2020-08-06 13:47:59.000000000","message":"nit: we could remove that line","commit_id":"7413a6e85f943fea05263a8aea3e3e87d1a7caf7"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"a862cb813b845084e02cb5ca414a60bf5a74ae00","unresolved":false,"context_lines":[{"line_number":227,"context_line":"                return"},{"line_number":228,"context_line":"        except Exception as e:"},{"line_number":229,"context_line":"            LOG.error(e)"},{"line_number":230,"context_line":"            pass"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        if int(time.time()) - start \u003e\u003d timeout:"},{"line_number":233,"context_line":"            message \u003d \"Timed out after {timeout} seconds waiting\".format("}],"source_content_type":"text/x-python","patch_set":11,"id":"9f560f44_1192974c","line":230,"range":{"start_line":230,"start_character":12,"end_line":230,"end_character":16},"in_reply_to":"9f560f44_646a381a","updated":"2020-08-06 16:35:42.000000000","message":"Done","commit_id":"7413a6e85f943fea05263a8aea3e3e87d1a7caf7"}]}