)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"ec12b6b471f6b7340220f5e57f11907fee55994b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2f6a795d_65f9dba8","updated":"2025-10-21 13:43:26.000000000","message":"I\u0027m sure we fixed that bug already by not sending the value if it didn\u0027t change.","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"b17b73bcef9aaca4288587ae397e7af3aec273bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7a713b3d_9bb5cbc4","in_reply_to":"2f6a795d_65f9dba8","updated":"2025-10-21 13:51:14.000000000","message":"@openstack@dopieralski.pl I think the bug only fixed the update the port part, but the create port issue still exist.","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"428f26b7b4c11f17231604cb4af4c32c072b08a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a44bb4b6_ecee80cc","in_reply_to":"3e135177_7d959809","updated":"2025-10-21 14:42:48.000000000","message":"For create port part, cannot check if the value changed or not. Another solution I am think if the enable_port_security is checked, don’t send the port_security_enabled attribute to neutron, as the default value is true. What do you think?","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"64909f43e666e586a0edc02cb477f61980aaa095","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3e135177_7d959809","in_reply_to":"7a713b3d_9bb5cbc4","updated":"2025-10-21 14:19:49.000000000","message":"In that case, would it be better to use the same solution in all cases?","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"778448c9c6b4bc820cd0e5c7600a5b0963a36c4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e18190f4_7013ee07","in_reply_to":"a44bb4b6_ecee80cc","updated":"2025-10-29 14:53:43.000000000","message":"Sorry for the delay. And sorry for a silly suggestion, I didn\u0027t really think about it very well.\n\nIdeally we would like to gray out the port checkbox, and add a tooltip on it explaining why it is grayed out. But we want to do this in a more generic way, for many other fields as well, so this will be more work.\n\nI think that for now, as a quick fix, simply not sending this value when it is not allowed is acceptable.","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"82da5621044914a3b954c5a236e58793270a0d1f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31f53d76_c3fee437","in_reply_to":"e18190f4_7013ee07","updated":"2025-10-31 05:02:41.000000000","message":"updated the patch that not sending this value when it is not allowed and the checkbox is true, please help review, thanks","commit_id":"6941955db9b90ebc27f44905359c7d20d98a50f0"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"a402dccbf190009039a3a60af5cc1ee58f4db869","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"39c550fd_04c386c5","updated":"2025-11-20 15:25:09.000000000","message":"Please don\u0027t +1 patches that are already merged. It adds nothing but confusion.","commit_id":"4b933df52ae64314b26c2d8028203b70091736c9"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"48f0e0df17466bcf77b5b2c2794d951f32cb6d0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"57c02778_dc364165","updated":"2025-11-03 10:05:20.000000000","message":"Thank you for the fix!","commit_id":"4b933df52ae64314b26c2d8028203b70091736c9"},{"author":{"_account_id":9725,"name":"Dong Ma","email":"winterma.dong@gmail.com","username":"larainema"},"change_message_id":"83745bc5e558ce943c12bd376c2d1f02479e46c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5cac5272_79211bc4","in_reply_to":"57c02778_dc364165","updated":"2025-11-07 16:32:46.000000000","message":"Thanks @openstack@dopieralski.pl, if we can merge this patch?","commit_id":"4b933df52ae64314b26c2d8028203b70091736c9"}]}