)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"3bf4fd8a38955f3a70311a7b4cba42c662689485","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"957271ee_15be1b8d","updated":"2025-06-09 11:57:50.000000000","message":"The `notify` should probably not have been removed and `before` just added. So IMO this change should keep `before` and add back `notify`, perhaps Takashi har other opinions so adding him.","commit_id":"2eada288298bb1982b148144797191254f72f717"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e1a8d2bf1b48df7df71f1298c59d449855247240","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"45bcab96_97717099","updated":"2025-06-09 12:07:22.000000000","message":"To be very honest I don\u0027t remember the actual reason why I changed this line, so it\u0027s probably accidental. One thing we may have to be aware of is that restarting northd may cause full sync between ovn and services but we already have notify from the package resource that may not be a severe problem.\n\nIIUC notify implies before so replacing before just by notify makes sense to me atm, but we may want to add back before in case we add a flag to defer restart, like we did in vswitch.","commit_id":"2eada288298bb1982b148144797191254f72f717"},{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"ddecbd5f30ec36848b10f98dddcd9299b581adac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"34bf1423_d941442e","in_reply_to":"45bcab96_97717099","updated":"2025-06-09 12:20:23.000000000","message":"Ack, thanks!","commit_id":"2eada288298bb1982b148144797191254f72f717"}]}