)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"2bbfc5bce21eba545f062da2c3d57505f538538c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f675dd36_59ec5337","updated":"2024-08-06 09:12:26.000000000","message":"need add unit tests","commit_id":"0db8a94f3c43b3f43dbfb99cd28b2ac44c6b4df8"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"a873687525c0d65ee79eea10fa8ba44074931a64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"8439c76b_d4ea6e88","updated":"2024-08-28 16:41:27.000000000","message":"Thanks again Rico for taking care of this! Still putting focus in the previous patch but this one looks fine","commit_id":"d82a8e0ec303b7a30c9c817511b487827a690416"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"fc7a595a5d922a11ca92eb09ea735ee59f2cea4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"53f2a10e_e0649cc9","updated":"2024-09-10 03:03:20.000000000","message":"recheck","commit_id":"a84cbd4729969d822b4be94107bf3da18950c325"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"be67c5fab766776b994419d79e41829551e10b0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"aa70f04e_4ac2d7ec","updated":"2024-09-09 13:26:56.000000000","message":"recheck","commit_id":"a84cbd4729969d822b4be94107bf3da18950c325"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"72f5529c504fb6c7cc19d116d49e377fd186f694","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7f68bbf1_994922c6","updated":"2024-09-10 08:24:09.000000000","message":"recheck ovn-octavia-provider-tempest-release unrelated","commit_id":"a84cbd4729969d822b4be94107bf3da18950c325"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"aa0ae3aa04e4145514d8d6285301dbaeb56235d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"05addf70_585289fe","updated":"2024-09-13 07:40:49.000000000","message":"recheck","commit_id":"dbd19cb27fe963b140df2e67ac92f777019ea4db"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"0c6294d335fc0f44f9e16d886b7366e9a9edf12a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"20897aed_07ec9bfe","updated":"2024-09-13 07:40:57.000000000","message":"recheck","commit_id":"dbd19cb27fe963b140df2e67ac92f777019ea4db"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"c2fe888cd7a600a6d1c2c48c469f11e794e59ec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"ba358140_bc64ec55","updated":"2024-09-16 03:24:01.000000000","message":"recheck","commit_id":"dbd19cb27fe963b140df2e67ac92f777019ea4db"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"044f2f5d178b8ca7a8197d8dc112637e4f6a35f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"6409ff2b_64c83fd4","updated":"2024-09-24 15:05:25.000000000","message":"recheck","commit_id":"7cb1c98d4f0a177e6b4a5c21e2925d447812d245"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"265b472e30f79dbe95b1cdff0b222e70099dc744","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"91baa631_56899f3b","updated":"2024-09-25 06:50:51.000000000","message":"Hi Fernando\n\nI can\u0027t reproduce the issue in this case\nI post the log and commands I ran to build loadbanacer and do sync in https://docs.google.com/document/d/1w547wsz8UUz9M9ajO6etADAuQYyHqxaG35FsnRZyLlY/edit?usp\u003dsharing\n\nCan you share with me how you setup env and test? Maybe something I missed here. \nAlso you can add `--debug` and share the logs with me.","commit_id":"660071ad8d8365eb704bb4d427e8d202a725cfa3"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"048470bb675696e71c4f52f9b26fad1a792810e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"1f565ecb_39407b26","updated":"2024-09-24 16:49:45.000000000","message":"This is what I get after some new round of tests:\n\nstack@ubuntu2204:~/devstack$ sudo ovn-nbctl list load_balancer\n_uuid               : 0fddd6d7-07ec-4949-88d6-797c8ad7a962\nexternal_ids        : {enabled\u003dTrue, listener_d3db3598-0396-44f3-af82-0744c4fc965b\u003d\"80:pool_5a54b9d3-129c-4984-adb7-c2d50f43d191\", lr_ref\u003dneutron-4764763c-49ac-4282-8e19-e12e5f7a3ea7, ls_refs\u003d\"{\\\"neutron-bfa9c1be-a38e-474b-8b25-b00e0913e62f\\\": 3}\", \"neutron:member_status\"\u003d\"{\\\"a196610e-441b-4f15-b104-5c71716a26bb\\\": \\\"NO_MONITOR\\\", \\\"e14f3ba6-a9c9-459f-b8f5-6ec1505ea27c\\\": \\\"NO_MONITOR\\\"}\", \"neutron:vip\"\u003d\"192.168.59.67\", \"neutron:vip_fip\"\u003d\"172.24.4.158\", \"neutron:vip_port_id\"\u003d\"c7eeacab-f48e-42ad-8a36-d123d9a14f2d\", pool_5a54b9d3-129c-4984-adb7-c2d50f43d191\u003d\"member_a196610e-441b-4f15-b104-5c71716a26bb_192.168.59.5:80_3a49a35b-6ff3-403b-9e69-cb5fd87f8520,member_e14f3ba6-a9c9-459f-b8f5-6ec1505ea27c_192.168.59.6:80_3a49a35b-6ff3-403b-9e69-cb5fd87f8520\"}\nhealth_check        : []\nip_port_mappings    : {}\nname                : \"e006005a-0a37-4b61-b2f0-39739898df07\"\noptions             : {}\nprotocol            : tcp\nselection_fields    : [ip_dst, ip_src, tp_dst, tp_src]\nvips                : {\"172.24.4.158:80\"\u003d\"192.168.59.5:80,192.168.59.6:80\", \"192.168.59.67:80\"\u003d\"192.168.59.5:80,192.168.59.6:80\"}\n\nstack@ubuntu2204:~/devstack$ sudo ovn-nbctl destroy load_balancer 0fddd6d7-07ec-4949-88d6-797c8ad7a962\n\n---\u003e run the syn now and getting this output:\n\n(venv) stack@ubuntu2204:~/ovn-octavia-provider$ octavia-ovn-db-sync-util\nINFO ovn_octavia_provider.driver [-] Starting sync OVN DB with Loadbalancer filter {}\nINFO ovn_octavia_provider.driver [-] Starting sync OVN DB with Loadbalancer e006005a-0a37-4b61-b2f0-39739898df07\nINFO ovn_octavia_provider.driver [-] Starting sync floating IP for Loadbalancer e006005a-0a37-4b61-b2f0-39739898df07\n\n\n\n\n----\u003e but result on OVN DB looks incomplete\n\nstack@ubuntu2204:~/devstack$ sudo ovn-nbctl list load_balancer\n_uuid               : 4b5c925b-7605-4713-8402-b9baa1030ae8\nexternal_ids        : {enabled\u003dTrue, listener_d3db3598-0396-44f3-af82-0744c4fc965b\u003d\"80:pool_5a54b9d3-129c-4984-adb7-c2d50f43d191\", lr_ref\u003dneutron-4764763c-49ac-4282-8e19-e12e5f7a3ea7, ls_refs\u003d\"{\\\"neutron-bfa9c1be-a38e-474b-8b25-b00e0913e62f\\\": 1}\", \"neutron:vip\"\u003d\"192.168.59.67\", \"neutron:vip_port_id\"\u003d\"c7eeacab-f48e-42ad-8a36-d123d9a14f2d\", pool_5a54b9d3-129c-4984-adb7-c2d50f43d191\u003d\"\"}\nhealth_check        : []\nip_port_mappings    : {}\nname                : \"e006005a-0a37-4b61-b2f0-39739898df07\"\noptions             : {}\nprotocol            : tcp\nselection_fields    : [ip_dst, ip_src, tp_dst, tp_src]\nvips                : {}","commit_id":"660071ad8d8365eb704bb4d427e8d202a725cfa3"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"eb454894ae374811e1b9cf955b9dd64ae0a88192","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"e3764345_81a352ec","updated":"2024-09-25 14:33:21.000000000","message":"recheck","commit_id":"07bf0cff82ef726e4478230d8d4ea31f515ef5d4"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"cfd2c05f97492df45b09b50a0a8b66acd05cabb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"6966ceaf_db6b2c55","updated":"2024-10-16 15:36:57.000000000","message":"Perhaps related bug rather than closes-bug, as this is the first from the chain and it is not fixing the bug yet","commit_id":"8fcd72e088939a5ba6e601649014f0836149e735"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"deb4bb5470365971631c5481abc723df83acaa4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"27873602_121ee29c","updated":"2024-10-14 10:51:02.000000000","message":"recheck ovn-octavia-provider-functional-release unrelated","commit_id":"8fcd72e088939a5ba6e601649014f0836149e735"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"00e81cdaa860f888829a6db96b839090ce37d70f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"15036bfa_516bb74f","updated":"2024-10-11 12:02:33.000000000","message":"recheck ovn-octavia-provider-tempest-release unrelated","commit_id":"8fcd72e088939a5ba6e601649014f0836149e735"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"5cd77ba4ad592e6b37f1350440f8f8e240c3146b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"dbe2c261_3c1b2780","updated":"2024-10-14 09:03:42.000000000","message":"recheck post_failure unrelated","commit_id":"8fcd72e088939a5ba6e601649014f0836149e735"}],"ovn_octavia_provider/common/clients.py":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"a873687525c0d65ee79eea10fa8ba44074931a64","unresolved":true,"context_lines":[{"line_number":159,"context_line":"        return OctaviaAuth().lbaas_proxy"},{"line_number":160,"context_line":"    except Exception as e:"},{"line_number":161,"context_line":"        msg \u003d _(\u0027Cannot initialize OpenStackSDK. Exception: %s. \u0027"},{"line_number":162,"context_line":"                \u0027Please verify Keystone service configuration \u0027"},{"line_number":163,"context_line":"                \u0027in Octavia API configuration.\u0027) % e"},{"line_number":164,"context_line":"        raise driver_exceptions.DriverError("},{"line_number":165,"context_line":"            operator_fault_string\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":12,"id":"bec90165_ad61da54","line":162,"updated":"2024-08-28 16:41:27.000000000","message":"\"Keystone -\u003e Octavia\" in the log msg?","commit_id":"d82a8e0ec303b7a30c9c817511b487827a690416"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"3832abb2675a5fbe1ece439f205f03b40fe64128","unresolved":true,"context_lines":[{"line_number":159,"context_line":"        return OctaviaAuth().lbaas_proxy"},{"line_number":160,"context_line":"    except Exception as e:"},{"line_number":161,"context_line":"        msg \u003d _(\u0027Cannot initialize OpenStackSDK. Exception: %s. \u0027"},{"line_number":162,"context_line":"                \u0027Please verify Keystone service configuration \u0027"},{"line_number":163,"context_line":"                \u0027in Octavia API configuration.\u0027) % e"},{"line_number":164,"context_line":"        raise driver_exceptions.DriverError("},{"line_number":165,"context_line":"            operator_fault_string\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":12,"id":"70f7e792_4032aa95","line":162,"in_reply_to":"007cab2f_e905a6ac","updated":"2024-08-29 08:09:27.000000000","message":"okay, make a little change on this to use `service_auth` phrase","commit_id":"d82a8e0ec303b7a30c9c817511b487827a690416"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"d183026a13f540d90afc9755a8e008df9b8f67fe","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        return OctaviaAuth().lbaas_proxy"},{"line_number":160,"context_line":"    except Exception as e:"},{"line_number":161,"context_line":"        msg \u003d _(\u0027Cannot initialize OpenStackSDK. Exception: %s. \u0027"},{"line_number":162,"context_line":"                \u0027Please verify Keystone service configuration \u0027"},{"line_number":163,"context_line":"                \u0027in Octavia API configuration.\u0027) % e"},{"line_number":164,"context_line":"        raise driver_exceptions.DriverError("},{"line_number":165,"context_line":"            operator_fault_string\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":12,"id":"212ee830_4a834b57","line":162,"in_reply_to":"70f7e792_4032aa95","updated":"2024-09-09 09:01:19.000000000","message":"Done","commit_id":"d82a8e0ec303b7a30c9c817511b487827a690416"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"5f032aa3b1c4082cf7d9be7e0239648eab6c2aa3","unresolved":true,"context_lines":[{"line_number":159,"context_line":"        return OctaviaAuth().lbaas_proxy"},{"line_number":160,"context_line":"    except Exception as e:"},{"line_number":161,"context_line":"        msg \u003d _(\u0027Cannot initialize OpenStackSDK. Exception: %s. \u0027"},{"line_number":162,"context_line":"                \u0027Please verify Keystone service configuration \u0027"},{"line_number":163,"context_line":"                \u0027in Octavia API configuration.\u0027) % e"},{"line_number":164,"context_line":"        raise driver_exceptions.DriverError("},{"line_number":165,"context_line":"            operator_fault_string\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":12,"id":"007cab2f_e905a6ac","line":162,"in_reply_to":"bec90165_ad61da54","updated":"2024-08-29 06:31:40.000000000","message":"actually I do refering `service_auth` setion in config which is what keystoneauth uses by default for service auth, also that\u0027s what been setup in octavia.conf\nIf there\u0027s better way to describe in message, I\u0027m fine with change it for sure :)","commit_id":"d82a8e0ec303b7a30c9c817511b487827a690416"}],"ovn_octavia_provider/driver.py":[{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"5cf9806829efc6e7b8225b9df74bcd70efa10da1","unresolved":true,"context_lines":[{"line_number":725,"context_line":"                self._ovn_helper._octavia_driver_lib.get_loadbalancer(lb.id)"},{"line_number":726,"context_line":"            provider_pools \u003d []"},{"line_number":727,"context_line":"            provider_listeners \u003d []"},{"line_number":728,"context_line":"            for p in provider_lb.pools:"},{"line_number":729,"context_line":"                provider_pool \u003d o_datamodels.Pool.from_dict(p)"},{"line_number":730,"context_line":"                # format healthmonitor provider"},{"line_number":731,"context_line":"                if not isinstance("}],"source_content_type":"text/x-python","patch_set":16,"id":"2c59973c_ba4ad075","line":728,"updated":"2024-09-06 12:42:06.000000000","message":"Here it would be necessary to check that the pools are not None in case it is a partial LB that does not yet have pools, so that it does not break the process.","commit_id":"cb4c48d109bddb9cc1415b5f483bfdc91ac73506"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"d183026a13f540d90afc9755a8e008df9b8f67fe","unresolved":false,"context_lines":[{"line_number":725,"context_line":"                self._ovn_helper._octavia_driver_lib.get_loadbalancer(lb.id)"},{"line_number":726,"context_line":"            provider_pools \u003d []"},{"line_number":727,"context_line":"            provider_listeners \u003d []"},{"line_number":728,"context_line":"            for p in provider_lb.pools:"},{"line_number":729,"context_line":"                provider_pool \u003d o_datamodels.Pool.from_dict(p)"},{"line_number":730,"context_line":"                # format healthmonitor provider"},{"line_number":731,"context_line":"                if not isinstance("}],"source_content_type":"text/x-python","patch_set":16,"id":"4c02a5c8_6fe37101","line":728,"in_reply_to":"2c59973c_ba4ad075","updated":"2024-09-09 09:01:19.000000000","message":"Done","commit_id":"cb4c48d109bddb9cc1415b5f483bfdc91ac73506"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"5cf9806829efc6e7b8225b9df74bcd70efa10da1","unresolved":true,"context_lines":[{"line_number":745,"context_line":"                    provider_pool.members \u003d provider_members"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"                provider_pools.append(provider_pool)"},{"line_number":748,"context_line":"            for l in provider_lb.listeners:"},{"line_number":749,"context_line":"                provider_listeners.append(o_datamodels.Listener.from_dict(l))"},{"line_number":750,"context_line":""},{"line_number":751,"context_line":"            provider_lb.pools \u003d provider_pools"}],"source_content_type":"text/x-python","patch_set":16,"id":"9278607e_d361a7c0","line":748,"updated":"2024-09-06 12:42:06.000000000","message":"ditto (for listeners)","commit_id":"cb4c48d109bddb9cc1415b5f483bfdc91ac73506"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"d183026a13f540d90afc9755a8e008df9b8f67fe","unresolved":false,"context_lines":[{"line_number":745,"context_line":"                    provider_pool.members \u003d provider_members"},{"line_number":746,"context_line":""},{"line_number":747,"context_line":"                provider_pools.append(provider_pool)"},{"line_number":748,"context_line":"            for l in provider_lb.listeners:"},{"line_number":749,"context_line":"                provider_listeners.append(o_datamodels.Listener.from_dict(l))"},{"line_number":750,"context_line":""},{"line_number":751,"context_line":"            provider_lb.pools \u003d provider_pools"}],"source_content_type":"text/x-python","patch_set":16,"id":"3c11dc37_33ac60e3","line":748,"in_reply_to":"9278607e_d361a7c0","updated":"2024-09-09 09:01:19.000000000","message":"Done","commit_id":"cb4c48d109bddb9cc1415b5f483bfdc91ac73506"}]}