)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"b395081e20b0df93a33edc79090cf817f1415357","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"xenapi: set vcpu cap to ensure weight is applied"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a bug in XenServer 6.1 and 6.2 that means the vcpu weight"},{"line_number":10,"context_line":"we set is ignored if the cap is not also set."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Setting the cap to its default of 0, meaning no cap, to ensure the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAS3%2F%2FrQ4%3D","line":9,"updated":"2013-09-22 15:03:21.000000000","message":"nit: it would be nice that we can paste the bug link here which allow to be digging in git history as well as to be helpful to review, thanks.","commit_id":"03d15b6498446db69ca150d4969d2e83b004bc64"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"739581465b545d4bf23030e087ae5087e291d798","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"xenapi: set vcpu cap to ensure weight is applied"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a bug in XenServer 6.1 and 6.2 that means the vcpu weight"},{"line_number":10,"context_line":"we set is ignored if the cap is not also set."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Setting the cap to its default of 0, meaning no cap, to ensure the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"AAAAS3%2F%2FpDQ%3D","line":9,"in_reply_to":"AAAAS3%2F%2FrQ4%3D","updated":"2013-09-23 14:54:08.000000000","message":"It would be good, but it doesn\u0027t have an external bug tracker today, so there is nothing really to reference here :(","commit_id":"03d15b6498446db69ca150d4969d2e83b004bc64"}],"nova/virt/xenapi/vm_utils.py":[{"author":{"_account_id":475,"name":"Rick Harris","email":"rick.harris@rackspace.com","username":"rconradharris"},"change_message_id":"efa1654b41af73da54988e1409ed40741bc4b0f6","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    vcpu_weight \u003d instance_type[\u0027vcpu_weight\u0027]"},{"line_number":212,"context_line":"    vcpu_params \u003d {}"},{"line_number":213,"context_line":"    if vcpu_weight is not None:"},{"line_number":214,"context_line":"        # NOTE(johngarbutt) bug in XenServer means we need to specify both"},{"line_number":215,"context_line":"        vcpu_params \u003d {\"weight\": str(vcpu_weight), \"cap\": \"0\"}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    rec \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2Fo6I%3D","line":214,"updated":"2013-09-23 15:26:47.000000000","message":"Might be worth including the explanation from the commit msg of  which hypervisor versions it affects and why it\u0027s needed.","commit_id":"03d15b6498446db69ca150d4969d2e83b004bc64"},{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"7139e75923dda4c8ced28803573110d3dbf68f25","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    vcpu_weight \u003d instance_type[\u0027vcpu_weight\u0027]"},{"line_number":212,"context_line":"    vcpu_params \u003d {}"},{"line_number":213,"context_line":"    if vcpu_weight is not None:"},{"line_number":214,"context_line":"        # NOTE(johngarbutt) bug in XenServer means we need to specify both"},{"line_number":215,"context_line":"        vcpu_params \u003d {\"weight\": str(vcpu_weight), \"cap\": \"0\"}"},{"line_number":216,"context_line":""},{"line_number":217,"context_line":"    rec \u003d {"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FoDg%3D","line":214,"in_reply_to":"AAAAS3%2F%2Fo6I%3D","updated":"2013-09-23 19:13:33.000000000","message":"Rick\u0027s suggestion seems worthwhile, then I think it\u0027s good to go.","commit_id":"03d15b6498446db69ca150d4969d2e83b004bc64"}]}