)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8f5f870acb20aa8eefbae2698a2b287cc5a32a83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e3de3222_a0388c5c","updated":"2025-07-08 15:24:27.000000000","message":"Hi Jaroslav, thanks for the fix.\n\nSince this also affects the master branch there is a slightly different workflow.\n\nYou need first create a change targeting the master branch, then once that merges we can cherry-pick to the stable branches using the review UI (there\u0027s a cherry-pick link in the upper right corner).\n\nAlso, if you could add a new unit tests so we cover this corner case it would be great, as we don\u0027t want to accidentally regress.\n\nThanks, Brian","commit_id":"f215669e121c3732009492599b79d83394ac41f0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ccfeca4c89c5ac187f7a396f61622387b7c81dde","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ee32f841_4a7b9207","in_reply_to":"c20751c8_c736b99f","updated":"2025-07-09 17:04:12.000000000","message":"Hi Jaroslav, I can add some unit tests, it should be pretty easy. Thanks for the fix. We will need to update this change once the master one merges.","commit_id":"f215669e121c3732009492599b79d83394ac41f0"},{"author":{"_account_id":16875,"name":"Jaroslav Pulchart","display_name":"Jaroslav Pulchart","email":"jaroslav.pulchart@gooddata.com","username":"jaroslav.pulchart"},"change_message_id":"317a073a105f741fb575aa8de1e3ffd273e22acc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c20751c8_c736b99f","in_reply_to":"e3de3222_a0388c5c","updated":"2025-07-09 13:51:23.000000000","message":"Hello Brian,\n\nI’ve cherry-picked the same fix into the master branch. I don’t have the bandwidth or familiarity with Neutron’s Python testing framework to write the new unit tests myself. If added tests are needed to cover this case, someone more versed in Neutron’s test suite will need to author them. By the way, this change doesn’t introduce any new functionality, it simply corrects a logic bug in an existing cleanup routine. Because it restores intended behavior rather than adding features, I’d appreciate if it could be merged as-is and supplemented with tests by the core team.\n\nBest,\nJaroslav P.","commit_id":"f215669e121c3732009492599b79d83394ac41f0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"23ef19b4ba23e6d22703adb6074b568cad35d477","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"da5f5ffc_a5297265","in_reply_to":"ee32f841_4a7b9207","updated":"2025-08-07 17:12:08.000000000","message":"Done","commit_id":"f215669e121c3732009492599b79d83394ac41f0"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"19f4620aff002c1a35a5fa3aaa4b6fd77c2ce31a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"458bb382_c1e08084","updated":"2025-08-20 20:04:28.000000000","message":"recheck POST_FAILURE","commit_id":"feea7e9f75ddd555b2af28a018111d3b6f45acb3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"4b05d709cc837c21623f2284d8ec061b76e44a06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e7d0b0b3_16cbab20","updated":"2025-08-20 17:51:37.000000000","message":"recheck unrelated functional failure","commit_id":"feea7e9f75ddd555b2af28a018111d3b6f45acb3"}]}
