)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"5ee1e90654903a29d03b3d70b52610c78e59a6f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6935772f_4c5482b4","updated":"2022-07-15 14:49:54.000000000","message":"Passes CI","commit_id":"3c6a61b49020b0d4066799afcdde55fddf962719"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"379546ff3156cc20c28f666fb85e9532a6c0a2d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d907bed1_82b58412","updated":"2022-07-21 03:36:10.000000000","message":"LGTM","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":34106,"name":"Prajakta","email":"prajakta.ambulkar@ibm.com","username":"prajaktab"},"change_message_id":"886bfb33a3c0619f56a82e5a46c8f61f385b6816","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7cea1f85_ce44768c","updated":"2022-08-30 17:53:06.000000000","message":"LGTM , +1","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d798d20bb8a1953faa58a60115f4ffcf9405e7a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d3e0e64f_52db9820","updated":"2022-09-16 14:38:04.000000000","message":"Suggestion for a clarification inline.","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"7717c23744d5416c0a8d0e4ad1469bc1057e17f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e927bf17_e151d673","updated":"2022-07-21 13:18:51.000000000","message":"Thanks!","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"7acfcd727c7cfbd4db975c5f2df931b791a655fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ecfee329_00f66d90","updated":"2022-09-12 15:37:23.000000000","message":"may be it\u0027s worth to add relevant information into bug or commit message why it\u0027s done? E.g. https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8/html/considerations_in_adopting_rhel_8/file-systems-and-storage_considerations-in-adopting-rhel-8#removal-of-lvmetad-daemon_file-systems-and-storage","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"9327bb098eb6be80e0faea7bdc78ac142968cf87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7f124219_94863e1d","updated":"2022-12-16 14:42:29.000000000","message":"Changes look good to me. ","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7d7cd762eff24828ca6376d9503f0daec4d5999b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4eec6d9c_d4ede471","updated":"2022-09-21 12:15:02.000000000","message":"LGTM","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":33473,"name":"Atsushi Kawai","display_name":"akawai","email":"atsushi.kawai.bu@hitachi.com","username":"akawai"},"change_message_id":"e549610dbb53086ee528700fd1f14a011f6123bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5833e5f9_a4365984","updated":"2022-10-21 09:19:18.000000000","message":"LGTM","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":34201,"name":"Mounika Sreeram","email":"sreeram.mounika@ibm.com","username":"sreerammounika"},"change_message_id":"5bd43f9cc553a6e7da32e5b192a34bc98e635156","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6c42966b_b844bd23","updated":"2022-12-19 05:39:00.000000000","message":"Looks good","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"661df0452f5a23852b87b977b6fdd6ab3e175cab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4c3e2511_2de8903c","updated":"2022-10-30 05:35:35.000000000","message":"hi core reviewers could you help me to merge this patch.","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"56b822f8289ed8c9900069b5528e4b5e3fc241b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"558b0f5e_9569111d","updated":"2022-10-31 13:38:19.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"af3e2ae260345d881a0f15f5dac937cd156c74f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6b9a7fdd_bfe95932","updated":"2022-10-11 10:04:21.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"a75a784832d05403c892e17ff3a360f025180daf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"81e76d84_5e26bc84","updated":"2022-10-25 12:14:20.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"99d13ec5d126b54bff6dfdec81fc47dcc38318a6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9e5e4fde_6e8f26c3","updated":"2022-10-27 08:35:28.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"5c50dafd36e6b8327d36f4104bb6e4a94c1ecef3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ae115990_bff7abdc","updated":"2022-10-13 12:16:36.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"b092ab2b30014eb76f4bdd71e70ef07fe537b397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cbba842b_4bb356bc","updated":"2022-10-22 13:16:51.000000000","message":"ping","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"f3df0b4a93087a2bf8ee0e0c50bf534c9891d2bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b496281f_3b258bf1","updated":"2022-12-16 14:25:23.000000000","message":"still looks good to me","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"c3dbd827006e031264c103b9c370eecf7816ffc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9c6faf19_985ee36d","updated":"2022-10-21 12:48:44.000000000","message":"thanks!","commit_id":"9fd41690b6473a6ccc6cb41ccdefe4c67ad1fd0a"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"df7a574a10a70c676db0f6808635352943af6f60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"37d6ad89_3a375d6f","updated":"2023-01-20 15:50:12.000000000","message":"Looks like the text was updated so many times and looks good!","commit_id":"90b298796db316a2d938abb61fca4afc3aac1722"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d490047c964c4cbe0eee850f1538f704f2d27200","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d0ad2fa0_ae7abe64","updated":"2023-02-17 15:58:27.000000000","message":"Sorry this has been sitting so long.  Thanks, Tushar.","commit_id":"90b298796db316a2d938abb61fca4afc3aac1722"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"b336ecbaa4aea41ddc23c34e0c5fd9eb60e0bd86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"342560e0_db602d9e","updated":"2023-02-13 07:57:20.000000000","message":"ping","commit_id":"90b298796db316a2d938abb61fca4afc3aac1722"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"becfcf3aebafc27304527179fa361fbf12af0afd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5da09b6b_78733503","updated":"2023-02-10 03:24:47.000000000","message":"ping","commit_id":"90b298796db316a2d938abb61fca4afc3aac1722"}],"doc/source/install/cinder-storage-install-rdo.rst":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5f36ca5289120859e31719a66253364f0eb352e4","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"     .. code-block:: console"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"        # yum install lvm2 thin-provisioning-tools"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"6f595cc9_0900d768","line":20,"range":{"start_line":20,"start_character":27,"end_line":20,"end_character":50},"updated":"2022-01-21 15:15:26.000000000","message":"I understand the removal of the references to lvm2-lvmetad, but I don\u0027t think this is correct.\nOn RHEL, Fedora and derived distributions the package is called device-mapper-persistent-data.\nhttps://repology.org/project/device-mapper-persistent-data/versions\nhttps://repology.org/project/thin-provisioning-tools/versions\n\nThis is the command for RDO, so RHEL/CentOS Stream.","commit_id":"fc8bd23ff6012559fa3dffb0bdbccd342aee3d52"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"b2eceb47e0f9a7280b8fde6c0921b31cdd25eef3","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"     .. code-block:: console"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"        # yum install lvm2 thin-provisioning-tools"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"a4be70ba_a21b2bd1","line":20,"range":{"start_line":20,"start_character":27,"end_line":20,"end_character":50},"in_reply_to":"00090a69_d9aef1c2","updated":"2022-07-01 04:29:02.000000000","message":"Done","commit_id":"fc8bd23ff6012559fa3dffb0bdbccd342aee3d52"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"3688f00f5226de0b8c02666fdf9d730ec4712b49","unresolved":true,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"     .. code-block:: console"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"        # yum install lvm2 thin-provisioning-tools"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"00090a69_d9aef1c2","line":20,"range":{"start_line":20,"start_character":27,"end_line":20,"end_character":50},"in_reply_to":"6f595cc9_0900d768","updated":"2022-01-21 15:18:04.000000000","message":"Moreover, the commit message just says that the change updates the installation command, which is the \"what\", even thought that part can be easily guessed by looking at the change itself. What\u0027s missing is the \"why\". Please explain the reason (which I guess is that the lvm2-lvmmetad service is not available anymore?)\nhttps://cbea.ms/git-commit/#why-not-how\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages","commit_id":"fc8bd23ff6012559fa3dffb0bdbccd342aee3d52"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"1cce3ef90ea536d1cf6162265f0b8e2dc8fd66cb","unresolved":true,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * Start the LVM metadata service and configure it to start when the"},{"line_number":25,"context_line":"     system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fb5fce0a_953eeaa6","side":"PARENT","line":24,"updated":"2022-07-19 16:17:12.000000000","message":"I don\u0027t recommend unilaterally deleting this section, because it is still valid when deploying RDO on hosts running CentOS 7. The lvmetad service was only removed as of CentOS 8. Perhaps a better approach would be to reword this sentence to indicate this step only applies when running CentOS 7.","commit_id":"de6d54108b4152beee952ff5789861811dbe0016"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"066b7ec73b71f67a79f713674033c9701f69888f","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * Start the LVM metadata service and configure it to start when the"},{"line_number":25,"context_line":"     system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"}],"source_content_type":"text/x-rst","patch_set":2,"id":"01b37481_99e8e70c","side":"PARENT","line":24,"in_reply_to":"fb5fce0a_953eeaa6","updated":"2022-07-21 03:52:47.000000000","message":"Done","commit_id":"de6d54108b4152beee952ff5789861811dbe0016"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"35595949cfc0e33440e95b02f7b846854d91359d","unresolved":true,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * For hosts running CentOS 7 start the LVM metadata service"},{"line_number":25,"context_line":"     and configure it to start when the system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"}],"source_content_type":"text/x-rst","patch_set":4,"id":"43ccc5e6_3e83303f","line":24,"range":{"start_line":24,"start_character":5,"end_line":24,"end_character":31},"updated":"2022-07-20 13:45:40.000000000","message":"nit: the content lgtm, but it would benefit from adding a comma:\n\n\"For hosts running CentOS 7, \"","commit_id":"cd1ff29e64f98641c0867412d38ad12ad16b17dd"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"44137598fccb19447e84cef7820023a38e5defcf","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * For hosts running CentOS 7 start the LVM metadata service"},{"line_number":25,"context_line":"     and configure it to start when the system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"}],"source_content_type":"text/x-rst","patch_set":4,"id":"b4038625_9e343a2f","line":24,"range":{"start_line":24,"start_character":5,"end_line":24,"end_character":31},"in_reply_to":"43ccc5e6_3e83303f","updated":"2022-07-21 03:52:59.000000000","message":"Done","commit_id":"cd1ff29e64f98641c0867412d38ad12ad16b17dd"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d798d20bb8a1953faa58a60115f4ffcf9405e7a3","unresolved":true,"context_lines":[{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * For hosts running CentOS 7, start the LVM metadata service"},{"line_number":25,"context_line":"     and configure it to start when the system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"6ecce066_b22adaf3","line":25,"updated":"2022-09-16 14:38:04.000000000","message":"I suggest adding \"This is not required for CentOS 8 or later, which comes with a version of LVM that does not use the lvmetad service.\"  (This will also address Vladislav\u0027s point about why you don\u0027t need it.)","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"52a76245533b4cd4022721da9bc8926462b8dca4","unresolved":false,"context_lines":[{"line_number":22,"context_line":"     .. end"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"   * For hosts running CentOS 7, start the LVM metadata service"},{"line_number":25,"context_line":"     and configure it to start when the system boots:"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"     .. code-block:: console"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"533f6c43_7ed13dea","line":25,"in_reply_to":"6ecce066_b22adaf3","updated":"2022-09-21 11:10:14.000000000","message":"Done","commit_id":"5f602dc568983d814a187aa40a21d5e764350601"}]}