)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"deb89d1c4cb1d95413dcde7e97a84c876c8b0753","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ec88e05e_38150fd0","updated":"2022-11-03 14:20:54.000000000","message":"I would preference this one over introducing puppet for it. Seems more in-line with our wider objectives to centralise on Ansible and do less with Puppet. ","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"fd3d1fa315c18ea25a77544fe3a24f44aceba77b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6c9ad0b7_8882cb30","updated":"2022-11-03 14:21:17.000000000","message":"looks like it generated the file correctly:\n\nhttps://ae72f448a0379aed3fdf-8317af439e76bee5b0b71e78f082054c.ssl.cf5.rackcdn.com/863432/2/check/tripleo-ci-centos-9-containers-multinode/53620df/logs/undercloud/var/lib/config-data/ansible-generated/heat_ephemeral/etc/heat/heat.conf\n\nwill need a tripleoclient patch to actually consume and test it","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"}],"deployment/heat/heat-ephemeral-container-ansible.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a12bddf19ba51a81c832d36f638a6f0256b2b7f9","unresolved":true,"context_lines":[{"line_number":163,"context_line":"                path: \"{{ heat_ephemeral_config_dir }}\""},{"line_number":164,"context_line":"                state: directory"},{"line_number":165,"context_line":"                recurse: true"},{"line_number":166,"context_line":"                mode: \u00270755\u0027"},{"line_number":167,"context_line":"            - name: Create heat.conf for ephemeral Heat"},{"line_number":168,"context_line":"              copy:"},{"line_number":169,"context_line":"                dest: \"{{ heat_ephemeral_config_dir }}/heat.conf\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"24832a2d_0c5a532c","line":166,"range":{"start_line":166,"start_character":23,"end_line":166,"end_character":27},"updated":"2022-11-09 10:25:04.000000000","message":"The problem is that we need to drop the a bits here to avoid leaking the password. \nWe probably have to create the file/directory with 0750 and 0640 and launch the container to chown the directory and the file.","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f4777a1aa0a7e8e8cc1a9fe220c4aec034f3c601","unresolved":true,"context_lines":[{"line_number":163,"context_line":"                path: \"{{ heat_ephemeral_config_dir }}\""},{"line_number":164,"context_line":"                state: directory"},{"line_number":165,"context_line":"                recurse: true"},{"line_number":166,"context_line":"                mode: \u00270755\u0027"},{"line_number":167,"context_line":"            - name: Create heat.conf for ephemeral Heat"},{"line_number":168,"context_line":"              copy:"},{"line_number":169,"context_line":"                dest: \"{{ heat_ephemeral_config_dir }}/heat.conf\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e86e3013_3120a75b","line":166,"range":{"start_line":166,"start_character":23,"end_line":166,"end_character":27},"in_reply_to":"24832a2d_0c5a532c","updated":"2022-11-09 15:06:30.000000000","message":"Let me check whether https://review.opendev.org/c/openstack/tripleo-heat-templates/+/864124/1 works.","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"ce35fc2cab29388374d74c231cba7f1896198336","unresolved":true,"context_lines":[{"line_number":164,"context_line":"                state: directory"},{"line_number":165,"context_line":"                recurse: true"},{"line_number":166,"context_line":"                mode: \u00270755\u0027"},{"line_number":167,"context_line":"            - name: Create heat.conf for ephemeral Heat"},{"line_number":168,"context_line":"              copy:"},{"line_number":169,"context_line":"                dest: \"{{ heat_ephemeral_config_dir }}/heat.conf\""},{"line_number":170,"context_line":"                mode: \u00270666\u0027"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2204a167_4402f29b","line":167,"updated":"2022-11-09 09:20:59.000000000","message":"probably stupid, but why not use the ini_file module here? OK, it would need 2 calls instead of one, but imho it would be cleaner... ?","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"502a38b55c9563b99d3933c4b1821e3c78efb0a5","unresolved":true,"context_lines":[{"line_number":164,"context_line":"                state: directory"},{"line_number":165,"context_line":"                recurse: true"},{"line_number":166,"context_line":"                mode: \u00270755\u0027"},{"line_number":167,"context_line":"            - name: Create heat.conf for ephemeral Heat"},{"line_number":168,"context_line":"              copy:"},{"line_number":169,"context_line":"                dest: \"{{ heat_ephemeral_config_dir }}/heat.conf\""},{"line_number":170,"context_line":"                mode: \u00270666\u0027"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d9917767_95a1c175","line":167,"in_reply_to":"2204a167_4402f29b","updated":"2022-11-15 20:10:01.000000000","message":"because copy with content does exactly what\u0027s needed?\nthere\u0027s always multiple ways to do things. cleaner to me is what I did, but I don\u0027t care either way.","commit_id":"108a2456f8fa0c5cf5cc716050e8029c089feda9"}]}