)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"788a740d33b5d19793c4f6d7e999495b1917047a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f4ed62ca_bfb923f6","updated":"2023-01-16 16:55:30.000000000","message":"There are some pending patches that will be merged in the next hours. I\u0027ll update this patch tomorrow.","commit_id":"c89a6d4413b7e35f714807c336ed17938c731426"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"02081fdd95598e332c575b1f6a5585efa43d5088","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c99bdf19_87caa71d","updated":"2023-01-19 18:54:35.000000000","message":"@ralonsoh, if by Miguel you refer to me, please see comments above. I wrote my opinion yesterday","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"8ca4b0d40557a5b6d0d8e257bf3887ba13021609","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"01ac9735_f8cba799","updated":"2023-01-18 08:26:43.000000000","message":"Backport queue looks good now for xena","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"84aa0eb1893285f3806f8566a1040bb04fe46aee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7281f27c_289ec852","updated":"2023-01-26 11:03:14.000000000","message":"Hi Elod, can you check this patch?","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"caac0f1c367669ec76a64a96afcdadd1e4de211e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7f75c78a_d6ccf65d","updated":"2023-01-20 07:37:27.000000000","message":"If release team fine with it let\u0027s do with the release as it is, but see my comment above, and we have to be more careful with backports","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"da4bdab4711755f656ac4fd44b27da2270f255c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"434a78a4_9d1e9a18","updated":"2023-01-19 00:00:23.000000000","message":"It seems to me that https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30 should be removed. I\u0027m ok with the rest","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"bd1817a76841da624843f544f4a30a67bd4906bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4ee5e9e3_d3572af2","updated":"2023-01-19 06:58:26.000000000","message":"Just my 2 cents regarding https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30. It is ok with me leaving it in or out. I would not consider that an actual feature, as it is basically just adding information on the external_ids, so not changing the behavior of anything (it is just there so that ovn-bgp-agent can get access directly to that information). That said I also agree this is something new, that is not addressing any existing bug. So, it is ok if you want to remove it from the list, but should not do any harm or modify any behavior from the use point of view if it is finally included.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"a79b88be5291c4c9c52167d643df9ce50956806d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d820da46_1a109454","updated":"2023-01-20 07:10:11.000000000","message":"Looks like there are no backward incompatible changes or anything that could break existing deployments after upgrade so I\u0027m ok with this release.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"bb9d2fbd84a26a5b0095b25aaa1a5140feda8b4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6d8297a0_dd0a4e6d","updated":"2023-02-01 15:11:53.000000000","message":"May we update the hash to include https://review.opendev.org/c/openstack/neutron/+/871637 to support recent ovn versions?.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f41545f51cc2cc4d1a9b7dfdd1ed2ed4f0895874","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cb637b8b_a61d38d5","updated":"2023-01-19 16:57:47.000000000","message":"Ping Akihiro, Miguel and Oleg. Do you mind sharing your opinion on this? Thanks in advance.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"37b12229ca199ed07cabfb11224159a9d099e2e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a031ab2f_bdebcb1e","updated":"2023-01-18 09:08:25.000000000","message":"There are a couple of comment I want to add to this release:\n* MINOR version bump, looks correct, as there are many changes that have bigger impact than a usual bugfix (and MAJOR version cannot be bumped):\n  * some bugfix have \u0027new feature\u0027 release notes [1]. in general, new features are not allowed to be backported to stable branches according to stable policy (i see that not every such change is really a \u0027new feature\u0027, but still...)\n  * i see default config change [2], is the team full aware of that? (again, default config change in general is not allowed on stable branches)\n  * there is a requirement\u0027s lower constraints bump, though it\u0027s a PATCH version bump so it could be OK\n\nIf the team want to release this as is, then I\u0027m also OK with it, but please do a double check based on my comments above. Giving now -1 to highlight my question.\n\n[1] https://zuul.opendev.org/t/openstack/build/d16c9341a2d044f2b0134d32f8cde41e/log/tox/list-changes/list-changes-results.log#2050-2085\n[2] https://zuul.opendev.org/t/openstack/build/d16c9341a2d044f2b0134d32f8cde41e/log/tox/list-changes/list-changes-results.log#2091-2097\n[3] https://zuul.opendev.org/t/openstack/build/d16c9341a2d044f2b0134d32f8cde41e/log/tox/list-changes/list-changes-results.log#281-282","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"63abae6d9104096478068b582e2202f931cfe792","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"568a8ab7_ca6f5928","updated":"2023-02-01 15:14:23.000000000","message":"We\u0027re investigating a possible issue in RDO where ovsdbapp is \"try again\" looping when upgrading from ovsdbapp 1.12.2 -\u003e 1.12.3, just putting it out there since there might be a commit in this large release that fixes support with ovsdbapp 1.12.3\n\nCan see logs here https://review.rdoproject.org/r/c/rdoinfo/+/46697 please note that newer patchset reverts to 1.12.2 thought and testing if that works.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2c57eb96cbd087ca8f4bd54c313607372d29b967","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"807a878c_32026034","in_reply_to":"00987e2f_577f237f","updated":"2023-01-19 10:45:23.000000000","message":"Thank you folks for replying. About [1], I would keep it because:\n1) This is actually fixing a problem with the ovn agent, populating some fields that we were not writing before. This is, IMO, not a new feature despite the RFE tag and the \"feature\" description.\n2) The change is affecting the OVN database and this patch is also providing a maintenance method to health any existing OVN NB DB. So IMO this is safe to be backported. When I\u0027m saying this is affecting the OVN database, this is not changing the structure but the data.\n3) We don\u0027t change APIs, DB definitions nor config options.\n\nSo, IMO, we can keep this backport and continue with this release. In any case, and I\u0027m the first who needs to be more careful on this, we should be more careful when backporting patches.\n\n[1]https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"4d825ba14364967cb8df157da7909ed7a67e63a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ff8b57bf_b01dae04","in_reply_to":"5357d40b_dc70a58a","updated":"2023-02-01 17:50:31.000000000","message":"There was https://review.opendev.org/c/openstack/ovsdbapp/+/862577 and the neutron patch that fixed wait_for_change that needed to be fixed. https://review.opendev.org/c/openstack/neutron/+/857774. IIRC, the neutron patch was requiredalso, more recently https://review.opendev.org/c/openstack/neutron/+/871990 in the same area, but not this exact issue).","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"43816818c198204bee40e8327efd45643eda392d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5357d40b_dc70a58a","in_reply_to":"568a8ab7_ca6f5928","updated":"2023-02-01 15:44:26.000000000","message":"I think this is addressed in https://review.opendev.org/c/openstack/ovsdbapp/+/862577. In any case, I\u0027ll ping Terry today.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"43816818c198204bee40e8327efd45643eda392d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ca0272e7_c77917e7","in_reply_to":"6d8297a0_dd0a4e6d","updated":"2023-02-01 15:44:26.000000000","message":"I\u0027ll update the hash but we need to freeze at some point.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"3a961527064537d05ff73febd689a0ab371a76df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c813f9a8_94547622","in_reply_to":"7281f27c_289ec852","updated":"2023-01-26 15:27:57.000000000","message":"Sorry, I\u0027m lost in the comments. Did the team agree?","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"27507bbc7f087327be727970a57e30900a8917e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ded27db4_bcac3d2b","in_reply_to":"73be1195_527a82c7","updated":"2023-01-18 21:42:55.000000000","message":"Thanks Rodolfo for detailed reply, I will just add for default config option change https://review.opendev.org/q/I8bc4f2784ccefe6078506bc27cc7a98c48192ad2, its is indeed OK: for users that changed the value the code will behave as before, for users using the default value they get a performance fix that is purely in neutron internals (nothing to change in deployment to match the change, no new port to open etc).\nAlso adding to reviewers ovn-bgp-agent for https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f741245de7e384d87c5cf561827eb323e91173bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"00987e2f_577f237f","in_reply_to":"8cb266dc_153ffe07","updated":"2023-01-19 08:32:18.000000000","message":"+1, I only see https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30 as a real feature (even the bug referenced is RFE), so let\u0027s revert that backport","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"2fa38af361bbe895fdcf12309f4a69530fdc6dcb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"73be1195_527a82c7","in_reply_to":"a031ab2f_bdebcb1e","updated":"2023-01-18 15:38:51.000000000","message":"Apart from my comments there, I\u0027ll ping other Neutron cores to review your questions. Thanks!\n\nNew features:\n* https://review.opendev.org/q/Ieb4cab11068747bd79868c6eb1778aab91447c30 is indeed a new feature. It was introduced to make the ovn-bgp-agent work and be able to filter by address scope. That is indeed a new feature and we should consider if we need to remove it.\n* https://review.opendev.org/q/I50091c84e67c172c94ce9140f23235421599185c is a bug fix, an improvement for live migration in ML2/OVN. This could be between the \"features\" a \"fixes\" sections. This is a legit backport.\n* https://review.opendev.org/q/I0de5fbb70eb852f82bd311616557985d1ce89bbf is a new tool to fix an existing problem is stable versions too. It introduces a config parameter that is only used by this script.\n* https://review.opendev.org/q/Idfe5e51007b9a3eaa48779cd01edbca2f586eee5 is a fix too, despite we are adding a new config paramter. But the default value (True) of this parameter has no effect in the current code. If an operator doesnt\u0027 change the configuration, the behaviour will be the same.\n\nUpgrades:\n* https://review.opendev.org/q/I8bc4f2784ccefe6078506bc27cc7a98c48192ad2 it is indeed changing the default value of a config option. But in this case this is fixing an existing issue. If the deployment is using the default value set in the code, with this upgrade the number of OVN DB connections will be reduced, improving the OVN performance and reducing the OVN DB load.\n\nLibrary bump:\n* https://review.opendev.org/q/Ib597b62017b56b41009dd4d7359e169f424272b0: the new ovsdbapp version, previously released, is needed for this patch. It is needed due to a new class implemented. This patch is fixing the OVN NB QoS registers, adding missing fields.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"43816818c198204bee40e8327efd45643eda392d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b90b9d01_4c133a5b","in_reply_to":"c813f9a8_94547622","updated":"2023-02-01 15:44:26.000000000","message":"Yes, the team agreed with the current code. In any case, I\u0027m going to update the hash to include new patches.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a25a836c3ad16d03e20af1272717beac030c4b1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8cb266dc_153ffe07","in_reply_to":"ded27db4_bcac3d2b","updated":"2023-01-19 08:09:50.000000000","message":"I agree with Rodolfo and Bernard there. Those \"new features\" are in fact changes to the internal implementation of Neutron things. Well, except script to remove duplicate port bindings but that just tool for operators and will not impact anyone for sure.\nRegarding change in default config option, it\u0027s like Bernard said - for people who were using default value it will simply improve performance. For others, it won\u0027t change anything.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b0d6ba0956c0839bc9b2ea09a8fe25c32e9cb17a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"37645ba8_b4ed9fed","in_reply_to":"ff8b57bf_b01dae04","updated":"2023-02-02 12:07:16.000000000","message":"Thanks for your feedback! We are safe now in Neutron with the current version of ovsdbapp.","commit_id":"e7ef4843ab981610ecd59d223ba43df63f0d79d9"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9284976fc64bacecbcd9535dbe15fde7d6bc9e8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"da6a4cc1_4c2eea98","updated":"2023-02-02 12:35:56.000000000","message":"As neutron team agreed to not to revert the mentioned patches I accept the team\u0027s decision.","commit_id":"797158d0a6958b8d8b036ef5564836e45908f175"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"07a9cea57eb320d87a4bf6d9421d5aae3c509f0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"66131290_b5f29fa7","updated":"2023-02-02 14:27:01.000000000","message":"Looks good with new commits (fixes, CI, optimisation, ovn versions compatibility)\n\n% git shortlog  80b37e7c3fb7466f70988a2529e6b328a0abb815..fc62d1ea8ef5024cd1b851dab2f765ba425279c3\nArnau Verdaguer (1):\n      [Trunk] Update the trunk status with the parent status\n\nEdward Hope-Morley (1):\n      Partially revert \"Do not link up HA router gateway in backup node\"\n\nMiro Tomaska (1):\n      Improve agent provision performance for large networks\n\nRodolfo Alonso Hernandez (1):\n      Since OVN 20.06, config is stored in \"Chassis.other_config\"\n\nSlawek Kaplonski (2):\n      Increase fullstack job\u0027s timeout\n      Use common wait_until_ha_router_has_state method everywhere\n\nTerry Wilson (1):\n      Never raise an exception in notify()\n","commit_id":"797158d0a6958b8d8b036ef5564836e45908f175"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e32c39328c09dd6c1f97c2b760dfd8cdc020b3a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7d8d98f4_63893fb6","updated":"2023-02-02 12:06:28.000000000","message":"Ping folks, please check this patch. This new hash has 4 more commits, including the one providing compatibility for new and old OVN versions.","commit_id":"797158d0a6958b8d8b036ef5564836e45908f175"}]}