)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"2280decc50a5f93398ff8cde15e5ac23c4277264","unresolved":true,"context_lines":[{"line_number":17,"context_line":"of original image has any external references before deleting/removing"},{"line_number":18,"context_line":"it\u0027s snapshot and returns 409 Conflict response if it has any."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"NOTE: To check this dependency glance osd needs \u0027read\u0027 access to"},{"line_number":21,"context_line":"cinder and nova side RBD pool."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Closes-Bug: #1954883"},{"line_number":24,"context_line":"Depends-on: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/819476"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"64bc9a64_e0d7dd8d","line":21,"range":{"start_line":20,"start_character":0,"end_line":21,"end_character":30},"updated":"2021-12-20 13:16:22.000000000","message":"When TripleO deploys Ceph our default is to create a keyring with this level of privilege. \n\n\nIn more detail, a recent run of one of our CI jobs [1] show that we create this data structure and pass it to our cephx ansible keys module [2]\n\ntripleo_cephadm_keys: [{\u0027name\u0027: \u0027client.openstack\u0027, \u0027key\u0027: \u0027AQC+vYNXgDAgAhAAc8UoYt+OTz5uhV7ItLdwUw\u003d\u003d\u0027, \u0027mode\u0027: \u00270600\u0027, \u0027caps\u0027: {\u0027mgr\u0027: \u0027allow *\u0027, \u0027mon\u0027: \u0027profile rbd\u0027, \u0027osd\u0027: \u0027profile rbd pool\u003dvms, profile rbd pool\u003dvolumes, profile rbd pool\u003dimages, profile rbd pool\u003dmetrics, profile rbd pool\u003dbackups, profile rbd pool\u003dpool2, profile rbd pool\u003dpool3, profile rbd pool\u003daltrbd\u0027}}, {\u0027name\u0027: \u0027client.radosgw\u0027, \u0027key\u0027: \u0027AQCHf79hAAAAABAAaRdkdcNGnpNEw26jPDnWxw\u003d\u003d\u0027, \u0027mode\u0027: \u00270600\u0027, \u0027caps\u0027: {\u0027mgr\u0027: \u0027allow *\u0027, \u0027mon\u0027: \u0027allow rw\u0027, \u0027osd\u0027: \u0027allow rwx\u0027}}, {\u0027caps\u0027: {\u0027mgr\u0027: \u0027allow *\u0027, \u0027mon\u0027: \u0027profile rbd\u0027, \u0027osd\u0027: \u0027profile rbd pool\u003dimages\u0027}, \u0027key\u0027: \u0027AQBRgQ9eAAAAABAAv84zEilJYZPNuJ0Iwn9Ndg\u003d\u003d\u0027, \u0027mode\u0027: \u00270600\u0027, \u0027name\u0027: \u0027client.glance\u0027}]\n\n\n[1] https://e142e8368045eb976a80-66c52d1469a99000c43c17a53dc4fc44.ssl.cf2.rackcdn.com/822240/1/check/tripleo-ci-centos-8-scenario001-standalone/e2b101b/logs/undercloud/home/zuul/tripleo-deploy/standalone-ansible-o17itm1c/cephadm/cephadm-extra-vars-ansible.yml\n\n[2] https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_cephadm/tasks/keys.yaml","commit_id":"91be31e8fdc148c1754fd89cc2d44166cfe124c2"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"9aba597d9e7dca6219b2c51dada369bcebdb6d0f","unresolved":true,"context_lines":[{"line_number":10,"context_line":"RBD backend, since then if you attempt to delete an image from"},{"line_number":11,"context_line":"glance that has a dependent volume, all future uses of that"},{"line_number":12,"context_line":"image will fail in error state. Despite the fact that image"},{"line_number":13,"context_line":"itself is still inside of Ceph/Glance. This issue is reproducible"},{"line_number":14,"context_line":"if you are using ceph client version greater than \u0027luminous\u0027"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"To resolve this issue glance RBD driver now checks whether snapshot"},{"line_number":17,"context_line":"of original image has any external references before deleting/removing"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a6b10fa1_f2c6f014","line":14,"range":{"start_line":13,"start_character":39,"end_line":14,"end_character":60},"updated":"2022-01-07 10:42:02.000000000","message":"Not related to the Ceph client version in use but the minimum supported client version configured in the Ceph side. If the Ceph cluster is set to support clients \u003c \u0027luminous\u0027 this issue is not reproducible by just opting into newer client.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"9aba597d9e7dca6219b2c51dada369bcebdb6d0f","unresolved":true,"context_lines":[{"line_number":17,"context_line":"of original image has any external references before deleting/removing"},{"line_number":18,"context_line":"it\u0027s snapshot and returns 409 Conflict response if it has any."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"NOTE: To check this dependency glance osd needs \u0027read\u0027 access to"},{"line_number":21,"context_line":"cinder and nova side RBD pool."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Closes-Bug: #1954883"},{"line_number":24,"context_line":"Depends-on: https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/819476"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dc0904c7_45ab628f","line":21,"range":{"start_line":20,"start_character":0,"end_line":21,"end_character":30},"updated":"2022-01-07 10:42:02.000000000","message":"Nothing to do with osds, but the Ceph client keys in use.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0abaa770e90f34a7e8fa36418ffc6df4b7f5654f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"84dd0eb3_015f0686","updated":"2021-12-13 14:06:00.000000000","message":"I think we need to do more investigation into other ways to detect that dependent images exist before implementing this method, which requires giving Glance access to the Cinder pool.","commit_id":"3a6c02a19b4ed2bcdab09209e98dcfbad30f97b9"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"acaa11a6b5fbc2d3be6d85329709ea7fe2daf2cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"573e9729_c8d9e421","updated":"2021-12-15 08:21:59.000000000","message":"Working on adding tempest side coverage (tempest scenario test to cover this usecase)","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"1d20fadf1753c8b94e3e57bd1cd316e2ca8dfa8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"75466c0d_42052041","updated":"2021-12-16 06:04:23.000000000","message":"recheck non-voting failure","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"dbf6e2e83023751aadd73d51612812dcca48fcaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b94d714c_f9e9fd16","updated":"2021-12-15 17:46:44.000000000","message":"recheck non-voting timeout","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c85fba9d97e4ceaaf8cd77d78a849cbf4018e915","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"39c3895f_bcb02de0","updated":"2021-12-21 17:06:59.000000000","message":"Clarification suggested for the release note, otherwise this looks fine.  I don\u0027t think Eric left a followup to his earlier comment, but he did investigate and could not find an alternative that didn\u0027t require changing the ceph pool ACLs.","commit_id":"91be31e8fdc148c1754fd89cc2d44166cfe124c2"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"95164f701f4b7edc81d2bd1ec672e553a93e3209","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d80f1ef_3eb5a73e","updated":"2021-12-20 05:29:28.000000000","message":"Thank you Benny and Eric for review, Made changes as per your suggestions!!","commit_id":"91be31e8fdc148c1754fd89cc2d44166cfe124c2"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"cc1de3ba7fdc77fe9c9dfe59b2f2b43c1ded20a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ea6d92c9_d75084dc","updated":"2021-12-22 14:19:43.000000000","message":"Brian +2?","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"faa018109782423bb489a69a048f56d8341ef244","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0a7efb11_8c34c7fa","updated":"2021-12-22 15:12:17.000000000","message":"LGTM, and the devstack-plugin-ceph patch would also be helpful for the future work regarding RBD clone optimization in upload volume to image operation.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b85edafa6a7f602ef92cc239179c341425003f71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4640ef21_111a2306","updated":"2021-12-22 18:56:41.000000000","message":"Ninja-approving as there are no other cores around (other than Abhishek, the author) during this holiday week.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2262ccbf238b7a428a805a6421c0e66b29ed2250","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"55e7af50_e568b674","updated":"2021-12-22 14:12:17.000000000","message":"Revision LGTM.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"58aa99869ccc5ef05bbd4e2e638409fd6f4f147d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1f390c2a_acc6fa46","updated":"2021-12-21 17:12:30.000000000","message":"Thank you Brian, made changes as per your suggestion!","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"6b646396f883ddfe4941173374c39dd93f3fca07","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3323e31c_9dabb136","updated":"2022-01-07 10:27:55.000000000","message":"Unfortunately due to the approach this will require major version bump on glance_store on next release as it\u0027s backwards incompatible breaking change. It also doesn\u0027t seem to handle the lacking permissions gracefully. \n\nDoesn\u0027t seem to have any testing for the same either.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a046a315d2c2d9d048e0a69d1af7d799b7773064","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ec3fd4c7_ec21d05c","updated":"2022-01-07 13:47:30.000000000","message":"We can submit a followup to modify the release note as per your inputs.\n","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"aac881750487a8076f56e6a9a9fc910e11211f5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bc04b375_67814189","updated":"2021-12-22 14:21:35.000000000","message":"oops, forgot to vote.  +2 !!!","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"ec4a9758b925b7603d626ccb4ad8834f1293a85e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2256395d_6b37a3bc","updated":"2022-01-04 04:12:00.000000000","message":"recheck cinder failure","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a046a315d2c2d9d048e0a69d1af7d799b7773064","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a8002a24_6f060f66","in_reply_to":"3323e31c_9dabb136","updated":"2022-01-07 13:47:30.000000000","message":"As we are in new cycle, we are going to release glance_store with major version change only.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"35c85e6c4162ec4fb4af6803de0475685e07e435","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cbb7cc4d_02bfd937","in_reply_to":"a8002a24_6f060f66","updated":"2022-01-18 17:40:35.000000000","message":"We do not normally release glance_store with new major every cycle. Last time this was in Ussuri, which is why this is important to remember and will signal to the consumers that breaking changes has happened.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"}],"glance_store/_drivers/rbd.py":[{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"82ad9f37e16af984759cc9eb15c732f90ccf19ff","unresolved":true,"context_lines":[{"line_number":440,"context_line":"        has_references \u003d False"},{"line_number":441,"context_line":"        image.set_snap(snapshot_name)"},{"line_number":442,"context_line":"        if image.list_children():"},{"line_number":443,"context_line":"            has_references \u003d True"},{"line_number":444,"context_line":"        image.set_snap(None)"},{"line_number":445,"context_line":""},{"line_number":446,"context_line":"        return has_references"}],"source_content_type":"text/x-python","patch_set":2,"id":"3548185d_98679b06","line":443,"updated":"2021-12-17 20:09:57.000000000","message":"Minor comment ,  maybe we can reduce lines here and without if,\nsomething like:\n\nimage.set_snap(snapshot_name)\nhas_references \u003d bool(image.list_children())\nimage.set_snap(None)\nreturn has_references","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"95164f701f4b7edc81d2bd1ec672e553a93e3209","unresolved":false,"context_lines":[{"line_number":440,"context_line":"        has_references \u003d False"},{"line_number":441,"context_line":"        image.set_snap(snapshot_name)"},{"line_number":442,"context_line":"        if image.list_children():"},{"line_number":443,"context_line":"            has_references \u003d True"},{"line_number":444,"context_line":"        image.set_snap(None)"},{"line_number":445,"context_line":""},{"line_number":446,"context_line":"        return has_references"}],"source_content_type":"text/x-python","patch_set":2,"id":"f87170d2_bbbfe9c7","line":443,"in_reply_to":"3548185d_98679b06","updated":"2021-12-20 05:29:28.000000000","message":"Done","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"9aba597d9e7dca6219b2c51dada369bcebdb6d0f","unresolved":true,"context_lines":[{"line_number":437,"context_line":"    def _snapshot_has_external_reference(self, image, snapshot_name):"},{"line_number":438,"context_line":"        \"\"\"Returns True if snapshot has external reference else False."},{"line_number":439,"context_line":"        \"\"\""},{"line_number":440,"context_line":"        image.set_snap(snapshot_name)"},{"line_number":441,"context_line":"        has_references \u003d bool(image.list_children())"},{"line_number":442,"context_line":"        image.set_snap(None)"},{"line_number":443,"context_line":"        return has_references"},{"line_number":444,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"efca7b0b_d496cb2a","line":441,"range":{"start_line":440,"start_character":0,"end_line":441,"end_character":52},"updated":"2022-01-07 10:42:02.000000000","message":"Are these calls available in all supported rbd client versions or do we need to bump the minimums as well?","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"faa018109782423bb489a69a048f56d8341ef244","unresolved":true,"context_lines":[{"line_number":467,"context_line":"                                        image, snapshot_name):"},{"line_number":468,"context_line":"                                    raise rbd.ImageBusy("},{"line_number":469,"context_line":"                                        \"Image snapshot has external \""},{"line_number":470,"context_line":"                                        \"references.\")"},{"line_number":471,"context_line":""},{"line_number":472,"context_line":"                                self._unprotect_snapshot(image, snapshot_name)"},{"line_number":473,"context_line":"                                image.remove_snap(snapshot_name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"eb837b9b_19e0f434","line":470,"range":{"start_line":470,"start_character":41,"end_line":470,"end_character":51},"updated":"2021-12-22 15:12:17.000000000","message":"or dependencies ? anyway both sound good","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"}],"glance_store/tests/unit/test_multistore_rbd.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"faa018109782423bb489a69a048f56d8341ef244","unresolved":true,"context_lines":[{"line_number":424,"context_line":"            self.called_commands_expected \u003d [\u0027unprotect_snap\u0027]"},{"line_number":425,"context_line":""},{"line_number":426,"context_line":"    def test_delete_image_snap_has_external_references(self):"},{"line_number":427,"context_line":"        with mock.patch.object(MockRBD.Image, \u0027list_children\u0027) as mocked:"},{"line_number":428,"context_line":"            mocked.return_value \u003d True"},{"line_number":429,"context_line":""},{"line_number":430,"context_line":"            self.assertRaises(exceptions.InUseByStore,"}],"source_content_type":"text/x-python","patch_set":4,"id":"42a147b1_7e0f1b35","line":427,"range":{"start_line":427,"start_character":66,"end_line":427,"end_character":72},"updated":"2021-12-22 15:12:17.000000000","message":"list_child or list_deps would\u0027ve been better than mocked but it works","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"}],"releasenotes/notes/bug-1954883-3666d63a3c0233f1.yaml":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"290c6fb711d13738ef72845eec4ffb407264fcdc","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Cinder V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD) and minimum ceph client"},{"line_number":12,"context_line":"    version is greater than \u0027luminous\u0027 need to access glance osd a read"},{"line_number":13,"context_line":"    access to cinder RBD pool."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"beb9ecf6_168a3541","line":10,"range":{"start_line":10,"start_character":32,"end_line":10,"end_character":55},"updated":"2021-12-17 21:18:32.000000000","message":"This could be worded better -- the v2 clone feature is a backend Ceph feature that is handled by Ceph and isn\u0027t really a Cinder feature.","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"95164f701f4b7edc81d2bd1ec672e553a93e3209","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Cinder V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD) and minimum ceph client"},{"line_number":12,"context_line":"    version is greater than \u0027luminous\u0027 need to access glance osd a read"},{"line_number":13,"context_line":"    access to cinder RBD pool."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f48f5311_b728b402","line":10,"range":{"start_line":10,"start_character":32,"end_line":10,"end_character":55},"in_reply_to":"beb9ecf6_168a3541","updated":"2021-12-20 05:29:28.000000000","message":"Ack","commit_id":"e642f4089cb67edf659470ec5ba94c6aed036cf3"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c85fba9d97e4ceaaf8cd77d78a849cbf4018e915","unresolved":true,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Ceph V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD or nova is using RBD driver)"},{"line_number":12,"context_line":"    and minimum ceph client version is greater than \u0027luminous\u0027 need to access"},{"line_number":13,"context_line":"    glance osd a read access to cinder and nova RBD pool."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"4ac3ac0c_0810c0e5","line":13,"range":{"start_line":12,"start_character":63,"end_line":13,"end_character":57},"updated":"2021-12-21 17:06:59.000000000","message":"\"need to grant glance osd read access to the cinder and nova RBD pool.\"","commit_id":"91be31e8fdc148c1754fd89cc2d44166cfe124c2"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"58aa99869ccc5ef05bbd4e2e638409fd6f4f147d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Ceph V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD or nova is using RBD driver)"},{"line_number":12,"context_line":"    and minimum ceph client version is greater than \u0027luminous\u0027 need to access"},{"line_number":13,"context_line":"    glance osd a read access to cinder and nova RBD pool."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5e541992_7f31cfb2","line":13,"range":{"start_line":12,"start_character":63,"end_line":13,"end_character":57},"in_reply_to":"4ac3ac0c_0810c0e5","updated":"2021-12-21 17:12:30.000000000","message":"Ack","commit_id":"91be31e8fdc148c1754fd89cc2d44166cfe124c2"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"9aba597d9e7dca6219b2c51dada369bcebdb6d0f","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Ceph V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD or nova is using RBD driver)"},{"line_number":12,"context_line":"    and minimum ceph client version is greater than \u0027luminous\u0027 need to grant"},{"line_number":13,"context_line":"    glance osd read access to the cinder and nova RBD pool."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5ed285cb_fa9954e2","line":12,"range":{"start_line":10,"start_character":4,"end_line":12,"end_character":62},"updated":"2022-01-07 10:42:02.000000000","message":"Ceph V2 clone feature is irrelevant to this issue, this behaviour is purely triggered with the Ceph option that sets the minimum client version.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"faa018109782423bb489a69a048f56d8341ef244","unresolved":true,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Deployments which are using Ceph V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD or nova is using RBD driver)"},{"line_number":12,"context_line":"    and minimum ceph client version is greater than \u0027luminous\u0027 need to grant"},{"line_number":13,"context_line":"    glance osd read access to the cinder and nova RBD pool."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"370715cf_5c56bd46","line":12,"range":{"start_line":12,"start_character":62,"end_line":12,"end_character":63},"updated":"2021-12-22 15:12:17.000000000","message":"nit: ,we","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"9aba597d9e7dca6219b2c51dada369bcebdb6d0f","unresolved":true,"context_lines":[{"line_number":10,"context_line":"    Deployments which are using Ceph V2 clone feature (i.e. RBD backend for"},{"line_number":11,"context_line":"    glance_store as well as cinder driver is RBD or nova is using RBD driver)"},{"line_number":12,"context_line":"    and minimum ceph client version is greater than \u0027luminous\u0027 need to grant"},{"line_number":13,"context_line":"    glance osd read access to the cinder and nova RBD pool."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"726f2132_10abc4cc","line":13,"range":{"start_line":13,"start_character":4,"end_line":13,"end_character":14},"updated":"2022-01-07 10:42:02.000000000","message":"ditto: glance rbd client, nothing to do with osds.","commit_id":"3d221ec529862d43ab303644e74ee9ad6ce8cd40"}]}