)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"5d8978577656ea960bae54f8d88eb3892bc1aaab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"715d1bd8_4d0cf373","updated":"2022-12-05 12:55:25.000000000","message":"In order to solve this bug quickly I\u0027d like to merge what we have but then follow up using ansible.inventory.manager","commit_id":"4b533d8b11307e64ae8218a7788e089f0d89ff69"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"b03fa0f29fdcef147a35008e168bb221c9d51399","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"559c1a2f_7c190ae1","updated":"2022-12-02 21:14:07.000000000","message":"Let me see if I can update to borrow the ideas in https://review.opendev.org/c/openstack/tripleo-ansible/+/866410","commit_id":"4b533d8b11307e64ae8218a7788e089f0d89ff69"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3f7538774871e1d6b4118b2a3b1beecdb00d62f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3171f710_ce6a8386","updated":"2022-12-05 13:00:14.000000000","message":"looks sane/didn\u0027t spot something ","commit_id":"4b533d8b11307e64ae8218a7788e089f0d89ff69"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1593abc7a4b5e77f29ff4b8e13373eff7856e9cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c860b82a_e9d055ab","updated":"2022-12-05 16:09:46.000000000","message":"currently in check so should go gate as soon as it completes\n\nhttps://zuul.openstack.org/status#866503","commit_id":"04c33a4747769cc414d163fb25cd0f8927ba9206"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"e1e9964165026104e523b25315203ee776ae993c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"34332ac5_370b56af","updated":"2022-12-05 20:40:50.000000000","message":"recheck","commit_id":"04c33a4747769cc414d163fb25cd0f8927ba9206"}],"tripleo_ansible/ansible_plugins/modules/ceph_spec_bootstrap.py":[{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"8b5576dcabcd473fd3af945831d50d774f989c24","unresolved":true,"context_lines":[{"line_number":189,"context_line":"    hosts_to_ips \u003d {}"},{"line_number":190,"context_line":"    for key in inventory:"},{"line_number":191,"context_line":"        if key in roles:"},{"line_number":192,"context_line":"            if \u0027children\u0027 in inventory[key] and \u0027hosts\u0027 in inventory[key]:"},{"line_number":193,"context_line":"                pass"},{"line_number":194,"context_line":"            if \u0027children\u0027 in inventory[key] and \u0027hosts\u0027 not in inventory[key]:"},{"line_number":195,"context_line":"                # e.g. if CephStorage has children (e.g. overcloud_CephStorage)"},{"line_number":196,"context_line":"                # then set the key to overcloud_CephStorage so we get its hosts"}],"source_content_type":"text/x-python","patch_set":1,"id":"69dc182c_1953bb8c","line":193,"range":{"start_line":192,"start_character":12,"end_line":193,"end_character":20},"updated":"2022-12-05 13:47:47.000000000","message":"Do we need this statement here? wondering if we can just omit this `if` here since we\u0027re not doing any action in case the condition is met","commit_id":"4b533d8b11307e64ae8218a7788e089f0d89ff69"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"39ded4c764e80028b7fa742b2243f0a9af334387","unresolved":false,"context_lines":[{"line_number":189,"context_line":"    hosts_to_ips \u003d {}"},{"line_number":190,"context_line":"    for key in inventory:"},{"line_number":191,"context_line":"        if key in roles:"},{"line_number":192,"context_line":"            if \u0027children\u0027 in inventory[key] and \u0027hosts\u0027 in inventory[key]:"},{"line_number":193,"context_line":"                pass"},{"line_number":194,"context_line":"            if \u0027children\u0027 in inventory[key] and \u0027hosts\u0027 not in inventory[key]:"},{"line_number":195,"context_line":"                # e.g. if CephStorage has children (e.g. overcloud_CephStorage)"},{"line_number":196,"context_line":"                # then set the key to overcloud_CephStorage so we get its hosts"}],"source_content_type":"text/x-python","patch_set":1,"id":"e1370409_35679f27","line":193,"range":{"start_line":192,"start_character":12,"end_line":193,"end_character":20},"in_reply_to":"69dc182c_1953bb8c","updated":"2022-12-05 14:24:43.000000000","message":"Done","commit_id":"4b533d8b11307e64ae8218a7788e089f0d89ff69"}]}