)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"a4d1838b83d43715493dc6b7cab72b2b2433ebc3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"5dd1f2c7_06a60236","updated":"2022-08-31 07:03:52.000000000","message":"recheck","commit_id":"c7fced51a55283fa813fb99fcc37645f62721166"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3d7028d3484da71495713f5afa020561df3b6747","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"8f69c543_e6b2c921","updated":"2022-09-05 09:19:30.000000000","message":"Let\u0027s wait with the other patches till the dust settles around Zed","commit_id":"04617051ee25abda96c3c50be12e1b9cc6ddeaf3"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"450cd9edc2e283ab721dc2368fe637165c0585ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":46,"id":"77cc826a_976fa192","updated":"2022-10-24 11:29:54.000000000","message":"recheck","commit_id":"6e060c571359c5546831c2da9c800bbdfcce4384"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"c2c690cd2739643a7611841522c8a598ba3cebd9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":59,"id":"ed660944_7eebee5a","updated":"2022-11-17 07:18:11.000000000","message":"interesting...\n\nAssertionError: Migration in expand branch contains drop command","commit_id":"196f8a156e9e1291e34cc37d5f8109e4ce59f7de"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"7c7ebc6b73a19d274c9a5e0b5d2dcdcc8301f318","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"56f0eb18_71a35034","updated":"2022-11-17 15:54:31.000000000","message":"Previsously two tests was failing, now just one...","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"d7f3902eb186148971200de2ae65c873498f49ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"08366873_6b8d4a11","updated":"2022-11-22 12:37:19.000000000","message":"recheck\nanother hit of https://bugs.launchpad.net/neutron/+bug/1997025","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"023bbcfc1fa1b3f6da13495c985e557f33e50108","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"13fed387_f09b5188","updated":"2022-11-22 19:48:45.000000000","message":"recheck fixed trunk live migration issue","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"346602ac09d25687eac387b5ed5c10a6105e3597","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"04934baa_e48f6aa2","updated":"2022-11-17 15:54:43.000000000","message":"recheck unrelated","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"c339d14c7f0f6cb5721588c160a74fc80b444017","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"0bf07837_28fccd7e","updated":"2022-11-20 11:03:36.000000000","message":"recheck unrelated","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"68bfba5a33ad8f89e85771ee5752390d310d1ece","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"0eab9a52_53c5b851","updated":"2022-11-18 07:30:51.000000000","message":"recheck unrelated","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"eb13adcd48ae34878dadc7acbf33d952e648adb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"6b32b091_c0c0c612","updated":"2022-11-18 11:10:52.000000000","message":"recheck unrelated","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"e06fc8bb537e4598a076df4959d2d7ba33bdd6d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":61,"id":"872e06cd_29c1fe3c","updated":"2022-11-17 12:58:19.000000000","message":"recheck unrelated","commit_id":"961def08d01b411965fb992d0aa79d7bc36259a4"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"9af67a38f6dfcb1cf8ac6b8f93e9fc3fab5445f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":62,"id":"bff4ffe9_5fe7cdd1","updated":"2022-11-23 09:58:05.000000000","message":"Zuul passed https://review.opendev.org/c/openstack/neutron/+/841838/70","commit_id":"eeff5b3c81ae7846a77c65edaafe58332090a621"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"117b8d1cce04170fb1741b0a3e101d5033f1ddac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":62,"id":"0a172874_73b7f844","updated":"2022-11-24 06:46:50.000000000","message":"recheck unrelated","commit_id":"eeff5b3c81ae7846a77c65edaafe58332090a621"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"a0064fcd22499b818d05bb5eb92a778c303876e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":62,"id":"4f1bdc95_f5c3852b","updated":"2022-11-23 09:56:45.000000000","message":"recheck unrelated","commit_id":"eeff5b3c81ae7846a77c65edaafe58332090a621"}],"neutron/cmd/upgrade_checks/checks.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0c030e7e25379dbb077f2f64164d5a5d6a494ea5","unresolved":true,"context_lines":[{"line_number":120,"context_line":"        segment.NetworkSegment.network_id,"},{"line_number":121,"context_line":"        segment.NetworkSegment.network_type,"},{"line_number":122,"context_line":"        segment.NetworkSegment.physical_network,"},{"line_number":123,"context_line":"        segment.NetworkSegment.segment_index"},{"line_number":124,"context_line":"    )"},{"line_number":125,"context_line":"    query \u003d query.having(func.count() \u003e 1)"},{"line_number":126,"context_line":"    return query.count()"}],"source_content_type":"text/x-python","patch_set":47,"id":"8cc209cc_5f24b3bc","line":123,"range":{"start_line":123,"start_character":43,"end_line":123,"end_character":44},"updated":"2022-11-04 08:43:57.000000000","message":"please, add another comma at the end to preserve the git history in future commits","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"48083dfbe942944241cd47d3bcc6c961f9c8a3a3","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        segment.NetworkSegment.network_id,"},{"line_number":121,"context_line":"        segment.NetworkSegment.network_type,"},{"line_number":122,"context_line":"        segment.NetworkSegment.physical_network,"},{"line_number":123,"context_line":"        segment.NetworkSegment.segment_index"},{"line_number":124,"context_line":"    )"},{"line_number":125,"context_line":"    query \u003d query.having(func.count() \u003e 1)"},{"line_number":126,"context_line":"    return query.count()"}],"source_content_type":"text/x-python","patch_set":47,"id":"47658e9b_7fd79400","line":123,"range":{"start_line":123,"start_character":43,"end_line":123,"end_character":44},"in_reply_to":"8cc209cc_5f24b3bc","updated":"2022-11-04 13:55:32.000000000","message":"Done","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"}],"neutron/db/migration/alembic_migrations/versions/antelope/expand/fc153938cdc1_update_segment_networks_constraint.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a1cc7b83544401fa77c5d4a0a761a571ebc5a096","unresolved":true,"context_lines":[{"line_number":31,"context_line":"revision \u003d \u0027fc153938cdc1\u0027"},{"line_number":32,"context_line":"down_revision \u003d \u00275881373af7f5\u0027"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"# milestone identifier, used by neutron-db-manage"},{"line_number":35,"context_line":"neutron_milestone \u003d [migration.ANTELOPE]"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"TABLE_NAME \u003d \u0027networksegments\u0027"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":58,"id":"215da24b_b115b027","line":35,"range":{"start_line":34,"start_character":0,"end_line":35,"end_character":40},"updated":"2022-11-16 18:05:17.000000000","message":"This is used only when a new branch is open and is added to the last commit of the previous branch. E.g.: https://review.opendev.org/c/openstack/neutron/+/859111","commit_id":"55bc7b2ca2d0dc2ac672ffb73ca5a428e2090a1d"}],"neutron/db/migration/alembic_migrations/versions/antilope/expand/e38852cbedef_update_segment_networks_uniq_contraint.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0c030e7e25379dbb077f2f64164d5a5d6a494ea5","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2022 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":47,"id":"2fa3505a_4a9802c2","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":8},"updated":"2022-11-04 08:43:57.000000000","message":"Directory name should be \"antelope\", not \"antilope\"","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"48083dfbe942944241cd47d3bcc6c961f9c8a3a3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2022 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":47,"id":"c99a18e0_f32846f3","line":1,"range":{"start_line":1,"start_character":0,"end_line":1,"end_character":8},"in_reply_to":"2fa3505a_4a9802c2","updated":"2022-11-04 13:55:32.000000000","message":"Done","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0c030e7e25379dbb077f2f64164d5a5d6a494ea5","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def upgrade():"},{"line_number":40,"context_line":"    op.drop_constraint("},{"line_number":41,"context_line":"        constraint_name\u003d\u0027\u0027,"},{"line_number":42,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":43,"context_line":"        type_\u003d\u0027foreignkey\u0027)"},{"line_number":44,"context_line":"    op.drop_constraint("},{"line_number":45,"context_line":"        constraint_name\u003d("},{"line_number":46,"context_line":"            \u0027uniq_networksegment0network_id0network_type0physical_network\u0027),"}],"source_content_type":"text/x-python","patch_set":47,"id":"d5dd9679_f0e3f646","line":43,"range":{"start_line":40,"start_character":5,"end_line":43,"end_character":27},"updated":"2022-11-04 08:43:57.000000000","message":"What are you dropping here?","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"308c9c4f67158e10551fc693a2d3dd1a5d196754","unresolved":true,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def upgrade():"},{"line_number":40,"context_line":"    op.drop_constraint("},{"line_number":41,"context_line":"        constraint_name\u003d\u0027\u0027,"},{"line_number":42,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":43,"context_line":"        type_\u003d\u0027foreignkey\u0027)"},{"line_number":44,"context_line":"    op.drop_constraint("},{"line_number":45,"context_line":"        constraint_name\u003d("},{"line_number":46,"context_line":"            \u0027uniq_networksegment0network_id0network_type0physical_network\u0027),"}],"source_content_type":"text/x-python","patch_set":47,"id":"be8343e4_e644e8d0","line":43,"range":{"start_line":40,"start_character":5,"end_line":43,"end_character":27},"in_reply_to":"0a44b70f_13fde6c9","updated":"2022-11-04 14:52:45.000000000","message":"Actually I think this part is needed:\n\n    raise errorclass(errno, errval)\nsqlalchemy.exc.OperationalError: (pymysql.err.OperationalError) (1553, \"Cannot drop index \u0027uniq_networksegment0network_id0network_type0physical_network\u0027: needed in a foreign key constraint\")\n[SQL: ALTER TABLE networksegments DROP INDEX uniq_networksegment0network_id0network_type0physical_network]\n(Background on this error at: https://sqlalche.me/e/14/e3q8)","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"6b95dc887e8d860b65df41502c4e39d8ef5efa9f","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def upgrade():"},{"line_number":40,"context_line":"    op.drop_constraint("},{"line_number":41,"context_line":"        constraint_name\u003d\u0027\u0027,"},{"line_number":42,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":43,"context_line":"        type_\u003d\u0027foreignkey\u0027)"},{"line_number":44,"context_line":"    op.drop_constraint("},{"line_number":45,"context_line":"        constraint_name\u003d("},{"line_number":46,"context_line":"            \u0027uniq_networksegment0network_id0network_type0physical_network\u0027),"}],"source_content_type":"text/x-python","patch_set":47,"id":"31d5500c_5a030013","line":43,"range":{"start_line":40,"start_character":5,"end_line":43,"end_character":27},"in_reply_to":"be8343e4_e644e8d0","updated":"2022-11-09 11:10:25.000000000","message":"Done","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"48083dfbe942944241cd47d3bcc6c961f9c8a3a3","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"def upgrade():"},{"line_number":40,"context_line":"    op.drop_constraint("},{"line_number":41,"context_line":"        constraint_name\u003d\u0027\u0027,"},{"line_number":42,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":43,"context_line":"        type_\u003d\u0027foreignkey\u0027)"},{"line_number":44,"context_line":"    op.drop_constraint("},{"line_number":45,"context_line":"        constraint_name\u003d("},{"line_number":46,"context_line":"            \u0027uniq_networksegment0network_id0network_type0physical_network\u0027),"}],"source_content_type":"text/x-python","patch_set":47,"id":"0a44b70f_13fde6c9","line":43,"range":{"start_line":40,"start_character":5,"end_line":43,"end_character":27},"in_reply_to":"d5dd9679_f0e3f646","updated":"2022-11-04 13:55:32.000000000","message":":-)","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0c030e7e25379dbb077f2f64164d5a5d6a494ea5","unresolved":true,"context_lines":[{"line_number":47,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":48,"context_line":"        type_\u003d\u0027unique\u0027)"},{"line_number":49,"context_line":"    op.create_unique_constraint("},{"line_number":50,"context_line":"        \u0027uniq_networksegment0net_id0type0phynet0segid\u0027,"},{"line_number":51,"context_line":"        TABLE_NAME,"},{"line_number":52,"context_line":"        [\u0027network_id\u0027, \u0027network_type\u0027, \u0027physical_network\u0027, \u0027segment_index\u0027])"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":47,"id":"896afca0_6992f3ce","line":50,"range":{"start_line":50,"start_character":41,"end_line":50,"end_character":47},"updated":"2022-11-04 08:43:57.000000000","message":"physnet (this is how we usually contract \"physical_network\")","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"48083dfbe942944241cd47d3bcc6c961f9c8a3a3","unresolved":false,"context_lines":[{"line_number":47,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":48,"context_line":"        type_\u003d\u0027unique\u0027)"},{"line_number":49,"context_line":"    op.create_unique_constraint("},{"line_number":50,"context_line":"        \u0027uniq_networksegment0net_id0type0phynet0segid\u0027,"},{"line_number":51,"context_line":"        TABLE_NAME,"},{"line_number":52,"context_line":"        [\u0027network_id\u0027, \u0027network_type\u0027, \u0027physical_network\u0027, \u0027segment_index\u0027])"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":47,"id":"4e8a41b5_26691ca1","line":50,"range":{"start_line":50,"start_character":41,"end_line":50,"end_character":47},"in_reply_to":"896afca0_6992f3ce","updated":"2022-11-04 13:55:32.000000000","message":"Done","commit_id":"1b71983b58a65d29a2f564db1d47ab10c0eb175d"}],"neutron/db/migration/alembic_migrations/versions/zed/expand/e38852cbedef_update_segment_networks_uniq_contraint.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"79cad5d0dcffbba6368e1623fa93c81a2f4bfd3b","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def upgrade():"},{"line_number":35,"context_line":"    op.drop_constraint("},{"line_number":36,"context_line":"        constraint_name\u003d\u0027networksegments_ibfk_1\u0027,"},{"line_number":37,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":38,"context_line":"        type_\u003d\u0027foreignkey\u0027)"}],"source_content_type":"text/x-python","patch_set":35,"id":"cf743f9f_0f9302bf","line":35,"updated":"2022-09-06 09:15:59.000000000","message":"this isn\u0027t allowed by default in the EXPAND phase. If You really need to do it, You should use expand_drop_exceptions() function, like e.g. in https://github.com/openstack/neutron/blob/master/neutron/db/migration/alembic_migrations/versions/zed/expand/5881373af7f5_delete_sgpb_on_sg_deletion.py#L47","commit_id":"04617051ee25abda96c3c50be12e1b9cc6ddeaf3"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"d2016584bf77bfd3af18f64006f24f9feaf73af7","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def upgrade():"},{"line_number":35,"context_line":"    op.drop_constraint("},{"line_number":36,"context_line":"        constraint_name\u003d\u0027networksegments_ibfk_1\u0027,"},{"line_number":37,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":38,"context_line":"        type_\u003d\u0027foreignkey\u0027)"}],"source_content_type":"text/x-python","patch_set":35,"id":"2baaeb67_44ba3f8c","line":35,"in_reply_to":"3fbb9b32_510fbeeb","updated":"2022-10-07 07:09:14.000000000","message":"Done, thank you Slawek!","commit_id":"04617051ee25abda96c3c50be12e1b9cc6ddeaf3"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"991466c1bc8a00852667a6aaaef11ef9a3560ad4","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def upgrade():"},{"line_number":35,"context_line":"    op.drop_constraint("},{"line_number":36,"context_line":"        constraint_name\u003d\u0027networksegments_ibfk_1\u0027,"},{"line_number":37,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":38,"context_line":"        type_\u003d\u0027foreignkey\u0027)"}],"source_content_type":"text/x-python","patch_set":35,"id":"dcf651e2_1ba16570","line":35,"in_reply_to":"cf743f9f_0f9302bf","updated":"2022-10-04 13:17:02.000000000","message":"I have tested locally and that was working. This fk has been created by mysql itself when has been created the unique constraint against `networksegment network_id network_type physical_network`.\n\nDo you still think I should use your suggestion?","commit_id":"04617051ee25abda96c3c50be12e1b9cc6ddeaf3"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"1adf38f0c399311ff98af3d4eef6f99a639d0bff","unresolved":true,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"def upgrade():"},{"line_number":35,"context_line":"    op.drop_constraint("},{"line_number":36,"context_line":"        constraint_name\u003d\u0027networksegments_ibfk_1\u0027,"},{"line_number":37,"context_line":"        table_name\u003dTABLE_NAME,"},{"line_number":38,"context_line":"        type_\u003d\u0027foreignkey\u0027)"}],"source_content_type":"text/x-python","patch_set":35,"id":"3fbb9b32_510fbeeb","line":35,"in_reply_to":"dcf651e2_1ba16570","updated":"2022-10-04 13:18:20.000000000","message":"Hum I should have checked CI before to respond you ;-)","commit_id":"04617051ee25abda96c3c50be12e1b9cc6ddeaf3"}],"neutron/tests/unit/plugins/ml2/test_plugin.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a1cc7b83544401fa77c5d4a0a761a571ebc5a096","unresolved":true,"context_lines":[{"line_number":3011,"context_line":"        retry_fixture.setUp()"},{"line_number":3012,"context_line":"        network_req \u003d self.new_create_request(\u0027networks\u0027, data)"},{"line_number":3013,"context_line":"        res \u003d network_req.get_response(self.api)"},{"line_number":3014,"context_line":"        self.assertEqual(201, res.status_int)"},{"line_number":3015,"context_line":""},{"line_number":3016,"context_line":"    def test_release_network_segments(self):"},{"line_number":3017,"context_line":"        data \u003d {\u0027network\u0027: {\u0027name\u0027: \u0027net1\u0027,"}],"source_content_type":"text/x-python","patch_set":58,"id":"7480011a_cb709f3e","line":3014,"range":{"start_line":3014,"start_character":8,"end_line":3014,"end_character":28},"updated":"2022-11-16 18:05:17.000000000","message":"I was going to say that this is reproducing again LP#1791233, but now to prevent this issue we have to define the seg_id too.","commit_id":"55bc7b2ca2d0dc2ac672ffb73ca5a428e2090a1d"}]}