)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"639adc891aaefe847753db7b69d4c8665e063812","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0961aabd_e0bff2a6","updated":"2022-10-25 11:41:20.000000000","message":"James, WDYT?","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"},{"author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"change_message_id":"7092f0b045bc376eb54bf322634eb8ac8e8c512f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"82d6f894_44fd26fd","updated":"2022-10-25 06:52:12.000000000","message":"thanks Ollie for looking into this","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"0b2c846b477ef7e8270ed5416be43e1d4b8c3ad0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"41e395a4_e20f19fc","updated":"2022-10-25 15:16:16.000000000","message":"works for our multi-cell testing","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"}],"tripleoclient/v1/overcloud_cell.py":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"05fb9698439bc4aad96a7554be44625dce131942","unresolved":true,"context_lines":[{"line_number":42,"context_line":"                                              default\u003d\u0027overcloud\u0027))"},{"line_number":43,"context_line":"        parser.add_argument(\u0027--cell-stack\u0027, \u0027-e\u0027, metavar\u003d\u0027\u003ccell stack\u003e\u0027,"},{"line_number":44,"context_line":"                            help\u003d_(\u0027Name of the controller cell Heat stack to \u0027"},{"line_number":45,"context_line":"                                   \u0027export information from. Used in case of: \u0027"},{"line_number":46,"context_line":"                                   \u0027control plane stack -\u003e cell controller \u0027"},{"line_number":47,"context_line":"                                   \u0027stack -\u003e multiple compute stacks\u0027))"},{"line_number":48,"context_line":"        parser.add_argument(\u0027--output-file\u0027, \u0027-o\u0027, metavar\u003d\u0027\u003coutput file\u003e\u0027,"},{"line_number":49,"context_line":"                            help\u003d_(\u0027Name of the output file for the cell data \u0027"},{"line_number":50,"context_line":"                                   \u0027export. It will default to \"\u003cname\u003e.yaml\"\u0027))"}],"source_content_type":"text/x-python","patch_set":5,"id":"e9f2ca02_8b033943","line":47,"range":{"start_line":45,"start_character":61,"end_line":47,"end_character":68},"updated":"2022-10-25 09:12:58.000000000","message":"I expect (in a follow-up patch) we could deprecate this and use overcloud export for the \u0027cell controller stack -\u003e multiple compute stacks\u0027 export as it\u0027s a regular multi-stack deployment relationship.","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"639adc891aaefe847753db7b69d4c8665e063812","unresolved":false,"context_lines":[{"line_number":42,"context_line":"                                              default\u003d\u0027overcloud\u0027))"},{"line_number":43,"context_line":"        parser.add_argument(\u0027--cell-stack\u0027, \u0027-e\u0027, metavar\u003d\u0027\u003ccell stack\u003e\u0027,"},{"line_number":44,"context_line":"                            help\u003d_(\u0027Name of the controller cell Heat stack to \u0027"},{"line_number":45,"context_line":"                                   \u0027export information from. Used in case of: \u0027"},{"line_number":46,"context_line":"                                   \u0027control plane stack -\u003e cell controller \u0027"},{"line_number":47,"context_line":"                                   \u0027stack -\u003e multiple compute stacks\u0027))"},{"line_number":48,"context_line":"        parser.add_argument(\u0027--output-file\u0027, \u0027-o\u0027, metavar\u003d\u0027\u003coutput file\u003e\u0027,"},{"line_number":49,"context_line":"                            help\u003d_(\u0027Name of the output file for the cell data \u0027"},{"line_number":50,"context_line":"                                   \u0027export. It will default to \"\u003cname\u003e.yaml\"\u0027))"}],"source_content_type":"text/x-python","patch_set":5,"id":"fdbd2c5a_c69c021a","line":47,"range":{"start_line":45,"start_character":61,"end_line":47,"end_character":68},"in_reply_to":"e9f2ca02_8b033943","updated":"2022-10-25 11:41:20.000000000","message":"Done","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"426fc01a0b57d509aaa56deaa01e0355da4ba783","unresolved":false,"context_lines":[{"line_number":94,"context_line":"            working_dir \u003d parsed_args.working_dir"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        if not parsed_args.config_download_dir:"},{"line_number":97,"context_line":"            config_download_dir \u003d os.path.join(os.environ.get(\u0027HOME\u0027),"},{"line_number":98,"context_line":"                                               \"overcloud-deploy\","},{"line_number":99,"context_line":"                                               stack_to_export,"},{"line_number":100,"context_line":"                                               \u0027config-download\u0027)"},{"line_number":101,"context_line":"        else:"},{"line_number":102,"context_line":"            config_download_dir \u003d parsed_args.config_download_dir"}],"source_content_type":"text/x-python","patch_set":5,"id":"4fd197c7_26b151b4","line":99,"range":{"start_line":97,"start_character":47,"end_line":99,"end_character":63},"updated":"2022-10-25 08:54:14.000000000","message":"We\u0027d be able to replace this by get_default_working_dir, but I see the same implementation is used in other commands such as overcloud export so we can consider that in a separate patch.","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"c6aa5ac19c759b7e33a37e20364a159ccd79dc26","unresolved":false,"context_lines":[{"line_number":94,"context_line":"            working_dir \u003d parsed_args.working_dir"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        if not parsed_args.config_download_dir:"},{"line_number":97,"context_line":"            config_download_dir \u003d os.path.join(os.environ.get(\u0027HOME\u0027),"},{"line_number":98,"context_line":"                                               \"overcloud-deploy\","},{"line_number":99,"context_line":"                                               stack_to_export,"},{"line_number":100,"context_line":"                                               \u0027config-download\u0027)"},{"line_number":101,"context_line":"        else:"},{"line_number":102,"context_line":"            config_download_dir \u003d parsed_args.config_download_dir"}],"source_content_type":"text/x-python","patch_set":5,"id":"0aeea3e0_ed03c999","line":99,"range":{"start_line":97,"start_character":47,"end_line":99,"end_character":63},"in_reply_to":"4fd197c7_26b151b4","updated":"2022-10-25 09:07:09.000000000","message":"yes, I intentionally kept this consistent with overcloud export","commit_id":"bcafdf7114903e26c4c02a44c09bfbac53574d69"}]}