)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"12f2469cf6cd852b2024753c51dd9d59e296a427","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2e847afb_9108d40b","updated":"2021-12-06 06:57:33.000000000","message":"As a confirmation, we don\u0027t see any denial related to that location here:\nhttps://e36beaa2ff297ebe7d5f-5944c3d62ed334b8cdf50b534c246731.ssl.cf5.rackcdn.com/820174/1/check/tripleo-ci-centos-8-undercloud-containers/f831423/logs/undercloud/var/log/extra/selinux_consolidated_avc.txt\n\nGuess we\u0027re all good to merge it!","commit_id":"a035de7d3fcb927cc16ecc18ea237510e73ce31c"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"4f3eb865a761883fc1c04b84fedbf0056f81b561","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"61498e1e_956722d1","updated":"2021-12-07 06:26:05.000000000","message":"lgtm","commit_id":"a035de7d3fcb927cc16ecc18ea237510e73ce31c"}],"common/common-container-setup-tasks.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9f0d954cd07220d15f331c33cd5a10f2e7897d67","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  file:"},{"line_number":4,"context_line":"    path: /var/log/containers"},{"line_number":5,"context_line":"    state: directory"},{"line_number":6,"context_line":"    setype: container_file_t"},{"line_number":7,"context_line":"    selevel: s0"},{"line_number":8,"context_line":"    mode: 0750"},{"line_number":9,"context_line":"  tags:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"20912b6d_4afcb935","side":"PARENT","line":6,"range":{"start_line":6,"start_character":12,"end_line":6,"end_character":28},"updated":"2021-12-07 07:58:07.000000000","message":"why don\u0027t we need to explicitly make that \u0027container_log_t\u0027 then?","commit_id":"b3e8e218f8aa5d66680118fecc8630ba3df80c47"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"99401a8ddbbd49a16084793cad804666f31c9106","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  file:"},{"line_number":4,"context_line":"    path: /var/log/containers"},{"line_number":5,"context_line":"    state: directory"},{"line_number":6,"context_line":"    setype: container_file_t"},{"line_number":7,"context_line":"    selevel: s0"},{"line_number":8,"context_line":"    mode: 0750"},{"line_number":9,"context_line":"  tags:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b6636cb7_2c14a721","side":"PARENT","line":6,"range":{"start_line":6,"start_character":12,"end_line":6,"end_character":28},"in_reply_to":"20912b6d_4afcb935","updated":"2021-12-07 12:16:17.000000000","message":"that\u0027s a good question. We use container_file_t historically for all entries managed by that path, maybe its time to reconsider (in followups)?","commit_id":"b3e8e218f8aa5d66680118fecc8630ba3df80c47"}]}