)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c1f064ae86d6a9b57aa04ea437a2cc85cd08f4f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e70864a7_7cb6ccc9","updated":"2024-05-18 08:31:58.000000000","message":"Makes sense to me.","commit_id":"c3354c9560648f5c883d5cf1c084ddfaff7af25c"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"a66e4e339e3ed866d4940194d109f728fcadd340","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fb326a8b_8177e81d","updated":"2024-08-06 08:04:01.000000000","message":"recheck\n\nUnclear why Zuul did not process this one yet","commit_id":"6b51b98282602f09ab5433f775395ceb84fd20c8"}],"ansible/roles/heat/templates/heat.conf.j2":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"be57d1df6976a2df3d19ff946c718b59d9552dc3","unresolved":true,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"{% if not enable_cinder_backup | bool %}"},{"line_number":108,"context_line":"[volumes]"},{"line_number":109,"context_line":"backups_enabled \u003d False"},{"line_number":110,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"c7a293a5_2c2408ec","line":109,"range":{"start_line":109,"start_character":0,"end_line":109,"end_character":23},"updated":"2024-06-26 17:33:06.000000000","message":"backups_enabled \u003d {{ enable_cinder_backup | bool }} is enough ...one line vs 4 lines of code","commit_id":"c3354c9560648f5c883d5cf1c084ddfaff7af25c"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"5f207678f05ac95ec64b151d5c4103a29623eb8b","unresolved":false,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"{% if not enable_cinder_backup | bool %}"},{"line_number":108,"context_line":"[volumes]"},{"line_number":109,"context_line":"backups_enabled \u003d False"},{"line_number":110,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"18a0c8fe_dd4dc14f","line":109,"range":{"start_line":109,"start_character":0,"end_line":109,"end_character":23},"in_reply_to":"b4330fb8_c053bc35","updated":"2024-06-26 20:18:53.000000000","message":"I chose this approach to avoid setting the option with its default value of True, but it could be better to be explicit: operators could notice more easily if it gets out of sync with the state of the deployment.","commit_id":"c3354c9560648f5c883d5cf1c084ddfaff7af25c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"4d94517586b46ce4a17d68fd8a32e4654801f4af","unresolved":true,"context_lines":[{"line_number":106,"context_line":""},{"line_number":107,"context_line":"{% if not enable_cinder_backup | bool %}"},{"line_number":108,"context_line":"[volumes]"},{"line_number":109,"context_line":"backups_enabled \u003d False"},{"line_number":110,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"b4330fb8_c053bc35","line":109,"range":{"start_line":109,"start_character":0,"end_line":109,"end_character":23},"in_reply_to":"c7a293a5_2c2408ec","updated":"2024-06-26 17:55:08.000000000","message":"sorry, 2 lines vs 4 ..of course there need to be also [volumes]","commit_id":"c3354c9560648f5c883d5cf1c084ddfaff7af25c"}],"releasenotes/notes/heat-volumes-backup-enabled-4e9799558677e6cd.yaml":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"7b64faebb328926622d9ffb8840c0216e1c04022","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Configures Heat with ``[volumes]/backups_enabled \u003d False`` when the"},{"line_number":5,"context_line":"    ``cinder-backup`` service is disabled."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"69f37be3_8983a5a0","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"updated":"2024-05-07 15:04:08.000000000","message":"Unsure if this should be a feature or a fix.","commit_id":"5f63afd20bccd2f4da16ffe0fad9ea254233eb28"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"4c0e88d51dd85eee40e09c55a0cc9aa6b102a933","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Configures Heat with ``[volumes]/backups_enabled \u003d False`` when the"},{"line_number":5,"context_line":"    ``cinder-backup`` service is disabled."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"93c6f9c4_aa0064e7","line":2,"range":{"start_line":2,"start_character":0,"end_line":2,"end_character":8},"in_reply_to":"69f37be3_8983a5a0","updated":"2024-05-16 07:25:29.000000000","message":"Settle on `fixes`.","commit_id":"5f63afd20bccd2f4da16ffe0fad9ea254233eb28"}]}