)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"49f027f6ea22d79bb0d7830165b4fc9379cb0a9e","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Shrikumar Sharma \u003cshrikumar.sharma@windriver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-15 15:27:55 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Handle exceptions in sysinv when sysinv fails due to missing docker"},{"line_number":8,"context_line":"service parameters"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When the ghcr docker service parameter is missing sysinv fails to"},{"line_number":11,"context_line":"download images and spams the logs with an exception. This"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d060ffb5_2ae893e1","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":18},"updated":"2021-11-15 21:17:35.000000000","message":"title should fit within a line","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Shrikumar Sharma \u003cshrikumar.sharma@windriver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-15 15:27:55 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Handle exceptions in sysinv when sysinv fails due to missing docker"},{"line_number":8,"context_line":"service parameters"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When the ghcr docker service parameter is missing sysinv fails to"},{"line_number":11,"context_line":"download images and spams the logs with an exception. This"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"21dda4ba_996849fe","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":18},"in_reply_to":"d060ffb5_2ae893e1","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"49f027f6ea22d79bb0d7830165b4fc9379cb0a9e","unresolved":true,"context_lines":[{"line_number":19,"context_line":"The code improvement here, is to initialize the local variables"},{"line_number":20,"context_line":"to None, and return gracefully with rc set to False (if an issue"},{"line_number":21,"context_line":"occurs with respect to these variables), and an error message."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Closes Bug: 1951014"},{"line_number":24,"context_line":"Change-Id: I77db038d2f33fd42b9b8963fc6b31ebb0c425f6c"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a3835a8_dd5b32e5","line":22,"updated":"2021-11-15 21:17:35.000000000","message":"Please include \"Test Plan\" as per:  https://docs.starlingx.io/developer_resources/code-submission-guide.html#pre-review-and-pre-submission-testing","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"The code improvement here, is to initialize the local variables"},{"line_number":20,"context_line":"to None, and return gracefully with rc set to False (if an issue"},{"line_number":21,"context_line":"occurs with respect to these variables), and an error message."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Closes Bug: 1951014"},{"line_number":24,"context_line":"Change-Id: I77db038d2f33fd42b9b8963fc6b31ebb0c425f6c"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"f152cf76_2f64c74c","line":22,"in_reply_to":"5a3835a8_dd5b32e5","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"cbbd0ceab4df705cba7b597313667001480fb25b","unresolved":true,"context_lines":[{"line_number":19,"context_line":"to None, and return gracefully with rc set to False (if an issue"},{"line_number":20,"context_line":"occurs with respect to these variables), and an error message."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes Bug: 1951014"},{"line_number":23,"context_line":"Change-Id: I77db038d2f33fd42b9b8963fc6b31ebb0c425f6c"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I720f63878f6672eccbe1a109f762966d73eef154"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dee4d443_98db0a13","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":10},"updated":"2021-11-15 21:34:12.000000000","message":"Closes-Bug","commit_id":"9a74647de9d356c20640d754b7b4041d46d9cab2"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"to None, and return gracefully with rc set to False (if an issue"},{"line_number":20,"context_line":"occurs with respect to these variables), and an error message."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes Bug: 1951014"},{"line_number":23,"context_line":"Change-Id: I77db038d2f33fd42b9b8963fc6b31ebb0c425f6c"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Change-Id: I720f63878f6672eccbe1a109f762966d73eef154"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"401eb6be_12f458a2","line":22,"range":{"start_line":22,"start_character":0,"end_line":22,"end_character":10},"in_reply_to":"dee4d443_98db0a13","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"9a74647de9d356c20640d754b7b4041d46d9cab2"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"5d35bc680e8adf212990b0e2b353da63c0c0f107","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This fix improves the original fix in"},{"line_number":21,"context_line":"https://opendev.org/starlingx/config/commit/88cacfc5d582a3bb7524a1a5599b289a52ab1f0d"},{"line_number":22,"context_line":"to fix the AttributeError scenario."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Test Plan:"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"6457ad6c_c87ca398","line":22,"range":{"start_line":22,"start_character":11,"end_line":22,"end_character":26},"updated":"2021-11-18 18:22:20.000000000","message":"UnboundLocalError ?","commit_id":"6b35fe48ae02e98f41d75de66bb7be48ebdf1960"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"566d2522fd590b974111521e69d5a2761b3f10b2","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"This fix improves the original fix in"},{"line_number":21,"context_line":"https://opendev.org/starlingx/config/commit/88cacfc5d582a3bb7524a1a5599b289a52ab1f0d"},{"line_number":22,"context_line":"to fix the AttributeError scenario."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Test Plan:"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"da4d01eb_3f462b2f","line":22,"range":{"start_line":22,"start_character":11,"end_line":22,"end_character":26},"in_reply_to":"6457ad6c_c87ca398","updated":"2021-11-18 19:21:40.000000000","message":"Done","commit_id":"6b35fe48ae02e98f41d75de66bb7be48ebdf1960"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28434,"name":"David Sullivan","email":"david.sullivan@windriver.com","username":"dsullivanwr"},"change_message_id":"ffc6d6a3155b2d50c8c85d07e63dd038441b1522","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9d12906a_d50498d8","updated":"2021-11-16 20:06:23.000000000","message":"It looks like this issue was already addressed by this commit\nhttps://opendev.org/starlingx/config/commit/88cacfc5d582a3bb7524a1a5599b289a52ab1f0d","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5cbd37bf_25ce5dc8","in_reply_to":"5377910d_f24645f8","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5377910d_f24645f8","in_reply_to":"9d12906a_d50498d8","updated":"2021-11-16 21:55:17.000000000","message":"The proposed change on line 3132 is a better change in my opinion in case of AttributeError.","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"}],"sysinv/sysinv/sysinv/sysinv/conductor/kube_app.py":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"49f027f6ea22d79bb0d7830165b4fc9379cb0a9e","unresolved":true,"context_lines":[{"line_number":3126,"context_line":""},{"line_number":3127,"context_line":"                    client.pull(target_img_tag, auth_config\u003dregistry_auth)"},{"line_number":3128,"context_line":""},{"line_number":3129,"context_line":"                except UnboundLocalError as e:"},{"line_number":3130,"context_line":"                    rc \u003d False"},{"line_number":3131,"context_line":"                    LOG.error(\"Image %s download failed from public/private\""},{"line_number":3132,"context_line":"                              \"due to Error: %s\" % (img_tag, e))"}],"source_content_type":"text/x-python","patch_set":1,"id":"45825c57_0349f368","line":3129,"updated":"2021-11-15 21:17:35.000000000","message":"UnboundLocalError exception should not occur with this fix?","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3126,"context_line":""},{"line_number":3127,"context_line":"                    client.pull(target_img_tag, auth_config\u003dregistry_auth)"},{"line_number":3128,"context_line":""},{"line_number":3129,"context_line":"                except UnboundLocalError as e:"},{"line_number":3130,"context_line":"                    rc \u003d False"},{"line_number":3131,"context_line":"                    LOG.error(\"Image %s download failed from public/private\""},{"line_number":3132,"context_line":"                              \"due to Error: %s\" % (img_tag, e))"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ce95654_9de90004","line":3129,"in_reply_to":"45825c57_0349f368","updated":"2021-11-18 17:09:49.000000000","message":"Done. Removed the exception.","commit_id":"e5eb844639dad23b21dfd9f09d4cfb5703cddf49"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":true,"context_lines":[{"line_number":3105,"context_line":"                             % img_tag)"},{"line_number":3106,"context_line":""},{"line_number":3107,"context_line":"                    # We should initialize the variables used"},{"line_number":3108,"context_line":"                    pub_img_tag \u003d None"},{"line_number":3109,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3110,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3111,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"4ed19c49_67993659","line":3108,"range":{"start_line":3108,"start_character":20,"end_line":3108,"end_character":38},"updated":"2021-11-16 21:55:17.000000000","message":"This is not necessary as it is already assigned below.","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3105,"context_line":"                             % img_tag)"},{"line_number":3106,"context_line":""},{"line_number":3107,"context_line":"                    # We should initialize the variables used"},{"line_number":3108,"context_line":"                    pub_img_tag \u003d None"},{"line_number":3109,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3110,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3111,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"c13c9a38_55f759cb","line":3108,"range":{"start_line":3108,"start_character":20,"end_line":3108,"end_character":38},"in_reply_to":"4ed19c49_67993659","updated":"2021-11-18 17:09:49.000000000","message":"Will be removed","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":true,"context_lines":[{"line_number":3109,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3110,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3111,"context_line":""},{"line_number":3112,"context_line":"                    if pub_img_tag is None:"},{"line_number":3113,"context_line":"                        LOG.error(\"Image %s download failed from public/private\""},{"line_number":3114,"context_line":"                                  \"unable to set pub_img_tag\" % img_tag)"},{"line_number":3115,"context_line":"                        return img_tag, None"}],"source_content_type":"text/x-python","patch_set":6,"id":"d115f4bd_ab58987c","line":3112,"range":{"start_line":3112,"start_character":20,"end_line":3112,"end_character":42},"updated":"2021-11-16 21:55:17.000000000","message":"This check is not necessary because if img_tag is None an AtributeError will be thrown.","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3109,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3110,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3111,"context_line":""},{"line_number":3112,"context_line":"                    if pub_img_tag is None:"},{"line_number":3113,"context_line":"                        LOG.error(\"Image %s download failed from public/private\""},{"line_number":3114,"context_line":"                                  \"unable to set pub_img_tag\" % img_tag)"},{"line_number":3115,"context_line":"                        return img_tag, None"}],"source_content_type":"text/x-python","patch_set":6,"id":"ecf1ce2e_fa4aee50","line":3112,"range":{"start_line":3112,"start_character":20,"end_line":3112,"end_character":42},"in_reply_to":"d115f4bd_ab58987c","updated":"2021-11-18 17:09:49.000000000","message":"Will be removed","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":true,"context_lines":[{"line_number":3114,"context_line":"                                  \"unable to set pub_img_tag\" % img_tag)"},{"line_number":3115,"context_line":"                        return img_tag, None"},{"line_number":3116,"context_line":""},{"line_number":3117,"context_line":"                    target_img_tag \u003d None"},{"line_number":3118,"context_line":"                    registry_auth \u003d None"},{"line_number":3119,"context_line":"                    target_img_tag, registry_auth \u003d \\"},{"line_number":3120,"context_line":"                        self._get_img_tag_with_registry(pub_img_tag, registries_info)"},{"line_number":3121,"context_line":""},{"line_number":3122,"context_line":"                    if target_img_tag is None or registry_auth is None:"}],"source_content_type":"text/x-python","patch_set":6,"id":"08da8d52_eb6fd90d","line":3119,"range":{"start_line":3117,"start_character":0,"end_line":3119,"end_character":0},"updated":"2021-11-16 21:55:17.000000000","message":"Same comment as for line 3108.","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3114,"context_line":"                                  \"unable to set pub_img_tag\" % img_tag)"},{"line_number":3115,"context_line":"                        return img_tag, None"},{"line_number":3116,"context_line":""},{"line_number":3117,"context_line":"                    target_img_tag \u003d None"},{"line_number":3118,"context_line":"                    registry_auth \u003d None"},{"line_number":3119,"context_line":"                    target_img_tag, registry_auth \u003d \\"},{"line_number":3120,"context_line":"                        self._get_img_tag_with_registry(pub_img_tag, registries_info)"},{"line_number":3121,"context_line":""},{"line_number":3122,"context_line":"                    if target_img_tag is None or registry_auth is None:"}],"source_content_type":"text/x-python","patch_set":6,"id":"f2059b9a_5956ad14","line":3119,"range":{"start_line":3117,"start_character":0,"end_line":3119,"end_character":0},"in_reply_to":"08da8d52_eb6fd90d","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":true,"context_lines":[{"line_number":3119,"context_line":"                    target_img_tag, registry_auth \u003d \\"},{"line_number":3120,"context_line":"                        self._get_img_tag_with_registry(pub_img_tag, registries_info)"},{"line_number":3121,"context_line":""},{"line_number":3122,"context_line":"                    if target_img_tag is None or registry_auth is None:"},{"line_number":3123,"context_line":"                        LOG.error(\"Image %s download failed from public/private\""},{"line_number":3124,"context_line":"                                  \"unable to get image tag with registry\" % img_tag)"},{"line_number":3125,"context_line":"                        return img_tag, None"},{"line_number":3126,"context_line":""},{"line_number":3127,"context_line":"                    client.pull(target_img_tag, auth_config\u003dregistry_auth)"},{"line_number":3128,"context_line":""},{"line_number":3129,"context_line":"                except Exception as e:"}],"source_content_type":"text/x-python","patch_set":6,"id":"26e948ba_f24ce420","line":3126,"range":{"start_line":3122,"start_character":0,"end_line":3126,"end_character":0},"updated":"2021-11-16 21:55:17.000000000","message":"The existing functionality is broken with this check as default REGISTRIES info is acceptable. Furthermore, the return value is also incorrect. The second value of the tuple must be True, False","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3119,"context_line":"                    target_img_tag, registry_auth \u003d \\"},{"line_number":3120,"context_line":"                        self._get_img_tag_with_registry(pub_img_tag, registries_info)"},{"line_number":3121,"context_line":""},{"line_number":3122,"context_line":"                    if target_img_tag is None or registry_auth is None:"},{"line_number":3123,"context_line":"                        LOG.error(\"Image %s download failed from public/private\""},{"line_number":3124,"context_line":"                                  \"unable to get image tag with registry\" % img_tag)"},{"line_number":3125,"context_line":"                        return img_tag, None"},{"line_number":3126,"context_line":""},{"line_number":3127,"context_line":"                    client.pull(target_img_tag, auth_config\u003dregistry_auth)"},{"line_number":3128,"context_line":""},{"line_number":3129,"context_line":"                except Exception as e:"}],"source_content_type":"text/x-python","patch_set":6,"id":"aeb1ae43_f3c00bdd","line":3126,"range":{"start_line":3122,"start_character":0,"end_line":3126,"end_character":0},"in_reply_to":"26e948ba_f24ce420","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"3543872a4aa827f3e0af9861706de613bb516cdd","unresolved":true,"context_lines":[{"line_number":3129,"context_line":"                except Exception as e:"},{"line_number":3130,"context_line":"                    rc \u003d False"},{"line_number":3131,"context_line":"                    LOG.error(\"Image %s download failed from public/private\""},{"line_number":3132,"context_line":"                              \"registry: %s\" % (img_tag, e))"},{"line_number":3133,"context_line":"                    return img_tag, rc"},{"line_number":3134,"context_line":""},{"line_number":3135,"context_line":"                try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"f462761e_0bd46523","line":3132,"range":{"start_line":3132,"start_character":0,"end_line":3132,"end_character":1},"updated":"2021-11-16 21:55:17.000000000","message":"The needed fix for UnboundLocalError is this line.","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"e9c61b1acf05580697df01679dfed0f782018c5a","unresolved":false,"context_lines":[{"line_number":3129,"context_line":"                except Exception as e:"},{"line_number":3130,"context_line":"                    rc \u003d False"},{"line_number":3131,"context_line":"                    LOG.error(\"Image %s download failed from public/private\""},{"line_number":3132,"context_line":"                              \"registry: %s\" % (img_tag, e))"},{"line_number":3133,"context_line":"                    return img_tag, rc"},{"line_number":3134,"context_line":""},{"line_number":3135,"context_line":"                try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"b57a0239_eb2a9a20","line":3132,"range":{"start_line":3132,"start_character":0,"end_line":3132,"end_character":1},"in_reply_to":"f462761e_0bd46523","updated":"2021-11-18 17:09:49.000000000","message":"Done","commit_id":"060d234debc81a90d6df52615b92597f6a114fe9"},{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"d659ae3891c7ee4c9052983558dfdc754a4f0d32","unresolved":true,"context_lines":[{"line_number":3104,"context_line":"                             \"download started from public/private registry\""},{"line_number":3105,"context_line":"                             % img_tag)"},{"line_number":3106,"context_line":""},{"line_number":3107,"context_line":"                    # We should initialize the variables used"},{"line_number":3108,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3109,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3110,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"42aa5f83_33db0400","line":3107,"range":{"start_line":3107,"start_character":0,"end_line":3107,"end_character":61},"updated":"2021-11-17 16:48:05.000000000","message":"Please remove this comment.","commit_id":"8eb6d23a513f411474e036036a2c5c31648a95b7"},{"author":{"_account_id":34119,"name":"Shrikumar Sharma","display_name":"Shrikumar Sharma","email":"shrikumar.sharma@windriver.com","username":"ssharma3"},"change_message_id":"da7f4ec92ca8804231ee1e0ff331a24a00cf4120","unresolved":false,"context_lines":[{"line_number":3104,"context_line":"                             \"download started from public/private registry\""},{"line_number":3105,"context_line":"                             % img_tag)"},{"line_number":3106,"context_line":""},{"line_number":3107,"context_line":"                    # We should initialize the variables used"},{"line_number":3108,"context_line":"                    pub_img_tag \u003d img_tag.replace("},{"line_number":3109,"context_line":"                        constants.DOCKER_REGISTRY_SERVER + \"/\", \"\")"},{"line_number":3110,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"6bcfd883_d595cdfe","line":3107,"range":{"start_line":3107,"start_character":0,"end_line":3107,"end_character":61},"in_reply_to":"42aa5f83_33db0400","updated":"2021-11-18 17:10:08.000000000","message":"Done","commit_id":"8eb6d23a513f411474e036036a2c5c31648a95b7"}]}