)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a0afdd546874fe135dcce7f8f77ccd0e58f9ef3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"589e7451_6bcd2cd2","updated":"2025-12-04 17:43:18.000000000","message":"Thanks Luigi for the review.","commit_id":"d23874627a144c23382677e831a55e53f7992226"}],"devstack/lib/cephadm":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"53398e7530e4af7caa03d872a14198070051c0dd","unresolved":true,"context_lines":[{"line_number":241,"context_line":"        test -e $CEPH_KEYRING"},{"line_number":242,"context_line":"    fi"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    if [[ \"$MULTI_CEPH\" \u003d\u003d \u0027True\u0027 ]]; then"},{"line_number":245,"context_line":"        $SUDO \"$CEPHADM\" --image \"$CONTAINER_IMAGE\" \\"},{"line_number":246,"context_line":"            bootstrap \\"},{"line_number":247,"context_line":"            --fsid $CEPH2_FSID \\"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"94eca2e3_9cb200f3","line":244,"updated":"2025-10-28 15:46:48.000000000","message":"more than MULTI_CEPH, shouldn\u0027t this be called \"ADDITIONAL_CEPH\"?\n\nIdeally the whole configuration would done through a variable with an integert value which would configure the requested amount of ceph pools.","commit_id":"d23874627a144c23382677e831a55e53f7992226"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a0afdd546874fe135dcce7f8f77ccd0e58f9ef3a","unresolved":true,"context_lines":[{"line_number":241,"context_line":"        test -e $CEPH_KEYRING"},{"line_number":242,"context_line":"    fi"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    if [[ \"$MULTI_CEPH\" \u003d\u003d \u0027True\u0027 ]]; then"},{"line_number":245,"context_line":"        $SUDO \"$CEPHADM\" --image \"$CONTAINER_IMAGE\" \\"},{"line_number":246,"context_line":"            bootstrap \\"},{"line_number":247,"context_line":"            --fsid $CEPH2_FSID \\"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3ffb741b_d83b8552","line":244,"in_reply_to":"94eca2e3_9cb200f3","updated":"2025-12-04 17:43:18.000000000","message":"I agree that this should be more scale-able but with the limited knowledge and time around devstack tooling, the current setup served my purpose which was to create a base infra for replication testing (without affecting the default 1 ceph deployment setup). I would be happy to review such an effort if someone picks it up but unfortunately I won\u0027t be able to pursue it.\nAlso the change for ADDITIONAL_CEPH is good but will require refactor of this and patches dependent on it so i will keep that as a note if needed to propose a revision.","commit_id":"d23874627a144c23382677e831a55e53f7992226"}]}