)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"92a4a89d71ada051664c68d81656ba6d0bbfde05","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This commit introduces a new dcmanager subcloud-backup command"},{"line_number":10,"context_line":"to restore from backup files for a subcloud or a group of subclouds"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Test Plan - WIP"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"1. PASS: Verify command without sysadmin password, ensure user is prompted"},{"line_number":15,"context_line":"2. PASS: Verfiy command with sysadmin password"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"10e472ce_d1b45784","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":15},"updated":"2022-10-14 01:37:50.000000000","message":"Please remove","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"4c22c31f7e9a60be3e939f1a622fcf22671c3aba","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This commit introduces a new dcmanager subcloud-backup command"},{"line_number":10,"context_line":"to restore from backup files for a subcloud or a group of subclouds"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Test Plan - WIP"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"1. PASS: Verify command without sysadmin password, ensure user is prompted"},{"line_number":15,"context_line":"2. PASS: Verfiy command with sysadmin password"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"eb558014_dfb4ce05","line":12,"range":{"start_line":12,"start_character":12,"end_line":12,"end_character":15},"in_reply_to":"10e472ce_d1b45784","updated":"2022-10-14 20:13:55.000000000","message":"Done","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"613d0a7bc0351e67a6cf20b6168911a717db4105","unresolved":true,"context_lines":[{"line_number":16,"context_line":"3. PASS: Verify command with subcloud option"},{"line_number":17,"context_line":"4. PASS: Verify command with group option"},{"line_number":18,"context_line":"5. PASS: Verfiy command with subcloud and group, ensure error msg is displayed"},{"line_number":19,"context_line":"6. Verify command with and without valid restore values file"},{"line_number":20,"context_line":"7. Verify command with and without local-only when combined with"},{"line_number":21,"context_line":"   or without registry-images"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Story: 2010116"},{"line_number":24,"context_line":"Task: 46222"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":23,"id":"a3810206_91bf334f","line":21,"range":{"start_line":19,"start_character":3,"end_line":21,"end_character":29},"updated":"2022-10-18 13:31:42.000000000","message":"PASS, or not run yet?","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"8280338686b33f06f953208ce2709c085468be5d","unresolved":false,"context_lines":[{"line_number":16,"context_line":"3. PASS: Verify command with subcloud option"},{"line_number":17,"context_line":"4. PASS: Verify command with group option"},{"line_number":18,"context_line":"5. PASS: Verfiy command with subcloud and group, ensure error msg is displayed"},{"line_number":19,"context_line":"6. Verify command with and without valid restore values file"},{"line_number":20,"context_line":"7. Verify command with and without local-only when combined with"},{"line_number":21,"context_line":"   or without registry-images"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Story: 2010116"},{"line_number":24,"context_line":"Task: 46222"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":23,"id":"e4d400b7_9c4550a3","line":21,"range":{"start_line":19,"start_character":3,"end_line":21,"end_character":29},"in_reply_to":"a3810206_91bf334f","updated":"2022-10-18 20:07:58.000000000","message":"It should be a pass","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"d8284cd7b8e76faecd397062f89a45bf27ac4520","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Test Plan:"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"1. Verify command without sysadmin password, ensure user is prompted"},{"line_number":15,"context_line":"2. Verfiy command with sysadmin password"},{"line_number":16,"context_line":"3. Verify command with subcloud option"},{"line_number":17,"context_line":"4. Verify command with group option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":26,"id":"0dae2d1d_095d82f4","line":14,"updated":"2022-10-19 12:52:27.000000000","message":"TestPlan should indicate result","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Test Plan:"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"1. Verify command without sysadmin password, ensure user is prompted"},{"line_number":15,"context_line":"2. Verfiy command with sysadmin password"},{"line_number":16,"context_line":"3. Verify command with subcloud option"},{"line_number":17,"context_line":"4. Verify command with group option"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":26,"id":"e1bdd43d_7dd53680","line":14,"in_reply_to":"0dae2d1d_095d82f4","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"58a3c351e94613f4672beda57397ec89a3b8178d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"75b157c7_b1b4f822","updated":"2022-09-13 20:50:22.000000000","message":"this needs to be rebased to resolve the merge conflict","commit_id":"d05836733c5ed08a1718bfb85103e0e7c927fa94"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"5f52c48456de6e19f2df1a1c6b44ff031c0c3e6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"572b74a4_84a94e8e","updated":"2022-09-21 12:55:29.000000000","message":"Looks okay to me, no vote since test plan is WIP","commit_id":"25b96bb84863b98df6f553b659670d04ffb229a8"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"ca51b4d8c1c6f46aaa332726af030bafab22588a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"0ad9f009_c29c6775","updated":"2022-10-24 15:49:05.000000000","message":"recheck","commit_id":"8279d2f59ca622d6a3ab4251c31aca04fb1cb8c6"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"cb63079698e4d0543a0337823fca6c050fe464a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"622813a1_ff3efef4","updated":"2022-10-25 13:31:01.000000000","message":"We need to add a new parameter: --bootstrap-address ","commit_id":"755b2ac0bb5f399239a0e2a36bd482f7fd174022"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"5acccdb4e89efb31d0d4058ec989db61dc861e19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"196f5c2b_8d38ded0","updated":"2022-10-25 13:29:06.000000000","message":"lgtm will +2 pending other reviewers","commit_id":"755b2ac0bb5f399239a0e2a36bd482f7fd174022"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"be7e9082c46bf3bee21e668f325cd8a2f4877c29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"002338d3_5091c5ab","updated":"2022-10-26 17:41:33.000000000","message":"recheck","commit_id":"6a585e0d657feac237f6ac7850e9bf4c80c5352f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"176e4460fd9da0416fcaa803522d00d702ade4d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"70ec048f_a71da5b2","updated":"2022-10-26 14:57:53.000000000","message":"recheck","commit_id":"6a585e0d657feac237f6ac7850e9bf4c80c5352f"}],"distributedcloud-client/dcmanagerclient/api/v1/subcloud_backup_manager.py":[{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"6a8f9bdf19f7b136e95236d5edcbfedc9b09a48e","unresolved":true,"context_lines":[{"line_number":41,"context_line":"            backup_datetime\u003djson_object[\u0027backup-datetime\u0027])"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def subcloud_backup_create(self, url, body, data):"},{"line_number":44,"context_line":"        if body:"},{"line_number":45,"context_line":"            fields \u003d dict()"},{"line_number":46,"context_line":"            for k, v in body.items():"},{"line_number":47,"context_line":"                fields.update({k: (v, open(v, \u0027rb\u0027),)})"},{"line_number":48,"context_line":"            fields.update(data)"},{"line_number":49,"context_line":"            enc \u003d MultipartEncoder(fields\u003dfields)"},{"line_number":50,"context_line":"            headers \u003d {\u0027Content-Type\u0027: enc.content_type}"}],"source_content_type":"text/x-python","patch_set":16,"id":"dc679a68_b3565d24","line":47,"range":{"start_line":44,"start_character":0,"end_line":47,"end_character":0},"updated":"2022-10-14 01:36:25.000000000","message":"Please rebase. See https://review.opendev.org/c/starlingx/distcloud-client/+/860725","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"4c22c31f7e9a60be3e939f1a622fcf22671c3aba","unresolved":false,"context_lines":[{"line_number":41,"context_line":"            backup_datetime\u003djson_object[\u0027backup-datetime\u0027])"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def subcloud_backup_create(self, url, body, data):"},{"line_number":44,"context_line":"        if body:"},{"line_number":45,"context_line":"            fields \u003d dict()"},{"line_number":46,"context_line":"            for k, v in body.items():"},{"line_number":47,"context_line":"                fields.update({k: (v, open(v, \u0027rb\u0027),)})"},{"line_number":48,"context_line":"            fields.update(data)"},{"line_number":49,"context_line":"            enc \u003d MultipartEncoder(fields\u003dfields)"},{"line_number":50,"context_line":"            headers \u003d {\u0027Content-Type\u0027: enc.content_type}"}],"source_content_type":"text/x-python","patch_set":16,"id":"2572a14c_92efcc7c","line":47,"range":{"start_line":44,"start_character":0,"end_line":47,"end_character":0},"in_reply_to":"dc679a68_b3565d24","updated":"2022-10-14 20:13:55.000000000","message":"Done","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"c18737e2801af02b4a614d003e57797adaf55baa","unresolved":true,"context_lines":[{"line_number":65,"context_line":"            return json.loads(resp.content)"},{"line_number":66,"context_line":"        return None"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def subcloud_backup_restore(self, url, body, data):"},{"line_number":69,"context_line":"        data \u003d json.dumps(data)"},{"line_number":70,"context_line":"        resp \u003d self.http_client.patch(url, data)"},{"line_number":71,"context_line":"        if resp.status_code not in [200, 204]:"}],"source_content_type":"text/x-python","patch_set":26,"id":"12fef3cc_052cb8bf","line":68,"updated":"2022-10-18 22:44:41.000000000","message":"We need to add the body info (see line 43).","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            return json.loads(resp.content)"},{"line_number":66,"context_line":"        return None"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    def subcloud_backup_restore(self, url, body, data):"},{"line_number":69,"context_line":"        data \u003d json.dumps(data)"},{"line_number":70,"context_line":"        resp \u003d self.http_client.patch(url, data)"},{"line_number":71,"context_line":"        if resp.status_code not in [200, 204]:"}],"source_content_type":"text/x-python","patch_set":26,"id":"7fc35ce7_843ac475","line":68,"in_reply_to":"12fef3cc_052cb8bf","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"eb812994f7158d0eccd7f1f856e2c17801fd9014","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            self._raise_api_exception(resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        resource \u003d []"},{"line_number":75,"context_line":"        if resp.status_code \u003d\u003d 200:"},{"line_number":76,"context_line":"            json_response_key \u003d get_json(resp)"},{"line_number":77,"context_line":"            json_objects \u003d json_response_key[\u0027subclouds\u0027]"},{"line_number":78,"context_line":"            for json_object in json_objects:"}],"source_content_type":"text/x-python","patch_set":26,"id":"4625b0f3_7ef7c2ef","line":75,"updated":"2022-10-19 00:12:43.000000000","message":"Do we only want to accept 200?","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"5770fa03ced70767ed38a27a494acaa9e603fda3","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            self._raise_api_exception(resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        resource \u003d []"},{"line_number":75,"context_line":"        if resp.status_code \u003d\u003d 200:"},{"line_number":76,"context_line":"            json_response_key \u003d get_json(resp)"},{"line_number":77,"context_line":"            json_objects \u003d json_response_key[\u0027subclouds\u0027]"},{"line_number":78,"context_line":"            for json_object in json_objects:"}],"source_content_type":"text/x-python","patch_set":26,"id":"17bf450e_a356d006","line":75,"in_reply_to":"0db84dc7_8a085a87","updated":"2022-10-24 17:47:17.000000000","message":"latest patchset (#29) appears to only accept 200 (not 204, etc) ?","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"62368ac8071a392de27c0b386fc800f938b60070","unresolved":true,"context_lines":[{"line_number":72,"context_line":"            self._raise_api_exception(resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        resource \u003d []"},{"line_number":75,"context_line":"        if resp.status_code \u003d\u003d 200:"},{"line_number":76,"context_line":"            json_response_key \u003d get_json(resp)"},{"line_number":77,"context_line":"            json_objects \u003d json_response_key[\u0027subclouds\u0027]"},{"line_number":78,"context_line":"            for json_object in json_objects:"}],"source_content_type":"text/x-python","patch_set":26,"id":"40781845_738ba72b","line":75,"in_reply_to":"17bf450e_a356d006","updated":"2022-10-24 20:05:29.000000000","message":"Yes, I changed it to keep the behavior consistent with that of the subcloud-backup create operation.","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"0d6ceed0602cacc1c8cf72024e8c0778001888a8","unresolved":false,"context_lines":[{"line_number":72,"context_line":"            self._raise_api_exception(resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        resource \u003d []"},{"line_number":75,"context_line":"        if resp.status_code \u003d\u003d 200:"},{"line_number":76,"context_line":"            json_response_key \u003d get_json(resp)"},{"line_number":77,"context_line":"            json_objects \u003d json_response_key[\u0027subclouds\u0027]"},{"line_number":78,"context_line":"            for json_object in json_objects:"}],"source_content_type":"text/x-python","patch_set":26,"id":"d4a4db1f_1c5c7748","line":75,"in_reply_to":"40781845_738ba72b","updated":"2022-10-25 15:23:46.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":72,"context_line":"            self._raise_api_exception(resp)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        resource \u003d []"},{"line_number":75,"context_line":"        if resp.status_code \u003d\u003d 200:"},{"line_number":76,"context_line":"            json_response_key \u003d get_json(resp)"},{"line_number":77,"context_line":"            json_objects \u003d json_response_key[\u0027subclouds\u0027]"},{"line_number":78,"context_line":"            for json_object in json_objects:"}],"source_content_type":"text/x-python","patch_set":26,"id":"0db84dc7_8a085a87","line":75,"in_reply_to":"4625b0f3_7ef7c2ef","updated":"2022-10-19 14:02:42.000000000","message":"No, a few lines above an exception is raised if the status_code is not 200 or 204. The content is only extracted from responses with code 200, 204 means no content.","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"}],"distributedcloud-client/dcmanagerclient/commands/v1/subcloud_backup_manager.py":[{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"6a8f9bdf19f7b136e95236d5edcbfedc9b09a48e","unresolved":true,"context_lines":[{"line_number":304,"context_line":"        parser.add_argument("},{"line_number":305,"context_line":"            \u0027--local-only\u0027,"},{"line_number":306,"context_line":"            required\u003dFalse,"},{"line_number":307,"context_line":"            help\u003d\u0027If included, backup files will be stored on the subcloud. \u0027"},{"line_number":308,"context_line":"                 \u0027Otherwise, they will be transferred and stored in \u0027"},{"line_number":309,"context_line":"                 \u0027dedicated location on the system controller.\u0027"},{"line_number":310,"context_line":"        )"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":16,"id":"547e5cac_5d66012f","line":309,"range":{"start_line":307,"start_character":18,"end_line":309,"end_character":63},"updated":"2022-10-14 01:36:25.000000000","message":"Need to update text for restore, looks like a copy and paste from backup\n\ne.g. If included, the subcloud will be restored from backup data stored on t subcloud. Otherwise, it will be restored from backup data stored in....\n\nPlease review all help messages in this file.","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"4c22c31f7e9a60be3e939f1a622fcf22671c3aba","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        parser.add_argument("},{"line_number":305,"context_line":"            \u0027--local-only\u0027,"},{"line_number":306,"context_line":"            required\u003dFalse,"},{"line_number":307,"context_line":"            help\u003d\u0027If included, backup files will be stored on the subcloud. \u0027"},{"line_number":308,"context_line":"                 \u0027Otherwise, they will be transferred and stored in \u0027"},{"line_number":309,"context_line":"                 \u0027dedicated location on the system controller.\u0027"},{"line_number":310,"context_line":"        )"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":16,"id":"43585d1c_8f5d6daf","line":309,"range":{"start_line":307,"start_character":18,"end_line":309,"end_character":63},"in_reply_to":"547e5cac_5d66012f","updated":"2022-10-14 20:13:55.000000000","message":"Done","commit_id":"61a70827cff35908c8f569257acc747bbff9dbfe"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"613d0a7bc0351e67a6cf20b6168911a717db4105","unresolved":true,"context_lines":[{"line_number":301,"context_line":"            required\u003dFalse,"},{"line_number":302,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":303,"context_line":"            help\u003d\u0027If present (or equal to true), remote install \u0027"},{"line_number":304,"context_line":"                 \u0027subcloud before proceding with restore.\u0027"},{"line_number":305,"context_line":"        )"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":23,"id":"c0a4e4a4_3c1f6a78","line":304,"range":{"start_line":304,"start_character":34,"end_line":304,"end_character":44},"updated":"2022-10-18 13:31:42.000000000","message":"proceeding","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"8280338686b33f06f953208ce2709c085468be5d","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            required\u003dFalse,"},{"line_number":302,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":303,"context_line":"            help\u003d\u0027If present (or equal to true), remote install \u0027"},{"line_number":304,"context_line":"                 \u0027subcloud before proceding with restore.\u0027"},{"line_number":305,"context_line":"        )"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":23,"id":"eda366f9_12b21c59","line":304,"range":{"start_line":304,"start_character":34,"end_line":304,"end_character":44},"in_reply_to":"c0a4e4a4_3c1f6a78","updated":"2022-10-18 20:07:58.000000000","message":"Done","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"613d0a7bc0351e67a6cf20b6168911a717db4105","unresolved":true,"context_lines":[{"line_number":326,"context_line":"            \u0027--restore-values\u0027,"},{"line_number":327,"context_line":"            required\u003dFalse,"},{"line_number":328,"context_line":"            help\u003d\u0027Reference to the restore playbook overrides yaml file, as \u0027"},{"line_number":329,"context_line":"                 \u0027listed in the product documentation for the ansible restore\u0027"},{"line_number":330,"context_line":"        )"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":23,"id":"5234e7af_df76dd2c","line":329,"range":{"start_line":329,"start_character":77,"end_line":329,"end_character":78},"updated":"2022-10-18 13:31:42.000000000","message":"nit: end with \u0027.\u0027 for consistency","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"8280338686b33f06f953208ce2709c085468be5d","unresolved":false,"context_lines":[{"line_number":326,"context_line":"            \u0027--restore-values\u0027,"},{"line_number":327,"context_line":"            required\u003dFalse,"},{"line_number":328,"context_line":"            help\u003d\u0027Reference to the restore playbook overrides yaml file, as \u0027"},{"line_number":329,"context_line":"                 \u0027listed in the product documentation for the ansible restore\u0027"},{"line_number":330,"context_line":"        )"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":23,"id":"6005d40a_6957708f","line":329,"range":{"start_line":329,"start_character":77,"end_line":329,"end_character":78},"in_reply_to":"5234e7af_df76dd2c","updated":"2022-10-18 20:07:58.000000000","message":"Done","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":33549,"name":"Kyle MacLeod","email":"Kyle.MacLeod@windriver.com","username":"kmacleod"},"change_message_id":"613d0a7bc0351e67a6cf20b6168911a717db4105","unresolved":true,"context_lines":[{"line_number":373,"context_line":"            data[\u0027group\u0027] \u003d parsed_args.group"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        if not parsed_args.local_only and parsed_args.registry_images:"},{"line_number":376,"context_line":"            error_msg \u003d (\u0027Option --registry-images can not be used without \u0027"},{"line_number":377,"context_line":"                         \u0027--local-only option.\u0027)"},{"line_number":378,"context_line":"            raise exceptions.DCManagerClientException(error_msg)"},{"line_number":379,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"2c038eb2_71d0296e","line":376,"range":{"start_line":376,"start_character":51,"end_line":376,"end_character":58},"updated":"2022-10-18 13:31:42.000000000","message":"nit: cannot","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"8280338686b33f06f953208ce2709c085468be5d","unresolved":false,"context_lines":[{"line_number":373,"context_line":"            data[\u0027group\u0027] \u003d parsed_args.group"},{"line_number":374,"context_line":""},{"line_number":375,"context_line":"        if not parsed_args.local_only and parsed_args.registry_images:"},{"line_number":376,"context_line":"            error_msg \u003d (\u0027Option --registry-images can not be used without \u0027"},{"line_number":377,"context_line":"                         \u0027--local-only option.\u0027)"},{"line_number":378,"context_line":"            raise exceptions.DCManagerClientException(error_msg)"},{"line_number":379,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"8bd4af4d_794b94b9","line":376,"range":{"start_line":376,"start_character":51,"end_line":376,"end_character":58},"in_reply_to":"2c038eb2_71d0296e","updated":"2022-10-18 20:07:58.000000000","message":"Done","commit_id":"5f2c9c8a44525e54b5089ed5bc684cf33f65d4bf"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"d8284cd7b8e76faecd397062f89a45bf27ac4520","unresolved":true,"context_lines":[{"line_number":288,"context_line":""},{"line_number":289,"context_line":""},{"line_number":290,"context_line":"class RestoreSubcloudBackup(base.DCManagerShowOne):"},{"line_number":291,"context_line":"    \"\"\"Restore a subcloud or group of subcloud from a backup\"\"\""},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"    def _get_format_function(self):"},{"line_number":294,"context_line":"        return detail_format"}],"source_content_type":"text/x-python","patch_set":26,"id":"b828b866_8f806216","line":291,"range":{"start_line":291,"start_character":38,"end_line":291,"end_character":60},"updated":"2022-10-19 12:52:27.000000000","message":"subclouds from backup\n\n(It would require more than \u0027a backup\u0027 to restore multiple subclouds)","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":288,"context_line":""},{"line_number":289,"context_line":""},{"line_number":290,"context_line":"class RestoreSubcloudBackup(base.DCManagerShowOne):"},{"line_number":291,"context_line":"    \"\"\"Restore a subcloud or group of subcloud from a backup\"\"\""},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"    def _get_format_function(self):"},{"line_number":294,"context_line":"        return detail_format"}],"source_content_type":"text/x-python","patch_set":26,"id":"7b833092_be9f5baa","line":291,"range":{"start_line":291,"start_character":38,"end_line":291,"end_character":60},"in_reply_to":"b828b866_8f806216","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":300,"context_line":"            \u0027--with-install\u0027,"},{"line_number":301,"context_line":"            required\u003dFalse,"},{"line_number":302,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":303,"context_line":"            help\u003d\u0027If present (or equal to true), remote install \u0027"},{"line_number":304,"context_line":"                 \u0027subcloud before proceeding with restore.\u0027"},{"line_number":305,"context_line":"        )"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"a2968a83_db1451aa","line":304,"range":{"start_line":303,"start_character":17,"end_line":304,"end_character":59},"updated":"2022-10-19 00:22:22.000000000","message":"If included, the subcloud will be reinstalled prior to being restored from backup data.","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":300,"context_line":"            \u0027--with-install\u0027,"},{"line_number":301,"context_line":"            required\u003dFalse,"},{"line_number":302,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":303,"context_line":"            help\u003d\u0027If present (or equal to true), remote install \u0027"},{"line_number":304,"context_line":"                 \u0027subcloud before proceeding with restore.\u0027"},{"line_number":305,"context_line":"        )"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"fe835385_8e3b9963","line":304,"range":{"start_line":303,"start_character":17,"end_line":304,"end_character":59},"in_reply_to":"a2968a83_db1451aa","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored in the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"1132fcc3_561db8d5","line":313,"range":{"start_line":313,"start_character":37,"end_line":313,"end_character":39},"updated":"2022-10-19 00:22:22.000000000","message":"on","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored in the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"b00ebf7b_76812f46","line":313,"range":{"start_line":313,"start_character":37,"end_line":313,"end_character":39},"in_reply_to":"1132fcc3_561db8d5","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"d8284cd7b8e76faecd397062f89a45bf27ac4520","unresolved":true,"context_lines":[{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("},{"line_number":317,"context_line":"            \u0027--registry-images\u0027,"},{"line_number":318,"context_line":"            required\u003dFalse,"},{"line_number":319,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":320,"context_line":"            help\u003d\u0027If included, container images backup file will also be \u0027"}],"source_content_type":"text/x-python","patch_set":26,"id":"994deb8c_e96aab28","line":317,"range":{"start_line":317,"start_character":15,"end_line":317,"end_character":30},"updated":"2022-10-19 12:52:27.000000000","message":"should this be --user-registry-images ?","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("},{"line_number":317,"context_line":"            \u0027--registry-images\u0027,"},{"line_number":318,"context_line":"            required\u003dFalse,"},{"line_number":319,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":320,"context_line":"            help\u003d\u0027If included, container images backup file will also be \u0027"}],"source_content_type":"text/x-python","patch_set":26,"id":"526164fb_dcbf40d1","line":317,"range":{"start_line":317,"start_character":15,"end_line":317,"end_character":30},"in_reply_to":"994deb8c_e96aab28","updated":"2022-10-19 14:02:42.000000000","message":"Not according to the HLD, https://confluence.wrs.com/display/CE/TCPG-1085+-+Centralized+Subcloud+Backup+and+Restore","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":317,"context_line":"            \u0027--registry-images\u0027,"},{"line_number":318,"context_line":"            required\u003dFalse,"},{"line_number":319,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":320,"context_line":"            help\u003d\u0027If included, container images backup file will also be \u0027"},{"line_number":321,"context_line":"                 \u0027generated. This option can only be used with --local-only \u0027"},{"line_number":322,"context_line":"                 \u0027option.\u0027"},{"line_number":323,"context_line":"        )"},{"line_number":324,"context_line":""}],"source_content_type":"text/x-python","patch_set":26,"id":"fb3187ef_a361de10","line":321,"range":{"start_line":320,"start_character":17,"end_line":321,"end_character":28},"updated":"2022-10-19 00:22:22.000000000","message":"If included, user images will be restored post platform restore.","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":317,"context_line":"            \u0027--registry-images\u0027,"},{"line_number":318,"context_line":"            required\u003dFalse,"},{"line_number":319,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":320,"context_line":"            help\u003d\u0027If included, container images backup file will also be \u0027"},{"line_number":321,"context_line":"                 \u0027generated. This option can only be used with --local-only \u0027"},{"line_number":322,"context_line":"                 \u0027option.\u0027"},{"line_number":323,"context_line":"        )"},{"line_number":324,"context_line":""}],"source_content_type":"text/x-python","patch_set":26,"id":"eb5342ff_67753d40","line":321,"range":{"start_line":320,"start_character":17,"end_line":321,"end_character":28},"in_reply_to":"fb3187ef_a361de10","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":332,"context_line":"        parser.add_argument("},{"line_number":333,"context_line":"            \u0027--sysadmin-password\u0027,"},{"line_number":334,"context_line":"            required\u003dFalse,"},{"line_number":335,"context_line":"            help\u003d\u0027sysadmin password of the subcloud to restore backup, \u0027"},{"line_number":336,"context_line":"                 \u0027if not provided you will be prompted.\u0027"},{"line_number":337,"context_line":"        )"},{"line_number":338,"context_line":""}],"source_content_type":"text/x-python","patch_set":26,"id":"703b313e_70ba3007","line":335,"range":{"start_line":335,"start_character":52,"end_line":335,"end_character":69},"updated":"2022-10-19 00:22:22.000000000","message":"to be restored","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":332,"context_line":"        parser.add_argument("},{"line_number":333,"context_line":"            \u0027--sysadmin-password\u0027,"},{"line_number":334,"context_line":"            required\u003dFalse,"},{"line_number":335,"context_line":"            help\u003d\u0027sysadmin password of the subcloud to restore backup, \u0027"},{"line_number":336,"context_line":"                 \u0027if not provided you will be prompted.\u0027"},{"line_number":337,"context_line":"        )"},{"line_number":338,"context_line":""}],"source_content_type":"text/x-python","patch_set":26,"id":"2c8e0736_bd2860fb","line":335,"range":{"start_line":335,"start_character":52,"end_line":335,"end_character":69},"in_reply_to":"703b313e_70ba3007","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":339,"context_line":"        parser.add_argument("},{"line_number":340,"context_line":"            \u0027--subcloud\u0027,"},{"line_number":341,"context_line":"            required\u003dFalse,"},{"line_number":342,"context_line":"            help\u003d\u0027Name or ID of the subcloud to restore backup.\u0027"},{"line_number":343,"context_line":"        )"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"fa61e881_21d1008e","line":342,"range":{"start_line":342,"start_character":45,"end_line":342,"end_character":62},"updated":"2022-10-19 00:22:22.000000000","message":"to restore","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":339,"context_line":"        parser.add_argument("},{"line_number":340,"context_line":"            \u0027--subcloud\u0027,"},{"line_number":341,"context_line":"            required\u003dFalse,"},{"line_number":342,"context_line":"            help\u003d\u0027Name or ID of the subcloud to restore backup.\u0027"},{"line_number":343,"context_line":"        )"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":26,"id":"8663fb14_cb66b990","line":342,"range":{"start_line":342,"start_character":45,"end_line":342,"end_character":62},"in_reply_to":"fa61e881_21d1008e","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"00eebd2e2a764866ef013b526b81a6b36e4a0c0c","unresolved":true,"context_lines":[{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \u0027--group\u0027,"},{"line_number":347,"context_line":"            required\u003dFalse,"},{"line_number":348,"context_line":"            help\u003d\u0027Name or ID of the subcloud to restore backup.\u0027"},{"line_number":349,"context_line":"        )"},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":26,"id":"89f64090_723427b7","line":348,"range":{"start_line":348,"start_character":36,"end_line":348,"end_character":62},"updated":"2022-10-19 00:22:22.000000000","message":"subcloud group to restore.","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":345,"context_line":"        parser.add_argument("},{"line_number":346,"context_line":"            \u0027--group\u0027,"},{"line_number":347,"context_line":"            required\u003dFalse,"},{"line_number":348,"context_line":"            help\u003d\u0027Name or ID of the subcloud to restore backup.\u0027"},{"line_number":349,"context_line":"        )"},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":26,"id":"c1643964_30b746c3","line":348,"range":{"start_line":348,"start_character":36,"end_line":348,"end_character":62},"in_reply_to":"89f64090_723427b7","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"c18737e2801af02b4a614d003e57797adaf55baa","unresolved":true,"context_lines":[{"line_number":404,"context_line":"                error_msg \u003d \"Restore_values file does not exist: %s\" % \\"},{"line_number":405,"context_line":"                            parsed_args.restore_values"},{"line_number":406,"context_line":"                raise exceptions.DCManagerClientException(error_msg)"},{"line_number":407,"context_line":"            files[\u0027restore_values\u0027] \u003d parsed_args.restore_values"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"        try:"},{"line_number":410,"context_line":"            return dcmanager_client.subcloud_backup_manager.\\"}],"source_content_type":"text/x-python","patch_set":26,"id":"0f6d2995_86daca7e","line":407,"updated":"2022-10-18 22:44:41.000000000","message":"Use utils.load_file(parsed_args.restore_values)","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a414fd0c17795d06db1ca2d400a64ff114abe202","unresolved":false,"context_lines":[{"line_number":404,"context_line":"                error_msg \u003d \"Restore_values file does not exist: %s\" % \\"},{"line_number":405,"context_line":"                            parsed_args.restore_values"},{"line_number":406,"context_line":"                raise exceptions.DCManagerClientException(error_msg)"},{"line_number":407,"context_line":"            files[\u0027restore_values\u0027] \u003d parsed_args.restore_values"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"        try:"},{"line_number":410,"context_line":"            return dcmanager_client.subcloud_backup_manager.\\"}],"source_content_type":"text/x-python","patch_set":26,"id":"4dd55bea_4ec26944","line":407,"in_reply_to":"0f6d2995_86daca7e","updated":"2022-10-19 14:02:42.000000000","message":"Done","commit_id":"d1cf4815bdf14848b6a6369747308db274cdea2f"},{"author":{"_account_id":14675,"name":"Hugo Nicodemos","email":"hugo.brito@windriver.com","username":"nicodemos"},"change_message_id":"282076af6ab9842ec3d29ed36cadb8b1fd65b1bc","unresolved":true,"context_lines":[{"line_number":378,"context_line":"            if parsed_args.bootstrap_address:"},{"line_number":379,"context_line":"                data[\u0027bootstrap_address\u0027] \u003d parsed_args.bootstrap_address"},{"line_number":380,"context_line":"            else:"},{"line_number":381,"context_line":"                error_msg \u003d (\u0027Option --bootstrap-address is required to \u0027"},{"line_number":382,"context_line":"                             \u0027restore a subcloud.\u0027)"},{"line_number":383,"context_line":"                raise exceptions.DCManagerClientException(error_msg)"},{"line_number":384,"context_line":""}],"source_content_type":"text/x-python","patch_set":30,"id":"3c05b706_04eca735","line":381,"updated":"2022-10-25 18:55:31.000000000","message":"This verification should be done in the API side, this is only required if the subcloud doesn\u0027t have the data_install.","commit_id":"005bae21937266fad8e822da4dbffc96a554c210"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"f6cf0104bd342f92b0d6df2bab59d30101bbd578","unresolved":true,"context_lines":[{"line_number":378,"context_line":"            if parsed_args.bootstrap_address:"},{"line_number":379,"context_line":"                data[\u0027bootstrap_address\u0027] \u003d parsed_args.bootstrap_address"},{"line_number":380,"context_line":"            else:"},{"line_number":381,"context_line":"                error_msg \u003d (\u0027Option --bootstrap-address is required to \u0027"},{"line_number":382,"context_line":"                             \u0027restore a subcloud.\u0027)"},{"line_number":383,"context_line":"                raise exceptions.DCManagerClientException(error_msg)"},{"line_number":384,"context_line":""}],"source_content_type":"text/x-python","patch_set":30,"id":"ae9047e2_b6dbe7a7","line":381,"in_reply_to":"3c05b706_04eca735","updated":"2022-10-27 16:22:07.000000000","message":"I removed it since the boostrap-address parameter is no longer needed.","commit_id":"005bae21937266fad8e822da4dbffc96a554c210"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"fb77ef8f5ff0821421ddbd5bd92cc9e8640edb1c","unresolved":false,"context_lines":[{"line_number":378,"context_line":"            if parsed_args.bootstrap_address:"},{"line_number":379,"context_line":"                data[\u0027bootstrap_address\u0027] \u003d parsed_args.bootstrap_address"},{"line_number":380,"context_line":"            else:"},{"line_number":381,"context_line":"                error_msg \u003d (\u0027Option --bootstrap-address is required to \u0027"},{"line_number":382,"context_line":"                             \u0027restore a subcloud.\u0027)"},{"line_number":383,"context_line":"                raise exceptions.DCManagerClientException(error_msg)"},{"line_number":384,"context_line":""}],"source_content_type":"text/x-python","patch_set":30,"id":"b2a59e65_58f175a8","line":381,"in_reply_to":"ae9047e2_b6dbe7a7","updated":"2022-10-27 16:22:20.000000000","message":"Done","commit_id":"005bae21937266fad8e822da4dbffc96a554c210"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"02bdee5f9bfc48b113156489113b3c7c1465affa","unresolved":true,"context_lines":[{"line_number":309,"context_line":"            required\u003dFalse,"},{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored on the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":37,"id":"f7c7adb2_56064b4d","line":312,"range":{"start_line":312,"start_character":18,"end_line":312,"end_character":24},"updated":"2022-10-27 19:10:35.000000000","message":"perhaps change to \"previously saved\"?","commit_id":"6c5ca654f83e9a8f917cffc274413b3488c186fa"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a28f6cfb8a8f94afbe4141c6924ee3ad901595da","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            required\u003dFalse,"},{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored on the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""}],"source_content_type":"text/x-python","patch_set":37,"id":"87cf52c0_dff6994c","line":312,"range":{"start_line":312,"start_character":18,"end_line":312,"end_character":24},"in_reply_to":"f7c7adb2_56064b4d","updated":"2022-10-27 19:38:31.000000000","message":"Done","commit_id":"6c5ca654f83e9a8f917cffc274413b3488c186fa"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"02bdee5f9bfc48b113156489113b3c7c1465affa","unresolved":true,"context_lines":[{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored on the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":37,"id":"6f9905b1_99c30760","line":313,"range":{"start_line":313,"start_character":30,"end_line":313,"end_character":36},"updated":"2022-10-27 19:10:35.000000000","message":"ditto","commit_id":"6c5ca654f83e9a8f917cffc274413b3488c186fa"},{"author":{"_account_id":35155,"name":"André Dexheimer Carneiro","email":"Andre.DexheimerCarneiro@windriver.com","username":"andre-dexheimercarneiro"},"change_message_id":"a28f6cfb8a8f94afbe4141c6924ee3ad901595da","unresolved":false,"context_lines":[{"line_number":310,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":311,"context_line":"            help\u003d\u0027If included, the subcloud will be restored from backup data \u0027"},{"line_number":312,"context_line":"                 \u0027stored on the subcloud. Otherwise, it will be restored from \u0027"},{"line_number":313,"context_line":"                 \u0027backup data stored on the system controller.\u0027"},{"line_number":314,"context_line":"        )"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":37,"id":"5b93a865_3840b1ec","line":313,"range":{"start_line":313,"start_character":30,"end_line":313,"end_character":36},"in_reply_to":"6f9905b1_99c30760","updated":"2022-10-27 19:38:31.000000000","message":"Done","commit_id":"6c5ca654f83e9a8f917cffc274413b3488c186fa"}]}