)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"844f42d7efc4e1cf60ff283a085c1a592f22b857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1cf7275f_bef7125e","updated":"2023-08-17 08:43:03.000000000","message":"Looks ok","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b2004093876ec11402d78e8ce93c444b3faed3ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ca32026b_412c86ad","updated":"2023-05-15 13:18:39.000000000","message":"The patch was merged on master so this is the first back-port in the series.\ncherry-picked line is correct and otherwise this is a clean back-port.","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"46106a272d588cafb9cd83a5a0d5727db9e02a7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"01ae8b80_5db34f46","updated":"2023-07-04 11:18:54.000000000","message":"clean cherry pick from master branch, indeed. I\u0027m a bit hesitant though as this adds a new (default) config value, which might cause changes when a deployed environment upgrades to this new code. Considering this is \u0027high\u0027 importance bug and the fact that this affects \"only\" OVS based deployments, maybe backporting this is acceptable. I let other stable cores review \u0026 decide.","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"92ff641f1f03a1e5f2aa8eab2d31d4686cdcb6bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"04177524_bb9d3026","updated":"2023-06-28 07:14:57.000000000","message":"clean cherry-pick, lgtm","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fb3bacb0ce69979a469c69a136bc4dcfd89ae915","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad461eec_515cbc80","updated":"2023-07-06 10:16:53.000000000","message":"droping to +1 while we disucss the upraged impact of the backport","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fb3bacb0ce69979a469c69a136bc4dcfd89ae915","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7e698c2b_417f5699","in_reply_to":"01ae8b80_5db34f46","updated":"2023-07-06 10:16:53.000000000","message":"so i wanted to fix this by default on master but i think we could remove the default for the config optionon stable so its entrily opt in.\n\nwe have done that in the past if i had tought of it at the time i would have done the orginal patch as 2 patches (1 to add the fix and 1 to enable it by default)\n\nwoudl you be more comfortable with not having a default for the backport?","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"92c239e73200f8d39bbeb4ab026d9513032be18c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a9c6eab8_59a3c0ed","in_reply_to":"2744cc4a_d708f8c7","updated":"2024-07-03 21:52:22.000000000","message":"ok this is rather low on my current priority list but ill see if i can find time to update this.\n\nwe backported this as is downstream, we are very close to releasing our next version of oepsntack but are still importing upstream changes.\n\nif i update this now and we import it, it would regress our product since we wil need an installer update to enable it by default for our customers.\n\nif others want to update it before then that is but i would like to hold of for 2-3 weeks until we line up the required installer changes too.\n\nits a pretty trivial change but not one i want to try and justfigy as a code freeze excpetion so ill loopback to this closer the end of the month","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"fbed722dbf9b6cd3e1913db743669cc278340329","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2744cc4a_d708f8c7","in_reply_to":"7e698c2b_417f5699","updated":"2024-07-03 13:47:54.000000000","message":"(first of all, sorry for not coming back to this earlier) to answer your question:\nyes, if there is no *default* value (so the user has to explicitly add the config option with a value) should be OK, so that an upgrade won\u0027t change any behaviour, but **can** opt-in, if they want.","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"1167b46fcb16c5148f05b7961a31053f6963d248","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"004e99ba_ac78b643","in_reply_to":"a9c6eab8_59a3c0ed","updated":"2024-07-04 08:55:33.000000000","message":"ACK, thanks for the explanation, completely understandable.","commit_id":"7fff20c973b54aebcadd103ccae563e5d9793508"}]}