)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e847d2e595a4612a3b9c057f53d57839f69a999b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch moves the creation of the rebuild_claim from rebuild_instance"},{"line_number":10,"context_line":"to _do_rebuild_instance_with_claim() helper so that in later patches"},{"line_number":11,"context_line":"more code can be added to _do_rebuild_instance_with_claim right"},{"line_number":12,"context_line":"before the rebuild_claim is created without dealing with deep nesting"},{"line_number":13,"context_line":"in rebuild_instance."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Icf445ed185086e7add31bc921fd3d3a45344a887"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_3e7c2471","line":12,"updated":"2019-10-23 19:28:49.000000000","message":"OK and that is this:\n\nhttps://review.opendev.org/#/c/688387/6/nova/compute/manager.py\n\nWhich doesn\u0027t seem like much but I could see how that would make it a lot easier to test that addition without having to mock out all of the stuff going on in rebuild_instance before we get to the claims code. rebuild_instance is one of those methods that could be benefit in general from being broken down into smaller parts for easier unit testing and error handling.","commit_id":"05c60fe70e743f4ca63e9c243cde9d5ff0c4c6e2"}]}