)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"e4a00b1cfba3335fff6d31b9a775748502901854","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1c4ee239_3ed9ba5f","updated":"2025-01-09 14:32:37.000000000","message":"Apparently the fix without revert is https://review.opendev.org/c/openstack/neutron/+/938800","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f2d172ee76f923270a7e90ef21532189dc86327a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f0c19a32_4a087ca8","updated":"2025-01-09 08:29:11.000000000","message":"As per ML thread - lets do the revert for now and later implement it in better way","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"58e1876b220fbde6b2e07fc6b43c1b905f0c671d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7c01d28a_e7e0ba95","updated":"2025-01-08 15:27:17.000000000","message":"Can\u0027t it be just yanked out of the python module to satisfy the runtime linkage requirement?","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"3356cd4de7b5e7b4c28534f114585e0681db77f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"82e556bb_5137f7c2","updated":"2025-01-08 15:45:59.000000000","message":"OK it pains me but that\u0027s the right thing to do.\n\nThe author should be advised to provide an alternative that is not in violation of governance and GPL (AFAIU either instructing deployer to copy file themselves from elsewhere; or re-implement the script in python?) and post again. I\u0027d advise reviewers then to speed up the merge since the feature patch took quite a long and frustrating path to get merged already.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b95b9fbd5b3e45696b469389eae823e702f0f60b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d44a325e_6ca215ce","updated":"2025-01-09 14:50:16.000000000","message":"Waiting for https://review.opendev.org/c/openstack/neutron/+/938800","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3c2c3460ac9a3f5df3ea3994a776055674dfddc3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7aa259de_03f5689e","updated":"2025-01-09 08:35:23.000000000","message":"recheck\ntempest integrated failure is not related (test_create_backup )","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"851dd6fa6fd6565a921cf5953fb92dfb4bec06a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"889d5929_19e07f35","updated":"2025-01-09 11:18:44.000000000","message":"recheck tempest-integrated-networking","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"278c0293594c94008f5034b6c5add9b1d539a02c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"353984ab_ca7b2bad","in_reply_to":"319debe3_725bd3b7","updated":"2025-01-08 20:26:39.000000000","message":"Thanks for letting us know! Please post and we\u0027ll try to expedite the merge. The worst case, we\u0027ll unrevert the revert. :) Thanks for working on a fix! I am sure your upcoming update will get a priority now.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"9b52f341b75465b122806563b1979a7ad8072301","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d8ca4874_2e9244fd","in_reply_to":"353984ab_ca7b2bad","updated":"2025-01-08 20:33:33.000000000","message":"UPD: I removed +W from the patch in anticipation of your other patch. I hope we don\u0027t need to go through revert then after all.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6cde413fcd6578984deb6181def1692d60393c12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d70fed3b_94220fb6","in_reply_to":"7c01d28a_e7e0ba95","updated":"2025-01-08 15:40:07.000000000","message":"I have this: https://review.opendev.org/c/openstack/neutron/+/938669\n\nIf that is enough, perfect. But we need a TC resolution and seems that this is not enough. Slawek proposed to request the user to manually add the script. I suggest to reimplement it in Python, create a new script and use it instead of the current code.\n\nIn any case, right now we need to revert this patch.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":11290,"name":"Gaudenz Steinlin","email":"gaudenz.steinlin@cloudscale.ch","username":"gaudenz"},"change_message_id":"6dadeabbe6b923e3ff8f2b67a2f638bd3b392af7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"319debe3_725bd3b7","in_reply_to":"82e556bb_5137f7c2","updated":"2025-01-08 20:08:34.000000000","message":"I was caught a bit by surprise that the change got merged over the christmas holidays. This problem was pointed out only very recently and I was working on replacing the script with a solution which uses the existing keepalived_state_change.py script. I would like to submit this change again with this fix. I think I could have that ready tonight or tomorrow, so if that\u0027s OK, it\u0027s not needed to revert this.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"851dd6fa6fd6565a921cf5953fb92dfb4bec06a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7f2914c1_6e94ce54","in_reply_to":"d8ca4874_2e9244fd","updated":"2025-01-09 11:18:44.000000000","message":"As Slawek commented, we\u0027ll proceed with the revert and wait for the TC resolution. The \"cleanest\" option could be to refactor the entire script, for example into Python code, but that requires time and testing.","commit_id":"ba9eaf6d21ffa80d2e2efbc0baa2acc4b27649a4"}]}