)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"cbd50f6e8ca40fec8bdaba4f80386699f0332b75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff8ba043_0d8d23fd","updated":"2026-04-14 06:14:54.000000000","message":"I would need some more justification for this patch.\n\nCommit message does not explain why disabling PrivateDevices is not applicable for EL10, as memcached unit file still contains directive:\n```\n# cat /usr/lib/systemd/system/memcached.service | grep PrivateDevices\nPrivateDevices\u003dtrue\n```\n\nWe need to ensure the upgrade path and that override is not present anymore on the upgrade.\n\nSo some extra task to remove the override might be needed here.\n\nAs we\u0027re changing the behavior and removing a variable, a release not should be issued.","commit_id":"cce94393a5a0f1f4c45d1ad77f294e83fc772cfd"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"0d90b3db4c673eb248265d4e65da51bc9c2f1b3e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"775f80e9_99e0b9c9","in_reply_to":"42a19cdb_d4435665","updated":"2026-04-20 13:30:46.000000000","message":"Hello, I additional checking few moments and update this MR.","commit_id":"cce94393a5a0f1f4c45d1ad77f294e83fc772cfd"},{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"821346cec7e13a78d73ff1f3b2dfa39a20c265f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8f6706dc_14e63980","in_reply_to":"775f80e9_99e0b9c9","updated":"2026-06-05 22:51:02.000000000","message":"Well, complete finished in:\nhttps://review.opendev.org/c/openstack/openstack-ansible-memcached_server/+/991951","commit_id":"cce94393a5a0f1f4c45d1ad77f294e83fc772cfd"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"91e861838c1dd07006eb0c0d346b0ce3e2467d1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"42a19cdb_d4435665","in_reply_to":"ff8ba043_0d8d23fd","updated":"2026-04-17 08:23:34.000000000","message":"Just to elaborate on this - I am not arguing this is the wrong thing to do in general. As I think ubuntu does the same.\n\nBut we need to have a reasoning in commit message for removing a variable and changing behavior, as well as the release note.","commit_id":"cce94393a5a0f1f4c45d1ad77f294e83fc772cfd"}]}
