)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6e7e7f58f3001a415f18ee86e6ac98c4774da527","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     songwenping \u003csongwenping@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-12-10 05:40:49 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Generate cyborgclient when there is device_prfoile in instance"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ida6d6323b1a217066dc6588966a315ed087c7f5d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1c274570_9ee4db75","line":7,"range":{"start_line":7,"start_character":36,"end_line":7,"end_character":50},"updated":"2021-12-13 10:59:22.000000000","message":"nit: device_profile","commit_id":"ae986da06cd285884ad13e6382a5c1705f167b2e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6e7e7f58f3001a415f18ee86e6ac98c4774da527","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"735c84ce_cd63e499","updated":"2021-12-13 10:59:22.000000000","message":"Please also add unit test coverage for the change","commit_id":"ae986da06cd285884ad13e6382a5c1705f167b2e"}],"nova/compute/manager.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6e7e7f58f3001a415f18ee86e6ac98c4774da527","unresolved":true,"context_lines":[{"line_number":9594,"context_line":"                    # TODO(brinzhang): After cyborg support batch query ARQs"},{"line_number":9595,"context_line":"                    # for more than one instances, we will improve efficiency"},{"line_number":9596,"context_line":"                    # with this implemention."},{"line_number":9597,"context_line":"                    cyclient \u003d cyborg.get_client(context)"},{"line_number":9598,"context_line":"                    accel_uuids \u003d cyclient.get_arq_uuids_for_instance(instance)"},{"line_number":9599,"context_line":"                self.shelve_offload_instance("},{"line_number":9600,"context_line":"                    context, instance, clean_shutdown\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1b91ae29_2871cf5a","line":9597,"updated":"2021-12-13 10:59:22.000000000","message":"This means you will recreate the client for every instance. That feels wasteful. The baseline solution creates one client for all the instances but it also creates a client if there is no instances. If you want to avoid the latter then you can simply exit from the task if there is no instance to gc:\n\n    if not to_gc:\n        return\n    \n    cyclient \u003d cyborg.get_client(context)\n    for instance in to_gc:\n        ...","commit_id":"ae986da06cd285884ad13e6382a5c1705f167b2e"}]}