)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"e34b32f4b147b3e0981fc1bc53fa39e0c8d17bc5","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"PASS: Tried to lock a host during a backup run"},{"line_number":18,"context_line":"PASS: Tried to lock a host after backup failed and flag was not cleared"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: 1990984"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4ef83100_e0b36775","line":17,"updated":"2022-09-27 15:25:35.000000000","message":"please describe what is the expected behavior (alarm? warning? command failure?) so the tests can be implemented properly when automating. Ditto for the 2nd TC.","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"},{"author":{"_account_id":34198,"name":"Luis Eduardo Angelini Marquitti","email":"luis.eduardoangelinimarquitti@windriver.com","username":"leduard1"},"change_message_id":"905c03b1f45431f12064a3f68018f3b8c01d47e8","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"PASS: Tried to lock a host during a backup run"},{"line_number":18,"context_line":"PASS: Tried to lock a host after backup failed and flag was not cleared"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: 1990984"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c01bfa2e_d2e7e7f6","line":17,"in_reply_to":"4ef83100_e0b36775","updated":"2022-10-21 20:35:42.000000000","message":"Done","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"58670789fe9523d1404151743c8b8b92c63ff340","unresolved":true,"context_lines":[{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"PASS: Tried to lock a host during a backup run"},{"line_number":18,"context_line":"PASS: Tried to lock a host after backup failed and flag was not cleared"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: 1990984"},{"line_number":21,"context_line":"Change-Id: I747d79f41cff41b4990cc40cff2150a73c10b056"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"eafe3aeb_dedfbe58","line":18,"updated":"2022-09-28 15:01:39.000000000","message":"please verify that after load-import and upgrade-start, that the flag is cleared before the upgrade-start goes to \u0027started\u0027; as do not want to block orchestration which performs the host-lock after upgrade-start.","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"},{"author":{"_account_id":34198,"name":"Luis Eduardo Angelini Marquitti","email":"luis.eduardoangelinimarquitti@windriver.com","username":"leduard1"},"change_message_id":"905c03b1f45431f12064a3f68018f3b8c01d47e8","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Test plan:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"PASS: Tried to lock a host during a backup run"},{"line_number":18,"context_line":"PASS: Tried to lock a host after backup failed and flag was not cleared"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Closes-Bug: 1990984"},{"line_number":21,"context_line":"Change-Id: I747d79f41cff41b4990cc40cff2150a73c10b056"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5bb2339b_67288798","line":18,"in_reply_to":"eafe3aeb_dedfbe58","updated":"2022-10-21 20:35:42.000000000","message":"Done","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"}],"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/host.py":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"4d4c8b6cb4ba8b2468c124d23f643e93f4661b3e","unresolved":true,"context_lines":[{"line_number":5525,"context_line":"                  \"\u0027locked\u0027 host %s\" % (hostupdate.ihost_patch[\u0027action\u0027],"},{"line_number":5526,"context_line":"                                        hostupdate.ihost_orig[\u0027hostname\u0027])))"},{"line_number":5527,"context_line":""},{"line_number":5528,"context_line":"        # Semantic Check: Avoid Lock when a backup is in progress"},{"line_number":5529,"context_line":"        if os.path.isfile(tsc.BACKUP_IN_PROGRESS_FLAG):"},{"line_number":5530,"context_line":"            raise wsme.exc.ClientSideError("},{"line_number":5531,"context_line":"                _(\"Avoiding lock action during \u0027backup in progress\u0027 on host %s. \""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f285177_38a18a2d","line":5528,"updated":"2022-09-27 15:26:10.000000000","message":"recommend to add tox unit test for code coverage:\ne.g. https://opendev.org/starlingx/config/src/branch/master/sysinv/sysinv/sysinv/sysinv/tests/api/test_host.py#L2182","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"},{"author":{"_account_id":34198,"name":"Luis Eduardo Angelini Marquitti","email":"luis.eduardoangelinimarquitti@windriver.com","username":"leduard1"},"change_message_id":"905c03b1f45431f12064a3f68018f3b8c01d47e8","unresolved":false,"context_lines":[{"line_number":5525,"context_line":"                  \"\u0027locked\u0027 host %s\" % (hostupdate.ihost_patch[\u0027action\u0027],"},{"line_number":5526,"context_line":"                                        hostupdate.ihost_orig[\u0027hostname\u0027])))"},{"line_number":5527,"context_line":""},{"line_number":5528,"context_line":"        # Semantic Check: Avoid Lock when a backup is in progress"},{"line_number":5529,"context_line":"        if os.path.isfile(tsc.BACKUP_IN_PROGRESS_FLAG):"},{"line_number":5530,"context_line":"            raise wsme.exc.ClientSideError("},{"line_number":5531,"context_line":"                _(\"Avoiding lock action during \u0027backup in progress\u0027 on host %s. \""}],"source_content_type":"text/x-python","patch_set":1,"id":"851d87a3_426f606e","line":5528,"in_reply_to":"3f285177_38a18a2d","updated":"2022-10-21 20:35:42.000000000","message":"Done","commit_id":"eb6b9f74a811f6f03526f0d1f8d28e3b51309640"}]}
