)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"29fc390734b61ad3d0efb70848044d5fc555100e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2f7dd049_e28a97b8","updated":"2022-08-26 19:15:49.000000000","message":"That is brilliant way to make this work. Thank you! Could you please rebase this on current master instead and also remove the HC parameter from condition? Otherwise +1.","commit_id":"7b261c41b03295097163c814a5437b3ca8944c42"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"988878749f68a32c6fb4716e5e7f1401baa2fc5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"9c8cb15c_db5ea643","updated":"2022-08-27 03:22:40.000000000","message":"just a side note:\n\nThe task to set ACL is not present in collectd-container-ansible.yaml. Is this expected ? I guess we need the same for ansible implementation.","commit_id":"614634f76854a10e2823e1cd5f5e4c52fd4ba166"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"a5933ffce9094c05e13bc32660bab236101a03c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"e39db59b_eb40c634","updated":"2022-08-29 10:01:42.000000000","message":"Thanks for this patch Takashi!","commit_id":"8e8efcfb3a1770cb7b0e5ab2b8bb8fb6d441b349"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"7864332ecd262001a9ac44dd596aff5a08f54f61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"c329c7d8_b07afacb","updated":"2022-08-29 09:01:54.000000000","message":"oh, nice indeed!","commit_id":"8e8efcfb3a1770cb7b0e5ab2b8bb8fb6d441b349"}],"deployment/metrics/collectd-container-puppet.yaml":[{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"29fc390734b61ad3d0efb70848044d5fc555100e","unresolved":true,"context_lines":[{"line_number":312,"context_line":"    default:"},{"line_number":313,"context_line":"      tag: collectd"},{"line_number":314,"context_line":"      file: /var/log/containers/collectd/collectd.log"},{"line_number":315,"context_line":"  ContainerHealthcheckDisabled:"},{"line_number":316,"context_line":"    type: boolean"},{"line_number":317,"context_line":"    description: Whether or not we disable the container healthcheck."},{"line_number":318,"context_line":"    default: false"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"b0283596_9ad94f34","line":315,"updated":"2022-08-26 19:15:49.000000000","message":"Sensubility needs the permission to podman socket even in the case when container HC are disabled. Otherwise it is useless.","commit_id":"7b261c41b03295097163c814a5437b3ca8944c42"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6ba2e880a5a1ae36f91c6e1db3c2e02815d5cf4f","unresolved":false,"context_lines":[{"line_number":312,"context_line":"    default:"},{"line_number":313,"context_line":"      tag: collectd"},{"line_number":314,"context_line":"      file: /var/log/containers/collectd/collectd.log"},{"line_number":315,"context_line":"  ContainerHealthcheckDisabled:"},{"line_number":316,"context_line":"    type: boolean"},{"line_number":317,"context_line":"    description: Whether or not we disable the container healthcheck."},{"line_number":318,"context_line":"    default: false"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6e01a6e5_e7442abe","line":315,"in_reply_to":"b0283596_9ad94f34","updated":"2022-08-27 03:21:00.000000000","message":"Done. Seems I misread the condition somehow.","commit_id":"7b261c41b03295097163c814a5437b3ca8944c42"}]}