)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"20019c495e14f0263c67b720dc537db2991a61b6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The HostPciSRIOVDevicesInfo test object provided a \u0027create_pci_devices\u0027"},{"line_number":10,"context_line":"function that did what \u0027__init__\u0027 usually does. Adopt the principle of"},{"line_number":11,"context_line":"least suprise and just use \u0027__init__\u0027 directly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I7723cfb6a2d2699d809739a4b03c46a300222897"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3a461143_9cd0a6f9","line":11,"updated":"2017-02-02 13:52:10.000000000","message":"s/suprise/surprise/\n\nsurprise!","commit_id":"33fbf35b211cfceee68258580832dd23c36444a3"}],"nova/tests/unit/virt/libvirt/fakelibvirt.py":[{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"ad6387dce27896c6cf132b1a04552f6d84de32e9","unresolved":false,"context_lines":[{"line_number":290,"context_line":""},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"class HostInfo(object):"},{"line_number":293,"context_line":""},{"line_number":294,"context_line":"    def __init__(self, arch\u003dobj_fields.Architecture.X86_64, kB_mem\u003d4096,"},{"line_number":295,"context_line":"                 cpus\u003d2, cpu_mhz\u003d800, cpu_nodes\u003d1,"},{"line_number":296,"context_line":"                 cpu_sockets\u003d1, cpu_cores\u003d2,"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a6eadb0_7f73ede7","line":293,"updated":"2016-12-19 08:51:02.000000000","message":"unrelated change","commit_id":"0f393530c3e6f146aa858825422ad9bacc6665d0"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"20019c495e14f0263c67b720dc537db2991a61b6","unresolved":false,"context_lines":[{"line_number":255,"context_line":"        def _calc_numa_node(dev):"},{"line_number":256,"context_line":"            return dev % total_numa_nodes if numa_node is None else numa_node"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"        self.devices \u003d {}"},{"line_number":259,"context_line":"        if num_vfs and not num_pfs:"},{"line_number":260,"context_line":"            raise ValueError(\u0027Cannot create VFs without PFs\u0027)"},{"line_number":261,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a461143_5c9e3e08","line":258,"updated":"2017-02-02 13:52:10.000000000","message":"In future, consider either putting style/variable-rename-only changes like this (and the many other LOCC in this patch due to this variable rename) in a separate patch marked as style-only. Helps reduce reviewer load.","commit_id":"33fbf35b211cfceee68258580832dd23c36444a3"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"20019c495e14f0263c67b720dc537db2991a61b6","unresolved":false,"context_lines":[{"line_number":257,"context_line":""},{"line_number":258,"context_line":"        self.devices \u003d {}"},{"line_number":259,"context_line":"        if num_vfs and not num_pfs:"},{"line_number":260,"context_line":"            raise ValueError(\u0027Cannot create VFs without PFs\u0027)"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"        vf_ratio \u003d num_vfs // num_pfs if num_pfs else 0"},{"line_number":263,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3a461143_fce1aa9b","line":260,"updated":"2017-02-02 13:52:10.000000000","message":"This is an unrelated functional change, probably best left to a separate patch to reduce reviewer cognitive load","commit_id":"33fbf35b211cfceee68258580832dd23c36444a3"}]}