)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"3a6678a96680a84e37ab5484289af5c4116656c0","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Pooja Ghumre \u003cpooja@platform9.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-18 15:40:02 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"If OS populated machine_id file is empty, use hardware serial for the libvirt driver."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Nova intends to treat absence of /etc/machine-id file same as the file being empty."},{"line_number":10,"context_line":"However, when sysinfo_serial option is set to \u0027auto\u0027, it only checks for presence of file and not whether id is present."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_aed224fe","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":85},"updated":"2016-07-20 03:25:41.000000000","message":"Welcome, Pooja!\n\nHere are a few conventions the OpenStack community tries to follow for commit messages:\n\nCommit messages should start with a short 50 character or less summary on a single line (without a period). Subsequent lines should be wrapped at 72 characters.\n\n- http://docs.openstack.org/infra/manual/developers.html#committing-a-change\n- https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"b621b7ab0571b8da1ce7c0d9d8b3b8d7c62855ff"},{"author":{"_account_id":22802,"name":"Pooja Ghumre","email":"poojaghumre@gmail.com","username":"poojaghumre"},"change_message_id":"7504c4fbf5739b5eccd417f429411d82c722ab0b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Pooja Ghumre \u003cpooja@platform9.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-18 15:40:02 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"If OS populated machine_id file is empty, use hardware serial for the libvirt driver."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Nova intends to treat absence of /etc/machine-id file same as the file being empty."},{"line_number":10,"context_line":"However, when sysinfo_serial option is set to \u0027auto\u0027, it only checks for presence of file and not whether id is present."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_5fb7d3f4","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":85},"in_reply_to":"dada55a8_aed224fe","updated":"2016-07-20 06:20:31.000000000","message":"Thanks Diana! I will update the commit summary and submit for review.","commit_id":"b621b7ab0571b8da1ce7c0d9d8b3b8d7c62855ff"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bce8f14095cc7a38bc46b5a7892b51dad198745a","unresolved":false,"context_lines":[{"line_number":3581,"context_line":"            # so we parse \u0026 reformat the value"},{"line_number":3582,"context_line":"            lines \u003d f.read().split()"},{"line_number":3583,"context_line":"            if not lines:"},{"line_number":3584,"context_line":"                msg \u003d _(\"Unable to get host UUID: /etc/machine-id is empty\")"},{"line_number":3585,"context_line":"                raise exception.NovaException(msg)"},{"line_number":3586,"context_line":""},{"line_number":3587,"context_line":"            return str(uuid.UUID(lines[0]))"},{"line_number":3588,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_90a02845","line":3585,"range":{"start_line":3584,"start_character":16,"end_line":3585,"end_character":50},"updated":"2016-07-19 22:27:17.000000000","message":"So I guess you\u0027re intending to handle this for auto too?","commit_id":"b621b7ab0571b8da1ce7c0d9d8b3b8d7c62855ff"},{"author":{"_account_id":22802,"name":"Pooja Ghumre","email":"poojaghumre@gmail.com","username":"poojaghumre"},"change_message_id":"7504c4fbf5739b5eccd417f429411d82c722ab0b","unresolved":false,"context_lines":[{"line_number":3581,"context_line":"            # so we parse \u0026 reformat the value"},{"line_number":3582,"context_line":"            lines \u003d f.read().split()"},{"line_number":3583,"context_line":"            if not lines:"},{"line_number":3584,"context_line":"                msg \u003d _(\"Unable to get host UUID: /etc/machine-id is empty\")"},{"line_number":3585,"context_line":"                raise exception.NovaException(msg)"},{"line_number":3586,"context_line":""},{"line_number":3587,"context_line":"            return str(uuid.UUID(lines[0]))"},{"line_number":3588,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_a76ecf93","line":3585,"range":{"start_line":3584,"start_character":16,"end_line":3585,"end_character":50},"in_reply_to":"dada55a8_90a02845","updated":"2016-07-20 06:20:31.000000000","message":"Yes, that is right. I will add a unit test for this case.","commit_id":"b621b7ab0571b8da1ce7c0d9d8b3b8d7c62855ff"}]}