)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"594cac9cd258cf0d9eb7863c816b677342a58ec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fea6d19a_a7913ac8","updated":"2022-08-29 02:43:36.000000000","message":"-1 because I believe https://review.opendev.org/c/openstack/tripleo-heat-templates/+/854814 is more stable and simpler approach.","commit_id":"a7c1c7d6285506027e5544d6822100bc60ea205e"}],"deployment/metrics/collectd-container-puppet.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3fa4b4f61ab05ca618d424c86d0872fe00baa828","unresolved":true,"context_lines":[{"line_number":641,"context_line":"            - enable_sensubility"},{"line_number":642,"context_line":"            - step|int \u003d\u003d 4"},{"line_number":643,"context_line":"          block:"},{"line_number":644,"context_line":"            - name: validate collectd container state"},{"line_number":645,"context_line":"              containers.podman.podman_container_info:"},{"line_number":646,"context_line":"                name: collectd"},{"line_number":647,"context_line":"              register: collectd_infos"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"67658d4b_dda19043","line":644,"range":{"start_line":644,"start_character":14,"end_line":644,"end_character":53},"updated":"2022-08-26 16:06:44.000000000","message":"just wondering why we need to modify the permission AFTER starting collectd ?\n\nIf the /run/podman is already present before collectd container is started, we can launch a temporal container before starting collectd to fix the permission.\n\nhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/854814","commit_id":"184e07fcdadcfe3cd3729f97de6042cece0c82d8"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"4686824a08c5fa39cba21d70f5808e57b6b6a3b5","unresolved":true,"context_lines":[{"line_number":641,"context_line":"            - enable_sensubility"},{"line_number":642,"context_line":"            - step|int \u003d\u003d 4"},{"line_number":643,"context_line":"          block:"},{"line_number":644,"context_line":"            - name: validate collectd container state"},{"line_number":645,"context_line":"              containers.podman.podman_container_info:"},{"line_number":646,"context_line":"                name: collectd"},{"line_number":647,"context_line":"              register: collectd_infos"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"973baf50_af2af476","line":644,"range":{"start_line":644,"start_character":14,"end_line":644,"end_character":53},"in_reply_to":"67658d4b_dda19043","updated":"2022-08-26 16:09:19.000000000","message":"But we don\u0027t know the collectd userid. That is a user from within the container, so we need to setfacl from within the container. Or am I wrong? Is there another way?","commit_id":"184e07fcdadcfe3cd3729f97de6042cece0c82d8"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"39ba3878d7d697a66766c0a6e588110ada40fe7c","unresolved":true,"context_lines":[{"line_number":641,"context_line":"            - enable_sensubility"},{"line_number":642,"context_line":"            - step|int \u003d\u003d 4"},{"line_number":643,"context_line":"          block:"},{"line_number":644,"context_line":"            - name: validate collectd container state"},{"line_number":645,"context_line":"              containers.podman.podman_container_info:"},{"line_number":646,"context_line":"                name: collectd"},{"line_number":647,"context_line":"              register: collectd_infos"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4c8411f2_8dc57c0d","line":644,"range":{"start_line":644,"start_character":14,"end_line":644,"end_character":53},"in_reply_to":"973baf50_af2af476","updated":"2022-08-26 16:23:46.000000000","message":"I\u0027m thinking of this approach .\n\nhttps://review.opendev.org/c/openstack/tripleo-heat-templates/+/854814/3/deployment/metrics/collectd-container-puppet.yaml","commit_id":"184e07fcdadcfe3cd3729f97de6042cece0c82d8"}]}