)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"41506fa11285d878ce0459c9ec71aebcbc8ef34c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"xenapi: ensure vcpu_weight configured correctly"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently the vcpu_weight specified the flavor is only set after the"},{"line_number":10,"context_line":"VM has started, and is not set during migrate_finish."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This changes moves the setting of the vcpu_weight to the same place that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAQn%2F%2F0%2BA%3D","line":9,"updated":"2013-07-22 15:53:53.000000000","message":"specified by the flavor?","commit_id":"34750dbe11b7a791ac9c1f8561a5774bd221e0f6"}],"nova/tests/virt/xenapi/test_xenapi.py":[{"author":{"_account_id":2835,"name":"Brian Elliott","email":"bdelliott@gmail.com","username":"belliott"},"change_message_id":"41506fa11285d878ce0459c9ec71aebcbc8ef34c","unresolved":false,"context_lines":[{"line_number":822,"context_line":"    def test_spawn_vhd_glance_windows(self):"},{"line_number":823,"context_line":"        self._test_spawn(IMAGE_VHD, None, None,"},{"line_number":824,"context_line":"                         os_type\u003d\"windows\", architecture\u003d\"i386\","},{"line_number":825,"context_line":"                         instance_type_id\u003d5)"},{"line_number":826,"context_line":"        self.check_vm_params_for_windows()"},{"line_number":827,"context_line":""},{"line_number":828,"context_line":"    def test_spawn_iso_glance(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F08I%3D","line":825,"updated":"2013-07-22 15:53:53.000000000","message":"I\u0027m assuming the case where an instance type has vcpu_weight \u003d\u003d None is already covered somewhere?","commit_id":"34750dbe11b7a791ac9c1f8561a5774bd221e0f6"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"c51d20adcab154f60603a9007644bd0b5dfefa8a","unresolved":false,"context_lines":[{"line_number":822,"context_line":"    def test_spawn_vhd_glance_windows(self):"},{"line_number":823,"context_line":"        self._test_spawn(IMAGE_VHD, None, None,"},{"line_number":824,"context_line":"                         os_type\u003d\"windows\", architecture\u003d\"i386\","},{"line_number":825,"context_line":"                         instance_type_id\u003d5)"},{"line_number":826,"context_line":"        self.check_vm_params_for_windows()"},{"line_number":827,"context_line":""},{"line_number":828,"context_line":"    def test_spawn_iso_glance(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F0yY%3D","line":825,"in_reply_to":"AAAAQn%2F%2F08I%3D","updated":"2013-07-22 16:36:11.000000000","message":"Yep, the default is instance_type_id\u003d3, which is None. Its a bit hidden though...","commit_id":"34750dbe11b7a791ac9c1f8561a5774bd221e0f6"}],"nova/virt/xenapi/vm_utils.py":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"change_message_id":"1a32bf61822bba2842de871a26e3b9f675ce0cef","unresolved":false,"context_lines":[{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    vcpu_weight \u003d instance_type[\u0027vcpu_weight\u0027]"},{"line_number":221,"context_line":"    vcpu_params \u003d {}"},{"line_number":222,"context_line":"    if vcpu_weight !\u003d None:"},{"line_number":223,"context_line":"        vcpu_params \u003d {\"weight\": str(vcpu_weight)}"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"    rec \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FzFE%3D","line":222,"updated":"2013-07-23 02:29:18.000000000","message":"PEP8 states that singletons (True, False, None) shoud use is/is not on comparsion.\n\n if vcpu_weight is not None:","commit_id":"34750dbe11b7a791ac9c1f8561a5774bd221e0f6"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"687ff79ed7297db4697ee24164bf96c51dbb1c8b","unresolved":false,"context_lines":[{"line_number":219,"context_line":""},{"line_number":220,"context_line":"    vcpu_weight \u003d instance_type[\u0027vcpu_weight\u0027]"},{"line_number":221,"context_line":"    vcpu_params \u003d {}"},{"line_number":222,"context_line":"    if vcpu_weight !\u003d None:"},{"line_number":223,"context_line":"        vcpu_params \u003d {\"weight\": str(vcpu_weight)}"},{"line_number":224,"context_line":""},{"line_number":225,"context_line":"    rec \u003d {"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FyQ4%3D","line":222,"in_reply_to":"AAAAQn%2F%2FzFE%3D","updated":"2013-07-23 09:06:13.000000000","message":"yeah, good spot.","commit_id":"34750dbe11b7a791ac9c1f8561a5774bd221e0f6"}]}