)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ae5e59e049afab9c87a49505b7e7d49bdee60d5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"191c156d_7c64c613","updated":"2026-06-15 15:03:56.000000000","message":"Just my one nit, not a deal breaker","commit_id":"74aaf149404b8ff95c1c2806c3e43fcea453c3bd"},{"author":{"_account_id":19298,"name":"Nicolas Bock","email":"nicolas.bock@canonical.com","username":"nicolasbock"},"change_message_id":"e06a93a31a89a37f92aec01907f466e784029984","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0faa86ac_209df8d4","updated":"2026-06-15 17:57:34.000000000","message":"LGTM","commit_id":"74aaf149404b8ff95c1c2806c3e43fcea453c3bd"}],"src/lib/charm/openstack/neutron_api_plugin_ovn.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ae5e59e049afab9c87a49505b7e7d49bdee60d5d","unresolved":true,"context_lines":[{"line_number":158,"context_line":"        return states_to_check"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    @property"},{"line_number":161,"context_line":"    def db_migration_needed(self):  # noqa"},{"line_number":162,"context_line":"        \"\"\"Determine whether DB migration is needed."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        The returned variable must be set in the release specifc charm classes."}],"source_content_type":"text/x-python","patch_set":1,"id":"3440587c_6e39d82b","line":161,"updated":"2026-06-15 15:03:56.000000000","message":"not sure why the # noqa is here, not on the master branch","commit_id":"74aaf149404b8ff95c1c2806c3e43fcea453c3bd"},{"author":{"_account_id":6737,"name":"Edward Hope-Morley","email":"edward.hope-morley@canonical.com","username":"hopem"},"change_message_id":"e742e789fc782c0c44249881f49eeed28140329c","unresolved":true,"context_lines":[{"line_number":158,"context_line":"        return states_to_check"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    @property"},{"line_number":161,"context_line":"    def db_migration_needed(self):  # noqa"},{"line_number":162,"context_line":"        \"\"\"Determine whether DB migration is needed."},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"        The returned variable must be set in the release specifc charm classes."}],"source_content_type":"text/x-python","patch_set":1,"id":"abf6bf65_d092e506","line":161,"in_reply_to":"3440587c_6e39d82b","updated":"2026-06-15 19:58:24.000000000","message":"so in the master and caracal branches it was fixed by changing var db_migration_needed -\u003e _db_migration_needed. I dont know why that was done and assume it is not fixing a bug so I went with this for now.","commit_id":"74aaf149404b8ff95c1c2806c3e43fcea453c3bd"}]}