)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"8ef7546484b75a27a61deeb16d8dc99abce1798d","unresolved":true,"context_lines":[{"line_number":19,"context_line":"safer to have an explicit parameter for when external"},{"line_number":20,"context_line":"ceph is used."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Id3e397d81dbca9a48d0456588784bcc20737093f"},{"line_number":23,"context_line":"Closes-Bug: #1986988"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5999754d_cc9a0d02","line":22,"updated":"2022-08-22 15:10:32.000000000","message":"maybe we also want\n\nDepends-On: https://review.opendev.org/c/openstack/tripleo-ansible/+/853791","commit_id":"f111a671d823e7e962922affc2f94a01e12dbe42"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"e2fb4b2446d58d183bcd33fab814f0385814d24f","unresolved":true,"context_lines":[{"line_number":19,"context_line":"safer to have an explicit parameter for when external"},{"line_number":20,"context_line":"ceph is used."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: Id3e397d81dbca9a48d0456588784bcc20737093f"},{"line_number":23,"context_line":"Closes-Bug: #1986988"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"341c80ba_23cb5be2","line":22,"in_reply_to":"5999754d_cc9a0d02","updated":"2022-08-22 15:11:28.000000000","message":"\u003e maybe we also want\n\u003e \n\u003e Depends-On: https://review.opendev.org/c/openstack/tripleo-ansible/+/853791\n\nWe have this Depends-On: I2651c2850debd8110da93df2adc5fd8768a00db0","commit_id":"f111a671d823e7e962922affc2f94a01e12dbe42"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5cc66e11667bf8a36cc2583aac66f490b780d70c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cda2563d_2409124c","updated":"2022-08-20 12:02:28.000000000","message":"I checked the combination of this patch and https://review.opendev.org/c/openstack/tripleo-ansible/+/853791 and they don\u0027t break the internal ceph deployment workflow.","commit_id":"e031058520ea36feff7e1956ae0d146ef5b19817"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"40d7d665b7665c49e0efcbc8bdc9ae15c2a4d459","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4f5fa584_d8cfbbcf","updated":"2022-08-19 18:51:37.000000000","message":"Thanks John!","commit_id":"e031058520ea36feff7e1956ae0d146ef5b19817"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"0577bc30644bf68f3960c871208e57066e2a936c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cf802d24_52921246","updated":"2022-08-20 07:19:06.000000000","message":"recheck","commit_id":"e031058520ea36feff7e1956ae0d146ef5b19817"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"e6293aa382f96acfe721cd56bc571aac862043ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e1043d04_397a3bb2","updated":"2022-08-19 17:48:40.000000000","message":"recheck","commit_id":"e031058520ea36feff7e1956ae0d146ef5b19817"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b1c9ea2a5631d2bc52f0654b852b9b0e809344ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"de81c877_b48df30b","updated":"2022-08-22 19:05:42.000000000","message":"recheck\n\nmy bad; i cherry-picked the predecessor patch too early","commit_id":"e031058520ea36feff7e1956ae0d146ef5b19817"}],"deployment/cephadm/ceph-base.yaml":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"e5a3fd78d27fb0515d8c8af0f0d9b2849d5675b3","unresolved":true,"context_lines":[{"line_number":613,"context_line":"        - name: ceph_base_external_deploy_task"},{"line_number":614,"context_line":"          when:"},{"line_number":615,"context_line":"            - (step | int) \u003d\u003d 3"},{"line_number":616,"context_line":"            - {get_param: DisableCephadm}"},{"line_number":617,"context_line":"          block:"},{"line_number":618,"context_line":"            - name: Pause cephadm"},{"line_number":619,"context_line":"              include_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2f16b0fc_76b9ad62","line":616,"updated":"2022-08-18 23:27:45.000000000","message":"not sure but looks like we want \"not external_ceph\" here as well?","commit_id":"c0322779e5b64e3188fa39c480e503e7dcf460ae"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"e3356a6fa496eb449d56a540428635d1fd9a5e4c","unresolved":false,"context_lines":[{"line_number":613,"context_line":"        - name: ceph_base_external_deploy_task"},{"line_number":614,"context_line":"          when:"},{"line_number":615,"context_line":"            - (step | int) \u003d\u003d 3"},{"line_number":616,"context_line":"            - {get_param: DisableCephadm}"},{"line_number":617,"context_line":"          block:"},{"line_number":618,"context_line":"            - name: Pause cephadm"},{"line_number":619,"context_line":"              include_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"04bb2444_06cf8b04","line":616,"in_reply_to":"2f16b0fc_76b9ad62","updated":"2022-08-19 06:52:14.000000000","message":"Done","commit_id":"c0322779e5b64e3188fa39c480e503e7dcf460ae"}]}