)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1bedef4b12bf9b226d715ea64b634fb30ff15b61","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Victoria Martinez de la Cruz \u003cvictoria@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-03-24 20:21:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix auth ID caps for manila with ceph-mgr"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now that we issue mgr/volume commands and rados"},{"line_number":10,"context_line":"get/put on pool with \"ganesha_rados_store_pool_name\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0e83bb50_daa198dc","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"updated":"2021-03-24 21:37:40.000000000","message":"nit: Reduce \n\nwould sound more appropriate here","commit_id":"403763e59ca9a5287c62ca9c5deff3f12816356a"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"ee7a92b5874b80128fb44cbf6c9bafe6332647f6","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Victoria Martinez de la Cruz \u003cvictoria@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-03-24 20:21:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix auth ID caps for manila with ceph-mgr"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Now that we issue mgr/volume commands and rados"},{"line_number":10,"context_line":"get/put on pool with \"ganesha_rados_store_pool_name\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"75fc52d2_e9faa8b6","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":3},"in_reply_to":"0e83bb50_daa198dc","updated":"2021-03-24 21:49:07.000000000","message":"Very well sir","commit_id":"403763e59ca9a5287c62ca9c5deff3f12816356a"}],"devstack/lib/ceph":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1bedef4b12bf9b226d715ea64b634fb30ff15b61","unresolved":true,"context_lines":[{"line_number":721,"context_line":"        ${CEPHFS_DATA_POOL}"},{"line_number":722,"context_line":"    sudo $DOCKER_EXEC ceph -c ${CEPH_CONF_FILE} auth get-or-create \\"},{"line_number":723,"context_line":"        client.${MANILA_CEPH_USER} \\"},{"line_number":724,"context_line":"        mon \"allow r\" osd \"allow rw pool\u003d${CEPHFS_DATA_POOL}\" mgr \"allow rw\" \\"},{"line_number":725,"context_line":"        -o ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"},{"line_number":726,"context_line":"    sudo chown ${STACK_USER}:$(id -g -n $whoami) \\"},{"line_number":727,"context_line":"        ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"c0182326_e1e7aca3","line":724,"range":{"start_line":724,"start_character":0,"end_line":724,"end_character":78},"updated":"2021-03-24 21:37:40.000000000","message":"This code is branchless. \n\nDon\u0027t we need the older caps for older branches?","commit_id":"403763e59ca9a5287c62ca9c5deff3f12816356a"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"ee7a92b5874b80128fb44cbf6c9bafe6332647f6","unresolved":true,"context_lines":[{"line_number":721,"context_line":"        ${CEPHFS_DATA_POOL}"},{"line_number":722,"context_line":"    sudo $DOCKER_EXEC ceph -c ${CEPH_CONF_FILE} auth get-or-create \\"},{"line_number":723,"context_line":"        client.${MANILA_CEPH_USER} \\"},{"line_number":724,"context_line":"        mon \"allow r\" osd \"allow rw pool\u003d${CEPHFS_DATA_POOL}\" mgr \"allow rw\" \\"},{"line_number":725,"context_line":"        -o ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"},{"line_number":726,"context_line":"    sudo chown ${STACK_USER}:$(id -g -n $whoami) \\"},{"line_number":727,"context_line":"        ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"fc344e06_efeca2c0","line":724,"range":{"start_line":724,"start_character":0,"end_line":724,"end_character":78},"in_reply_to":"c0182326_e1e7aca3","updated":"2021-03-24 21:49:07.000000000","message":"That\u0027s a good point. Yes. If we are getting ceph-mgr in Wallaby and we are not backporting this we need to keep the older caps for older branches","commit_id":"403763e59ca9a5287c62ca9c5deff3f12816356a"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"89a5b5ba8ed6c9beae561a882decbb066604aa43","unresolved":true,"context_lines":[{"line_number":721,"context_line":"        ${CEPHFS_DATA_POOL}"},{"line_number":722,"context_line":"    sudo $DOCKER_EXEC ceph -c ${CEPH_CONF_FILE} auth get-or-create \\"},{"line_number":723,"context_line":"        client.${MANILA_CEPH_USER} \\"},{"line_number":724,"context_line":"        mon \"allow r\" osd \"allow rw pool\u003d${CEPHFS_DATA_POOL}\" mgr \"allow rw\" \\"},{"line_number":725,"context_line":"        -o ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"},{"line_number":726,"context_line":"    sudo chown ${STACK_USER}:$(id -g -n $whoami) \\"},{"line_number":727,"context_line":"        ${CEPH_CONF_DIR}/ceph.client.${MANILA_CEPH_USER}.keyring"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"b07d4a68_afc6eee3","line":724,"range":{"start_line":724,"start_character":7,"end_line":724,"end_character":76},"updated":"2021-04-23 06:36:26.000000000","message":"I have big concerns regarding usecase where cephfs_protocol_helper_type\u003dCEPHFS\nAs it will require both capabilities for creating ceph users and access to mds.","commit_id":"234d2d333fc6ff1470be915b119206273da3bc46"}]}