)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"92953781bd74918554265f4bcdf91283f693f96c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"246ade18_d48e078d","updated":"2026-03-11 18:01:03.000000000","message":"https://review.opendev.org/c/openstack/kolla-ansible/+/980007 reworks it to make it respect the yamllint line-length rule.","commit_id":"b396fc7a8b02eea8ec1f82f7ee8dccdc95e9d360"}],"roles/cephadm/tasks/main.yml":[{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"3e3904e73a05390178ef28dd73665506fcac4724","unresolved":true,"context_lines":[{"line_number":88,"context_line":"    ceph_conf_fixed: |"},{"line_number":89,"context_line":"      [global]"},{"line_number":90,"context_line":"      fsid \u003d {{ ceph_fsid }}"},{"line_number":91,"context_line":"      mon_host \u003d {% for host in groups[\u0027all\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + api_interface_name | replace(\u0027-\u0027, \u0027_\u0027)].ipv4.address }}{% if not loop.last %},{% endif %}{% endfor %}"},{"line_number":92,"context_line":"  copy:"},{"line_number":93,"context_line":"    content: \"{{ ceph_conf_fixed }}\""},{"line_number":94,"context_line":"    dest: \"/etc/ceph/ceph.conf.fixed\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"dcd4c261_878140f9","line":91,"range":{"start_line":91,"start_character":6,"end_line":91,"end_character":180},"updated":"2026-01-29 10:59:10.000000000","message":"Wouldn\u0027t it be better to use: \" mon_host \u003d {%- for host in groups[\u0027all\u0027] -%} {{ hostvars[host][\u0027ansible_\u0027 + api_interface_name | replace(\u0027-\u0027, \u0027_\u0027)].ipv4.address }} {%- if not loop.last %},{% endif %} {%- endfor %}\" Using {%- and -%} (minus signs) automatically removes white spaces (spaces and newlines) around Jinja2 control blocks as an additional safeguard","commit_id":"b396fc7a8b02eea8ec1f82f7ee8dccdc95e9d360"},{"author":{"_account_id":37306,"name":"Piotr Milewski","display_name":"Piotr Milewski","email":"vurmil@gmail.com","username":"vurmil"},"change_message_id":"136d10d237ff36268a4050a36ae0466ba4616054","unresolved":true,"context_lines":[{"line_number":88,"context_line":"    ceph_conf_fixed: |"},{"line_number":89,"context_line":"      [global]"},{"line_number":90,"context_line":"      fsid \u003d {{ ceph_fsid }}"},{"line_number":91,"context_line":"      mon_host \u003d {% for host in groups[\u0027all\u0027] %}{{ hostvars[host][\u0027ansible_\u0027 + api_interface_name | replace(\u0027-\u0027, \u0027_\u0027)].ipv4.address }}{% if not loop.last %},{% endif %}{% endfor %}"},{"line_number":92,"context_line":"  copy:"},{"line_number":93,"context_line":"    content: \"{{ ceph_conf_fixed }}\""},{"line_number":94,"context_line":"    dest: \"/etc/ceph/ceph.conf.fixed\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8825a669_bea52639","line":91,"range":{"start_line":91,"start_character":6,"end_line":91,"end_character":180},"in_reply_to":"dcd4c261_878140f9","updated":"2026-01-29 11:00:07.000000000","message":"these configurations are very sensitive to spaces etc. and this is a long test so I think it\u0027s worth it","commit_id":"b396fc7a8b02eea8ec1f82f7ee8dccdc95e9d360"}]}
