)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"1a4a9f1552d3f40a68990d0b6d8cbb9eb71d5f74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a177a1ed_71d3dc69","updated":"2023-06-20 09:19:51.000000000","message":"Thanks Miguel ! are you expecting to implement everything in one release?","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a88ae17b95641dfc9e146f3acce40b25bce5a123","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8522cd05_f02958f7","updated":"2023-06-22 11:31:54.000000000","message":"thanks, looks ok","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"c097208ab8e510b29d5efd435e05bdea95d78883","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0ee3331a_be2d9559","in_reply_to":"a177a1ed_71d3dc69","updated":"2023-06-26 23:46:37.000000000","message":"Yes, one release","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"eba9bb458acdfa8b190b923a063d03c710594bfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"26d483a7_eefc1ce3","updated":"2023-06-27 10:25:24.000000000","message":"Thanks for this proposal. Now I\u0027m waiting for the big refactor ahead in the OVN L3 plugin!","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"}],"specs/2023.2/ml2ovn-router-flavors.rst":[{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"1a4a9f1552d3f40a68990d0b6d8cbb9eb71d5f74","unresolved":true,"context_lines":[{"line_number":86,"context_line":"                           |       +-------------+       |"},{"line_number":87,"context_line":"                           |       |             |       |"},{"line_number":88,"context_line":"                   Updates +------\u003e|   Neutron   |\u003c------+ Reads"},{"line_number":89,"context_line":"                                   |   databse   |"},{"line_number":90,"context_line":"                                   |             |"},{"line_number":91,"context_line":"                                   +-------------+"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"e60c24ee_3dfad152","line":89,"range":{"start_line":89,"start_character":39,"end_line":89,"end_character":46},"updated":"2023-06-20 09:19:51.000000000","message":"database","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"756f19ce39a2a794f05383cf14072f784e9826f2","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                           |       +-------------+       |"},{"line_number":87,"context_line":"                           |       |             |       |"},{"line_number":88,"context_line":"                   Updates +------\u003e|   Neutron   |\u003c------+ Reads"},{"line_number":89,"context_line":"                                   |   databse   |"},{"line_number":90,"context_line":"                                   |             |"},{"line_number":91,"context_line":"                                   +-------------+"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-rst","patch_set":9,"id":"3aa1c83f_c47866d3","line":89,"range":{"start_line":89,"start_character":39,"end_line":89,"end_character":46},"in_reply_to":"e60c24ee_3dfad152","updated":"2023-06-27 01:10:11.000000000","message":"Done","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a88ae17b95641dfc9e146f3acce40b25bce5a123","unresolved":true,"context_lines":[{"line_number":112,"context_line":" | e47c1c5c-629b-4c48-b49a-78abe6ac7696 | user-defined-router-flavor | True    | L3_ROUTER_NAT | User defined flavor for routers in the L3 OVN plugin |"},{"line_number":113,"context_line":" +--------------------------------------+----------------------------+---------+---------------+------------------------------------------------------+"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":" $ openstack router create router-of-user-defined-flavor --flavor-id e47c1c5c-629b-4c48-b49a-78abe6ac7696"},{"line_number":116,"context_line":" +-------------------------+--------------------------------------+"},{"line_number":117,"context_line":" | Field                   | Value                                |"},{"line_number":118,"context_line":" +-------------------------+--------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":9,"id":"267c484a_e9102f78","line":115,"range":{"start_line":115,"start_character":60,"end_line":115,"end_character":68},"updated":"2023-06-22 11:31:54.000000000","message":"nit: this can be just flavor, and on the cli a uuid or name can be used","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"756f19ce39a2a794f05383cf14072f784e9826f2","unresolved":false,"context_lines":[{"line_number":112,"context_line":" | e47c1c5c-629b-4c48-b49a-78abe6ac7696 | user-defined-router-flavor | True    | L3_ROUTER_NAT | User defined flavor for routers in the L3 OVN plugin |"},{"line_number":113,"context_line":" +--------------------------------------+----------------------------+---------+---------------+------------------------------------------------------+"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":" $ openstack router create router-of-user-defined-flavor --flavor-id e47c1c5c-629b-4c48-b49a-78abe6ac7696"},{"line_number":116,"context_line":" +-------------------------+--------------------------------------+"},{"line_number":117,"context_line":" | Field                   | Value                                |"},{"line_number":118,"context_line":" +-------------------------+--------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":9,"id":"fab391e5_f2674b05","line":115,"range":{"start_line":115,"start_character":60,"end_line":115,"end_character":68},"in_reply_to":"267c484a_e9102f78","updated":"2023-06-27 01:10:11.000000000","message":"Done","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"a88ae17b95641dfc9e146f3acce40b25bce5a123","unresolved":true,"context_lines":[{"line_number":138,"context_line":"As of the writing of this specification, the openstack client doesn\u0027t allow the"},{"line_number":139,"context_line":"specification of a flavor ID when creating a vrouter. Adding this functionality"},{"line_number":140,"context_line":"to the client will be part of this specification\u0027s implementation."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"The OVN flavor driver will be loaded by default when the L3 plugin starts."},{"line_number":143,"context_line":"This is an example of the steps a cloud administrator will follow to configure"},{"line_number":144,"context_line":"and load drivers for other flavors:"}],"source_content_type":"text/x-rst","patch_set":9,"id":"be858b90_c856d05f","line":141,"updated":"2023-06-22 11:31:54.000000000","message":"good catch, it is ready in SDK at leaset","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"756f19ce39a2a794f05383cf14072f784e9826f2","unresolved":false,"context_lines":[{"line_number":138,"context_line":"As of the writing of this specification, the openstack client doesn\u0027t allow the"},{"line_number":139,"context_line":"specification of a flavor ID when creating a vrouter. Adding this functionality"},{"line_number":140,"context_line":"to the client will be part of this specification\u0027s implementation."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"The OVN flavor driver will be loaded by default when the L3 plugin starts."},{"line_number":143,"context_line":"This is an example of the steps a cloud administrator will follow to configure"},{"line_number":144,"context_line":"and load drivers for other flavors:"}],"source_content_type":"text/x-rst","patch_set":9,"id":"a73b24ca_115810da","line":141,"in_reply_to":"be858b90_c856d05f","updated":"2023-06-27 01:10:11.000000000","message":"Ack","commit_id":"ae0ce90c84369c0be0b5c65a08408316cad65b29"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"dc72e387fa964350bc1eddc3f5a65ffbf125c36b","unresolved":true,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"To achieve this, all the OVN specific functionality in the ML2/OVN L3 plugin"},{"line_number":51,"context_line":"related to vrouters and floating IPs will be refactored to a separate driver."},{"line_number":52,"context_line":"Once this is done, the L3 plugin will be responsible for performing only the"},{"line_number":53,"context_line":"Neutron DB processing steps related to vrouters and floating IPs, while letting"},{"line_number":54,"context_line":"separate drivers to take care of all the backend processing. These drivers will"},{"line_number":55,"context_line":"listen and act on events notifications sent by the refactored L3 plugin for the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"99dd6704_c4f913b7","line":52,"range":{"start_line":52,"start_character":23,"end_line":52,"end_character":32},"updated":"2023-06-28 07:57:04.000000000","message":"does it means that we will have only one L3 plugin when this will be done? Or is this sentence about ovn-l3 service plugin only?","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f45d66e93b2e300a88ac66667c601b05219d7317","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"To achieve this, all the OVN specific functionality in the ML2/OVN L3 plugin"},{"line_number":51,"context_line":"related to vrouters and floating IPs will be refactored to a separate driver."},{"line_number":52,"context_line":"Once this is done, the L3 plugin will be responsible for performing only the"},{"line_number":53,"context_line":"Neutron DB processing steps related to vrouters and floating IPs, while letting"},{"line_number":54,"context_line":"separate drivers to take care of all the backend processing. These drivers will"},{"line_number":55,"context_line":"listen and act on events notifications sent by the refactored L3 plugin for the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"b30e4253_d57a9185","line":52,"range":{"start_line":52,"start_character":23,"end_line":52,"end_character":32},"in_reply_to":"99dd6704_c4f913b7","updated":"2023-07-03 07:29:00.000000000","message":"I\u0027m almost 100% sure he is referring to the OVN L3 service only, that is the scope of this RFE.","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0d47ac06b7b588802556de4d308100069262bbe7","unresolved":true,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"To achieve this, all the OVN specific functionality in the ML2/OVN L3 plugin"},{"line_number":51,"context_line":"related to vrouters and floating IPs will be refactored to a separate driver."},{"line_number":52,"context_line":"Once this is done, the L3 plugin will be responsible for performing only the"},{"line_number":53,"context_line":"Neutron DB processing steps related to vrouters and floating IPs, while letting"},{"line_number":54,"context_line":"separate drivers to take care of all the backend processing. These drivers will"},{"line_number":55,"context_line":"listen and act on events notifications sent by the refactored L3 plugin for the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"effdf686_cca7edb4","line":52,"range":{"start_line":52,"start_character":23,"end_line":52,"end_character":32},"in_reply_to":"b30e4253_d57a9185","updated":"2023-07-03 07:29:18.000000000","message":"(not resolved yet)","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"59d86598813e5253bca6f4726b801b35202e184c","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"To achieve this, all the OVN specific functionality in the ML2/OVN L3 plugin"},{"line_number":51,"context_line":"related to vrouters and floating IPs will be refactored to a separate driver."},{"line_number":52,"context_line":"Once this is done, the L3 plugin will be responsible for performing only the"},{"line_number":53,"context_line":"Neutron DB processing steps related to vrouters and floating IPs, while letting"},{"line_number":54,"context_line":"separate drivers to take care of all the backend processing. These drivers will"},{"line_number":55,"context_line":"listen and act on events notifications sent by the refactored L3 plugin for the"}],"source_content_type":"text/x-rst","patch_set":10,"id":"e6963f97_fbb2b37e","line":52,"range":{"start_line":52,"start_character":23,"end_line":52,"end_character":32},"in_reply_to":"effdf686_cca7edb4","updated":"2023-07-03 21:41:49.000000000","message":"Correct, the scope is OVN L3 only","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"eba9bb458acdfa8b190b923a063d03c710594bfa","unresolved":true,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"#. Create a service profile for the router flavor::"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"     $ openstack network flavor profile create --description \"User defined router flavor profile\" --enable --driver neutron.services.ovn_l3.service_providers.user_defined.UserDefined"},{"line_number":165,"context_line":"     +-------------+--------------------------------------------------------------------+"},{"line_number":166,"context_line":"     | Field       | Value                                                              |"},{"line_number":167,"context_line":"     +-------------+--------------------------------------------------------------------+"}],"source_content_type":"text/x-rst","patch_set":10,"id":"9d385ae2_a70302fb","line":164,"range":{"start_line":164,"start_character":116,"end_line":164,"end_character":182},"updated":"2023-06-27 10:25:24.000000000","message":"We should have stevedore entry points to make this name shorter. But this is more an implementation consideration.","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"eba9bb458acdfa8b190b923a063d03c710594bfa","unresolved":false,"context_lines":[{"line_number":194,"context_line":"REST API impact"},{"line_number":195,"context_line":"---------------"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"No REST API impact is expected."},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"DB Impact"},{"line_number":200,"context_line":"---------"}],"source_content_type":"text/x-rst","patch_set":10,"id":"fe378c25_90162b49","line":197,"range":{"start_line":197,"start_character":1,"end_line":197,"end_character":29},"updated":"2023-06-27 10:25:24.000000000","message":"It is surprising that we don\u0027t need any new API, but I think it is correct. Same for the DB schema.","commit_id":"8a0eaebf9a26b86cd63515ef304899d308f369c7"}]}