)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a15144bb25110cf6d3741766711868c126f08b96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a40e47fb_ed20b5a0","updated":"2024-12-07 10:39:45.000000000","message":"Okay, this won\u0027t hurt, let\u0027s move on.","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"9c2bd81479ff5c2c1642f557419a3b624ad4c4dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5720bbe6_c65eabe9","updated":"2024-11-11 17:12:27.000000000","message":"recheck","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"02671ca07038cd32973e81bfc12ca4bfbc05041e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2cf57d48_f264d663","updated":"2025-01-07 21:52:32.000000000","message":"recheck bifrost had a pin on stable/2023.1","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"be370b381651824e5c3b778932c4513f2490e62b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"094e436d_44ef83ec","updated":"2024-11-14 15:02:50.000000000","message":"recheck bifrost-upgrade-keystone-centos9 FAILURE","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"842a0900a8cead7cf99c86fecd86f295f2aa921b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"10d7258b_9f4b0091","in_reply_to":"2cf57d48_f264d663","updated":"2025-01-07 22:13:44.000000000","message":"We need https://review.opendev.org/c/openstack/bifrost/+/938125 to merge first.","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"}],"playbooks/roles/bifrost-ironic-install/tasks/create_tftpboot.yml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a6f4af4b127c772f671cb487484bcc444062899f","unresolved":false,"context_lines":[{"line_number":183,"context_line":"    group: ironic"},{"line_number":184,"context_line":"    mode: \"0644\""},{"line_number":185,"context_line":"  vars:"},{"line_number":186,"context_line":"    inspection_callback_url: \u003e-"},{"line_number":187,"context_line":"      {%- if enable_inspector | bool -%}"},{"line_number":188,"context_line":"      {{ api_protocol }}://{{ internal_ip }}:5050/v1/continue"},{"line_number":189,"context_line":"      {%- else -%}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4510b09b_f389a506","line":186,"updated":"2024-12-06 13:40:32.000000000","message":"\u003e- is supposed to do the stripping\n\nAlso, this code path is covered by the CI. How does it work now?","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"9162d4856f41250144be3c2577b0e732cb09e24c","unresolved":false,"context_lines":[{"line_number":183,"context_line":"    group: ironic"},{"line_number":184,"context_line":"    mode: \"0644\""},{"line_number":185,"context_line":"  vars:"},{"line_number":186,"context_line":"    inspection_callback_url: \u003e-"},{"line_number":187,"context_line":"      {%- if enable_inspector | bool -%}"},{"line_number":188,"context_line":"      {{ api_protocol }}://{{ internal_ip }}:5050/v1/continue"},{"line_number":189,"context_line":"      {%- else -%}"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ff351561_993e24e0","line":186,"in_reply_to":"4510b09b_f389a506","updated":"2024-12-06 13:56:20.000000000","message":"I don\u0027t know why this isn\u0027t caught by CI, but the stripping with `\u003e-` (which is what you wrote, but it is displayed as a quoted bullet item by Gerrit web UI) only removes trailing newlines, not spaces before/after.\n\nSee https://yaml-multiline.info or the YAML spec.\n\nSimple reproducer:\n\n```\n---\n- hosts: localhost\n  vars:\n    api_protocol: \"https\"\n    enable_inspector: false\n    internal_ip: \"127.0.0.1\"\n    inspection_callback_url: \u003e-\n      {% if enable_inspector | bool %}\n      {{ api_protocol }}://{{ internal_ip }}:5050/v1/continue\n      {% else %}\n      {{ api_protocol }}://{{ internal_ip }}:6385/v1/continue_inspection\n      {% endif %}\n  tasks:\n    - debug:\n        var: inspection_callback_url\n```","commit_id":"e4a6792a26f05059fe5e6de6414fc81f99bd2d15"}]}