)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"009ab703c03b44f968b21c95df0081c5e388cc49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"843761f7_2654fff2","updated":"2023-02-03 05:06:07.000000000","message":"it’s only an opendev problem, in real life you don’t have apparmor enabled without having it installed, do you?","commit_id":"45df2881335c489e6a82f5528be1f124cca11f16"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ea860de0a6ad227c04f2c374e141075b29ebea27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4899184f_c041c0c9","updated":"2023-02-03 05:34:37.000000000","message":"ok, tested, actually possible - but still we need to fix kolla_docker in kolla-ansible and CI in kolla (because it doesn’t use that role).","commit_id":"45df2881335c489e6a82f5528be1f124cca11f16"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"c4c6f07eb5564c9c958e426f93e5d183096daf61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eaef3ced_1d044879","updated":"2023-02-03 08:02:17.000000000","message":"Another thought it that we might want to install apparmor only on systems with apparmor enabled? Probably docker does not try to use apparmor_parser when apparmor is disabled.","commit_id":"249aff8aacffadb143cd513e757e573bf3f70ef1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"2276d1e7f166f70670092adbe3208b55043ddd40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e2ebc4fe_2d23a134","updated":"2023-02-03 08:25:37.000000000","message":"Ok, docker-ce seems to check only if apparmor is supported, if it is - it\u0027s trying to check the status, and since it can\u0027t - well, we have to install it.\nThere\u0027s some approach here to make it better, but let\u0027s install apparmor for now: https://github.com/moby/moby/pull/44902","commit_id":"249aff8aacffadb143cd513e757e573bf3f70ef1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6031e41b03f8933573f6a7bef6a290d535718402","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4e5e97ea_2c572470","updated":"2023-02-03 14:59:35.000000000","message":"recheck (should pass now)","commit_id":"249aff8aacffadb143cd513e757e573bf3f70ef1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"ad1d30b8aa5c85e84b877a794282fd4268f74803","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f7b85049_8dcfd774","updated":"2023-02-04 16:37:53.000000000","message":"recheck (rocky 9 failed)","commit_id":"b3673b38b494fbacdbdf2ecc940545aefd957f5e"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"7907020a8f6f92ca6515d5b821aeef2750bc2f23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d10a3b2e_08c404f7","updated":"2023-02-07 08:16:08.000000000","message":"It looks this can\u0027t merge because of rocky linux kayobe issue ? :( ","commit_id":"706f7c3ae00361a949781352c9e5b4507e1444a4"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"fdf2dbd4cc1431f12596dfe21186eaf072a3dd09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"685c3c62_f44e4e25","in_reply_to":"d10a3b2e_08c404f7","updated":"2023-02-07 08:29:51.000000000","message":"Yes, removing those jobs for now in a different patch.","commit_id":"706f7c3ae00361a949781352c9e5b4507e1444a4"}],"roles/packages/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"efcd986b09d2d344ca3dd90abbf853fcf6217238","unresolved":true,"context_lines":[{"line_number":12,"context_line":" - iputils-ping"},{"line_number":13,"context_line":" - \"{% if enable_multipathd|bool %}sg3-utils-udev{% endif %}\""},{"line_number":14,"context_line":" - tzdata"},{"line_number":15,"context_line":" - apparmor"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"openeuler_pkg_install:"},{"line_number":18,"context_line":" - python3-docker"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c7c57d10_617e0ca1","line":15,"updated":"2023-02-03 07:50:12.000000000","message":"Shouldn\u0027t we rather do it in docker role?","commit_id":"45df2881335c489e6a82f5528be1f124cca11f16"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9605cb14bb277ce773970cf7283ed95acbe789da","unresolved":false,"context_lines":[{"line_number":12,"context_line":" - iputils-ping"},{"line_number":13,"context_line":" - \"{% if enable_multipathd|bool %}sg3-utils-udev{% endif %}\""},{"line_number":14,"context_line":" - tzdata"},{"line_number":15,"context_line":" - apparmor"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"openeuler_pkg_install:"},{"line_number":18,"context_line":" - python3-docker"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0bd9b54b_2f5f40cb","line":15,"in_reply_to":"c7c57d10_617e0ca1","updated":"2023-02-03 20:07:27.000000000","message":"Done","commit_id":"45df2881335c489e6a82f5528be1f124cca11f16"}]}