)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"ebf242592528b55a34f138bec23742525504d896","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephane Wartel \u003cstephane.wartel@deveryware.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-31 21:13:20 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a condition on fact if ovs is not already installed"},{"line_number":8,"context_line":"Add a condition to manifest if fact is null"},{"line_number":9,"context_line":"Closes-Bug: #1540118"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a5de9d1_8f1bc8c9","line":7,"updated":"2016-01-31 21:37:28.000000000","message":"Can you fix this commit message to follow a correct format?  https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"10f999353bb86273810642d0108f07b959092d18"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"2632dd71e415935290cd1fe28f78d93dd02f6a44","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephane Wartel \u003cstephane.wartel@deveryware.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-01-31 22:11:01 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Condition if ovs is not already installed"},{"line_number":8,"context_line":"Closes-Bug: #1540118"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I2e1b721aca8b7385f24184a9a51cc620c70e5075"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a5de9d1_b2df0b68","line":7,"updated":"2016-01-31 22:54:58.000000000","message":"You need a blank line after this. The other one was kind of OK but you needed a format similar to:\n\n  Short commit summary\n  \u003cblank line\u003e\n  Longer commit description\n  Closes-Bug: #bugid\n\n\nIf you don\u0027t put a blank line after the first line, all of the lines get merged together","commit_id":"06aec3cf3ebc9248cb97f27bc30520a58b137369"}],"manifests/ovs.pp":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"39db5a372c1ed24b33c24515748c86a2cd2c892f","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        } else {"},{"line_number":72,"context_line":"          $kernel_mod_file \u003d \"/lib/modules/${::kernelrelease}/updates/dkms/openvswitch.ko\""},{"line_number":73,"context_line":"        }"},{"line_number":74,"context_line":"      }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    }"},{"line_number":77,"context_line":"    \u0027Redhat\u0027: {"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"7a5de9d1_ae445049","line":74,"updated":"2016-02-01 23:29:58.000000000","message":"I would argue that we should set $kernel_mod_file outside of the if $::ovs_version conditional, but as far as I can tell we forgot to use $kernel_mod_file anywhere, so we can figure that out in a different patch.","commit_id":"231e5ca7d7e3c60972b6504abaa7ed98b8f88edd"}]}
