)]}'
{"cinder/volume/drivers/rbd.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f38bce426898d4c7fe738d348197f2a19df668d1","unresolved":true,"context_lines":[{"line_number":702,"context_line":"        Image data goes to rbd_pool unless a data pool is configured on it"},{"line_number":703,"context_line":"        outside Cinder with"},{"line_number":704,"context_line":""},{"line_number":705,"context_line":"            rbd config pool set \u003crbd_pool\u003e rbd_default_data_pool \u003cpool\u003e"},{"line_number":706,"context_line":""},{"line_number":707,"context_line":"        Existing images keep the pool they were created with, so this is"},{"line_number":708,"context_line":"        where the next allocation goes."}],"source_content_type":"text/x-python","patch_set":1,"id":"f102627a_c2f3ff15","line":705,"updated":"2026-11-03 14:29:53.000000000","message":"Or if you have set the rbd_data_pool option once we land https://review.opendev.org/c/openstack/cinder/+/914930","commit_id":"86004f7f039121f8a0a7e46fa7589c3f93b2abde"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"ae8ec3bc0c3d384de7ba6801e860b492c2445cd8","unresolved":true,"context_lines":[{"line_number":702,"context_line":"        Image data goes to rbd_pool unless a data pool is configured on it"},{"line_number":703,"context_line":"        outside Cinder with"},{"line_number":704,"context_line":""},{"line_number":705,"context_line":"            rbd config pool set \u003crbd_pool\u003e rbd_default_data_pool \u003cpool\u003e"},{"line_number":706,"context_line":""},{"line_number":707,"context_line":"        Existing images keep the pool they were created with, so this is"},{"line_number":708,"context_line":"        where the next allocation goes."}],"source_content_type":"text/x-python","patch_set":1,"id":"b72976f1_651bb0f1","line":705,"in_reply_to":"f102627a_c2f3ff15","updated":"2026-11-05 06:50:33.000000000","message":"I\u0027d prefer to keep this bug-fix on capacity reporting independent of 914930. it fixes incorrect capacity reporting for exisiting deployment that configure `rbd_default_data_pool` directly in ceph, without cinder side data-pool option\n\nfor example - here[1], as reported in this openstack-discuss ML. 914930 introduces a separate configuration path, which can be handled with that change.\n\n[1] https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/CYA234ZWGXEX7HWIZ2YZMEWD64XNRL2R/","commit_id":"86004f7f039121f8a0a7e46fa7589c3f93b2abde"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f38bce426898d4c7fe738d348197f2a19df668d1","unresolved":true,"context_lines":[{"line_number":733,"context_line":"        with RADOSClient(self) as client:"},{"line_number":734,"context_line":"            pool_name \u003d self._get_capacity_pool(client)"},{"line_number":735,"context_line":"            if pool_name !\u003d self.configuration.rbd_pool:"},{"line_number":736,"context_line":"                LOG.debug(\u0027Image data for pool %(pool)s is stored in \u0027"},{"line_number":737,"context_line":"                          \u0027%(data_pool)s, reporting its capacity.\u0027,"},{"line_number":738,"context_line":"                          {\u0027pool\u0027: self.configuration.rbd_pool,"},{"line_number":739,"context_line":"                           \u0027data_pool\u0027: pool_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"ae0f27a2_d211877d","line":736,"range":{"start_line":736,"start_character":16,"end_line":736,"end_character":25},"updated":"2026-11-03 14:29:53.000000000","message":"This is excessive to log on every stats update even at the debug level -- it seems ok to just remove it since people who setup data pools will need to know how this works.","commit_id":"86004f7f039121f8a0a7e46fa7589c3f93b2abde"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"ae8ec3bc0c3d384de7ba6801e860b492c2445cd8","unresolved":true,"context_lines":[{"line_number":733,"context_line":"        with RADOSClient(self) as client:"},{"line_number":734,"context_line":"            pool_name \u003d self._get_capacity_pool(client)"},{"line_number":735,"context_line":"            if pool_name !\u003d self.configuration.rbd_pool:"},{"line_number":736,"context_line":"                LOG.debug(\u0027Image data for pool %(pool)s is stored in \u0027"},{"line_number":737,"context_line":"                          \u0027%(data_pool)s, reporting its capacity.\u0027,"},{"line_number":738,"context_line":"                          {\u0027pool\u0027: self.configuration.rbd_pool,"},{"line_number":739,"context_line":"                           \u0027data_pool\u0027: pool_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"7619086a_0f3ea6ce","line":736,"range":{"start_line":736,"start_character":16,"end_line":736,"end_character":25},"in_reply_to":"ae0f27a2_d211877d","updated":"2026-11-05 06:50:33.000000000","message":"so the reason of adding this at here is while debuging I think this will help us where data-pool is configured in ceph outside cinder configuration\nso cinder is not aware of full picture, and this runs only if extra pool is setup outside cinder knowledge. if ec-pool (or any new pool) is enabled from cinder side, this will never be logged.\nso data pool configued by 914930 wont trigger it.\n\nbut the yeah w.r.t prod deployment - agreed, it can be repeated during stats refreshes via `_update_volume_stats` so lot of redundancy. I\u0027ll remove it.","commit_id":"86004f7f039121f8a0a7e46fa7589c3f93b2abde"}]}