)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":39224,"name":"Bhavana S","display_name":"Bhavana","email":"bhavanas@cdac.in","username":"bhavana-s"},"change_message_id":"ed209685dee35576f54642e6b6695f602d8dd2fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c1996eb3_1f5ea873","updated":"2026-06-24 08:44:22.000000000","message":"Thanks for the review! I\u0027ve moved the context to a reno and added the introductory paragraph to the docs. PTAL (Please Take A Look).","commit_id":"7bc9d935653d056f08710f63da91a885013e55f7"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"c20a103dba36c1503c670a29b39df1f50b0f876f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"577a235c_2cc8477d","updated":"2026-06-24 08:50:03.000000000","message":"thanks","commit_id":"af4e5f2bb8c70da1ca886b7abf071fafdb9a61d0"}],"ansible/group_vars/all/cinder.yml":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"3be1e16332e6e7fa3401498a4f35156b64e61372","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"enable_cinder: false"},{"line_number":3,"context_line":"# Default to false when no Ceph backend is configured."},{"line_number":4,"context_line":"# Ceph backup requires /etc/ceph/ceph.conf which is only present"},{"line_number":5,"context_line":"# when cinder_backend_ceph is true."},{"line_number":6,"context_line":"# See LP#1849827"},{"line_number":7,"context_line":"enable_cinder_backup: \"{{ cinder_backend_ceph | bool or cinder_backup_driver !\u003d \u0027ceph\u0027 }}\""},{"line_number":8,"context_line":"enable_cinder_backend_iscsi: \"{{ enable_cinder_backend_lvm | bool }}\""},{"line_number":9,"context_line":"enable_cinder_backend_lvm: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b03a1bd8_a7b97c43","line":6,"range":{"start_line":3,"start_character":0,"end_line":6,"end_character":16},"updated":"2026-06-24 07:41:23.000000000","message":"This comment should be moved to the reno.","commit_id":"7bc9d935653d056f08710f63da91a885013e55f7"},{"author":{"_account_id":39224,"name":"Bhavana S","display_name":"Bhavana","email":"bhavanas@cdac.in","username":"bhavana-s"},"change_message_id":"ed209685dee35576f54642e6b6695f602d8dd2fc","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"enable_cinder: false"},{"line_number":3,"context_line":"# Default to false when no Ceph backend is configured."},{"line_number":4,"context_line":"# Ceph backup requires /etc/ceph/ceph.conf which is only present"},{"line_number":5,"context_line":"# when cinder_backend_ceph is true."},{"line_number":6,"context_line":"# See LP#1849827"},{"line_number":7,"context_line":"enable_cinder_backup: \"{{ cinder_backend_ceph | bool or cinder_backup_driver !\u003d \u0027ceph\u0027 }}\""},{"line_number":8,"context_line":"enable_cinder_backend_iscsi: \"{{ enable_cinder_backend_lvm | bool }}\""},{"line_number":9,"context_line":"enable_cinder_backend_lvm: false"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"12f8662c_51481246","line":6,"range":{"start_line":3,"start_character":0,"end_line":6,"end_character":16},"in_reply_to":"b03a1bd8_a7b97c43","updated":"2026-06-24 08:44:22.000000000","message":"Done","commit_id":"7bc9d935653d056f08710f63da91a885013e55f7"}],"doc/source/reference/storage/cinder-guide.rst":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"3be1e16332e6e7fa3401498a4f35156b64e61372","unresolved":true,"context_lines":[{"line_number":241,"context_line":"   configured. The default Ceph backup driver requires ``/etc/ceph/ceph.conf``"},{"line_number":242,"context_line":"   to be present. Without Ceph, explicitly set ``cinder_backup_driver`` to"},{"line_number":243,"context_line":"   use a different backend like NFS. See `bug 1849827"},{"line_number":244,"context_line":"   \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1849827\u003e`_."},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"Cinder-Backup with S3 Backend"},{"line_number":247,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa617ee3_34144ee9","line":244,"updated":"2026-06-24 07:41:23.000000000","message":"There is no paragraph about cinder-backup and the \"enable_cinder_backup\" variable mentioned before. So, if this is a note about that, we need at least some words about how to enable cinder-backup.\nAlso I\u0027m unsure whether the launchpad bug should be referenced here, relno is enough.","commit_id":"7bc9d935653d056f08710f63da91a885013e55f7"},{"author":{"_account_id":39224,"name":"Bhavana S","display_name":"Bhavana","email":"bhavanas@cdac.in","username":"bhavana-s"},"change_message_id":"ed209685dee35576f54642e6b6695f602d8dd2fc","unresolved":false,"context_lines":[{"line_number":241,"context_line":"   configured. The default Ceph backup driver requires ``/etc/ceph/ceph.conf``"},{"line_number":242,"context_line":"   to be present. Without Ceph, explicitly set ``cinder_backup_driver`` to"},{"line_number":243,"context_line":"   use a different backend like NFS. See `bug 1849827"},{"line_number":244,"context_line":"   \u003chttps://bugs.launchpad.net/kolla-ansible/+bug/1849827\u003e`_."},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"Cinder-Backup with S3 Backend"},{"line_number":247,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"358db3fc_9c95d72e","line":244,"in_reply_to":"fa617ee3_34144ee9","updated":"2026-06-24 08:44:22.000000000","message":"Done","commit_id":"7bc9d935653d056f08710f63da91a885013e55f7"}]}