)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"fba329215665ba3387d5e183ad19ba906ac0dad0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a6f3a6b3_fcbc29e7","updated":"2023-07-12 16:36:58.000000000","message":"i know we kept backwards compatiblity, but i think we should update our roles to use new variables before we forget to do this:\n\netcd uses `systemd_Restart` and `systemd_RestartSec`:\nhttps://github.com/noonedeadpunk/ansible-role-etcd/blob/master/tasks/etcd_post_install_gateway.yml#L20-L21\n\nMost of the roles use: `systemd_BlockIOAccounting`, `systemd_MemoryAccounting`, `systemd_TasksAccounting`\nexample: https://opendev.org/openstack/openstack-ansible-os_placement/src/branch/master/tasks/placement_install.yml#L77-L80","commit_id":"b763b73ffc49204f88213366ca38e2bbd4be2d25"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"cf17888bed5c61520e8095454bdef1d942c11893","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b3d96bcd_42d66afd","in_reply_to":"a6f3a6b3_fcbc29e7","updated":"2023-07-13 11:40:00.000000000","message":"Sure, but patching these roles would depend on this change. So they all will have depends-on, like uwsgi one does.\nAnd all roles should be patches for linters anyway - so will get new vars applied soon - I just haven\u0027t started patching them yet.","commit_id":"b763b73ffc49204f88213366ca38e2bbd4be2d25"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"a80f921e662dd94412bdcf25161ba15b6594c417","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"deb7877d_df781601","in_reply_to":"b3d96bcd_42d66afd","updated":"2023-07-13 12:23:03.000000000","message":"ack, i just wanted to make sure you are aware","commit_id":"b763b73ffc49204f88213366ca38e2bbd4be2d25"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"ea04538e334c72209531610bbd8fe44d37a34c15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9551ded0_9ef1c06c","in_reply_to":"deb7877d_df781601","updated":"2023-07-13 13:43:52.000000000","message":"Yeah, I did that change because these vars were highlighted by linter in uwsgi role...","commit_id":"b763b73ffc49204f88213366ca38e2bbd4be2d25"}]}
