)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"28c095e5a47adb750b62749237c907d50e55739c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     nhk \u003cnguyenhuukhoinw@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-01-27 08:42:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ceph_volume_backend_name."},{"line_number":8,"context_line":"It is useful for multi AZ and each AZ has same volume backend type."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I2fab6be1f35b233ae376835d78228b57bb75098e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8c46f992_2555e0a4","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":29},"updated":"2025-01-27 09:25:13.000000000","message":"Please separate the Git commit summary from the rest of the commit message. You don\u0027t need a dot at the end of the summary line.","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"},{"author":{"_account_id":35086,"name":"Khoi","email":"nguyenhuukhoinw@gmail.com","username":"nhk"},"change_message_id":"aad7a2b28cde3ed5f7b7e4615f92ca744f3347c2","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     nhk \u003cnguyenhuukhoinw@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-01-27 08:42:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ceph_volume_backend_name."},{"line_number":8,"context_line":"It is useful for multi AZ and each AZ has same volume backend type."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I2fab6be1f35b233ae376835d78228b57bb75098e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eb1b958e_bafd912e","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":29},"in_reply_to":"8c46f992_2555e0a4","updated":"2025-01-28 00:25:00.000000000","message":"You mean I need recommit, right? I am not familiar with it.","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"28c095e5a47adb750b62749237c907d50e55739c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4eefc1cd_4cdadcc2","updated":"2025-01-27 09:25:13.000000000","message":"Thank you for your first contribution.","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"51977dab44d87be4e6340f1a75eeafa076fc216d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e5585cd4_5b225ac5","updated":"2025-01-28 20:35:56.000000000","message":"https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"d481fe0369bd318f75ab80b6bb8bb0bbe865f0e2"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"08e23400488c8c71847f575868f52d536ebeb8c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0c427f5e_f0f3dadb","updated":"2025-02-07 15:51:23.000000000","message":"there is a patch ready to merge https://review.opendev.org/c/openstack/kolla-ansible/+/940608","commit_id":"d481fe0369bd318f75ab80b6bb8bb0bbe865f0e2"}],"ansible/roles/cinder/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"2f6f9e3c9cf9a8eac9ab0914d46dc3f5dce70275","unresolved":true,"context_lines":[{"line_number":267,"context_line":"cinder_backend_pure_nvme_tcp_name: \"Pure-FlashArray-nvme-tcp\""},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"cinder_ceph_backends:"},{"line_number":270,"context_line":"  - name: \"{{ cinder_backend_ceph_name }}\""},{"line_number":271,"context_line":"    ceph_volume_backend_name: \"{{ ceph_volume_backend_name }}\""},{"line_number":272,"context_line":"    cluster: \"{{ ceph_cluster }}\""},{"line_number":273,"context_line":"    user: \"{{ ceph_cinder_user }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e0b6691b_b8e51666","line":270,"updated":"2025-01-27 11:12:44.000000000","message":"What\u0027s wrong with using name?","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"},{"author":{"_account_id":35086,"name":"Khoi","email":"nguyenhuukhoinw@gmail.com","username":"nhk"},"change_message_id":"89a4072f672ac9b8d067f796163ca1160085accc","unresolved":true,"context_lines":[{"line_number":267,"context_line":"cinder_backend_pure_nvme_tcp_name: \"Pure-FlashArray-nvme-tcp\""},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"cinder_ceph_backends:"},{"line_number":270,"context_line":"  - name: \"{{ cinder_backend_ceph_name }}\""},{"line_number":271,"context_line":"    ceph_volume_backend_name: \"{{ ceph_volume_backend_name }}\""},{"line_number":272,"context_line":"    cluster: \"{{ ceph_cluster }}\""},{"line_number":273,"context_line":"    user: \"{{ ceph_cinder_user }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d245a125_10881bf2","line":270,"in_reply_to":"af665208_5e04d83e","updated":"2025-01-30 03:23:52.000000000","message":"I crate a volume type with volume_backend_name\u003dcephssd. I have same backend type at two AZ if we use \"name\" then it cannot fulfill this logic.","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e1ee425b96ce8799a29c50092e2be3d17d9aa02e","unresolved":true,"context_lines":[{"line_number":267,"context_line":"cinder_backend_pure_nvme_tcp_name: \"Pure-FlashArray-nvme-tcp\""},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"cinder_ceph_backends:"},{"line_number":270,"context_line":"  - name: \"{{ cinder_backend_ceph_name }}\""},{"line_number":271,"context_line":"    ceph_volume_backend_name: \"{{ ceph_volume_backend_name }}\""},{"line_number":272,"context_line":"    cluster: \"{{ ceph_cluster }}\""},{"line_number":273,"context_line":"    user: \"{{ ceph_cinder_user }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5ff5d1f4_b1011c28","line":270,"in_reply_to":"d245a125_10881bf2","updated":"2025-02-03 11:19:26.000000000","message":"Hi, yes, you’re right, we also just ran into this small problem.\n\n@mnasiadka@gmail.com, maybe I’ll clarify what he’s trying to achieve. He simply needs to have a volume_backend_name that will handle the creation of Cinder volumes, but he needs to define it twice for two different availability zones (AZs). However, the Cinder block configuration section for these two Cinder backends cannot be identical - You simply have to distinguish them. (essentially the same backend but in different AZs).\n\n```\n[ceph-geographic-AZ1]\nvolume_driver \u003d cinder.volume.drivers.rbd.RBDDriver\nvolume_backend_name \u003d ceph-geographic\nrbd_pool \u003d volumes\nrbd_keyring_conf \u003d /etc/ceph/ceph-geographic.client.cinder.keyring\nrbd_secret_uuid \u003d THIS_IS_SECRET\nbackend_availability_zone \u003d AZ1\n\n[ceph-geographic-AZ2]\nvolume_driver \u003d cinder.volume.drivers.rbd.RBDDriver\nvolume_backend_name \u003d ceph-geographic\nrbd_pool \u003d volumes\nrbd_keyring_conf \u003d /etc/ceph/ceph-geographic.client.cinder.keyring\nrbd_secret_uuid \u003d THIS_IS_SECRET\nbackend_availability_zone \u003d AZ2\n```\n\nOr similar.","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"},{"author":{"_account_id":35086,"name":"Khoi","email":"nguyenhuukhoinw@gmail.com","username":"nhk"},"change_message_id":"aad7a2b28cde3ed5f7b7e4615f92ca744f3347c2","unresolved":true,"context_lines":[{"line_number":267,"context_line":"cinder_backend_pure_nvme_tcp_name: \"Pure-FlashArray-nvme-tcp\""},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"cinder_ceph_backends:"},{"line_number":270,"context_line":"  - name: \"{{ cinder_backend_ceph_name }}\""},{"line_number":271,"context_line":"    ceph_volume_backend_name: \"{{ ceph_volume_backend_name }}\""},{"line_number":272,"context_line":"    cluster: \"{{ ceph_cluster }}\""},{"line_number":273,"context_line":"    user: \"{{ ceph_cinder_user }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"af665208_5e04d83e","line":270,"in_reply_to":"e0b6691b_b8e51666","updated":"2025-01-28 00:25:00.000000000","message":"For example,\n\nIf we have multi AZ we cannot use same ceph_volume_backend_name in each AZ..\n\n  - name: \"ssd1\"\n    ceph_volume_backend_name: \"cephssd\"\n    cluster: \"az1\"\n    user: \"cinder\"\n    pool: \"volumes\"\n    availability_zone: \"az1\"\n    enabled: \"{{ cinder_backend_ceph | bool }}\"\n  - name: \"ssd2\"\n    ceph_volume_backend_name: \"cephssd\"\n    cluster: \"az3\"\n    user: \"cinder\"\n    pool: \"volumes\"\n    availability_zone: \"az2\"\n    enabled: \"{{ cinder_backend_ceph | bool }}\"","commit_id":"ed8cd7004e9538e9aa21cd2842d84ac628c7e0ea"}]}