)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c63ee8929463d427b36963ae60792eb9fa2a1cce","unresolved":true,"context_lines":[{"line_number":19,"context_line":"do not want to delete the node in Ironic, but are removing it"},{"line_number":20,"context_line":"from nova management. But on balance, it seems safer to leave"},{"line_number":21,"context_line":"these cases up to the operator to resolve manually, and collect"},{"line_number":22,"context_line":"feedback on how to better help those users."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"blueprint ironic-shards"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"670feb7c_b1e18b4e","line":22,"updated":"2024-02-25 12:10:55.000000000","message":"nit it would have been nice to call out the virt driver api change here\n\nbut given its additive its fine.","commit_id":"4e16f26a9a70229d396178fda35651fb49d9d7eb"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"c88faff6bfc4be816cdca09f736444b28695d0fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"28af34a3_3da8857c","updated":"2024-02-29 16:14:47.000000000","message":"Eventually approving the series, now we checked it was working by Tempest and we\u0027ll have an experimental/periodic test for it with https://review.opendev.org/c/openstack/nova/+/910333","commit_id":"947bb5f641095e726c80c87d8430c6e99a58ddf0"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"b50cd1be8c837f271de4d06bdeccaeac89170551","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"64a6ea76_4f780464","updated":"2024-02-28 17:51:46.000000000","message":"I still like this change. It stops us deleting compute nodes we shouldn\u0027t ever delete.","commit_id":"947bb5f641095e726c80c87d8430c6e99a58ddf0"}],"nova/tests/unit/compute/test_compute_mgr.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c63ee8929463d427b36963ae60792eb9fa2a1cce","unresolved":true,"context_lines":[{"line_number":509,"context_line":"        db_nodes[0].destroy.side_effect \u003d exception.ObjectActionError("},{"line_number":510,"context_line":"            action\u003d\u0027destroy\u0027, reason\u003d\u0027host changed\u0027)"},{"line_number":511,"context_line":"        get_avail_nodes.return_value \u003d set()"},{"line_number":512,"context_line":"        mock_deleted.return_value \u003d False"},{"line_number":513,"context_line":"        self.compute.update_available_resource(self.context)"},{"line_number":514,"context_line":"        get_db_nodes.assert_called_once_with(self.context, set(),"},{"line_number":515,"context_line":"                                             use_slave\u003dTrue, startup\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":4,"id":"cdcab07f_f36d701c","line":512,"updated":"2024-02-25 12:10:55.000000000","message":"ah here is the assignemtn  of false. ok so we have coverage of both branches.","commit_id":"4e16f26a9a70229d396178fda35651fb49d9d7eb"}]}