)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"598274062f9aa7f804bfe4ebc844df8326ed5260","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"95b6bde8_9998b78c","updated":"2021-11-08 16:05:01.000000000","message":"ah missed the cherry pick, yea this likely needs fixing","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"1750d95aeb3adc0d62d88f591a1e801280fac892","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c12f2797_99c69a2c","in_reply_to":"95b6bde8_9998b78c","updated":"2021-11-08 16:09:10.000000000","message":"I\u0027m on it, thx","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"}],"tripleoclient/v1/overcloud_restore.py":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"ec4e16759a96108d30a17293d55bb1c231c7235f","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__ + \".RestoreOvercloud\")"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"INVENTORY \u003d \u0027~/overcloud-deploy/overcloud/\u0027+constants.TRIPLEO_STATIC_INVENTORY"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class RestoreOvercloud(command.Command):"}],"source_content_type":"text/x-python","patch_set":1,"id":"c55d1522_97cac4ba","line":29,"range":{"start_line":29,"start_character":13,"end_line":29,"end_character":15},"updated":"2021-11-08 15:01:35.000000000","message":"Isn\u0027t there some extra processing to handle this.  Also do we want to make the overcloud stack name configurable in this instead of the file? or maybe both?","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"445728aaba56732f603000ce310ece6aad548689","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__ + \".RestoreOvercloud\")"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"INVENTORY \u003d \u0027~/overcloud-deploy/overcloud/\u0027+constants.TRIPLEO_STATIC_INVENTORY"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class RestoreOvercloud(command.Command):"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfef4cf1_0ff86880","line":29,"range":{"start_line":29,"start_character":13,"end_line":29,"end_character":15},"in_reply_to":"2307ffa0_b2cd5bd5","updated":"2021-11-08 15:12:47.000000000","message":"https://review.opendev.org/c/openstack/python-tripleoclient/+/816998/3/tripleoclient/v2/overcloud_delete.py#118","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"2b2c1015c031f67996a62c5673f5121bbcc040c9","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__ + \".RestoreOvercloud\")"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"INVENTORY \u003d \u0027~/overcloud-deploy/overcloud/\u0027+constants.TRIPLEO_STATIC_INVENTORY"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class RestoreOvercloud(command.Command):"}],"source_content_type":"text/x-python","patch_set":1,"id":"2307ffa0_b2cd5bd5","line":29,"range":{"start_line":29,"start_character":13,"end_line":29,"end_character":15},"in_reply_to":"29171250_e0fa89f4","updated":"2021-11-08 15:09:59.000000000","message":"I thought there was another constant for the THT inventory that took a stack name as a param to give you the correct location.  What i mean here is the ~/ handling needs to be evaulated I think?","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"65ef86f7bd16a017cff9307a512a6a591762a4fb","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__ + \".RestoreOvercloud\")"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"INVENTORY \u003d \u0027~/overcloud-deploy/overcloud/\u0027+constants.TRIPLEO_STATIC_INVENTORY"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class RestoreOvercloud(command.Command):"}],"source_content_type":"text/x-python","patch_set":1,"id":"29171250_e0fa89f4","line":29,"range":{"start_line":29,"start_character":13,"end_line":29,"end_character":15},"in_reply_to":"c55d1522_97cac4ba","updated":"2021-11-08 15:07:16.000000000","message":"The inventory can be configured with --inventory. line 42.\nAre you asking for some \"stackname\" parameter to configure the inventory file?","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"},{"author":{"_account_id":22954,"name":"Juan Badia Payno","email":"jbadiapa@redhat.com","username":"jbadiapa"},"change_message_id":"33df46261a0fa03ed7260bf510ff923d3674ccc6","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"LOG \u003d logging.getLogger(__name__ + \".RestoreOvercloud\")"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"INVENTORY \u003d \u0027~/overcloud-deploy/overcloud/\u0027+constants.TRIPLEO_STATIC_INVENTORY"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"class RestoreOvercloud(command.Command):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf2d615f_f555d77c","line":29,"range":{"start_line":29,"start_character":13,"end_line":29,"end_character":15},"in_reply_to":"dfef4cf1_0ff86880","updated":"2021-11-08 15:18:29.000000000","message":"I see,\nShould we merge this patch and do a follow up? as this patch is a cherry pick?","commit_id":"d71f64edddb0126d5aaad533d422c3fa8168c37b"}]}
